mbox series

[0/7] bugfix for hns3 PMD

Message ID 20220406092240.52900-1-humin29@huawei.com (mailing list archive)
Headers
Series bugfix for hns3 PMD |

Message

humin (Q) April 6, 2022, 9:22 a.m. UTC 
  This patch set contains 7 bugfix for hns3 PMD.

Chengwen Feng (1):
  net/hns3: fix more mbufs are freed when Tx done cleanup

Huisong Li (6):
  net/hns3: fix order of clearing imissed register in PF
  net/hns3: fix MAC and queues HW statistics reversion
  net/hns3: fix pseudo-sharing between threads
  net/hns3: fix RSS disable
  net/hns3: fix undo rollback when update RSS hash
  net/hns3: remove redundant RSS tuple field

 drivers/net/hns3/hns3_ethdev.c    |  20 ++--
 drivers/net/hns3/hns3_ethdev.h    |   6 ++
 drivers/net/hns3/hns3_ethdev_vf.c |  20 ++--
 drivers/net/hns3/hns3_flow.c      |  10 +-
 drivers/net/hns3/hns3_rss.c       | 124 ++++++-----------------
 drivers/net/hns3/hns3_rss.h       |   5 +-
 drivers/net/hns3/hns3_rxtx.c      |   8 +-
 drivers/net/hns3/hns3_rxtx.h      |   4 +-
 drivers/net/hns3/hns3_stats.c     | 163 ++++++++++++++++++++++--------
 drivers/net/hns3/hns3_stats.h     |   6 +-
 10 files changed, 183 insertions(+), 183 deletions(-)

Comments

humin (Q) April 25, 2022, 7:03 a.m. UTC | #1 
Hi, Ferruh,
	what do you think of this set of patches?


在 2022/4/6 17:22, Min Hu (Connor) 写道:
> This patch set contains 7 bugfix for hns3 PMD.
> 
> Chengwen Feng (1):
>    net/hns3: fix more mbufs are freed when Tx done cleanup
> 
> Huisong Li (6):
>    net/hns3: fix order of clearing imissed register in PF
>    net/hns3: fix MAC and queues HW statistics reversion
>    net/hns3: fix pseudo-sharing between threads
>    net/hns3: fix RSS disable
>    net/hns3: fix undo rollback when update RSS hash
>    net/hns3: remove redundant RSS tuple field
> 
>   drivers/net/hns3/hns3_ethdev.c    |  20 ++--
>   drivers/net/hns3/hns3_ethdev.h    |   6 ++
>   drivers/net/hns3/hns3_ethdev_vf.c |  20 ++--
>   drivers/net/hns3/hns3_flow.c      |  10 +-
>   drivers/net/hns3/hns3_rss.c       | 124 ++++++-----------------
>   drivers/net/hns3/hns3_rss.h       |   5 +-
>   drivers/net/hns3/hns3_rxtx.c      |   8 +-
>   drivers/net/hns3/hns3_rxtx.h      |   4 +-
>   drivers/net/hns3/hns3_stats.c     | 163 ++++++++++++++++++++++--------
>   drivers/net/hns3/hns3_stats.h     |   6 +-
>   10 files changed, 183 insertions(+), 183 deletions(-)
>
Ferruh Yigit May 3, 2022, 7:07 p.m. UTC | #2 
On 4/6/2022 10:22 AM, Min Hu (Connor) wrote:
> This patch set contains 7 bugfix for hns3 PMD.
> 
> Chengwen Feng (1):
>    net/hns3: fix more mbufs are freed when Tx done cleanup
> 
> Huisong Li (6):
>    net/hns3: fix order of clearing imissed register in PF
>    net/hns3: fix MAC and queues HW statistics reversion
>    net/hns3: fix pseudo-sharing between threads
>    net/hns3: fix RSS disable
>    net/hns3: fix undo rollback when update RSS hash
>    net/hns3: remove redundant RSS tuple field
> 

Hi Connor,

I need to resolve some conflicts while merging this set, I assume it is 
based on another patch, can you please document it in cover letter next 
time.

Also I can see CI had same issue, to enable CI to check the patch, each 
set should be send on top of latest next-net, so the CI can apply them.
There is a defined 'Depends-on:' keyword [1] for commit log, but as far 
as I know CI not implemented it yet.

[1]
https://doc.dpdk.org/guides/contributing/patches.html#patch-dependencies
humin (Q) May 5, 2022, 12:29 p.m. UTC | #3 
Hi, Ferruh,
  v2 has been sent, thanks.

在 2022/4/6 17:22, Min Hu (Connor) 写道:
> This patch set contains 7 bugfix for hns3 PMD.
> 
> Chengwen Feng (1):
>    net/hns3: fix more mbufs are freed when Tx done cleanup
> 
> Huisong Li (6):
>    net/hns3: fix order of clearing imissed register in PF
>    net/hns3: fix MAC and queues HW statistics reversion
>    net/hns3: fix pseudo-sharing between threads
>    net/hns3: fix RSS disable
>    net/hns3: fix undo rollback when update RSS hash
>    net/hns3: remove redundant RSS tuple field
> 
>   drivers/net/hns3/hns3_ethdev.c    |  20 ++--
>   drivers/net/hns3/hns3_ethdev.h    |   6 ++
>   drivers/net/hns3/hns3_ethdev_vf.c |  20 ++--
>   drivers/net/hns3/hns3_flow.c      |  10 +-
>   drivers/net/hns3/hns3_rss.c       | 124 ++++++-----------------
>   drivers/net/hns3/hns3_rss.h       |   5 +-
>   drivers/net/hns3/hns3_rxtx.c      |   8 +-
>   drivers/net/hns3/hns3_rxtx.h      |   4 +-
>   drivers/net/hns3/hns3_stats.c     | 163 ++++++++++++++++++++++--------
>   drivers/net/hns3/hns3_stats.h     |   6 +-
>   10 files changed, 183 insertions(+), 183 deletions(-)
>