[PATCH] net/mlx5: fix GENEVE TLV option management

Raslan Darawsheh rasland at nvidia.com
Wed Jan 17 18:14:51 CET 2024


Hi,
> -----Original Message-----
> From: Michael Baum <michaelba at nvidia.com>
> Sent: Monday, January 15, 2024 2:13 PM
> To: dev at dpdk.org
> Cc: Matan Azrad <matan at nvidia.com>; Dariusz Sosnowski
> <dsosnowski at nvidia.com>; Raslan Darawsheh <rasland at nvidia.com>; Slava
> Ovsiienko <viacheslavo at nvidia.com>; Ori Kam <orika at nvidia.com>;
> Suanming Mou <suanmingm at nvidia.com>; Shiri Kuzin <shirik at nvidia.com>;
> stable at dpdk.org
> Subject: [PATCH] net/mlx5: fix GENEVE TLV option management
> 
> In SW steering, the GENEVE TLV option matching flows must be created using
> a translation function.
> This function checks whether this option has already created a DevX object for
> the matching and either creates the objects or updates the reference counter.
> After translation, a flag in flow structure is turned on indicating the destroy
> function to release this DevX object.
> 
> When the flow rule has meter, the rule may be split, and call translate function
> more than once per flow causing object reference counter to increase each
> time without updating the flow flag accordingly.
> 
> This patch uses this flag as a reference counter which is increased every
> translation and indicates the destroy function how many destroy DevX to do.
> 
> Fixes: f15f0c3806d0 ("net/mlx5: create GENEVE TLV option management")
> Fixes: e440d6cf589e ("net/mlx5: add GENEVE TLV option flow translation")
> Cc: shirik at nvidia.com
> Cc: stable at dpdk.org
> 
> Signed-off-by: Michael Baum <michaelba at nvidia.com>
> Acked-by: Matan Azrad <matan at nvidia.com>
Patch applied to next-net-mlx,

Kindest regards
Raslan Darawsheh



More information about the stable mailing list