net: dsa: support cross-chip FDB operations
authorVivien Didelot <vivien.didelot@savoirfairelinux.com>
Thu, 30 Nov 2017 17:56:43 +0000 (12:56 -0500)
committerDavid S. Miller <davem@davemloft.net>
Sun, 3 Dec 2017 02:21:17 +0000 (21:21 -0500)
When a MAC address is added to or removed from a switch port in the
fabric, the target switch must program its port and adjacent switches
must program their local DSA port used to reach the target switch.

For this purpose, use the dsa_towards_port() helper to identify the
local switch port which must be programmed.

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

index 9a01514ea9f3312c41ba94d08da63943c8a2ceae..b93511726069e9d7059c8f8dc23b37cb0236744a 100644 (file)
@@ -83,29 +83,23 @@ static int dsa_switch_bridge_leave(struct dsa_switch *ds,
 static int dsa_switch_fdb_add(struct dsa_switch *ds,
                              struct dsa_notifier_fdb_info *info)
 {
-       /* Do not care yet about other switch chips of the fabric */
-       if (ds->index != info->sw_index)
-               return 0;
+       int port = dsa_towards_port(ds, info->sw_index, info->port);
 
        if (!ds->ops->port_fdb_add)
                return -EOPNOTSUPP;
 
-       return ds->ops->port_fdb_add(ds, info->port, info->addr,
-                                    info->vid);
+       return ds->ops->port_fdb_add(ds, port, info->addr, info->vid);
 }
 
 static int dsa_switch_fdb_del(struct dsa_switch *ds,
                              struct dsa_notifier_fdb_info *info)
 {
-       /* Do not care yet about other switch chips of the fabric */
-       if (ds->index != info->sw_index)
-               return 0;
+       int port = dsa_towards_port(ds, info->sw_index, info->port);
 
        if (!ds->ops->port_fdb_del)
                return -EOPNOTSUPP;
 
-       return ds->ops->port_fdb_del(ds, info->port, info->addr,
-                                    info->vid);
+       return ds->ops->port_fdb_del(ds, port, info->addr, info->vid);
 }
 
 static int