net: dsa: sja1105: deny more than one VLAN-aware bridge

With tag_sja1105.c's only ability being to perform an imprecise RX
procedure and identify whether a packet comes from a VLAN-aware bridge
or not, we have no way to determine whether a packet with VLAN ID 5
comes from, say, br0 or br1. Actually we could, but it would mean that
we need to restrict all VLANs from br0 to be different from all VLANs
from br1, and this includes the default_pvid, which makes a setup with 2
VLAN-aware bridges highly imprectical.

The fact of the matter is that this isn't even that big of a practical
limitation, since even with a single VLAN-aware bridge we can pretty
much enforce forwarding isolation based on the VLAN port membership.

So in the end, tell the user that they need to model their setup using a
single VLAN-aware bridge.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/dsa/sja1105/sja1105_main.c b/drivers/net/dsa/sja1105/sja1105_main.c
index a380f37..ef63226 100644 (file)
--- a/drivers/net/dsa/sja1105/sja1105_main.c
+++ b/drivers/net/dsa/sja1105/sja1105_main.c
@@ -2257,12 +2257,25 @@ static int sja1105_prechangeupper(struct dsa_switch *ds, int port,
 {
        struct netlink_ext_ack *extack = info->info.extack;
        struct net_device *upper = info->upper_dev;
+       struct dsa_switch_tree *dst = ds->dst;
+       struct dsa_port *dp;
 
        if (is_vlan_dev(upper)) {
                NL_SET_ERR_MSG_MOD(extack, "8021q uppers are not supported");
                return -EBUSY;
        }
 
+       if (netif_is_bridge_master(upper)) {
+               list_for_each_entry(dp, &dst->ports, list) {
+                       if (dp->bridge_dev && dp->bridge_dev != upper &&
+                           br_vlan_enabled(dp->bridge_dev)) {
+                               NL_SET_ERR_MSG_MOD(extack,
+                                                  "Only one VLAN-aware bridge is supported");
+                               return -EBUSY;
+                       }
+               }
+       }
+
        return 0;
 }