I didn't plan to have it in this release since it's not critical and I'm afraid of such changes... I didn't mention this in the status mail today. @Raslan Darawsheh<mailto:[email protected]> can we check tomorrow regression result? If not good we'll need to revert it.
Regards, Asaf Penso ________________________________ From: Thomas Monjalon <[email protected]> Sent: Tuesday, May 18, 2021 12:43:34 PM To: Alexander Kozyrev <[email protected]> Cc: [email protected] <[email protected]>; [email protected] <[email protected]>; Raslan Darawsheh <[email protected]>; Matan Azrad <[email protected]>; Slava Ovsiienko <[email protected]>; Asaf Penso <[email protected]> Subject: Re: [dpdk-dev] [PATCH] net/mlx5: check meta register width for modify field > > The modify_field Flow API assumes that the META item is 32 bits wide. > > But the C Register that is used for Meta item can be 16 or 32 bits wide > > depending on kernel and firmware configurations. > > Take this into consideration and use the appropriate META width. > > > > Fixes: 641dbe4fb0 ("net/mlx5: support modify field flow action") > > Cc: [email protected] > > > > Signed-off-by: Alexander Kozyrev <[email protected]> > Acked-by: Viacheslav Ovsiienko <[email protected]> Applied in next-net-mlx, thanks.
