From: Ferruh Yigit > 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?
There is no a bug fix. The behavior will not be changed. It is just for comfort. > > 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