* [dpdk-dev] [PATCH] fib: promote experimental API's to stable
@ 2021-09-06 16:01 Vladimir Medvedkin
2021-09-07 0:34 ` Stephen Hemminger
2021-09-08 17:02 ` Conor Walsh
0 siblings, 2 replies; 6+ messages in thread
From: Vladimir Medvedkin @ 2021-09-06 16:01 UTC (permalink / raw)
To: dev; +Cc: konstantin.ananyev
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(-)
diff --git a/lib/fib/rte_fib.h b/lib/fib/rte_fib.h
index 570b4b6..b3c59df 100644
--- a/lib/fib/rte_fib.h
+++ b/lib/fib/rte_fib.h
@@ -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);
diff --git a/lib/fib/rte_fib6.h b/lib/fib/rte_fib6.h
index a9d6d04..95879af 100644
--- a/lib/fib/rte_fib6.h
+++ b/lib/fib/rte_fib6.h
@@ -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);
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;
--
2.7.4
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] fib: promote experimental API's to stable
2021-09-06 16:01 [dpdk-dev] [PATCH] fib: promote experimental API's to stable Vladimir Medvedkin
@ 2021-09-07 0:34 ` Stephen Hemminger
2021-09-08 13:57 ` Medvedkin, Vladimir
2021-09-08 17:02 ` Conor Walsh
1 sibling, 1 reply; 6+ messages in thread
From: Stephen Hemminger @ 2021-09-07 0:34 UTC (permalink / raw)
To: Vladimir Medvedkin; +Cc: dev, konstantin.ananyev
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] fib: promote experimental API's to stable
2021-09-07 0:34 ` Stephen Hemminger
@ 2021-09-08 13:57 ` Medvedkin, Vladimir
2021-09-08 15:10 ` Stephen Hemminger
0 siblings, 1 reply; 6+ messages in thread
From: Medvedkin, Vladimir @ 2021-09-08 13:57 UTC (permalink / raw)
To: Stephen Hemminger; +Cc: dev, konstantin.ananyev
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.
--
Regards,
Vladimir
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] fib: promote experimental API's to stable
2021-09-08 13:57 ` Medvedkin, Vladimir
@ 2021-09-08 15:10 ` Stephen Hemminger
0 siblings, 0 replies; 6+ messages in thread
From: Stephen Hemminger @ 2021-09-08 15:10 UTC (permalink / raw)
To: Medvedkin, Vladimir; +Cc: dev, konstantin.ananyev
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.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] fib: promote experimental API's to stable
2021-09-06 16:01 [dpdk-dev] [PATCH] fib: promote experimental API's to stable Vladimir Medvedkin
2021-09-07 0:34 ` Stephen Hemminger
@ 2021-09-08 17:02 ` Conor Walsh
2021-10-02 9:01 ` David Marchand
1 sibling, 1 reply; 6+ messages in thread
From: Conor Walsh @ 2021-09-08 17:02 UTC (permalink / raw)
To: Vladimir Medvedkin, dev; +Cc: konstantin.ananyev
> 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>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] fib: promote experimental API's to stable
2021-09-08 17:02 ` Conor Walsh
@ 2021-10-02 9:01 ` David Marchand
0 siblings, 0 replies; 6+ messages in thread
From: David Marchand @ 2021-10-02 9:01 UTC (permalink / raw)
To: Vladimir Medvedkin
Cc: dev, Ananyev, Konstantin, Conor Walsh, Stephen Hemminger
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.
--
David Marchand
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-10-02 9:01 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-06 16:01 [dpdk-dev] [PATCH] fib: promote experimental API's to stable Vladimir Medvedkin
2021-09-07 0:34 ` Stephen Hemminger
2021-09-08 13:57 ` Medvedkin, Vladimir
2021-09-08 15:10 ` Stephen Hemminger
2021-09-08 17:02 ` Conor Walsh
2021-10-02 9:01 ` David Marchand
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).