Re: [PATCH v1] bbdev: allow operation type enum for growth

2022-06-23 Thread Ray Kinsella
Thomas Monjalon writes: > This solution is what I proposed to the techboard some years ago, > but the preference was to completely remove the MAX values. I am mindful we don't have an explicit guidance in the documentation. I am including to add something to the abi_versioning document, that

RE: [PATCH v1] bbdev: allow operation type enum for growth

2022-06-17 Thread Chautru, Nicolas
> From: Thomas Monjalon > > This solution is what I proposed to the techboard some years ago, but the > preference was to completely remove the MAX values. > Thanks, good to see you had similar thought! I don't believe there is an actual recommendation captured in term of how to remove compl

Re: [PATCH v1] bbdev: allow operation type enum for growth

2022-06-17 Thread Thomas Monjalon
This solution is what I proposed to the techboard some years ago, but the preference was to completely remove the MAX values. 13/06/2022 20:24, Nicolas Chautru: > Updating the last enum for rte_bbdev_op_type > to allow for enum insertion. Please explain that the reason is to keep ABI compatible,

RE: [PATCH v1] bbdev: allow operation type enum for growth

2022-06-13 Thread Chautru, Nicolas
xp.com; > david.march...@redhat.com > Subject: Re: [PATCH v1] bbdev: allow operation type enum for growth > > On Mon, 13 Jun 2022 11:24:35 -0700 > Nicolas Chautru wrote: > > > Updating the last enum for rte_bbdev_op_type to allow for enum > > insertion. > > >

Re: [PATCH v1] bbdev: allow operation type enum for growth

2022-06-13 Thread Stephen Hemminger
On Mon, 13 Jun 2022 11:24:35 -0700 Nicolas Chautru wrote: > Updating the last enum for rte_bbdev_op_type > to allow for enum insertion. > > Signed-off-by: Nicolas Chautru Only allowed if you check now for the any of the reserved types and fail.