mbox series

[0/4] updates for hns3 ethernet pmd driver

Message ID 20191009141653.39364-1-xavier.huwei@tom.com (mailing list archive)
Headers
Series updates for hns3 ethernet pmd driver |

Message

Wei Hu (Xavier) Oct. 9, 2019, 2:16 p.m. UTC
  This series adds some updates for hns3 ethernet pmd driver.

The No.1 patch modifies the statistics information for sending
and receiving packets.
The No.2 patch changes the return value of firmware processing
timeout from -EBADE to -ETIME.
The No.3 patch adds commands between driver and firmware.
The No.4 patch restores bus_master_en and msix_enable during
PF FLR reset.

Chunsong Feng (1):
  net/hns3: restores bus_master_en and msix_enable during PF FLR reset

Hao Chen (1):
  net/hns3: modify the statistics for sending and receiving messages

Hongbo Zheng (1):
  net/hns3: change the return value of firmware processing timeout from
    -EBADE to -ETIME

humin (1):
  net/hns3: Renew command and desc structure

 drivers/net/hns3/hns3_cmd.c       |  17 +---
 drivers/net/hns3/hns3_cmd.h       |   9 +-
 drivers/net/hns3/hns3_ethdev.c    |   5 ++
 drivers/net/hns3/hns3_ethdev_vf.c | 135 +++++++++++++++++++++++++++++-
 drivers/net/hns3/hns3_intr.c      |   5 +-
 drivers/net/hns3/hns3_rxtx.c      |  11 +--
 drivers/net/hns3/hns3_rxtx.h      |   3 -
 drivers/net/hns3/hns3_stats.c     |  84 +++++--------------
 8 files changed, 168 insertions(+), 101 deletions(-)
  

Comments

Wei Hu (Xavier) Oct. 10, 2019, 3:28 a.m. UTC | #1
Hi, Ferruh Yigit

  The page in patches.dpdk.org indicates that CI checking against this
series failed, the page as follows:

http://mails.dpdk.org/archives/test-report/2019-October/101655.html

  Just like the previous failure of patch CI checking, this series based
on dpdk-next-net repo but CI checking based on dpdk repo.

  Thanks.

 
  Regards
Xavier

On 2019/10/9 22:16, Wei Hu (Xavier) wrote:
> This series adds some updates for hns3 ethernet pmd driver.
>
> The No.1 patch modifies the statistics information for sending
> and receiving packets.
> The No.2 patch changes the return value of firmware processing
> timeout from -EBADE to -ETIME.
> The No.3 patch adds commands between driver and firmware.
> The No.4 patch restores bus_master_en and msix_enable during
> PF FLR reset.
>
> Chunsong Feng (1):
>   net/hns3: restores bus_master_en and msix_enable during PF FLR reset
>
> Hao Chen (1):
>   net/hns3: modify the statistics for sending and receiving messages
>
> Hongbo Zheng (1):
>   net/hns3: change the return value of firmware processing timeout from
>     -EBADE to -ETIME
>
> humin (1):
>   net/hns3: Renew command and desc structure
>
>  drivers/net/hns3/hns3_cmd.c       |  17 +---
>  drivers/net/hns3/hns3_cmd.h       |   9 +-
>  drivers/net/hns3/hns3_ethdev.c    |   5 ++
>  drivers/net/hns3/hns3_ethdev_vf.c | 135 +++++++++++++++++++++++++++++-
>  drivers/net/hns3/hns3_intr.c      |   5 +-
>  drivers/net/hns3/hns3_rxtx.c      |  11 +--
>  drivers/net/hns3/hns3_rxtx.h      |   3 -
>  drivers/net/hns3/hns3_stats.c     |  84 +++++--------------
>  8 files changed, 168 insertions(+), 101 deletions(-)
>
  
Ferruh Yigit Oct. 15, 2019, 8:46 a.m. UTC | #2
On 10/9/2019 3:16 PM, Wei Hu (Xavier) wrote:
> This series adds some updates for hns3 ethernet pmd driver.
> 
> The No.1 patch modifies the statistics information for sending
> and receiving packets.
> The No.2 patch changes the return value of firmware processing
> timeout from -EBADE to -ETIME.
> The No.3 patch adds commands between driver and firmware.
> The No.4 patch restores bus_master_en and msix_enable during
> PF FLR reset.
> 
> Chunsong Feng (1):
>   net/hns3: restores bus_master_en and msix_enable during PF FLR reset
> 
> Hao Chen (1):
>   net/hns3: modify the statistics for sending and receiving messages
> 
> Hongbo Zheng (1):
>   net/hns3: change the return value of firmware processing timeout from
>     -EBADE to -ETIME
> 
> humin (1):
>   net/hns3: Renew command and desc structure
> 

Hi Xavier,

'check-git-log.sh' is giving some errors, can you please check them?

$ /devtools/check-git-log.sh -4
Wrong headline format:
        net/hns3: restores bus_master_en and msix_enable during PF FLR reset
Wrong headline uppercase:
        net/hns3: Renew command and desc structure
Headline too long:
        net/hns3: modify the statistics for sending and receiving messages
        net/hns3: change the return value of firmware processing timeout from
-EBADE to -ETIME
        net/hns3: restores bus_master_en and msix_enable during PF FLR reset
  
Wei Hu (Xavier) Oct. 18, 2019, 8:24 a.m. UTC | #3
Hi, Ferruh Yigit


On 2019/10/15 16:46, Ferruh Yigit wrote:
> On 10/9/2019 3:16 PM, Wei Hu (Xavier) wrote:
>> This series adds some updates for hns3 ethernet pmd driver.
>>
>> The No.1 patch modifies the statistics information for sending
>> and receiving packets.
>> The No.2 patch changes the return value of firmware processing
>> timeout from -EBADE to -ETIME.
>> The No.3 patch adds commands between driver and firmware.
>> The No.4 patch restores bus_master_en and msix_enable during
>> PF FLR reset.
>>
>> Chunsong Feng (1):
>>   net/hns3: restores bus_master_en and msix_enable during PF FLR reset
>>
>> Hao Chen (1):
>>   net/hns3: modify the statistics for sending and receiving messages
>>
>> Hongbo Zheng (1):
>>   net/hns3: change the return value of firmware processing timeout from
>>     -EBADE to -ETIME
>>
>> humin (1):
>>   net/hns3: Renew command and desc structure
>>
> Hi Xavier,
>
> 'check-git-log.sh' is giving some errors, can you please check them?
>
> $ /devtools/check-git-log.sh -4
> Wrong headline format:
>         net/hns3: restores bus_master_en and msix_enable during PF FLR reset
> Wrong headline uppercase:
>         net/hns3: Renew command and desc structure
> Headline too long:
>         net/hns3: modify the statistics for sending and receiving messages
>         net/hns3: change the return value of firmware processing timeout from
> -EBADE to -ETIME
>         net/hns3: restores bus_master_en and msix_enable during PF FLR reset
>
  We will fix it and send Patch V2.
  Thanks for your comments.

  Regards
Xavier