net: mscc: ocelot: allow offloading of bridge on top of LAG
authorVladimir Oltean <vladimir.oltean@nxp.com>
Mon, 18 Jan 2021 13:52:10 +0000 (15:52 +0200)
committerJakub Kicinski <kuba@kernel.org>
Mon, 18 Jan 2021 19:41:35 +0000 (11:41 -0800)
commit79267ae22615496655feee2db0848f6786bcf67a
treee9aa03bc4b66fed4e7b4a21f455bbfa38b3750d4
parent66c556025d687dbdd0f748c5e1df89c977b6c02a
net: mscc: ocelot: allow offloading of bridge on top of LAG

The blamed commit was too aggressive, and it made ocelot_netdevice_event
react only to network interface events emitted for the ocelot switch
ports.

In fact, only the PRECHANGEUPPER should have had that check.

When we ignore all events that are not for us, we miss the fact that the
upper of the LAG changes, and the bonding interface gets enslaved to a
bridge. This is an operation we could offload under certain conditions.

Fixes: 7afb3e575e5a ("net: mscc: ocelot: don't handle netdev events for other netdevs")
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Link: https://lore.kernel.org/r/20210118135210.2666246-1-olteanv@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/mscc/ocelot_net.c