diff mbox series

[v6,13/33] net/ixgbe: rename vector txq free mbufs

Message ID	8b66b094e3a32e39c031c1bdeb7d0ca7808c82ce.1749483382.git.anatoly.burakov@intel.com (mailing list archive)
State	Superseded
Delegated to:	Bruce Richardson
Headers	From: Anatoly Burakov <anatoly.burakov@intel.com> To: dev@dpdk.org, Vladimir Medvedkin <vladimir.medvedkin@intel.com> Cc: bruce.richardson@intel.com Subject: [PATCH v6 13/33] net/ixgbe: rename vector txq free mbufs Date: Mon, 9 Jun 2025 16:37:11 +0100 Message-ID: <8b66b094e3a32e39c031c1bdeb7d0ca7808c82ce.1749483382.git.anatoly.burakov@intel.com> In-Reply-To: <cover.1749483381.git.anatoly.burakov@intel.com> <cover.1749483381.git.anatoly.burakov@intel.com> References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: list Errors-To: dev-bounces@dpdk.org
Series	Intel PMD drivers Rx cleanup \| [v6,00/33] Intel PMD drivers Rx cleanup [v6,01/33] net/ixgbe: remove unused field in Rx queue struct [v6,02/33] net/iavf: make IPsec stats dynamically allocated [v6,03/33] net/ixgbe: match variable names to other drivers [v6,04/33] net/i40e: match variable name to other drivers [v6,05/33] net/ice: match variable name to other drivers [v6,06/33] net/i40e: rename 16-byte descriptor define [v6,07/33] net/ice: rename 16-byte descriptor define [v6,08/33] net/iavf: remove 16-byte descriptor define [v6,09/33] net/ixgbe: simplify packet type support check [v6,10/33] net/ixgbe: adjust indentation [v6,11/33] net/ixgbe: remove unnecessary platform checks [v6,12/33] net/ixgbe: decouple scalar and vec rxq free mbufs [v6,13/33] net/ixgbe: rename vector txq free mbufs [v6,14/33] net/ixgbe: refactor vector common code [v6,15/33] net/ixgbe: move vector Rx/Tx code to vec common [v6,16/33] net/ixgbe: simplify vector PMD compilation [v6,17/33] net/ixgbe: replace always-true check [v6,18/33] net/ixgbe: add a desc done function [v6,19/33] net/ixgbe: clean up definitions [v6,20/33] net/i40e: clean up definitions [v6,21/33] net/ice: clean up definitions [v6,22/33] net/iavf: clean up definitions [v6,23/33] net/ixgbe: create common Rx queue structure [v6,24/33] net/i40e: use the common Rx queue structure [v6,25/33] net/ice: use the common Rx queue structure [v6,26/33] net/iavf: use the common Rx queue structure [v6,27/33] net/intel: generalize vectorized Rx rearm [v6,28/33] net/i40e: use common Rx rearm code [v6,29/33] net/iavf: use common Rx rearm code [v6,30/33] net/ixgbe: use common Rx rearm code [v6,31/33] net/intel: support wider x86 vectors for Rx rearm [v6,32/33] net/intel: add common Rx mbuf recycle [v6,33/33] net/intel: add common Tx mbuf recycle

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Burakov, Anatoly June 9, 2025, 3:37 p.m. UTC

There is no reason why scalar and vector implementations of tx free mbufs
function have to share the same name, in fact it is counter-productive
because including `ixgbe_rxtx_vec_common.h` from `ixgbe_rxtx.c` will cause
naming clashes. Rename the vector implementation to avoid naming clashes.

Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
---

Notes:
    v5:
    - Add this patch

 drivers/net/intel/ixgbe/ixgbe_rxtx_vec_common.h | 2 +-
 drivers/net/intel/ixgbe/ixgbe_rxtx_vec_neon.c   | 2 +-
 drivers/net/intel/ixgbe/ixgbe_rxtx_vec_sse.c    | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff mbox series

Patch

diff --git a/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_common.h b/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_common.h
index 018010820f..9e1abf4449 100644
--- a/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_common.h
+++ b/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_common.h
@@ -12,7 +12,7 @@ 
 #include "ixgbe_rxtx.h"
 
 static __rte_always_inline int
-ixgbe_tx_free_bufs(struct ci_tx_queue *txq)
+ixgbe_tx_free_bufs_vec(struct ci_tx_queue *txq)
 {
 	struct ci_tx_entry_vec *txep;
 	uint32_t status;
diff --git a/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_neon.c b/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_neon.c
index f8916d44e8..3a0b2909a7 100644
--- a/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_neon.c
+++ b/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_neon.c
@@ -583,7 +583,7 @@  ixgbe_xmit_fixed_burst_vec(void *tx_queue, struct rte_mbuf **tx_pkts,
 	nb_pkts = RTE_MIN(nb_pkts, txq->tx_rs_thresh);
 
 	if (txq->nb_tx_free < txq->tx_free_thresh)
-		ixgbe_tx_free_bufs(txq);
+		ixgbe_tx_free_bufs_vec(txq);
 
 	nb_commit = nb_pkts = (uint16_t)RTE_MIN(txq->nb_tx_free, nb_pkts);
 	if (unlikely(nb_pkts == 0))
diff --git a/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_sse.c b/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_sse.c
index 9417e5b11f..1e063bb243 100644
--- a/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_sse.c
+++ b/drivers/net/intel/ixgbe/ixgbe_rxtx_vec_sse.c
@@ -703,7 +703,7 @@  ixgbe_xmit_fixed_burst_vec(void *tx_queue, struct rte_mbuf **tx_pkts,
 	nb_pkts = RTE_MIN(nb_pkts, txq->tx_rs_thresh);
 
 	if (txq->nb_tx_free < txq->tx_free_thresh)
-		ixgbe_tx_free_bufs(txq);
+		ixgbe_tx_free_bufs_vec(txq);
 
 	nb_commit = nb_pkts = (uint16_t)RTE_MIN(txq->nb_tx_free, nb_pkts);
 	if (unlikely(nb_pkts == 0))