fib: promote experimental API's to stable
Checks
Commit Message
The fib and fib6 API's have been in since 19.11 and
should be marked as stable.
Signed-off-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>
---
Notes:
This patch should be applied after
https://patches.dpdk.org/project/dpdk/patch/1630943759-362969-1-git-send-email-vladimir.medvedkin@intel.com/
lib/fib/rte_fib.h | 13 -------------
lib/fib/rte_fib6.h | 13 -------------
lib/fib/version.map | 2 +-
3 files changed, 1 insertion(+), 27 deletions(-)
Comments
On Mon, 6 Sep 2021 17:01:15 +0100
Vladimir Medvedkin <vladimir.medvedkin@intel.com> wrote:
> diff --git a/lib/fib/version.map b/lib/fib/version.map
> index be975ea..af76add 100644
> --- a/lib/fib/version.map
> +++ b/lib/fib/version.map
> @@ -1,4 +1,4 @@
> -EXPERIMENTAL {
> +DPDK_22 {
> global:
>
> rte_fib_add;
> --
I think you should add to existing version number
Hi Stephen,
On 07/09/2021 02:34, Stephen Hemminger wrote:
> On Mon, 6 Sep 2021 17:01:15 +0100
> Vladimir Medvedkin <vladimir.medvedkin@intel.com> wrote:
>
>> diff --git a/lib/fib/version.map b/lib/fib/version.map
>> index be975ea..af76add 100644
>> --- a/lib/fib/version.map
>> +++ b/lib/fib/version.map
>> @@ -1,4 +1,4 @@
>> -EXPERIMENTAL {
>> +DPDK_22 {
>> global:
>>
>> rte_fib_add;
>> --
>
> I think you should add to existing version number
> \
Could you please clarify what did you mean? There was no existing stable
API section in the fib's version.map and current ABI version is DPDK_22.
On Wed, 8 Sep 2021 15:57:54 +0200
"Medvedkin, Vladimir" <vladimir.medvedkin@intel.com> wrote:
> Hi Stephen,
>
> On 07/09/2021 02:34, Stephen Hemminger wrote:
> > On Mon, 6 Sep 2021 17:01:15 +0100
> > Vladimir Medvedkin <vladimir.medvedkin@intel.com> wrote:
> >
> >> diff --git a/lib/fib/version.map b/lib/fib/version.map
> >> index be975ea..af76add 100644
> >> --- a/lib/fib/version.map
> >> +++ b/lib/fib/version.map
> >> @@ -1,4 +1,4 @@
> >> -EXPERIMENTAL {
> >> +DPDK_22 {
> >> global:
> >>
> >> rte_fib_add;
> >> --
> >
> > I think you should add to existing version number
> > \
>
> Could you please clarify what did you mean? There was no existing stable
> API section in the fib's version.map and current ABI version is DPDK_22.
>
Ok, I missed that. Was looking at so many other files where
version was DPDK_20.
> The fib and fib6 API's have been in since 19.11 and
> should be marked as stable.
>
> Signed-off-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>
Acked-by: Conor Walsh <conor.walsh@intel.com>
On Wed, Sep 8, 2021 at 7:04 PM Conor Walsh <conor.walsh@intel.com> wrote:
> > The fib and fib6 API's have been in since 19.11 and
> > should be marked as stable.
> >
> > Signed-off-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>
> Acked-by: Conor Walsh <conor.walsh@intel.com>
Updated MAINTAINERS accordingly.
Applied, thanks.
@@ -11,10 +11,6 @@
*
* RTE FIB library.
*
- * @warning
- * @b EXPERIMENTAL:
- * All functions in this file may be changed or removed without prior notice.
- *
* FIB (Forwarding information base) implementation
* for IPv4 Longest Prefix Match
*/
@@ -107,7 +103,6 @@ struct rte_fib_conf {
* Handle to the FIB object on success
* NULL otherwise with rte_errno set to an appropriate values.
*/
-__rte_experimental
struct rte_fib *
rte_fib_create(const char *name, int socket_id, struct rte_fib_conf *conf);
@@ -121,7 +116,6 @@ rte_fib_create(const char *name, int socket_id, struct rte_fib_conf *conf);
* set appropriately. Possible rte_errno values include:
* - ENOENT - required entry not available to return.
*/
-__rte_experimental
struct rte_fib *
rte_fib_find_existing(const char *name);
@@ -133,7 +127,6 @@ rte_fib_find_existing(const char *name);
* @return
* None
*/
-__rte_experimental
void
rte_fib_free(struct rte_fib *fib);
@@ -151,7 +144,6 @@ rte_fib_free(struct rte_fib *fib);
* @return
* 0 on success, negative value otherwise
*/
-__rte_experimental
int
rte_fib_add(struct rte_fib *fib, uint32_t ip, uint8_t depth, uint64_t next_hop);
@@ -167,7 +159,6 @@ rte_fib_add(struct rte_fib *fib, uint32_t ip, uint8_t depth, uint64_t next_hop);
* @return
* 0 on success, negative value otherwise
*/
-__rte_experimental
int
rte_fib_delete(struct rte_fib *fib, uint32_t ip, uint8_t depth);
@@ -188,7 +179,6 @@ rte_fib_delete(struct rte_fib *fib, uint32_t ip, uint8_t depth);
* @return
* -EINVAL for incorrect arguments, otherwise 0
*/
-__rte_experimental
int
rte_fib_lookup_bulk(struct rte_fib *fib, uint32_t *ips,
uint64_t *next_hops, int n);
@@ -201,7 +191,6 @@ rte_fib_lookup_bulk(struct rte_fib *fib, uint32_t *ips,
* Pointer on the dataplane struct on success
* NULL othervise
*/
-__rte_experimental
void *
rte_fib_get_dp(struct rte_fib *fib);
@@ -214,7 +203,6 @@ rte_fib_get_dp(struct rte_fib *fib);
* Pointer on the RIB on success
* NULL othervise
*/
-__rte_experimental
struct rte_rib *
rte_fib_get_rib(struct rte_fib *fib);
@@ -230,7 +218,6 @@ rte_fib_get_rib(struct rte_fib *fib);
* 0 on success
* -EINVAL on failure
*/
-__rte_experimental
int
rte_fib_select_lookup(struct rte_fib *fib, enum rte_fib_lookup_type type);
@@ -11,10 +11,6 @@
*
* RTE FIB6 library.
*
- * @warning
- * @b EXPERIMENTAL:
- * All functions in this file may be changed or removed without prior notice.
- *
* FIB (Forwarding information base) implementation
* for IPv6 Longest Prefix Match
*/
@@ -98,7 +94,6 @@ struct rte_fib6_conf {
* Handle to FIB object on success
* NULL otherwise with rte_errno set to an appropriate values.
*/
-__rte_experimental
struct rte_fib6 *
rte_fib6_create(const char *name, int socket_id, struct rte_fib6_conf *conf);
@@ -112,7 +107,6 @@ rte_fib6_create(const char *name, int socket_id, struct rte_fib6_conf *conf);
* set appropriately. Possible rte_errno values include:
* - ENOENT - required entry not available to return.
*/
-__rte_experimental
struct rte_fib6 *
rte_fib6_find_existing(const char *name);
@@ -124,7 +118,6 @@ rte_fib6_find_existing(const char *name);
* @return
* None
*/
-__rte_experimental
void
rte_fib6_free(struct rte_fib6 *fib);
@@ -142,7 +135,6 @@ rte_fib6_free(struct rte_fib6 *fib);
* @return
* 0 on success, negative value otherwise
*/
-__rte_experimental
int
rte_fib6_add(struct rte_fib6 *fib, const uint8_t ip[RTE_FIB6_IPV6_ADDR_SIZE],
uint8_t depth, uint64_t next_hop);
@@ -159,7 +151,6 @@ rte_fib6_add(struct rte_fib6 *fib, const uint8_t ip[RTE_FIB6_IPV6_ADDR_SIZE],
* @return
* 0 on success, negative value otherwise
*/
-__rte_experimental
int
rte_fib6_delete(struct rte_fib6 *fib,
const uint8_t ip[RTE_FIB6_IPV6_ADDR_SIZE], uint8_t depth);
@@ -181,7 +172,6 @@ rte_fib6_delete(struct rte_fib6 *fib,
* @return
* -EINVAL for incorrect arguments, otherwise 0
*/
-__rte_experimental
int
rte_fib6_lookup_bulk(struct rte_fib6 *fib,
uint8_t ips[][RTE_FIB6_IPV6_ADDR_SIZE],
@@ -196,7 +186,6 @@ rte_fib6_lookup_bulk(struct rte_fib6 *fib,
* Pointer on the dataplane struct on success
* NULL othervise
*/
-__rte_experimental
void *
rte_fib6_get_dp(struct rte_fib6 *fib);
@@ -209,7 +198,6 @@ rte_fib6_get_dp(struct rte_fib6 *fib);
* Pointer on the RIB6 on success
* NULL othervise
*/
-__rte_experimental
struct rte_rib6 *
rte_fib6_get_rib(struct rte_fib6 *fib);
@@ -225,7 +213,6 @@ rte_fib6_get_rib(struct rte_fib6 *fib);
* 0 on success
* -EINVAL on failure
*/
-__rte_experimental
int
rte_fib6_select_lookup(struct rte_fib6 *fib, enum rte_fib6_lookup_type type);
@@ -1,4 +1,4 @@
-EXPERIMENTAL {
+DPDK_22 {
global:
rte_fib_add;