[PATCH] net/mlx5: remove unneeded GENEVE option length assert

Slava Ovsiienko viacheslavo at nvidia.com
Wed Nov 9 08:27:55 CET 2022


> -----Original Message-----
> From: Suanming Mou <suanmingm at nvidia.com>
> Sent: Wednesday, November 9, 2022 3:40
> To: Matan Azrad <matan at nvidia.com>; Slava Ovsiienko <viacheslavo at nvidia.com>
> Cc: dev at dpdk.org; Raslan Darawsheh <rasland at nvidia.com>
> Subject: [PATCH] net/mlx5: remove unneeded GENEVE option length assert
> 
> In order to share flow items translate code, flow items translation of spec
> and mask was split individually.
> 
> In that case, the assert for GENEVE option length with mask becomes invalid,
> since the length in mask is bitmask. And as memcpy around the assert already
> checks the GENEVE option length, the assert looks redundant.
> 
> This commit removes the unneeded GENEVE option length assert.
> 
> Fixes: cd4ab742064a ("net/mlx5: split flow item matcher and value
> translation")
> 
> Signed-off-by: Suanming Mou <suanmingm at nvidia.com>
Acked-by: Viacheslav Ovsiienko <viacheslavo at nvidia.com>



More information about the dev mailing list