Hi,
> -----Original Message-----
> From: Michael Baum <[email protected]>
> Sent: Monday, November 27, 2023 2:43 PM
> To: [email protected]
> Cc: Matan Azrad <[email protected]>; Raslan Darawsheh
> <[email protected]>; Slava Ovsiienko <[email protected]>; Ori Kam
> <[email protected]>; Suanming Mou <[email protected]>;
> [email protected]; [email protected]
> Subject: [PATCH v2] net/mlx5: fix jump action validation
>
> Currently PMD doesn't allow to jump to the same group in order to
> avoid dead loop. But this also prevent experienced user to create
> flow with less Hops in order to have better performance.
>
> For example, rules in [1] should have better performance then [2].
>
> Furthermore, this protection will not really prevent dead loop, i.e
> [3]. So just remove this protection and user should take the
> responsibility to avoid dead loop.
>
> This patch enables jumping to the same group.
>
> [1]:
> flow create 0 group 1 priority 1 pattern eth / ipv4 / udp / gtp / end
> actions raw_decap / raw_encap / jump group 1 / end
> flow create 0 group 1 priority 0 pattern eth / ipv4 src is 1.0.0.1 / tcp
> / end actions queues index 1 / end
>
> [2]:
> flow create 0 group 1 priority 0 pattern eth / ipv4 / udp / gtp / end
> actions raw_decap / raw_encap / jump group 2 / end
> flow create 0 group 2 priority 0 pattern eth / ipv4 src is 1.0.0.1 / tcp
> / end actions queues index 1 / end
>
> [3]:
> flow create 0 group 1 pattern eth / end actions jump group 2 / end
> flow create 0 group 2 pattern eth / end actions jump group 1 / end
>
> Fixes: f78f747f41d0 ("net/mlx5: allow jump to group lower than current")
> Cc: [email protected]
> Cc: [email protected]
>
> Signed-off-by: Michael Baum <[email protected]>
> Acked-by: Matan Azrad <[email protected]>
> ---
>
> V2: change commit message to fix template.
Patch applied to next-net-mlx,
Kindest regards
Raslan Darawsheh
