Commit 3116ad06 authored by Vladimir Oltean's avatar Vladimir Oltean Committed by David S. Miller

net: bridge: vlan: don't notify to switchdev master VLANs without BRENTRY flag

When a VLAN is added to a bridge port and it doesn't exist on the bridge
device yet, it gets created for the multicast context, but it is
'hidden', since it doesn't have the BRENTRY flag yet:

ip link add br0 type bridge && ip link set swp0 master br0
bridge vlan add dev swp0 vid 100 # the master VLAN 100 gets created
bridge vlan add dev br0 vid 100 self # that VLAN becomes brentry just now

All switchdev drivers ignore switchdev notifiers for VLAN entries which
have the BRENTRY unset, and for good reason: these are merely private
data structures used by the bridge driver. So we might just as well not
notify those at all.

Cleanup in the switchdev drivers that check for the BRENTRY flag is now
possible, and will be handled separately, since those checks just became
dead code.
Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
Acked-by: default avatarNikolay Aleksandrov <nikolay@nvidia.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent b2bc58d4
...@@ -284,9 +284,12 @@ static int __vlan_add(struct net_bridge_vlan *v, u16 flags, ...@@ -284,9 +284,12 @@ static int __vlan_add(struct net_bridge_vlan *v, u16 flags,
} }
br_multicast_port_ctx_init(p, v, &v->port_mcast_ctx); br_multicast_port_ctx_init(p, v, &v->port_mcast_ctx);
} else { } else {
err = br_switchdev_port_vlan_add(dev, v->vid, flags, extack); if (br_vlan_should_use(v)) {
if (err && err != -EOPNOTSUPP) err = br_switchdev_port_vlan_add(dev, v->vid, flags,
goto out; extack);
if (err && err != -EOPNOTSUPP)
goto out;
}
br_multicast_ctx_init(br, v, &v->br_mcast_ctx); br_multicast_ctx_init(br, v, &v->br_mcast_ctx);
v->priv_flags |= BR_VLFLAG_GLOBAL_MCAST_ENABLED; v->priv_flags |= BR_VLFLAG_GLOBAL_MCAST_ENABLED;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment