diff mbox series

[v2] ethdev: declare Tx prepare API as not experimental

Message ID 1547215897-22272-1-git-send-email-david.marchand@redhat.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series [v2] ethdev: declare Tx prepare API as not experimental |

Checks

Context Check Description
ci/Intel-compilation success Compilation OK
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/intel-Performance-Testing success Performance Testing PASS

Commit Message

David Marchand Jan. 11, 2019, 2:11 p.m. UTC 
  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(-)

Comments

Ferruh Yigit Jan. 11, 2019, 4:52 p.m. UTC | #1 
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. UTC | #2 
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.
diff mbox series

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