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 <dsosnow...@nvidia.com>
Acked-by: Suanming Mou <suanmi...@nvidia.com>
---
 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 fe7df7305f..39d1cd96d4 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;
-- 
2.39.2

Reply via email to