]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
net: dsa: rollback bridging on error
authorVivien Didelot <vivien.didelot@savoirfairelinux.com>
Fri, 3 Feb 2017 18:20:18 +0000 (13:20 -0500)
committerDavid S. Miller <davem@davemloft.net>
Mon, 6 Feb 2017 21:53:28 +0000 (16:53 -0500)
When an error is returned during the bridging of a port in a
NETDEV_CHANGEUPPER event, net/core/dev.c rolls back the operation.

Be consistent and unassign dp->bridge_dev when this happens.

In the meantime, add comments to document this behavior.

Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/dsa/slave.c

index 332eb234dc21c9bc5f497be6cd9f1b4f466fc52a..d726307c77959bda6f55902ff6fbab48896589d3 100644 (file)
@@ -562,12 +562,21 @@ static int dsa_slave_bridge_port_join(struct net_device *dev,
        struct dsa_switch *ds = p->dp->ds;
        int ret = -EOPNOTSUPP;
 
+       /* Here the port is already bridged. Reflect the current configuration
+        * so that drivers can program their chips accordingly.
+        */
        p->dp->bridge_dev = br;
 
        if (ds->ops->port_bridge_join)
                ret = ds->ops->port_bridge_join(ds, p->dp->index, br);
 
-       return ret == -EOPNOTSUPP ? 0 : ret;
+       /* The bridging is rolled back on error */
+       if (ret && ret != -EOPNOTSUPP) {
+               p->dp->bridge_dev = NULL;
+               return ret;
+       }
+
+       return 0;
 }
 
 static void dsa_slave_bridge_port_leave(struct net_device *dev,
@@ -576,6 +585,9 @@ static void dsa_slave_bridge_port_leave(struct net_device *dev,
        struct dsa_slave_priv *p = netdev_priv(dev);
        struct dsa_switch *ds = p->dp->ds;
 
+       /* Here the port is already unbridged. Reflect the current configuration
+        * so that drivers can program their chips accordingly.
+        */
        p->dp->bridge_dev = NULL;
 
        if (ds->ops->port_bridge_leave)