[INTERNAL]RE: [PATCH] net/mlx5/hws: fix ESP matching validation

[Slava Ovsiienko]

HWS guys claimed this check was needed to make sure matcher works well on ESP.
Are we sure we can remove this ?


> -----Original Message-----
> From: Raslan Darawsheh <rasland at nvidia.com>
> Sent: Wednesday, January 17, 2024 7:12 PM
> To: Michael Baum <michaelba at nvidia.com>; dev at dpdk.org
> Cc: Matan Azrad <matan at nvidia.com>; Dariusz Sosnowski
> <dsosnowski at nvidia.com>; Slava Ovsiienko <viacheslavo at nvidia.com>; Ori
> Kam <orika at nvidia.com>; Suanming Mou <suanmingm at nvidia.com>; Hamdan
> Igbaria <hamdani at nvidia.com>; stable at dpdk.org
> Subject: RE: [PATCH] net/mlx5/hws: fix ESP matching validation
> 
> Hi,
> > -----Original Message-----
> > From: Michael Baum <michaelba at nvidia.com>
> > Sent: Monday, January 15, 2024 2:10 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>; Hamdan Igbaria
> > <hamdani at nvidia.com>; stable at dpdk.org
> > Subject: [PATCH] net/mlx5/hws: fix ESP matching validation
> >
> > The "mlx5dr_definer_conv_item_esp()" function validates first whether
> > "ipsec_offload" PRM flag is on, if the flag is off the function
> > returns error.
> >
> > The "ipsec_offload" PRM flag indicates whether IPsec encrypt/decrypt
> > is supported, IPsec matching may be supported even when this flag is off.
> >
> > This patch removes this validation.
> >
> > Fixes: 81cf20a25abf ("net/mlx5/hws: support match on ESP item")
> > Cc: hamdani at nvidia.com
> > Cc: stable at dpdk.org
> >
> > Signed-off-by: Michael Baum <michaelba at nvidia.com>
> > Acked-by: Hamdan Igbaria <hamdani at nvidia.com>
> > Acked-by: Matan Azrad <matan at nvidia.com>
> Patch applied to next-net-mlx,
> 
> Kindest regards
> Raslan Darawsheh