net/mlx5: fix VLAN ID action offset

Currently, the set VLAN ID header modify action has already got the
action pointer with offset from the header modify action array, but
the configuration saves the detail to the memory of action with the
offset again. It causes double offset to set the VLAN ID action to
the wrong place in the header modify array.

Remove the offset when get the action pointer to fix that issue.

Fixes: 5f163d520cff ("net/mlx5: support modify VLAN ID on existing VLAN header")
Cc: stable@dpdk.org
Signed-off-by: Suanming Mou <suanmingm@mellanox.com>
Acked-by: Dekel Peled <dekelp@mellanox.com>
Acked-by: Matan Azrad <matan@mellanox.com>

diff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_dv.c
index 467d1ce6b1e7b23b741bd0ec262840308f9d2cb2..99d668f5d35840cf117c63389e87e1c6342f0a2f 100644 (file)
--- a/drivers/net/mlx5/mlx5_flow_dv.c
+++ b/drivers/net/mlx5/mlx5_flow_dv.c
@@ -573,7 +573,7 @@ flow_dv_convert_action_modify_vlan_vid
        const struct rte_flow_action_of_set_vlan_vid *conf =
                (const struct rte_flow_action_of_set_vlan_vid *)(action->conf);
        int i = resource->actions_num;
-       struct mlx5_modification_cmd *actions = &resource->actions[i];
+       struct mlx5_modification_cmd *actions = resource->actions;
        struct field_modify_info *field = modify_vlan_out_first_vid;
 
        if (i >= MLX5_MAX_MODIFY_NUM)