[PATCH net-next 4/6] net: dsa: check bridge VLAN in slave operations

[Vivien Didelot]

The bridge VLANs are not offloaded by dsa_port_vlan_* if the port is
not bridged or if its bridge is not VLAN aware.

This is a good thing but other corners of DSA, such as the tag_8021q
driver, may need to program VLANs regardless the bridge state.

And also because bridge_dev is specific to user ports anyway, move
these checks were it belongs, one layer up in the slave code.

Signed-off-by: Vivien Didelot <vivien.dide...@gmail.com>
Suggested-by: Vladimir Oltean <olte...@gmail.com>
---
 net/dsa/port.c  | 10 ++--------
 net/dsa/slave.c | 12 ++++++++++++
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/net/dsa/port.c b/net/dsa/port.c
index ef28df7ecbde..9b54e5a76297 100644
--- a/net/dsa/port.c
+++ b/net/dsa/port.c
@@ -348,10 +348,7 @@ int dsa_port_vlan_add(struct dsa_port *dp,
                .vlan = vlan,
        };
 
-       if (!dp->bridge_dev || br_vlan_enabled(dp->bridge_dev))
-               return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_ADD, &info);
-
-       return 0;
+       return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_ADD, &info);
 }
 
 int dsa_port_vlan_del(struct dsa_port *dp,
@@ -363,10 +360,7 @@ int dsa_port_vlan_del(struct dsa_port *dp,
                .vlan = vlan,
        };
 
-       if (!dp->bridge_dev || br_vlan_enabled(dp->bridge_dev))
-               return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_DEL, &info);
-
-       return 0;
+       return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_DEL, &info);
 }
 
 int dsa_port_vid_add(struct dsa_port *dp, u16 vid, u16 flags)
diff --git a/net/dsa/slave.c b/net/dsa/slave.c
index 8f5126c41282..82e48d247b81 100644
--- a/net/dsa/slave.c
+++ b/net/dsa/slave.c
@@ -323,6 +323,9 @@ static int dsa_slave_vlan_add(struct net_device *dev,
        if (obj->orig_dev != dev)
                return -EOPNOTSUPP;
 
+       if (dp->bridge_dev && !br_vlan_enabled(dp->bridge_dev))
+               return 0;
+
        vlan = *SWITCHDEV_OBJ_PORT_VLAN(obj);
 
        err = dsa_port_vlan_add(dp, &vlan, trans);
@@ -377,6 +380,9 @@ static int dsa_slave_vlan_del(struct net_device *dev,
        if (obj->orig_dev != dev)
                return -EOPNOTSUPP;
 
+       if (dp->bridge_dev && !br_vlan_enabled(dp->bridge_dev))
+               return 0;
+
        return dsa_port_vlan_del(dp, SWITCHDEV_OBJ_PORT_VLAN(obj));
 }
 
@@ -1099,6 +1105,9 @@ static int dsa_slave_vlan_rx_add_vid(struct net_device 
*dev, __be16 proto,
         * need to emulate the switchdev prepare + commit phase.
         */
        if (dp->bridge_dev) {
+               if (!br_vlan_enabled(dp->bridge_dev))
+                       return 0;
+
                /* br_vlan_get_info() returns -EINVAL or -ENOENT if the
                 * device, respectively the VID is not found, returning
                 * 0 means success, which is a failure for us here.
@@ -1126,6 +1135,9 @@ static int dsa_slave_vlan_rx_kill_vid(struct net_device 
*dev, __be16 proto,
         * need to emulate the switchdev prepare + commit phase.
         */
        if (dp->bridge_dev) {
+               if (!br_vlan_enabled(dp->bridge_dev))
+                       return 0;
+
                /* br_vlan_get_info() returns -EINVAL or -ENOENT if the
                 * device, respectively the VID is not found, returning
                 * 0 means success, which is a failure for us here.
-- 
2.23.0