mlxsw: spectrum: Add an option to flood mc by mc_router_port
authorNogah Frankel <nogahf@mellanox.com>
Thu, 9 Feb 2017 13:54:47 +0000 (14:54 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 10 Feb 2017 16:46:40 +0000 (11:46 -0500)
The decision whether to flood a multicast packet to a port dependent
on three flags: mc_disabled, mc_router_port, mc_flood.

If mc_disabled is on, the port will be flooded according to mc_flood,
otherwise, according to mc_router_port. To accomplish that, add those
flags into the mlxsw_sp_port struct and update the mc flood table
accordingly.

Update mc_router_port by switchdev attribute
SWITCHDEV_ATTR_ID_PORT_MC_ROUTER_PORT.

Signed-off-by: Nogah Frankel <nogahf@mellanox.com>
Signed-off-by: Yotam Gigi <yotamg@mellanox.com>
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlxsw/spectrum.c
drivers/net/ethernet/mellanox/mlxsw/spectrum.h
drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c

index 1d9a8c5948e7fa41231f3867f3d6e7f6985a4e9e..37c018bea5e2059b34108484d5b8d73eb290b981 100644 (file)
@@ -4073,6 +4073,8 @@ static int mlxsw_sp_port_bridge_join(struct mlxsw_sp_port *mlxsw_sp_port,
        mlxsw_sp_port->learning_sync = 1;
        mlxsw_sp_port->uc_flood = 1;
        mlxsw_sp_port->mc_flood = 1;
+       mlxsw_sp_port->mc_router = 0;
+       mlxsw_sp_port->mc_disabled = 1;
        mlxsw_sp_port->bridged = 1;
 
        return 0;
@@ -4090,6 +4092,7 @@ static void mlxsw_sp_port_bridge_leave(struct mlxsw_sp_port *mlxsw_sp_port)
        mlxsw_sp_port->learning_sync = 0;
        mlxsw_sp_port->uc_flood = 0;
        mlxsw_sp_port->mc_flood = 0;
+       mlxsw_sp_port->mc_router = 0;
        mlxsw_sp_port->bridged = 0;
 
        /* Add implicit VLAN interface in the device, so that untagged
@@ -4753,6 +4756,8 @@ static int mlxsw_sp_vport_bridge_join(struct mlxsw_sp_port *mlxsw_sp_vport,
        mlxsw_sp_vport->learning_sync = 1;
        mlxsw_sp_vport->uc_flood = 1;
        mlxsw_sp_vport->mc_flood = 1;
+       mlxsw_sp_vport->mc_router = 0;
+       mlxsw_sp_vport->mc_disabled = 1;
        mlxsw_sp_vport->bridged = 1;
 
        return 0;
@@ -4774,6 +4779,7 @@ static void mlxsw_sp_vport_bridge_leave(struct mlxsw_sp_port *mlxsw_sp_vport)
        mlxsw_sp_vport->learning_sync = 0;
        mlxsw_sp_vport->uc_flood = 0;
        mlxsw_sp_vport->mc_flood = 0;
+       mlxsw_sp_vport->mc_router = 0;
        mlxsw_sp_vport->bridged = 0;
 }
 
index 7b8d1cfe5cf45367b993ce6dd6ce96334a26c421..13ec85e7c392f8941ecf6441333d416ba4609f3a 100644 (file)
@@ -339,10 +339,12 @@ struct mlxsw_sp_port {
        struct mlxsw_sp *mlxsw_sp;
        u8 local_port;
        u8 stp_state;
-       u8 learning:1,
+       u16 learning:1,
           learning_sync:1,
           uc_flood:1,
           mc_flood:1,
+          mc_router:1,
+          mc_disabled:1,
           bridged:1,
           lagged:1,
           split:1;
index fe3cf568e1d5cb457891a02da78de0f19b5a47b8..53702464db2eadd20fd09545a411e827a6d223c2 100644 (file)
@@ -264,14 +264,20 @@ err_port_flood_set:
 int mlxsw_sp_vport_flood_set(struct mlxsw_sp_port *mlxsw_sp_vport, u16 fid,
                             bool set)
 {
+       bool mc_set = set;
        u16 vfid;
 
        /* In case of vFIDs, index into the flooding table is relative to
         * the start of the vFIDs range.
         */
        vfid = mlxsw_sp_fid_to_vfid(fid);
+
+       if (set)
+               mc_set = mlxsw_sp_vport->mc_disabled ?
+                        mlxsw_sp_vport->mc_flood : mlxsw_sp_vport->mc_router;
+
        return __mlxsw_sp_port_flood_set(mlxsw_sp_vport, vfid, vfid, set, set,
-                                        set);
+                                        mc_set);
 }
 
 static int mlxsw_sp_port_learning_set(struct mlxsw_sp_port *mlxsw_sp_port,
@@ -393,6 +399,22 @@ static int mlxsw_sp_port_attr_br_vlan_set(struct mlxsw_sp_port *mlxsw_sp_port,
        return 0;
 }
 
+static int mlxsw_sp_port_attr_mc_router_set(struct mlxsw_sp_port *mlxsw_sp_port,
+                                           struct switchdev_trans *trans,
+                                           bool is_port_mc_router)
+{
+       if (switchdev_trans_ph_prepare(trans))
+               return 0;
+
+       mlxsw_sp_port->mc_router = is_port_mc_router;
+       if (!mlxsw_sp_port->mc_disabled)
+               return mlxsw_sp_port_flood_table_set(mlxsw_sp_port,
+                                                    MLXSW_SP_FLOOD_TABLE_MC,
+                                                    is_port_mc_router);
+
+       return 0;
+}
+
 static int mlxsw_sp_port_attr_set(struct net_device *dev,
                                  const struct switchdev_attr *attr,
                                  struct switchdev_trans *trans)
@@ -422,6 +444,10 @@ static int mlxsw_sp_port_attr_set(struct net_device *dev,
                                                     attr->orig_dev,
                                                     attr->u.vlan_filtering);
                break;
+       case SWITCHDEV_ATTR_ID_PORT_MROUTER:
+               err = mlxsw_sp_port_attr_mc_router_set(mlxsw_sp_port, trans,
+                                                      attr->u.mrouter);
+               break;
        default:
                err = -EOPNOTSUPP;
                break;
@@ -567,6 +593,7 @@ static int mlxsw_sp_port_fid_map(struct mlxsw_sp_port *mlxsw_sp_port, u16 fid,
 static int mlxsw_sp_port_fid_join(struct mlxsw_sp_port *mlxsw_sp_port,
                                  u16 fid_begin, u16 fid_end)
 {
+       bool mc_flood;
        int fid, err;
 
        for (fid = fid_begin; fid <= fid_end; fid++) {
@@ -575,9 +602,12 @@ static int mlxsw_sp_port_fid_join(struct mlxsw_sp_port *mlxsw_sp_port,
                        goto err_port_fid_join;
        }
 
+       mc_flood = mlxsw_sp_port->mc_disabled ?
+                       mlxsw_sp_port->mc_flood : mlxsw_sp_port->mc_router;
+
        err = __mlxsw_sp_port_flood_set(mlxsw_sp_port, fid_begin, fid_end,
                                        mlxsw_sp_port->uc_flood, true,
-                                       mlxsw_sp_port->mc_flood);
+                                       mc_flood);
        if (err)
                goto err_port_flood_set;