[PATCH] app/testpmd: fix tunnel offload validation

[Ferruh Yigit]

On 11/17/2021 10:27 AM, Slava Ovsiienko wrote:
>> -----Original Message-----
>> From: Ferruh Yigit <ferruh.yigit at intel.com>
>> Sent: Wednesday, November 17, 2021 12:10
>> To: Gregory Etelson <getelson at nvidia.com>; dev at dpdk.org; Ori Kam
>> <orika at nvidia.com>; Slava Ovsiienko <viacheslavo at nvidia.com>
>> Cc: Matan Azrad <matan at nvidia.com>; Raslan Darawsheh
>> <rasland at nvidia.com>; NBU-Contact-Thomas Monjalon
>> <thomas at monjalon.net>
>> Subject: Re: [PATCH] app/testpmd: fix tunnel offload validation
>>
>> On 11/17/2021 8:57 AM, Gregory Etelson wrote:
>>> Hello Ferruh,
>>>
>>> Can you estimate when this patch will be merged into 21.11 ?
>>>
>>
>> Hi Gregory,
>>
>> It is not merged because it is missing review.
>>
>> Ori, Slava, can you please help with the review?
>>
>>
>>> Thank you.
>>>
>>> Regards,
>>> Gregory
>>>
>>>> -----Original Message-----
>>>> From: Gregory Etelson <getelson at nvidia.com>
>>>> Sent: Tuesday, November 2, 2021 14:24
>>>> To: dev at dpdk.org; Gregory Etelson
>>>> <getelson at nvidia.com>
>>>> Cc: Matan Azrad <matan at nvidia.com>; Raslan Darawsheh
>>>> <rasland at nvidia.com>; stable at dpdk.org; Xiaoyun Li
>>>> <xiaoyun.li at intel.com>
>>>> Subject: [PATCH] app/testpmd: fix tunnel offload validation
>>>>
>>>> Tunnel offload API allows application to restore packet to its
>>>> original form if chain of flows missed after DECAP action.
>>>> The main idea of the tunnel offload API was to query port PMD to
>>>> provide flow elements - actions or items.
>>>> Flow elements supplied by PMD are merged with original flow rule
>>>> elements provided by testpmd operator to create a new flow rule,
>>>> optimal for PMD, to implement the tunnel offload API.
>>>> That flow rule transformation is hidden form testpmd operator and
>>>> uses internal testpmd resources.
>>>>
>>>> Current testpmd did not release tunnel offload resources if flow rule
>>>> validation failed.
>>>>
>>>> The patch always releases tunnel offload resources after flow rule
>>>> validation returns.
>>>>
>>>> Cc: stable at dpdk.org
>>>>
>>>> Fixes: 1b9f274623b8 ("app/testpmd: add commands for tunnel offload")
>>>>
>>>> Signed-off-by: Gregory Etelson
>>>> <getelson at nvidia.com>
> Reviewed-by: Viacheslav Ovsiienko <viacheslavo at nvidia.com>
> 
> 

Applied to dpdk-next-net/main, thanks.