From: Ciara Loftus <ciara.loftus@intel.com>
To: dev@dpdk.org
Cc: Ciara Loftus <ciara.loftus@intel.com>,
Bruce Richardson <bruce.richardson@intel.com>
Subject: [PATCH v3 08/15] net/i40e: use the new common vector capability function
Date: Mon, 18 Aug 2025 10:59:07 +0000 [thread overview]
Message-ID: <20250818105914.169732-9-ciara.loftus@intel.com> (raw)
In-Reply-To: <20250818105914.169732-1-ciara.loftus@intel.com>
Use the new function for determining the maximum simd bitwidth in
the i40e driver.
Signed-off-by: Ciara Loftus <ciara.loftus@intel.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
---
v3:
* removed unnecessary #ifdef CC_AVX512_SUPPORT
* added error log for case when tx simd width is invalid
---
drivers/net/intel/i40e/i40e_ethdev.h | 5 +-
drivers/net/intel/i40e/i40e_rxtx.c | 66 +++++-----------------
drivers/net/intel/i40e/i40e_rxtx.h | 1 +
drivers/net/intel/i40e/i40e_rxtx_vec_sse.c | 6 ++
4 files changed, 23 insertions(+), 55 deletions(-)
diff --git a/drivers/net/intel/i40e/i40e_ethdev.h b/drivers/net/intel/i40e/i40e_ethdev.h
index c38cb5f340..3fca089d6c 100644
--- a/drivers/net/intel/i40e/i40e_ethdev.h
+++ b/drivers/net/intel/i40e/i40e_ethdev.h
@@ -15,6 +15,7 @@
#include <rte_flow_driver.h>
#include <rte_tm_driver.h>
#include "rte_pmd_i40e.h"
+#include <rte_vect.h>
#include "base/i40e_register.h"
#include "base/i40e_type.h"
@@ -1279,9 +1280,7 @@ struct i40e_adapter {
/* For RSS reta table update */
uint8_t rss_reta_updated;
- /* used only on x86, zero on other architectures */
- bool tx_use_avx2;
- bool tx_use_avx512;
+ enum rte_vect_max_simd tx_simd_width;
};
/**
diff --git a/drivers/net/intel/i40e/i40e_rxtx.c b/drivers/net/intel/i40e/i40e_rxtx.c
index c89359e625..76d90be931 100644
--- a/drivers/net/intel/i40e/i40e_rxtx.c
+++ b/drivers/net/intel/i40e/i40e_rxtx.c
@@ -3284,32 +3284,6 @@ i40e_recycle_rxq_info_get(struct rte_eth_dev *dev, uint16_t queue_id,
}
}
-#ifdef RTE_ARCH_X86
-static inline bool
-get_avx_supported(bool request_avx512)
-{
- if (request_avx512) {
- if (rte_vect_get_max_simd_bitwidth() >= RTE_VECT_SIMD_512 &&
- rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX512F) == 1 &&
- rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX512BW) == 1)
-#ifdef CC_AVX512_SUPPORT
- return true;
-#else
- PMD_DRV_LOG(NOTICE,
- "AVX512 is not supported in build env");
- return false;
-#endif
- } else {
- if (rte_vect_get_max_simd_bitwidth() >= RTE_VECT_SIMD_256 &&
- rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX2) == 1 &&
- rte_cpu_get_flag_enabled(RTE_CPUFLAG_AVX512F) == 1)
- return true;
- }
-
- return false;
-}
-#endif /* RTE_ARCH_X86 */
-
static const struct {
eth_rx_burst_t pkt_burst;
const char *info;
@@ -3351,7 +3325,7 @@ i40e_set_rx_function(struct rte_eth_dev *dev)
* conditions to be met and Rx Bulk Allocation should be allowed.
*/
#ifdef RTE_ARCH_X86
- bool rx_use_avx512 = false, rx_use_avx2 = false;
+ enum rte_vect_max_simd rx_simd_width = i40e_get_max_simd_bitwidth();
#endif
if (i40e_rx_vec_dev_conf_condition_check(dev) || !ad->rx_bulk_alloc_allowed) {
PMD_INIT_LOG(DEBUG, "Port[%d] doesn't meet"
@@ -3370,35 +3344,25 @@ i40e_set_rx_function(struct rte_eth_dev *dev)
break;
}
}
-#ifdef RTE_ARCH_X86
- rx_use_avx512 = get_avx_supported(1);
-
- if (!rx_use_avx512)
- rx_use_avx2 = get_avx_supported(0);
-#endif
}
if (ad->rx_vec_allowed && rte_vect_get_max_simd_bitwidth() >= RTE_VECT_SIMD_128) {
#ifdef RTE_ARCH_X86
if (dev->data->scattered_rx) {
- if (rx_use_avx512) {
-#ifdef CC_AVX512_SUPPORT
+ if (rx_simd_width == RTE_VECT_SIMD_512) {
ad->rx_func_type = I40E_RX_AVX512_SCATTERED;
-#endif
} else {
- ad->rx_func_type = rx_use_avx2 ?
+ ad->rx_func_type = (rx_simd_width == RTE_VECT_SIMD_256) ?
I40E_RX_AVX2_SCATTERED :
I40E_RX_SCATTERED;
dev->recycle_rx_descriptors_refill =
i40e_recycle_rx_descriptors_refill_vec;
}
} else {
- if (rx_use_avx512) {
-#ifdef CC_AVX512_SUPPORT
+ if (rx_simd_width == RTE_VECT_SIMD_512) {
ad->rx_func_type = I40E_RX_AVX512;
-#endif
} else {
- ad->rx_func_type = rx_use_avx2 ?
+ ad->rx_func_type = (rx_simd_width == RTE_VECT_SIMD_256) ?
I40E_RX_AVX2 :
I40E_RX_SSE;
dev->recycle_rx_descriptors_refill =
@@ -3509,8 +3473,7 @@ i40e_set_tx_function(struct rte_eth_dev *dev)
if (rte_eal_process_type() == RTE_PROC_PRIMARY) {
#ifdef RTE_ARCH_X86
- ad->tx_use_avx2 = false;
- ad->tx_use_avx512 = false;
+ ad->tx_simd_width = i40e_get_max_simd_bitwidth();
#endif
if (ad->tx_vec_allowed) {
for (i = 0; i < dev->data->nb_tx_queues; i++) {
@@ -3522,12 +3485,6 @@ i40e_set_tx_function(struct rte_eth_dev *dev)
break;
}
}
-#ifdef RTE_ARCH_X86
- ad->tx_use_avx512 = get_avx_supported(1);
-
- if (!ad->tx_use_avx512)
- ad->tx_use_avx2 = get_avx_supported(0);
-#endif
}
}
@@ -3537,17 +3494,22 @@ i40e_set_tx_function(struct rte_eth_dev *dev)
if (ad->tx_simple_allowed) {
if (ad->tx_vec_allowed) {
#ifdef RTE_ARCH_X86
- if (ad->tx_use_avx512) {
+ if (ad->tx_simd_width == RTE_VECT_SIMD_512) {
#ifdef CC_AVX512_SUPPORT
PMD_DRV_LOG(NOTICE, "Using AVX512 Vector Tx (port %d).",
dev->data->port_id);
dev->tx_pkt_burst = i40e_xmit_pkts_vec_avx512;
+#else
+ PMD_DRV_LOG(ERR, "Invalid Tx SIMD width reported, defaulting to "
+ "using scalar Tx (port %d).",
+ dev->data->port_id);
+ dev->tx_pkt_burst = i40e_xmit_pkts;
#endif
} else {
PMD_INIT_LOG(DEBUG, "Using %sVector Tx (port %d).",
- ad->tx_use_avx2 ? "avx2 " : "",
+ ad->tx_simd_width == RTE_VECT_SIMD_256 ? "avx2 " : "",
dev->data->port_id);
- dev->tx_pkt_burst = ad->tx_use_avx2 ?
+ dev->tx_pkt_burst = ad->tx_simd_width == RTE_VECT_SIMD_256 ?
i40e_xmit_pkts_vec_avx2 :
i40e_xmit_pkts_vec;
dev->recycle_tx_mbufs_reuse = i40e_recycle_tx_mbufs_reuse_vec;
diff --git a/drivers/net/intel/i40e/i40e_rxtx.h b/drivers/net/intel/i40e/i40e_rxtx.h
index 984532c507..b867e18daf 100644
--- a/drivers/net/intel/i40e/i40e_rxtx.h
+++ b/drivers/net/intel/i40e/i40e_rxtx.h
@@ -167,6 +167,7 @@ uint16_t i40e_recv_scattered_pkts_vec_avx512(void *rx_queue,
uint16_t i40e_xmit_pkts_vec_avx512(void *tx_queue,
struct rte_mbuf **tx_pkts,
uint16_t nb_pkts);
+enum rte_vect_max_simd i40e_get_max_simd_bitwidth(void);
/* For each value it means, datasheet of hardware can tell more details
*
diff --git a/drivers/net/intel/i40e/i40e_rxtx_vec_sse.c b/drivers/net/intel/i40e/i40e_rxtx_vec_sse.c
index 15cf07e548..c035408dcc 100644
--- a/drivers/net/intel/i40e/i40e_rxtx_vec_sse.c
+++ b/drivers/net/intel/i40e/i40e_rxtx_vec_sse.c
@@ -715,3 +715,9 @@ i40e_rx_vec_dev_conf_condition_check(struct rte_eth_dev *dev)
{
return i40e_rx_vec_dev_conf_condition_check_default(dev);
}
+
+enum rte_vect_max_simd
+i40e_get_max_simd_bitwidth(void)
+{
+ return ci_get_x86_max_simd_bitwidth();
+}
--
2.34.1
next prev parent reply other threads:[~2025-08-18 11:00 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-18 10:58 [PATCH v3 00/15] net/intel: rx path selection simplification Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 01/15] net/ice: use the same Rx path across process types Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 02/15] net/iavf: rename Rx/Tx function type variables Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 03/15] net/iavf: use the same Rx path across process types Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 04/15] net/i40e: " Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 05/15] net/intel: introduce common vector capability function Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 06/15] net/ice: use the new " Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 07/15] net/iavf: " Ciara Loftus
2025-08-18 10:59 ` Ciara Loftus [this message]
2025-08-18 10:59 ` [PATCH v3 09/15] net/iavf: remove redundant field from iavf adapter struct Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 10/15] net/ice: remove unsupported Rx offload Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 11/15] net/iavf: reorder enum of Rx function types Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 12/15] net/intel: introduce infrastructure for Rx path selection Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 13/15] net/ice: use the common Rx path selection infrastructure Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 14/15] net/iavf: " Ciara Loftus
2025-08-18 10:59 ` [PATCH v3 15/15] net/i40e: " Ciara Loftus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250818105914.169732-9-ciara.loftus@intel.com \
--to=ciara.loftus@intel.com \
--cc=bruce.richardson@intel.com \
--cc=dev@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).