[dpdk-dev] [PATCH v4] net/mlx5: fix tunnel offload private items location

Ferruh Yigit ferruh.yigit at intel.com
Mon May 10 18:10:48 CEST 2021


On 5/10/2021 1:29 PM, Gregory Etelson wrote:
> Hello Ferruh,
> 
> The patch has failed ci/Intel-compilation test.
> (http://mails.dpdk.org/archives/test-report/2021-May/193327.html)
> I've checked that with Thomas. He confirmed that was a known fault and 
> should not affect the current patch.
> Can we proceed with the patch integration ?
> 

Hi Gregory,

Thanks for the new version and additional effort for clarification,
agree that CI issue is something else, patch is already assigned to Raslan and
we can proceed with his tree.

Thanks,
ferruh

> Thank you.
> 
> Regards,
> Gregory 
> 
>> -----Original Message-----
>> From: Gregory Etelson <getelson at nvidia.com>
>> Sent: Thursday, May 6, 2021 12:58
>> To: dev at dpdk.org
>> Cc: Gregory Etelson <getelson at nvidia.com>; Matan Azrad
>> <matan at nvidia.com>; Ori Kam <orika at nvidia.com>; Raslan Darawsheh
>> <rasland at nvidia.com>; ferruh.yigit at intel.com; stable at dpdk.org; Slava
>> Ovsiienko <viacheslavo at nvidia.com>; Shahaf Shuler
>> <shahafs at nvidia.com>
>> Subject: [PATCH v4] net/mlx5: fix tunnel offload private items location
>>
>> Tunnel offload API requires application to query PMD for specific flow items
>> and actions. Application uses these PMD specific elements to build flow
>> rules according to the tunnel offload model.
>> The model does not restrict private elements location in a flow rule, but the
>> current MLX5 PMD implementation expects that tunnel offload rule will
>> begin with PMD specific elements.
>> The patch removes that placement limitation.
>>
>> Cc: stable at dpdk.org
>>
>> Fixes: 4ec6360de37d ("net/mlx5: implement tunnel offload")
>>
>> Signed-off-by: Gregory Etelson <getelson at nvidia.com>
>> Acked-by: Viacheslav Ovsiienko <viacheslavo at nvidia.com>
>> ---
>> v2: Update the patch comment.
>> v3: Remove testpmd patch from the series.
>> v4: Remove redundant verifications in flow_dv_validate.

<...>


More information about the dev mailing list