Skip to content

Commit

Permalink
net/mlx5: fix action configuration validation
Browse files Browse the repository at this point in the history
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: 57c7b94 ("net/mlx5: add async flow operation validation")

Signed-off-by: Dariusz Sosnowski <dsosnowski@nvidia.com>
Acked-by: Suanming Mou <suanmingm@nvidia.com>
  • Loading branch information
sodar authored and raslandarawsheh committed Jul 22, 2024
1 parent 60d122b commit 213d4b9
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion drivers/net/mlx5/mlx5_flow_hw.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down

0 comments on commit 213d4b9

Please sign in to comment.