[RFC PATCH 02/14] net/iavf: rename Rx/Tx function type variables
Ciara Loftus
ciara.loftus at intel.com
Fri Jul 25 14:49:07 CEST 2025
Rename variables from burst_type to func_type to better reflect
the information the variables are storing.
Signed-off-by: Ciara Loftus <ciara.loftus at intel.com>
---
drivers/net/intel/iavf/iavf.h | 8 +-
drivers/net/intel/iavf/iavf_rxtx.c | 114 ++++++++++++++---------------
2 files changed, 61 insertions(+), 61 deletions(-)
diff --git a/drivers/net/intel/iavf/iavf.h b/drivers/net/intel/iavf/iavf.h
index f81c939c96..dacfb92d5f 100644
--- a/drivers/net/intel/iavf/iavf.h
+++ b/drivers/net/intel/iavf/iavf.h
@@ -320,7 +320,7 @@ struct iavf_devargs {
struct iavf_security_ctx;
-enum iavf_rx_burst_type {
+enum iavf_rx_func_type {
IAVF_RX_DISABLED,
IAVF_RX_DEFAULT,
IAVF_RX_FLEX_RXD,
@@ -349,7 +349,7 @@ enum iavf_rx_burst_type {
IAVF_RX_AVX512_SCATTERED_FLEX_RXD_OFFLOAD,
};
-enum iavf_tx_burst_type {
+enum iavf_tx_func_type {
IAVF_TX_DISABLED,
IAVF_TX_DEFAULT,
IAVF_TX_SSE,
@@ -381,8 +381,8 @@ struct iavf_adapter {
bool stopped;
bool closed;
bool no_poll;
- enum iavf_rx_burst_type rx_burst_type;
- enum iavf_tx_burst_type tx_burst_type;
+ enum iavf_rx_func_type rx_func_type;
+ enum iavf_tx_func_type tx_func_type;
uint16_t fdir_ref_cnt;
struct iavf_devargs devargs;
};
diff --git a/drivers/net/intel/iavf/iavf_rxtx.c b/drivers/net/intel/iavf/iavf_rxtx.c
index 7033a74610..57f7a4b67d 100644
--- a/drivers/net/intel/iavf/iavf_rxtx.c
+++ b/drivers/net/intel/iavf/iavf_rxtx.c
@@ -3826,14 +3826,14 @@ iavf_recv_pkts_no_poll(void *rx_queue, struct rte_mbuf **rx_pkts,
uint16_t nb_pkts)
{
struct ci_rx_queue *rxq = rx_queue;
- enum iavf_rx_burst_type rx_burst_type;
+ enum iavf_rx_func_type rx_func_type;
if (!rxq->iavf_vsi || rxq->iavf_vsi->adapter->no_poll)
return 0;
- rx_burst_type = rxq->iavf_vsi->adapter->rx_burst_type;
+ rx_func_type = rxq->iavf_vsi->adapter->rx_func_type;
- return iavf_rx_pkt_burst_ops[rx_burst_type].pkt_burst(rx_queue,
+ return iavf_rx_pkt_burst_ops[rx_func_type].pkt_burst(rx_queue,
rx_pkts, nb_pkts);
}
@@ -3842,14 +3842,14 @@ iavf_xmit_pkts_no_poll(void *tx_queue, struct rte_mbuf **tx_pkts,
uint16_t nb_pkts)
{
struct ci_tx_queue *txq = tx_queue;
- enum iavf_tx_burst_type tx_burst_type;
+ enum iavf_tx_func_type tx_func_type;
if (!txq->iavf_vsi || txq->iavf_vsi->adapter->no_poll)
return 0;
- tx_burst_type = txq->iavf_vsi->adapter->tx_burst_type;
+ tx_func_type = txq->iavf_vsi->adapter->tx_func_type;
- return iavf_tx_pkt_burst_ops[tx_burst_type].pkt_burst(tx_queue,
+ return iavf_tx_pkt_burst_ops[tx_func_type].pkt_burst(tx_queue,
tx_pkts, nb_pkts);
}
@@ -3866,8 +3866,8 @@ iavf_xmit_pkts_check(void *tx_queue, struct rte_mbuf **tx_pkts,
bool pkt_error = false;
struct ci_tx_queue *txq = tx_queue;
struct iavf_adapter *adapter = txq->iavf_vsi->adapter;
- enum iavf_tx_burst_type tx_burst_type =
- txq->iavf_vsi->adapter->tx_burst_type;
+ enum iavf_tx_func_type tx_func_type =
+ txq->iavf_vsi->adapter->tx_func_type;
for (idx = 0; idx < nb_pkts; idx++) {
mb = tx_pkts[idx];
@@ -3934,7 +3934,7 @@ iavf_xmit_pkts_check(void *tx_queue, struct rte_mbuf **tx_pkts,
return 0;
}
- return iavf_tx_pkt_burst_ops[tx_burst_type].pkt_burst(tx_queue, tx_pkts, good_pkts);
+ return iavf_tx_pkt_burst_ops[tx_func_type].pkt_burst(tx_queue, tx_pkts, good_pkts);
}
/* choose rx function*/
@@ -3944,7 +3944,7 @@ iavf_set_rx_function(struct rte_eth_dev *dev)
struct iavf_adapter *adapter =
IAVF_DEV_PRIVATE_TO_ADAPTER(dev->data->dev_private);
struct iavf_info *vf = IAVF_DEV_PRIVATE_TO_VF(dev->data->dev_private);
- enum iavf_rx_burst_type rx_burst_type;
+ enum iavf_rx_func_type rx_func_type;
int no_poll_on_link_down = adapter->devargs.no_poll_on_link_down;
int i;
struct ci_rx_queue *rxq;
@@ -4015,42 +4015,42 @@ iavf_set_rx_function(struct rte_eth_dev *dev)
}
}
if (use_flex) {
- rx_burst_type = IAVF_RX_SSE_SCATTERED_FLEX_RXD;
+ rx_func_type = IAVF_RX_SSE_SCATTERED_FLEX_RXD;
if (use_avx2) {
if (check_ret == IAVF_VECTOR_PATH)
- rx_burst_type =
+ rx_func_type =
IAVF_RX_AVX2_SCATTERED_FLEX_RXD;
else
- rx_burst_type =
+ rx_func_type =
IAVF_RX_AVX2_SCATTERED_FLEX_RXD_OFFLOAD;
}
#ifdef CC_AVX512_SUPPORT
if (use_avx512) {
if (check_ret == IAVF_VECTOR_PATH)
- rx_burst_type =
+ rx_func_type =
IAVF_RX_AVX512_SCATTERED_FLEX_RXD;
else
- rx_burst_type =
+ rx_func_type =
IAVF_RX_AVX512_SCATTERED_FLEX_RXD_OFFLOAD;
}
#endif
} else {
- rx_burst_type = IAVF_RX_SSE_SCATTERED;
+ rx_func_type = IAVF_RX_SSE_SCATTERED;
if (use_avx2) {
if (check_ret == IAVF_VECTOR_PATH)
- rx_burst_type =
+ rx_func_type =
IAVF_RX_AVX2_SCATTERED;
else
- rx_burst_type =
+ rx_func_type =
IAVF_RX_AVX2_SCATTERED_OFFLOAD;
}
#ifdef CC_AVX512_SUPPORT
if (use_avx512) {
if (check_ret == IAVF_VECTOR_PATH)
- rx_burst_type =
+ rx_func_type =
IAVF_RX_AVX512_SCATTERED;
else
- rx_burst_type =
+ rx_func_type =
IAVF_RX_AVX512_SCATTERED_OFFLOAD;
}
#endif
@@ -4081,46 +4081,46 @@ iavf_set_rx_function(struct rte_eth_dev *dev)
}
}
if (use_flex) {
- rx_burst_type = IAVF_RX_SSE_FLEX_RXD;
+ rx_func_type = IAVF_RX_SSE_FLEX_RXD;
if (use_avx2) {
if (check_ret == IAVF_VECTOR_PATH)
- rx_burst_type = IAVF_RX_AVX2_FLEX_RXD;
+ rx_func_type = IAVF_RX_AVX2_FLEX_RXD;
else
- rx_burst_type = IAVF_RX_AVX2_FLEX_RXD_OFFLOAD;
+ rx_func_type = IAVF_RX_AVX2_FLEX_RXD_OFFLOAD;
}
#ifdef CC_AVX512_SUPPORT
if (use_avx512) {
if (check_ret == IAVF_VECTOR_PATH)
- rx_burst_type = IAVF_RX_AVX512_FLEX_RXD;
+ rx_func_type = IAVF_RX_AVX512_FLEX_RXD;
else
- rx_burst_type =
+ rx_func_type =
IAVF_RX_AVX512_FLEX_RXD_OFFLOAD;
}
#endif
} else {
- rx_burst_type = IAVF_RX_SSE;
+ rx_func_type = IAVF_RX_SSE;
if (use_avx2) {
if (check_ret == IAVF_VECTOR_PATH)
- rx_burst_type = IAVF_RX_AVX2;
+ rx_func_type = IAVF_RX_AVX2;
else
- rx_burst_type = IAVF_RX_AVX2_OFFLOAD;
+ rx_func_type = IAVF_RX_AVX2_OFFLOAD;
}
#ifdef CC_AVX512_SUPPORT
if (use_avx512) {
if (check_ret == IAVF_VECTOR_PATH)
- rx_burst_type = IAVF_RX_AVX512;
+ rx_func_type = IAVF_RX_AVX512;
else
- rx_burst_type = IAVF_RX_AVX512_OFFLOAD;
+ rx_func_type = IAVF_RX_AVX512_OFFLOAD;
}
#endif
}
}
if (no_poll_on_link_down) {
- adapter->rx_burst_type = rx_burst_type;
+ adapter->rx_func_type = rx_func_type;
dev->rx_pkt_burst = iavf_recv_pkts_no_poll;
} else {
- dev->rx_pkt_burst = iavf_rx_pkt_burst_ops[rx_burst_type].pkt_burst;
+ dev->rx_pkt_burst = iavf_rx_pkt_burst_ops[rx_func_type].pkt_burst;
}
return;
}
@@ -4136,13 +4136,13 @@ iavf_set_rx_function(struct rte_eth_dev *dev)
rxq = dev->data->rx_queues[i];
(void)iavf_rxq_vec_setup(rxq);
}
- rx_burst_type = IAVF_RX_SSE;
+ rx_func_type = IAVF_RX_SSE;
if (no_poll_on_link_down) {
- adapter->rx_burst_type = rx_burst_type;
+ adapter->rx_func_type = rx_func_type;
dev->rx_pkt_burst = iavf_recv_pkts_no_poll;
} else {
- dev->rx_pkt_burst = iavf_rx_pkt_burst_ops[rx_burst_type].pkt_burst;
+ dev->rx_pkt_burst = iavf_rx_pkt_burst_ops[rx_func_type].pkt_burst;
}
return;
}
@@ -4151,27 +4151,27 @@ iavf_set_rx_function(struct rte_eth_dev *dev)
PMD_DRV_LOG(DEBUG, "Using a Scattered Rx callback (port=%d).",
dev->data->port_id);
if (use_flex)
- rx_burst_type = IAVF_RX_SCATTERED_FLEX_RXD;
+ rx_func_type = IAVF_RX_SCATTERED_FLEX_RXD;
else
- rx_burst_type = IAVF_RX_SCATTERED;
+ rx_func_type = IAVF_RX_SCATTERED;
} else if (adapter->rx_bulk_alloc_allowed) {
PMD_DRV_LOG(DEBUG, "Using bulk Rx callback (port=%d).",
dev->data->port_id);
- rx_burst_type = IAVF_RX_BULK_ALLOC;
+ rx_func_type = IAVF_RX_BULK_ALLOC;
} else {
PMD_DRV_LOG(DEBUG, "Using Basic Rx callback (port=%d).",
dev->data->port_id);
if (use_flex)
- rx_burst_type = IAVF_RX_FLEX_RXD;
+ rx_func_type = IAVF_RX_FLEX_RXD;
else
- rx_burst_type = IAVF_RX_DEFAULT;
+ rx_func_type = IAVF_RX_DEFAULT;
}
if (no_poll_on_link_down) {
- adapter->rx_burst_type = rx_burst_type;
+ adapter->rx_func_type = rx_func_type;
dev->rx_pkt_burst = iavf_recv_pkts_no_poll;
} else {
- dev->rx_pkt_burst = iavf_rx_pkt_burst_ops[rx_burst_type].pkt_burst;
+ dev->rx_pkt_burst = iavf_rx_pkt_burst_ops[rx_func_type].pkt_burst;
}
}
@@ -4181,7 +4181,7 @@ iavf_set_tx_function(struct rte_eth_dev *dev)
{
struct iavf_adapter *adapter =
IAVF_DEV_PRIVATE_TO_ADAPTER(dev->data->dev_private);
- enum iavf_tx_burst_type tx_burst_type;
+ enum iavf_tx_func_type tx_func_type;
int mbuf_check = adapter->devargs.mbuf_check;
int no_poll_on_link_down = adapter->devargs.no_poll_on_link_down;
#ifdef RTE_ARCH_X86
@@ -4217,11 +4217,11 @@ iavf_set_tx_function(struct rte_eth_dev *dev)
if (use_sse) {
PMD_DRV_LOG(DEBUG, "Using Vector Tx (port %d).",
dev->data->port_id);
- tx_burst_type = IAVF_TX_SSE;
+ tx_func_type = IAVF_TX_SSE;
}
if (use_avx2) {
if (check_ret == IAVF_VECTOR_PATH) {
- tx_burst_type = IAVF_TX_AVX2;
+ tx_func_type = IAVF_TX_AVX2;
PMD_DRV_LOG(DEBUG, "Using AVX2 Vector Tx (port %d).",
dev->data->port_id);
} else if (check_ret == IAVF_VECTOR_CTX_OFFLOAD_PATH) {
@@ -4229,7 +4229,7 @@ iavf_set_tx_function(struct rte_eth_dev *dev)
"AVX2 does not support outer checksum offload.");
goto normal;
} else {
- tx_burst_type = IAVF_TX_AVX2_OFFLOAD;
+ tx_func_type = IAVF_TX_AVX2_OFFLOAD;
PMD_DRV_LOG(DEBUG, "Using AVX2 OFFLOAD Vector Tx (port %d).",
dev->data->port_id);
}
@@ -4237,19 +4237,19 @@ iavf_set_tx_function(struct rte_eth_dev *dev)
#ifdef CC_AVX512_SUPPORT
if (use_avx512) {
if (check_ret == IAVF_VECTOR_PATH) {
- tx_burst_type = IAVF_TX_AVX512;
+ tx_func_type = IAVF_TX_AVX512;
PMD_DRV_LOG(DEBUG, "Using AVX512 Vector Tx (port %d).",
dev->data->port_id);
} else if (check_ret == IAVF_VECTOR_OFFLOAD_PATH) {
- tx_burst_type = IAVF_TX_AVX512_OFFLOAD;
+ tx_func_type = IAVF_TX_AVX512_OFFLOAD;
PMD_DRV_LOG(DEBUG, "Using AVX512 OFFLOAD Vector Tx (port %d).",
dev->data->port_id);
} else if (check_ret == IAVF_VECTOR_CTX_PATH) {
- tx_burst_type = IAVF_TX_AVX512_CTX;
+ tx_func_type = IAVF_TX_AVX512_CTX;
PMD_DRV_LOG(DEBUG, "Using AVX512 CONTEXT Vector Tx (port %d).",
dev->data->port_id);
} else {
- tx_burst_type = IAVF_TX_AVX512_CTX_OFFLOAD;
+ tx_func_type = IAVF_TX_AVX512_CTX_OFFLOAD;
PMD_DRV_LOG(DEBUG, "Using AVX512 CONTEXT OFFLOAD Vector Tx (port %d).",
dev->data->port_id);
}
@@ -4264,13 +4264,13 @@ iavf_set_tx_function(struct rte_eth_dev *dev)
}
if (no_poll_on_link_down) {
- adapter->tx_burst_type = tx_burst_type;
+ adapter->tx_func_type = tx_func_type;
dev->tx_pkt_burst = iavf_xmit_pkts_no_poll;
} else if (mbuf_check) {
- adapter->tx_burst_type = tx_burst_type;
+ adapter->tx_func_type = tx_func_type;
dev->tx_pkt_burst = iavf_xmit_pkts_check;
} else {
- dev->tx_pkt_burst = iavf_tx_pkt_burst_ops[tx_burst_type].pkt_burst;
+ dev->tx_pkt_burst = iavf_tx_pkt_burst_ops[tx_func_type].pkt_burst;
}
return;
}
@@ -4279,16 +4279,16 @@ iavf_set_tx_function(struct rte_eth_dev *dev)
#endif
PMD_DRV_LOG(DEBUG, "Using Basic Tx callback (port=%d).",
dev->data->port_id);
- tx_burst_type = IAVF_TX_DEFAULT;
+ tx_func_type = IAVF_TX_DEFAULT;
if (no_poll_on_link_down) {
- adapter->tx_burst_type = tx_burst_type;
+ adapter->tx_func_type = tx_func_type;
dev->tx_pkt_burst = iavf_xmit_pkts_no_poll;
} else if (mbuf_check) {
- adapter->tx_burst_type = tx_burst_type;
+ adapter->tx_func_type = tx_func_type;
dev->tx_pkt_burst = iavf_xmit_pkts_check;
} else {
- dev->tx_pkt_burst = iavf_tx_pkt_burst_ops[tx_burst_type].pkt_burst;
+ dev->tx_pkt_burst = iavf_tx_pkt_burst_ops[tx_func_type].pkt_burst;
}
}
--
2.34.1
More information about the dev
mailing list