From 213d4b9ee5b754569436e15a1e6ca85770c77f00 Mon Sep 17 00:00:00 2001 From: Dariusz Sosnowski Date: Thu, 18 Jul 2024 11:57:16 +0200 Subject: [PATCH] net/mlx5: fix action configuration validation Checking if action configuration is required should be checked based on action type recorded in the actions template, not on user action. Also, adds a missing internal RSS action type to configuration check skip list. Fixes: 57c7b94301ee ("net/mlx5: add async flow operation validation") Signed-off-by: Dariusz Sosnowski Acked-by: Suanming Mou --- drivers/net/mlx5/mlx5_flow_hw.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/mlx5/mlx5_flow_hw.c b/drivers/net/mlx5/mlx5_flow_hw.c index fe7df7305fc..39d1cd96d4e 100644 --- a/drivers/net/mlx5/mlx5_flow_hw.c +++ b/drivers/net/mlx5/mlx5_flow_hw.c @@ -16388,10 +16388,11 @@ flow_hw_validate_rule_actions(struct rte_eth_dev *dev, user_action = &actions[act_data->action_src]; /* Skip actions which do not require conf. */ - switch ((int)user_action->type) { + switch ((int)act_data->type) { case RTE_FLOW_ACTION_TYPE_COUNT: case MLX5_RTE_FLOW_ACTION_TYPE_COUNT: case MLX5_RTE_FLOW_ACTION_TYPE_METER_MARK: + case MLX5_RTE_FLOW_ACTION_TYPE_RSS: continue; default: break;