net/mlx5e: Add post_parse() op to tc action infrastructure

The post_parse() op should be called after the parse op was called
for all actions. It could be an action state is dependent on other
actions. In the new op an action can fail the parse if the state
is not valid anymore.

Signed-off-by: Roi Dayan <roid@nvidia.com>
Reviewed-by: Oz Shlomo <ozsh@nvidia.com>
Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en/tc/act/act.h b/drivers/net/ethernet/mellanox/mlx5/core/en/tc/act/act.h
index 0aa995a9f6743753c48f1978eae36e28c444e314..26efa33de56fbfc98df1d41ef9c6d956c1e8f6f8 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx5/core/en/tc/act/act.h
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en/tc/act/act.h
@@ -36,6 +36,10 @@ struct mlx5e_tc_act {
                            const struct flow_action_entry *act,
                            struct mlx5e_priv *priv,
                            struct mlx5_flow_attr *attr);
+
+       int (*post_parse)(struct mlx5e_tc_act_parse_state *parse_state,
+                         struct mlx5e_priv *priv,
+                         struct mlx5_flow_attr *attr);
 };
 
 extern struct mlx5e_tc_act mlx5e_tc_act_drop;

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
index c6c6d20ecd09178b60b9d28c81ade5cf605193eb..2ece349592cd3991f602314a9c400371525fb148 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
@@ -3168,6 +3168,17 @@ parse_tc_actions(struct mlx5e_tc_act_parse_state *parse_state,
                        return err;
        }
 
+       flow_action_for_each(i, act, flow_action) {
+               tc_act = mlx5e_tc_act_get(act->id, ns_type);
+               if (!tc_act || !tc_act->post_parse ||
+                   !tc_act->can_offload(parse_state, act, i))
+                       continue;
+
+               err = tc_act->post_parse(parse_state, priv, attr);
+               if (err)
+                       return err;
+       }
+
        return 0;
 }