[PATCH] net/mlx5: fix age position in hairpin split
Raslan Darawsheh
rasland at nvidia.com
Wed Mar 13 08:48:14 CET 2024
Hi,
> -----Original Message-----
> From: Bing Zhao <bingz at nvidia.com>
> Sent: Thursday, March 7, 2024 10:09 AM
> To: Slava Ovsiienko <viacheslavo at nvidia.com>; dev at dpdk.org; Raslan
> Darawsheh <rasland at nvidia.com>
> Cc: Ori Kam <orika at nvidia.com>; Dariusz Sosnowski
> <dsosnowski at nvidia.com>; Suanming Mou <suanmingm at nvidia.com>;
> Matan Azrad <matan at nvidia.com>; Michael Baum <michaelba at nvidia.com>;
> stable at dpdk.org
> Subject: [PATCH] net/mlx5: fix age position in hairpin split
>
> When splitting a hairpin rule implicitly, the count action will be on either Tx or
> Rx subflow based on the encapsulation checking.
>
> Once there is a flow rule with both count and age action, one counter will be
> reused. If there is only age action and the ASO flow hit is supported, the flow
> hit will be chosen instead of a counter.
>
> In the previous flow splitting, the age would always be in the Rx part, while the
> count would be on the Tx part when there is an encap.
>
> Before this commit, 2 issues can be observed with a hairpin split:
> 1. On the root table, one counter was used on both Rx and Tx parts
> for age and count actions. Then one ingress packet will be
> counted twice.
> 2. On the non-root table, an extra ASO flow hit was used on the Rx
> part. This would cause some overhead.
>
> The age and count actions should be in the same subflow instead of 2.
>
> Fixes: daed4b6e3db2 ("net/mlx5: use aging by counter when counter exists")
> Cc: michaelba at nvidia.com
> Cc: stable at dpdk.org
>
> Signed-off-by: Bing Zhao <bingz at nvidia.com>
> Acked-by: Ori Kam <orika at nvidia.com>
Patch applied to next-net-mlx,
Kindest regards
Raslan Darawsheh
More information about the stable
mailing list