DPDK patches and discussions
 help / color / mirror / Atom feed
* [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).