Message ID | 20170330153106.14344-1-salil.mehta@huawei.com |
---|---|
Headers | show |
Series | net: hns: Misc. HNS Bug Fixes & Code Improvements | expand |
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.
> -----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
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.
> -----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.
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!
> -----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