mlxsw: spectrum: Allow different traps to have different actions
authorIdo Schimmel <idosch@mellanox.com>
Thu, 25 Aug 2016 16:42:39 +0000 (18:42 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 26 Aug 2016 20:13:36 +0000 (13:13 -0700)
Up until now we only trapped packets to CPU, but we are going to allow
some packets to be mirrored (trap & forward) to CPU.

Extend the Rx listener with 'action' member.

Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlxsw/core.h
drivers/net/ethernet/mellanox/mlxsw/spectrum.c

index d3476ead998285b68c95eb4ad3ac2a38f0b703d2..d2e32979319c93ce8be7a9591c92212b4d69ed9a 100644 (file)
@@ -87,6 +87,7 @@ struct mlxsw_rx_listener {
        void (*func)(struct sk_buff *skb, u8 local_port, void *priv);
        u8 local_port;
        u16 trap_id;
+       enum mlxsw_reg_hpkt_action action;
 };
 
 struct mlxsw_event_listener {
index cc83cd221c562d38d77173e9a68131b743928911..ea4086ac3e9a2190bd1b46dda160a8f3e074bd42 100644 (file)
@@ -2570,39 +2570,40 @@ static void mlxsw_sp_rx_listener_func(struct sk_buff *skb, u8 local_port,
        netif_receive_skb(skb);
 }
 
-#define MLXSW_SP_RXL(_func, _trap_id)                  \
-       {                                               \
-               .func = _func,                          \
-               .local_port = MLXSW_PORT_DONT_CARE,     \
-               .trap_id = MLXSW_TRAP_ID_##_trap_id,    \
+#define MLXSW_SP_RXL(_func, _trap_id, _action)                 \
+       {                                                       \
+               .func = _func,                                  \
+               .local_port = MLXSW_PORT_DONT_CARE,             \
+               .trap_id = MLXSW_TRAP_ID_##_trap_id,            \
+               .action = MLXSW_REG_HPKT_ACTION_##_action,      \
        }
 
 static const struct mlxsw_rx_listener mlxsw_sp_rx_listener[] = {
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, FDB_MC),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, FDB_MC, TRAP_TO_CPU),
        /* Traps for specific L2 packet types, not trapped as FDB MC */
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, STP),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, LACP),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, EAPOL),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, LLDP),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, MMRP),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, MVRP),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, RPVST),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, DHCP),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IGMP_QUERY),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IGMP_V1_REPORT),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IGMP_V2_REPORT),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IGMP_V2_LEAVE),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IGMP_V3_REPORT),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, ARPBC),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, ARPUC),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, STP, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, LACP, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, EAPOL, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, LLDP, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, MMRP, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, MVRP, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, RPVST, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, DHCP, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IGMP_QUERY, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IGMP_V1_REPORT, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IGMP_V2_REPORT, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IGMP_V2_LEAVE, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IGMP_V3_REPORT, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, ARPBC, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, ARPUC, TRAP_TO_CPU),
        /* L3 traps */
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, MTUERROR),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, TTLERROR),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, LBERROR),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, OSPF),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IP2ME),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, RTR_INGRESS0),
-       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, HOST_MISS_IPV4),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, MTUERROR, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, TTLERROR, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, LBERROR, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, OSPF, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, IP2ME, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, RTR_INGRESS0, TRAP_TO_CPU),
+       MLXSW_SP_RXL(mlxsw_sp_rx_listener_func, HOST_MISS_IPV4, TRAP_TO_CPU),
 };
 
 static int mlxsw_sp_traps_init(struct mlxsw_sp *mlxsw_sp)
@@ -2629,7 +2630,7 @@ static int mlxsw_sp_traps_init(struct mlxsw_sp *mlxsw_sp)
                if (err)
                        goto err_rx_listener_register;
 
-               mlxsw_reg_hpkt_pack(hpkt_pl, MLXSW_REG_HPKT_ACTION_TRAP_TO_CPU,
+               mlxsw_reg_hpkt_pack(hpkt_pl, mlxsw_sp_rx_listener[i].action,
                                    mlxsw_sp_rx_listener[i].trap_id);
                err = mlxsw_reg_write(mlxsw_sp->core, MLXSW_REG(hpkt), hpkt_pl);
                if (err)