On 6/16/2020 5:06 PM, Matan Azrad wrote:
> Hi
>
> Ferruh, Raslan
>
> This patch is not a fix, we don't need to backport it.
Why not backport it?
> I don't think we need fixes and cc stable here.
>
> Matan
>
> From: Raslan Darawsheh
>> Hi,
>>
>>> -----Original Message-----
>>> From: Ferruh Yigit <ferruh.yi...@intel.com>
>>> Sent: Tuesday, June 16, 2020 1:28 PM
>>> To: Dekel Peled <dek...@mellanox.com>; Matan Azrad
>>> <ma...@mellanox.com>; Slava Ovsiienko <viachesl...@mellanox.com>;
>>> Raslan Darawsheh <rasl...@mellanox.com>
>>> Cc: dev@dpdk.org
>>> Subject: Re: [dpdk-dev] [PATCH] net/mlx5: move GTP mask define to
>>> correct location
>>>
>>> On 6/10/2020 2:25 PM, Dekel Peled wrote:
>>>> Recent patch [1] added definition of mask MLX5_GTP_FLAGS_MASK, just
>>>> above function flow_dv_validate_item_gtp(), where it is used.
>>>>
>>>> Patch was applied together with other patches which modified the
>>>> same file, so the mask was located further away from the function it
>>>> is used in.
>>>>
>>>> This patch moves the mask definition to the proper location.
>>>>
>>>> Signed-off-by: Dekel Peled <dek...@mellanox.com>
>>>> Acked-by: Matan Azrad <ma...@mellanox.com>
>>>>
>>>> [1] Commit 563ac307a46b ("net/mlx5: support match on GTP flags")
>>>
>>>
>>> Better to add this as 'Fixes', instead of reference, so this way it
>>> helps for the backporting etc..
>>>
>>> Fixes: b88341ca35fc ("net/mlx5: convert flow dev handle to indexed")
>>> Cc: sta...@dpdk.org
>>
>> I'll change it in next-net-mlx accordingly,
>>
>> Kindest regards,
>> Raslan Darawsheh
