[dpdk-dev] [PATCH] mbuf: enforce no option for dynamic fields and flags

2021-10-12 Thread David Marchand
As stated in the API, dynamic field and flags should be created with no additional flag (simply in the API for future changes). Fix the dynamic flag register helper which was not enforcing it and add unit tests. Fixes: 4958ca3a443a ("mbuf: support dynamic fields and flags") Signed-off-by: David

Re: [dpdk-dev] [PATCH] mbuf: enforce no option for dynamic fields and flags

2021-10-12 Thread Stephen Hemminger
On Tue, 12 Oct 2021 21:39:57 +0200 David Marchand wrote: > As stated in the API, dynamic field and flags should be created with no > additional flag (simply in the API for future changes). > > Fix the dynamic flag register helper which was not enforcing it and add > unit tests. > > Fixes: 4958c

Re: [dpdk-dev] [PATCH] mbuf: enforce no option for dynamic fields and flags

2021-10-13 Thread Andrew Rybchenko
On 10/12/21 11:14 PM, Stephen Hemminger wrote: > On Tue, 12 Oct 2021 21:39:57 +0200 > David Marchand wrote: > >> As stated in the API, dynamic field and flags should be created with no >> additional flag (simply in the API for future changes). >> >> Fix the dynamic flag register helper which was

Re: [dpdk-dev] [PATCH] mbuf: enforce no option for dynamic fields and flags

2021-10-13 Thread Kinsella, Ray
On 12/10/2021 20:39, David Marchand wrote: > As stated in the API, dynamic field and flags should be created with no > additional flag (simply in the API for future changes). > > Fix the dynamic flag register helper which was not enforcing it and add > unit tests. > > Fixes: 4958ca3a443a ("mbu

Re: [dpdk-dev] [PATCH] mbuf: enforce no option for dynamic fields and flags

2021-10-15 Thread David Marchand
On Wed, Oct 13, 2021 at 9:06 AM Andrew Rybchenko wrote: > On 10/12/21 11:14 PM, Stephen Hemminger wrote: > > On Tue, 12 Oct 2021 21:39:57 +0200 > > David Marchand wrote: > > > >> As stated in the API, dynamic field and flags should be created with no > >> additional flag (simply in the API for fu