Re: [dpdk-stable] [PATCH] net/mlx5: fix meter header modify before decap issue

[Matan Azrad <matan@mellanox.com>]

From: Suanming Mou
> The meter flows are split into three subflows each, the prefix subflow with
> meter action color the packet, the meter subflow filters out the colored
> packets, the suffix subflow applies all the remaining actions to the passed
> packets. The tag header modify action is added to the prefix subflow to make
> the suffix subflow to match the packets from the prefix subflow.
> 
> Currently, the tag header modify action is added at the beginning in the
> prefix subflow even before decap action. The header modify action does not
> make sense to the later decap action, so the flow create will be validated as
> incorrect flow rule and failed.
> 
> Move the tag header modify action just before meter action in the prefix
> subflow to make the flow with decap action to do the decap first, then do
> the tag and meter to fix that issue.
> 
> Fixes: 9ea9b049a960 ("net/mlx5: split meter flow")
> Cc: stable@dpdk.org
> 
> Reported-by: Tonghao Zhang <xiangxia.m.yue@gmail.com>
> Signed-off-by: Suanming Mou <suanmingm@mellanox.com>
> Acked-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>
Acked-by: Matan Azrad <matan@mellanox.com>