Message ID | 20190430131429.19361-1-maxime.chevallier@bootlin.com |
---|---|
Headers | show |
Series | net: mvpp2: cls: Add classification | expand |
From: Maxime Chevallier <maxime.chevallier@bootlin.com> Date: Tue, 30 Apr 2019 15:14:25 +0200 > This series is a rework of the previously standalone patch adding > classification support for mvpp2 : ... Series applied, thanks.
On Tue, 30 Apr 2019 15:14:25 +0200, Maxime Chevallier wrote: > Compared to the first submissions, the NETIF_F_NTUPLE flag was also > removed, following Saeed's comment. You should probably add it back, even though the stack only uses NETIF_F_NTUPLE for aRFS the ethtool APIs historically depend on the drivers doing a lot of the validation. The flag was added by: 15682bc488d4 ("ethtool: Introduce n-tuple filter programming support") your initial use of the flag was correct.
Hello Jakub, On Sat, 4 May 2019 02:53:53 -0400 Jakub Kicinski <jakub.kicinski@netronome.com> wrote: >On Tue, 30 Apr 2019 15:14:25 +0200, Maxime Chevallier wrote: >> Compared to the first submissions, the NETIF_F_NTUPLE flag was also >> removed, following Saeed's comment. > >You should probably add it back, even though the stack only uses >NETIF_F_NTUPLE for aRFS the ethtool APIs historically depend on the >drivers doing a lot of the validation. OK my bad, reading your previous comments again, I should indeed have left it. I'll re-add the flag, do you think this should go through -net or wait until net-next reopens ? Thanks, Maxime
On Mon, 6 May 2019 10:00:26 +0200, Maxime Chevallier wrote: > Hello Jakub, > > On Sat, 4 May 2019 02:53:53 -0400 > Jakub Kicinski <jakub.kicinski@netronome.com> wrote: > > >On Tue, 30 Apr 2019 15:14:25 +0200, Maxime Chevallier wrote: > >> Compared to the first submissions, the NETIF_F_NTUPLE flag was also > >> removed, following Saeed's comment. > > > >You should probably add it back, even though the stack only uses > >NETIF_F_NTUPLE for aRFS the ethtool APIs historically depend on the > >drivers doing a lot of the validation. > > OK my bad, reading your previous comments again, I should indeed have > left it. > > I'll re-add the flag, do you think this should go through -net or wait > until net-next reopens ? I think the patch should be relatively simple and clean? So I'd try for net, with a Fixes tag, it's a slight ABI correction and we are still in the merge window period. So I'd go for net :)