net/mlx5e: Replace TC VLAN pop and push actions with VLAN modify

Changing the VLAN header may be implemented by pop the existing header
and push a new one. Translate those operations as VLAN modify.
Applicable for use cases such as OVS where the controller translates a
vlan modify meta (OF) rule to DP pop+push actions rule.

Signed-off-by: Eli Britstein <elibr@mellanox.com>
Reviewed-by: Roi Dayan <roid@mellanox.com>
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
index 0f4e9615f666fb1b325c600a3d35c9620999baa1..c68edcc84af8c5538498e3bb01ac81a50366311f 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
@@ -2664,7 +2664,17 @@ static int parse_tc_fdb_actions(struct mlx5e_priv *priv,
                        break;
                case FLOW_ACTION_VLAN_PUSH:
                case FLOW_ACTION_VLAN_POP:
-                       err = parse_tc_vlan_action(priv, act, attr, &action);
+                       if (act->id == FLOW_ACTION_VLAN_PUSH &&
+                           (action & MLX5_FLOW_CONTEXT_ACTION_VLAN_POP)) {
+                               /* Replace vlan pop+push with vlan modify */
+                               action &= ~MLX5_FLOW_CONTEXT_ACTION_VLAN_POP;
+                               err = add_vlan_rewrite_action(priv,
+                                                             MLX5_FLOW_NAMESPACE_FDB,
+                                                             act, parse_attr, hdrs,
+                                                             &action, extack);
+                       } else {
+                               err = parse_tc_vlan_action(priv, act, attr, &action);
+                       }
                        if (err)
                                return err;