diff mbox series

[v3,1/1] bbdev: removing unnecessary symbols from version map

Message ID 20240827230648.3943434-2-nicolas.chautru@intel.com (mailing list archive)
State Awaiting Upstream
Delegated to: Maxime Coquelin
Headers
Series bbdev: removing unnecessaray symbols |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/github-robot: build success github build: passed
ci/intel-Functional success Functional PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-marvell-Functional success Functional Testing PASS
ci/iol-sample-apps-testing success Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-compile-amd64-testing success Testing PASS
ci/iol-compile-arm64-testing success Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS

Commit Message

Chautru, Nicolas Aug. 27, 2024, 11:06 p.m. UTC 
A number of inline functions should not be in the
version map since ABI versioning would be irrelevant.

Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
---
 lib/bbdev/rte_bbdev.h    |  1 -
 lib/bbdev/rte_bbdev_op.h |  2 --
 lib/bbdev/version.map    | 24 +-----------------------
 3 files changed, 1 insertion(+), 26 deletions(-)

Comments

Maxime Coquelin Sept. 13, 2024, 1:51 p.m. UTC | #1 
On 8/28/24 01:06, Nicolas Chautru wrote:
> A number of inline functions should not be in the
> version map since ABI versioning would be irrelevant.
> 
> Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
> ---
>   lib/bbdev/rte_bbdev.h    |  1 -
>   lib/bbdev/rte_bbdev_op.h |  2 --
>   lib/bbdev/version.map    | 24 +-----------------------
>   3 files changed, 1 insertion(+), 26 deletions(-)
> 

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime
Hemant Agrawal Sept. 18, 2024, 5:58 a.m. UTC | #2 
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
Maxime Coquelin Sept. 19, 2024, 8:26 a.m. UTC | #3 
On 8/28/24 01:06, Nicolas Chautru wrote:
> A number of inline functions should not be in the
> version map since ABI versioning would be irrelevant.
> 
> Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
> ---
>   lib/bbdev/rte_bbdev.h    |  1 -
>   lib/bbdev/rte_bbdev_op.h |  2 --
>   lib/bbdev/version.map    | 24 +-----------------------
>   3 files changed, 1 insertion(+), 26 deletions(-)
> 

Applied to next-baseband/for-main.

Thanks,
Maxime
diff mbox series

Patch

diff --git a/lib/bbdev/rte_bbdev.h b/lib/bbdev/rte_bbdev.h
index 25514c58ac..bd49a0a304 100644
--- a/lib/bbdev/rte_bbdev.h
+++ b/lib/bbdev/rte_bbdev.h
@@ -897,7 +897,6 @@  rte_bbdev_dequeue_fft_ops(uint16_t dev_id, uint16_t queue_id,
  *   The number of operations actually dequeued (this is the number of entries
  *   copied into the @p ops array).
  */
-__rte_experimental
 static inline uint16_t
 rte_bbdev_dequeue_mldts_ops(uint16_t dev_id, uint16_t queue_id,
 		struct rte_bbdev_mldts_op **ops, uint16_t num_ops)
diff --git a/lib/bbdev/rte_bbdev_op.h b/lib/bbdev/rte_bbdev_op.h
index 459631d0d0..5b862c13a6 100644
--- a/lib/bbdev/rte_bbdev_op.h
+++ b/lib/bbdev/rte_bbdev_op.h
@@ -1159,7 +1159,6 @@  rte_bbdev_fft_op_alloc_bulk(struct rte_mempool *mempool,
  *   - 0 on success.
  *   - EINVAL if invalid mempool is provided.
  */
-__rte_experimental
 static inline int
 rte_bbdev_mldts_op_alloc_bulk(struct rte_mempool *mempool,
 		struct rte_bbdev_mldts_op **ops, uint16_t num_ops)
@@ -1236,7 +1235,6 @@  rte_bbdev_fft_op_free_bulk(struct rte_bbdev_fft_op **ops, unsigned int num_ops)
  * @param num_ops
  *   Number of structures
  */
-__rte_experimental
 static inline void
 rte_bbdev_mldts_op_free_bulk(struct rte_bbdev_mldts_op **ops, unsigned int num_ops)
 {
diff --git a/lib/bbdev/version.map b/lib/bbdev/version.map
index e0d82ff752..2a5baacd90 100644
--- a/lib/bbdev/version.map
+++ b/lib/bbdev/version.map
@@ -6,21 +6,9 @@  DPDK_25 {
 	rte_bbdev_callback_unregister;
 	rte_bbdev_close;
 	rte_bbdev_count;
-	rte_bbdev_dec_op_alloc_bulk;
-	rte_bbdev_dec_op_free_bulk;
-	rte_bbdev_dequeue_dec_ops;
-	rte_bbdev_dequeue_enc_ops;
-	rte_bbdev_dequeue_fft_ops;
 	rte_bbdev_device_status_str;
 	rte_bbdev_devices;
-	rte_bbdev_enc_op_alloc_bulk;
-	rte_bbdev_enc_op_free_bulk;
-	rte_bbdev_enqueue_dec_ops;
-	rte_bbdev_enqueue_enc_ops;
-	rte_bbdev_enqueue_fft_ops;
 	rte_bbdev_enqueue_status_str;
-	rte_bbdev_fft_op_alloc_bulk;
-	rte_bbdev_fft_op_free_bulk;
 	rte_bbdev_find_next;
 	rte_bbdev_get_named_dev;
 	rte_bbdev_info_get;
@@ -44,14 +32,4 @@  DPDK_25 {
 	rte_bbdev_stop;
 
 	local: *;
-};
-
-EXPERIMENTAL {
-	global:
-
-	# added in 23.11
-	rte_bbdev_dequeue_mldts_ops;
-	rte_bbdev_enqueue_mldts_ops;
-	rte_bbdev_mldts_op_alloc_bulk;
-	rte_bbdev_mldts_op_free_bulk;
-};
+};
\ No newline at end of file