[PATCH 03/54] net/bnxt: enable vector mode processing
Manish Kurup
manish.kurup at broadcom.com
Tue Sep 30 02:35:13 CEST 2025
From: Kishore Padmanabha <kishore.padmanabha at broadcom.com>
When representors are not enabled, then truflow could be run in
vector mode processing to achieve higher throughput. Added logic
to check representors to enable or disable vector mode.
Signed-off-by: Kishore Padmanabha <kishore.padmanabha at broadcom.com>
Reviewed-by: Shuanglin Wang <shuanglin.wang at broadcom.com>
---
drivers/net/bnxt/bnxt.h | 7 +++
drivers/net/bnxt/bnxt_ethdev.c | 88 +++++++++++++++++++++++++++++++++-
2 files changed, 93 insertions(+), 2 deletions(-)
diff --git a/drivers/net/bnxt/bnxt.h b/drivers/net/bnxt/bnxt.h
index c9fdd36d3e..72ac66b0db 100644
--- a/drivers/net/bnxt/bnxt.h
+++ b/drivers/net/bnxt/bnxt.h
@@ -852,6 +852,13 @@ struct bnxt {
((bp)->flags2 & BNXT_FLAGS2_MULTIROOT_EN)
#define BNXT_FLAGS2_COMPRESSED_RX_CQE BIT(5)
+#define BNXT_FLAGS2_USE_MPC BIT(6)
+#define BNXT_USE_MPC(bp) \
+ ((bp)->flags2 & BNXT_FLAGS2_USE_MPC)
+#define BNXT_FLAGS2_REP_MODE BIT(7)
+#define BNXT_REP_MODE_EN(bp) \
+ ((bp)->flags2 & BNXT_FLAGS2_REP_MODE)
+
uint32_t fw_cap;
#define BNXT_FW_CAP_HOT_RESET BIT(0)
#define BNXT_FW_CAP_IF_CHANGE BIT(1)
diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index cb0d99b9ee..0836be3b1e 100644
--- a/drivers/net/bnxt/bnxt_ethdev.c
+++ b/drivers/net/bnxt/bnxt_ethdev.c
@@ -107,6 +107,8 @@ static const struct rte_pci_id bnxt_pci_id_map[] = {
#define BNXT_DEVARG_IEEE_1588 "ieee-1588"
#define BNXT_DEVARG_CQE_MODE "cqe-mode"
#define BNXT_DEVARG_MPC "mpc"
+#define BNXT_DEVARG_SCALAR_MODE "scalar-mode"
+#define BNXT_DEVARD_APP_INST_ID "app-instance-id"
static const char *const bnxt_dev_args[] = {
BNXT_DEVARG_REPRESENTOR,
@@ -122,6 +124,8 @@ static const char *const bnxt_dev_args[] = {
BNXT_DEVARG_IEEE_1588,
BNXT_DEVARG_CQE_MODE,
BNXT_DEVARG_MPC,
+ BNXT_DEVARG_SCALAR_MODE,
+ BNXT_DEVARD_APP_INST_ID,
NULL
};
@@ -1447,7 +1451,7 @@ bnxt_receive_function(struct rte_eth_dev *eth_dev)
* asynchronous completions and receive completions can be placed in
* the same completion ring.
*/
- if ((BNXT_TRUFLOW_EN(bp) && !BNXT_CHIP_P7(bp)) ||
+ if ((BNXT_TRUFLOW_EN(bp) && BNXT_REP_MODE_EN(bp)) ||
!BNXT_NUM_ASYNC_CPR(bp))
goto use_scalar_rx;
@@ -1523,7 +1527,7 @@ bnxt_transmit_function(__rte_unused struct rte_eth_dev *eth_dev)
*/
if (eth_dev->data->scattered_rx ||
(offloads & ~RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE) ||
- (BNXT_TRUFLOW_EN(bp) && !BNXT_CHIP_P7(bp)) ||
+ (BNXT_TRUFLOW_EN(bp) && BNXT_REP_MODE_EN(bp)) ||
bp->ieee_1588)
goto use_scalar_tx;
@@ -6385,6 +6389,64 @@ bnxt_parse_devarg_rep_fc_f2r(__rte_unused const char *key,
return 0;
}
+static int
+bnxt_parse_devarg_representor_mode(__rte_unused const char *key,
+ const char *value, void *opaque_arg)
+{
+ struct bnxt *bp = opaque_arg;
+ unsigned long rep;
+ char *end = NULL;
+
+ if (!value || !opaque_arg) {
+ PMD_DRV_LOG_LINE(ERR,
+ "Invalid param passed to rep mode in devargs");
+ return -EINVAL;
+ }
+
+ rep = strtoul(value, &end, 10);
+ if (end == NULL || *end != '\0' ||
+ (rep == ULONG_MAX && errno == ERANGE)) {
+ PMD_DRV_LOG_LINE(ERR,
+ "Invalid param passed to rep mode in devargs");
+ return -EINVAL;
+ }
+
+ if (rep > 0)
+ bp->flags2 |= BNXT_FLAGS2_REP_MODE;
+ PMD_DRV_LOG_LINE(INFO, "representor feature enabled");
+
+ return 0;
+}
+
+static int
+bnxt_parse_devarg_scalar_mode(__rte_unused const char *key,
+ const char *value, void *opaque_arg)
+{
+ struct bnxt *bp = opaque_arg;
+ unsigned long rep;
+ char *end = NULL;
+
+ if (!value || !opaque_arg) {
+ PMD_DRV_LOG_LINE(ERR,
+ "Invalid param passed to scalar mode in devargs");
+ return -EINVAL;
+ }
+
+ rep = strtoul(value, &end, 10);
+ if (end == NULL || *end != '\0' ||
+ (rep == ULONG_MAX && errno == ERANGE)) {
+ PMD_DRV_LOG_LINE(ERR,
+ "Invalid param passed to scalar mode in devargs");
+ return -EINVAL;
+ }
+
+ if (rep > 0)
+ bp->flags2 |= BNXT_FLAGS2_REP_MODE;
+ PMD_DRV_LOG_LINE(INFO, "Scalar mode enabled");
+
+ return 0;
+}
+
static int
bnxt_parse_dev_args(struct bnxt *bp, struct rte_devargs *devargs)
{
@@ -6445,6 +6507,28 @@ bnxt_parse_dev_args(struct bnxt *bp, struct rte_devargs *devargs)
rte_kvargs_process(kvlist, BNXT_DEVARG_CQE_MODE,
bnxt_parse_devarg_cqe_mode, bp);
+ /*
+ * Handler for "representor" devarg.
+ * Invoked as for ex: "-a 000:00:0d.0,representor=1"
+ */
+ rte_kvargs_process(kvlist, BNXT_DEVARG_REPRESENTOR,
+ bnxt_parse_devarg_representor_mode, bp);
+
+ /*
+ * Handler for "scalar-mode" devarg.
+ * Invoked as for ex: "-a 000:00:0d.0,scalar-mode=1"
+ */
+ rte_kvargs_process(kvlist, BNXT_DEVARG_SCALAR_MODE,
+ bnxt_parse_devarg_scalar_mode, bp);
+
+ /*
+ * Handler for "app-instance-id" devarg.
+ * Invoked as for ex: "-a 000:00:0d.0,app-instance-id=1"
+ * This argument is required for enabling truflow hot upgrade feature.
+ */
+ rte_kvargs_process(kvlist, BNXT_DEVARD_APP_INST_ID,
+ bnxt_parse_devarg_app_instance_id, bp);
+
rte_kvargs_free(kvlist);
return ret;
}
--
2.39.5 (Apple Git-154)
More information about the dev
mailing list