Thread (22 messages) 22 messages, 2 authors, 2022-02-15

Re: [PATCH v2 net-next 1/8] net: bridge: vlan: notify switchdev only when something changed

flat view

From: Vladimir Oltean <vladimir.oltean@nxp.com>
Date: 2022-02-15 15:52:08

On Tue, Feb 15, 2022 at 05:44:52PM +0200, Nikolay Aleksandrov wrote:
quoted
Would you mind if I added an extra patch that also does this (it would
also remove the need for this check in drivers, plus it would change the
definition of "changed" to something IMO more reasonable, i.e. a master
VLAN that isn't brentry doesn't really exist, so even though the
!BRENTRY->BRENTRY is a flag change, it isn't a change of a switchdev
VLAN object that anybody offloads), or is there some reason I'm missing?
I agree, patch looks good to me. I see that everyone already filters the
!BRENTRY case anyway.
Thanks, this simplifies things quite a bit.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help