diff mbox

[dpdk-dev] eal: deprecate function to set default mbuf pool

Message ID 20180524094407.6969-1-olivier.matz@6wind.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Olivier Matz May 24, 2018, 9:44 a.m. UTC 
  Deprecate rte_eal_mbuf_default_mempool_ops(), it shall be replaced by
rte_mbuf_best_mempool_ops().

Cc: Hemant Agrawal <hemant.agrawal@nxp.com>
Cc: Jerin Jacob <jerin.jacob@caviumnetworks.com>

Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---
 doc/guides/rel_notes/deprecation.rst    | 4 ++--
 lib/librte_eal/common/include/rte_eal.h | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

Comments

Jerin Jacob May 24, 2018, 9:59 a.m. UTC | #1 
-----Original Message-----
> Date: Thu, 24 May 2018 11:44:07 +0200
> From: Olivier Matz <olivier.matz@6wind.com>
> To: dev@dpdk.org
> CC: Hemant Agrawal <hemant.agrawal@nxp.com>, Jerin Jacob
>  <jerin.jacob@caviumnetworks.com>
> Subject: [PATCH] eal: deprecate function to set default mbuf pool
> X-Mailer: git-send-email 2.11.0
> 
> Deprecate rte_eal_mbuf_default_mempool_ops(), it shall be replaced by
> rte_mbuf_best_mempool_ops().
> 
> Cc: Hemant Agrawal <hemant.agrawal@nxp.com>
> Cc: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> 
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>

Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Hemant Agrawal May 24, 2018, 10:08 a.m. UTC | #2 
> Deprecate rte_eal_mbuf_default_mempool_ops(), it shall be replaced by 
> rte_mbuf_best_mempool_ops().
> 
> Cc: Hemant Agrawal <hemant.agrawal@nxp.com>
> Cc: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> 
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>

Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
Thomas Monjalon May 28, 2018, 12:31 a.m. UTC | #3 
24/05/2018 12:08, Hemant Agrawal:
> > Deprecate rte_eal_mbuf_default_mempool_ops(), it shall be replaced by 
> > rte_mbuf_best_mempool_ops().
> > 
> > Cc: Hemant Agrawal <hemant.agrawal@nxp.com>
> > Cc: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> > 
> > Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
> 
> Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>

Acked-by: Thomas Monjalon <thomas@monjalon.net>

Applied, thanks
diff mbox

Patch

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 1e2443c76..781cc6894 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -32,8 +32,8 @@  Deprecation Notices
   mempool names have been defined in ``rte_mbuf`` in v18.02. The uses of
   ``rte_eal_mbuf_default_mempool_ops`` shall be replaced by
   ``rte_mbuf_best_mempool_ops``.
-  The following function is now redundant and it is target to be deprecated
-  in 18.05:
+  The following function is deprecated since 18.05, and will be removed
+  in 18.08:
 
   - ``rte_eal_mbuf_default_mempool_ops``
 
diff --git a/lib/librte_eal/common/include/rte_eal.h b/lib/librte_eal/common/include/rte_eal.h
index c698b3155..8de5d69e8 100644
--- a/lib/librte_eal/common/include/rte_eal.h
+++ b/lib/librte_eal/common/include/rte_eal.h
@@ -502,11 +502,13 @@  const char * __rte_experimental
 rte_eal_mbuf_user_pool_ops(void);
 
 /**
+ * @deprecated
  * Get default pool ops name for mbuf
  *
  * @return
  *   returns default pool ops name.
  */
+__rte_deprecated
 const char *
 rte_eal_mbuf_default_mempool_ops(void);