mbox series

[net,00/19] net: hns: Misc. HNS Bug Fixes & Code Improvements

Message ID 20170330153106.14344-1-salil.mehta@huawei.com
Headers show
Series net: hns: Misc. HNS Bug Fixes & Code Improvements | expand

Message

Salil Mehta March 30, 2017, 3:30 p.m. UTC
This patch set introduces various HNS bug fixes, optimizations and code
improvements.

Daode Huang (1):
  net: hns: bug fix of ethtool show the speed

Kejian Yan (7):
  net: hns: Remove the redundant adding and deleting mac function
  net: hns: Remove redundant mac_get_id()
  net: hns: Remove redundant mac table operations
  net: hns: Clean redundant code from hns_mdio.c file
  net: hns: Optimise the code in hns_mdio_wait_ready()
  net: hns: Simplify the exception sequence in hns_ppe_init()
  net: hns: Adjust the SBM module buffer threshold

Salil Mehta (1):
  net: hns: Some checkpatch.pl script & warning fixes

lipeng (10):
  net: hns: Fix the implementation of irq affinity function
  net: hns: Modify GMAC init TX threshold value
  net: hns: Optimize the code for GMAC pad and crc Config
  net: hns: Change the TX queue selection algorithm
  net: hns: Remove redundant memset during buffer release
  net: hns: Optimize hns_nic_common_poll for better performance
  net: hns: Fix to adjust buf_size of ring according to mtu
  net: hns: Replace netif_tx_lock to ring spin lock
  net: hns: Correct HNS RSS key set function
  net: hns: Avoid Hip06 chip TX packet line bug

 drivers/net/ethernet/hisilicon/hns/hnae.c          |   7 +-
 drivers/net/ethernet/hisilicon/hns/hnae.h          |  49 ++-
 drivers/net/ethernet/hisilicon/hns/hns_ae_adapt.c  | 132 ++++--
 drivers/net/ethernet/hisilicon/hns/hns_dsaf_gmac.c |  61 ++-
 drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c  |  41 +-
 drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.h  |   5 +-
 drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 249 +----------
 drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.h |  14 +-
 drivers/net/ethernet/hisilicon/hns/hns_dsaf_ppe.c  |  17 +-
 drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.c  | 143 +++++--
 drivers/net/ethernet/hisilicon/hns/hns_dsaf_rcb.h  |  26 +-
 drivers/net/ethernet/hisilicon/hns/hns_dsaf_reg.h  |   6 +-
 .../net/ethernet/hisilicon/hns/hns_dsaf_xgmac.c    |  13 -
 drivers/net/ethernet/hisilicon/hns/hns_enet.c      | 463 +++++++++++++++++----
 drivers/net/ethernet/hisilicon/hns/hns_enet.h      |   3 +-
 drivers/net/ethernet/hisilicon/hns/hns_ethtool.c   |  34 +-
 drivers/net/ethernet/hisilicon/hns_mdio.c          |  20 +-
 17 files changed, 723 insertions(+), 560 deletions(-)

-- 
2.7.4

Comments

David Miller March 30, 2017, 5:08 p.m. UTC | #1
From: Salil Mehta <salil.mehta@huawei.com>

Date: Thu, 30 Mar 2017 16:30:47 +0100

> This patch set introduces various HNS bug fixes, optimizations and code

> improvements.


What tree are you targetting?

You say "net" in your Subject lines, but this series has cleanups and all sorts
of other things which are absolutely not appropriate for 'net' and are 'net-next'
material.
Salil Mehta March 30, 2017, 5:19 p.m. UTC | #2
> -----Original Message-----

> From: David Miller [mailto:davem@davemloft.net]

> Sent: Thursday, March 30, 2017 6:09 PM

> To: Salil Mehta

> Cc: Zhuangyuzeng (Yisen); mehta.salil.lnk@gmail.com;

> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Linuxarm

> Subject: Re: [PATCH net 00/19] net: hns: Misc. HNS Bug Fixes & Code

> Improvements

> 

> From: Salil Mehta <salil.mehta@huawei.com>

> Date: Thu, 30 Mar 2017 16:30:47 +0100

> 

> > This patch set introduces various HNS bug fixes, optimizations and code

> > improvements.

> 

> What tree are you targetting?

> 

> You say "net" in your Subject lines, but this series has cleanups and all

> sorts

> of other things which are absolutely not appropriate for 'net' and are

> 'net-next'

> material.

Hi David,
Sorry David, These bug fixes are for the next merge window and for net-next.
Should I resend the patches with below change?

>>>>[PATCH net-next 00/19] net: hns: Misc. HNS Bug Fixes & Code Improvements  


Best regards
Salil
David Miller March 30, 2017, 5:22 p.m. UTC | #3
From: Salil Mehta <salil.mehta@huawei.com>

Date: Thu, 30 Mar 2017 17:19:44 +0000

> 

>> -----Original Message-----

>> From: David Miller [mailto:davem@davemloft.net]

>> Sent: Thursday, March 30, 2017 6:09 PM

>> To: Salil Mehta

>> Cc: Zhuangyuzeng (Yisen); mehta.salil.lnk@gmail.com;

>> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Linuxarm

>> Subject: Re: [PATCH net 00/19] net: hns: Misc. HNS Bug Fixes & Code

>> Improvements

>> 

>> From: Salil Mehta <salil.mehta@huawei.com>

>> Date: Thu, 30 Mar 2017 16:30:47 +0100

>> 

>> > This patch set introduces various HNS bug fixes, optimizations and code

>> > improvements.

>> 

>> What tree are you targetting?

>> 

>> You say "net" in your Subject lines, but this series has cleanups and all

>> sorts

>> of other things which are absolutely not appropriate for 'net' and are

>> 'net-next'

>> material.

> Hi David,

> Sorry David, These bug fixes are for the next merge window and for net-next.

> Should I resend the patches with below change?


It is not necessary to resend, thanks for clarifying.
Salil Mehta March 30, 2017, 5:23 p.m. UTC | #4
> -----Original Message-----

> From: David Miller [mailto:davem@davemloft.net]

> Sent: Thursday, March 30, 2017 6:22 PM

> To: Salil Mehta

> Cc: Zhuangyuzeng (Yisen); mehta.salil.lnk@gmail.com;

> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Linuxarm

> Subject: Re: [PATCH net 00/19] net: hns: Misc. HNS Bug Fixes & Code

> Improvements

> 

> From: Salil Mehta <salil.mehta@huawei.com>

> Date: Thu, 30 Mar 2017 17:19:44 +0000

> 

> >

> >> -----Original Message-----

> >> From: David Miller [mailto:davem@davemloft.net]

> >> Sent: Thursday, March 30, 2017 6:09 PM

> >> To: Salil Mehta

> >> Cc: Zhuangyuzeng (Yisen); mehta.salil.lnk@gmail.com;

> >> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Linuxarm

> >> Subject: Re: [PATCH net 00/19] net: hns: Misc. HNS Bug Fixes & Code

> >> Improvements

> >>

> >> From: Salil Mehta <salil.mehta@huawei.com>

> >> Date: Thu, 30 Mar 2017 16:30:47 +0100

> >>

> >> > This patch set introduces various HNS bug fixes, optimizations and

> code

> >> > improvements.

> >>

> >> What tree are you targetting?

> >>

> >> You say "net" in your Subject lines, but this series has cleanups

> and all

> >> sorts

> >> of other things which are absolutely not appropriate for 'net' and

> are

> >> 'net-next'

> >> material.

> > Hi David,

> > Sorry David, These bug fixes are for the next merge window and for

> net-next.

> > Should I resend the patches with below change?

> 

> It is not necessary to resend, thanks for clarifying.

Sure thanks.
David Miller March 31, 2017, 3:03 a.m. UTC | #5
From: Salil Mehta <salil.mehta@huawei.com>

Date: Thu, 30 Mar 2017 16:30:47 +0100

> This patch set introduces various HNS bug fixes, optimizations and code

> improvements.


There is no way you should do such an expensive calculation for every
single transmit packet as you are doing in your select_queue() routine.

That's really crazy.

Just use the networking stack's queue selection scheme, or suggest
ways to improve it.  Don't do private hashing like this in your
driver, please!
Salil Mehta March 31, 2017, 9:58 a.m. UTC | #6
> -----Original Message-----

> From: David Miller [mailto:davem@davemloft.net]

> Sent: Friday, March 31, 2017 4:03 AM

> To: Salil Mehta

> Cc: Zhuangyuzeng (Yisen); mehta.salil.lnk@gmail.com;

> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Linuxarm

> Subject: Re: [PATCH net 00/19] net: hns: Misc. HNS Bug Fixes & Code

> Improvements

> 

> From: Salil Mehta <salil.mehta@huawei.com>

> Date: Thu, 30 Mar 2017 16:30:47 +0100

> 

> > This patch set introduces various HNS bug fixes, optimizations and

> code

> > improvements.

> 

> There is no way you should do such an expensive calculation for every

> single transmit packet as you are doing in your select_queue() routine.

> 

> That's really crazy.

> 

> Just use the networking stack's queue selection scheme, or suggest

> ways to improve it.  Don't do private hashing like this in your

> driver, please!

Hi David,
I got your point. I will drop this patch from this patch-set for now.

Thanks
Salil