Patchwork [dpdk-dev,v2] ethdev: declare Tx prepare API as not experimental

login
register
mail settings
Submitter David Marchand
Date Jan. 11, 2019, 2:11 p.m.
Message ID <1547215897-22272-1-git-send-email-david.marchand@redhat.com>
Download mbox | patch
Permalink /patch/697737/
State New
Headers show

Comments

David Marchand - Jan. 11, 2019, 2:11 p.m.
The Tx prepare API was introduced at a time when the experimental API
tag mechanism did not exist yet and is missing the tag.
However, this API has been there since 17.02 and did not suffer any
change since its introduction.

Let's just remove the warning from the documentation.

Signed-off-by: David Marchand <david.marchand@redhat.com>
Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: Andrew Rybchenko <arybchenko@solarflare.com>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
---
 lib/librte_ethdev/rte_ethdev.h | 3 ---
 1 file changed, 3 deletions(-)
Ferruh Yigit - Jan. 11, 2019, 4:52 p.m.
On 1/11/2019 2:11 PM, David Marchand wrote:
> The Tx prepare API was introduced at a time when the experimental API
> tag mechanism did not exist yet and is missing the tag.
> However, this API has been there since 17.02 and did not suffer any
> change since its introduction.
> 
> Let's just remove the warning from the documentation.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> Acked-by: Andrew Rybchenko <arybchenko@solarflare.com>
> Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>

Applied to dpdk-next-net/master, thanks.
David Marchand - Jan. 11, 2019, 5:19 p.m.
On Fri, Jan 11, 2019 at 5:52 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> On 1/11/2019 2:11 PM, David Marchand wrote:
> > The Tx prepare API was introduced at a time when the experimental API
> > tag mechanism did not exist yet and is missing the tag.
> > However, this API has been there since 17.02 and did not suffer any
> > change since its introduction.
> >
> > Let's just remove the warning from the documentation.
> >
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> > Acked-by: Andrew Rybchenko <arybchenko@solarflare.com>
> > Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
>
> Applied to dpdk-next-net/master, thanks.
>

As discussed offlist and on ovs ml, once it reaches master, could we
backport at least to 18.11 ?
Thanks.

Patch

diff --git a/lib/librte_ethdev/rte_ethdev.h b/lib/librte_ethdev/rte_ethdev.h
index 1960f3a..76266ad 100644
--- a/lib/librte_ethdev/rte_ethdev.h
+++ b/lib/librte_ethdev/rte_ethdev.h
@@ -4159,9 +4159,6 @@  static inline int rte_eth_tx_descriptor_status(uint16_t port_id,
 }
 
 /**
- * @warning
- * @b EXPERIMENTAL: this API may change without prior notice
- *
  * Process a burst of output packets on a transmit queue of an Ethernet device.
  *
  * The rte_eth_tx_prepare() function is invoked to prepare output packets to be