From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 2D2A3606A31E for ; Fri, 23 Oct 2026 13:44:04 +0000 (UTC) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0C32440679; Fri, 23 Oct 2026 15:43:56 +0200 (CEST) Received: from inva020.nxp.com (inva020.nxp.com [92.121.34.13]) by mails.dpdk.org (Postfix) with ESMTP id D83D94066C for ; Fri, 23 Oct 2026 15:43:53 +0200 (CEST) Received: from inva020.nxp.com (localhost [127.0.0.1]) by inva020.eu-rdc02.nxp.com (Postfix) with ESMTP id B003B1A000A; Fri, 23 Oct 2026 15:43:53 +0200 (CEST) Received: from aprdc01srsp001v.ap-rdc01.nxp.com (aprdc01srsp001v.ap-rdc01.nxp.com [165.114.16.16]) by inva020.eu-rdc02.nxp.com (Postfix) with ESMTP id 7968F1A0040; Fri, 23 Oct 2026 15:43:53 +0200 (CEST) Received: from lsv031405.swis.in-blr01.nxp.com (lsv031405.swis.in-blr01.nxp.com [92.120.147.93]) by aprdc01srsp001v.ap-rdc01.nxp.com (Postfix) with ESMTP id 6797618000B5; Fri, 23 Oct 2026 21:43:52 +0800 (+08) From: Prashant Gupta To: stephen@networkplumber.org, dev@dpdk.org Cc: Jun Yang Subject: [PATCH v4-S3 02/65] net/dpaa2: generate devargs table and string from one list Date: Fri, 23 Oct 2026 19:12:42 +0530 Message-ID: <20261023134346.37727-3-prashant.gupta_3@nxp.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20261023134346.37727-1-prashant.gupta_3@nxp.com> References: <20260915120208.4185877-1-prashant.gupta_3@nxp.com> <20261023134346.37727-1-prashant.gupta_3@nxp.com> X-Archived-By: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Virus-Scanned: ClamAV using ClamSMTP X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Archive: List-Post: List-Help: List-Subscribe: , X-Spam-Status: No, score=-16.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=4.0.1 X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-26) on aws-us-west-2-korg-listserv-alma10-1 From: Jun Yang Add a common dpaax_devarg.h header providing the dpaax_devargs type, the dpaax_get_devargs() parse helper, and two X-macro operators: DPAAX_DEVARG_TABLE_ENTRY(key, handle, flag) -> {key, handle, flag}, DPAAX_DEVARG_STRING_ENTRY(key, handle, flag) -> key "= " Convert the dpaa2 net driver to declare its device arguments once via a single DPAA2_DEVARGS(ENTRY) list. The same list now builds both the parse table (s_dpaa2_devargs) and the RTE_PMD_REGISTER_PARAM_STRING valid-args string, so every key is spelled exactly once. The per-arg string defines and the hand-written dpaa2_get_devargs()/ check_devargs_handler() lookups are replaced by a single dpaax_get_devargs() call that sets priv->flags. Signed-off-by: Jun Yang --- drivers/common/dpaax/dpaax_devarg.h | 69 ++++++++++++ drivers/net/dpaa2/dpaa2_ethdev.c | 158 +++++++--------------------- 2 files changed, 106 insertions(+), 121 deletions(-) create mode 100644 drivers/common/dpaax/dpaax_devarg.h diff --git a/drivers/common/dpaax/dpaax_devarg.h b/drivers/common/dpaax/dpaax_devarg.h new file mode 100644 index 0000000000..2481a399f5 --- /dev/null +++ b/drivers/common/dpaax/dpaax_devarg.h @@ -0,0 +1,69 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright 2026 NXP + */ + +#ifndef _DPAAX_DEVARG_H_ +#define _DPAAX_DEVARG_H_ +#include +#include +#include + +struct dpaax_devargs { + const char *arg_key; + void (*handle)(void *vdev, const struct rte_kvargs_pair *pair); + uint64_t flag_bit; +}; + +/* Shared expansions for a driver's devargs X-macro list. + * + * A driver declares its argument list once as a list-macro that takes the + * per-entry operator as a parameter, for example: + * + * #define MY_DEVARGS(ENTRY) \ + * ENTRY("drv_foo", foo_handler, 0) \ + * ENTRY("drv_bar", NULL, MY_BAR_FLAG) + * + * The same list then generates both the parse table and the registered + * valid-args string by passing the matching operator below: + * + * static const struct dpaax_devargs s_my_devargs[] = { + * MY_DEVARGS(DPAAX_DEVARG_TABLE_ENTRY) + * }; + * RTE_PMD_REGISTER_PARAM_STRING(name, MY_DEVARGS(DPAAX_DEVARG_STRING_ENTRY)); + * + * This keeps each key spelled exactly once. + */ +#define DPAAX_DEVARG_TABLE_ENTRY(key, handle, flag) {key, handle, flag}, +#define DPAAX_DEVARG_STRING_ENTRY(key, handle, flag) key "= " + +#define dpaax_get_devargs(dev_args, dev, flags) \ +({ \ + uint32_t arg_num = RTE_DIM(dev_args), arg_idx, kv_idx;\ + struct rte_device *rdev = dev->device; \ + struct rte_kvargs *kvlist = NULL; \ + struct rte_kvargs_pair *pair; \ + uint32_t flag_bit; \ + \ + if (rdev->devargs) \ + kvlist = rte_kvargs_parse(rdev->devargs->args, NULL); \ + if (kvlist) { \ + for (arg_idx = 0; arg_idx < arg_num; arg_idx++) { \ + for (kv_idx = 0; kv_idx < kvlist->count; kv_idx++) { \ + pair = &kvlist->pairs[kv_idx]; \ + if (strcmp(pair->key, dev_args[arg_idx].arg_key)) \ + continue; \ + if (dev_args[arg_idx].handle) \ + dev_args[arg_idx].handle(dev, pair); \ + flag_bit = dev_args[arg_idx].flag_bit; \ + if (flags && pair->value) { \ + if (atoi(pair->value)) \ + (*((uint32_t *)flags)) |= flag_bit; \ + else \ + (*((uint32_t *)flags)) &= ~flag_bit; \ + } \ + } \ + } \ + rte_kvargs_free(kvlist); \ + } \ +}) +#endif /* _DPAAX_DEVARG_H_ */ diff --git a/drivers/net/dpaa2/dpaa2_ethdev.c b/drivers/net/dpaa2/dpaa2_ethdev.c index e69dc5b9a2..cd39c7874e 100644 --- a/drivers/net/dpaa2/dpaa2_ethdev.c +++ b/drivers/net/dpaa2/dpaa2_ethdev.c @@ -31,18 +31,32 @@ #include "dpaa2_ethdev.h" #include "dpaa2_sparser.h" #include +#include + +/* Single source of truth for dpaa2 devargs: + * ENTRY(key-string, handle, flag-bit) + * handle is an optional per-arg callback (NULL if the arg only sets a flag). + * Both the parse table (s_dpaa2_devargs) and the registered valid-args + * string are generated from this list, so each key is spelled once. The + * ENTRY operator (table vs string form) is provided by dpaax_devarg.h. + */ +#define DPAA2_DEVARGS(ENTRY) \ + ENTRY("drv_loopback", NULL, DPAA2_RX_LOOPBACK_MODE) \ + ENTRY("drv_no_prefetch", NULL, DPAA2_NO_PREFETCH_RX) \ + ENTRY("drv_tx_conf", NULL, DPAA2_TX_CONF_ENABLE) \ + ENTRY("drv_rx_parse_drop", NULL, DPAA2_PARSE_ERR_DROP) \ + ENTRY("drv_rx_print_parser_result", NULL, DPAA2_RX_PRINT_PSR_RESULT_FLAG) \ + ENTRY("drv_err_queue", NULL, DPAA2_RX_ERROR_QUEUE_FLAG) \ + ENTRY("drv_no_taildrop", NULL, DPAA2_RX_TAILDROP_OFF) \ + ENTRY("drv_no_data_stashing", NULL, DPAA2_RX_DATA_STASHING_OFF_FLAG) -#define DRIVER_LOOPBACK_MODE "drv_loopback" -#define DRIVER_NO_PREFETCH_MODE "drv_no_prefetch" -#define DRIVER_TX_CONF "drv_tx_conf" -#define DRIVER_RX_PARSE_ERR_DROP "drv_rx_parse_drop" -#define DRIVER_RX_PRINT_PARSER_RESULT "drv_rx_print_parser_result" -#define DRIVER_ERROR_QUEUE "drv_err_queue" -#define DRIVER_NO_TAILDROP "drv_no_taildrop" -#define DRIVER_NO_DATA_STASHING "drv_no_data_stashing" #define CHECK_INTERVAL 100 /* 100ms */ #define MAX_REPEAT_TIME 90 /* 9s (90 * 100ms) in total */ +static const struct dpaax_devargs s_dpaa2_devargs[] = { + DPAA2_DEVARGS(DPAAX_DEVARG_TABLE_ENTRY) +}; + /* Supported Rx offloads */ static uint64_t dev_rx_offloads_sup = RTE_ETH_RX_OFFLOAD_CHECKSUM | @@ -3398,47 +3412,9 @@ populate_mac_addr(struct fsl_mc_io *dpni_dev, return ret; } -static int -check_devargs_handler(__rte_unused const char *key, const char *value, - __rte_unused void *opaque) -{ - if (strcmp(value, "1")) - return -1; - - return 0; -} - -static int -dpaa2_get_devargs(struct rte_devargs *devargs, const char *key) -{ - struct rte_kvargs *kvlist; - - if (!devargs) - return 0; - - kvlist = rte_kvargs_parse(devargs->args, NULL); - if (!kvlist) - return 0; - - if (!rte_kvargs_count(kvlist, key)) { - rte_kvargs_free(kvlist); - return 0; - } - - if (rte_kvargs_process(kvlist, key, - check_devargs_handler, NULL) < 0) { - rte_kvargs_free(kvlist); - return 0; - } - rte_kvargs_free(kvlist); - - return 1; -} - static int dpaa2_dev_init(struct rte_eth_dev *eth_dev) { - struct rte_device *dev = eth_dev->device; struct rte_dpaa2_device *dpaa2_dev; struct fsl_mc_io *dpni_dev; struct dpni_attr attr; @@ -3456,42 +3432,23 @@ dpaa2_dev_init(struct rte_eth_dev *eth_dev) dpni_dev->regs = dpaa2_get_mcp_ptr(MC_PORTAL_INDEX); eth_dev->process_private = dpni_dev; - /* RX no prefetch mode? */ - if (dpaa2_get_devargs(dev->devargs, DRIVER_NO_PREFETCH_MODE)) { - priv->flags |= DPAA2_NO_PREFETCH_RX; - DPAA2_PMD_INFO("No RX prefetch mode"); - } - - if (dpaa2_get_devargs(dev->devargs, DRIVER_LOOPBACK_MODE)) { - priv->flags |= DPAA2_RX_LOOPBACK_MODE; - DPAA2_PMD_INFO("Rx loopback mode"); - } - - if (dpaa2_get_devargs(dev->devargs, DRIVER_NO_TAILDROP)) { - priv->flags |= DPAA2_RX_TAILDROP_OFF; - DPAA2_PMD_INFO("Rx taildrop disabled"); - } + priv->flags = 0; + dpaax_get_devargs(s_dpaa2_devargs, eth_dev, &priv->flags); - if (dpaa2_get_devargs(dev->devargs, DRIVER_NO_DATA_STASHING) || - getenv("DPAA2_DATA_STASHING_OFF")) { - priv->flags |= DPAA2_RX_DATA_STASHING_OFF_FLAG; - DPAA2_PMD_INFO("Data stashing disabled"); - } + eth_dev->dev_ops = &dpaa2_ethdev_ops; + eth_dev->rx_queue_count = dpaa2_dev_rx_queue_count; + if (priv->flags & DPAA2_RX_LOOPBACK_MODE) + eth_dev->rx_pkt_burst = dpaa2_dev_loopback_rx; + else if (priv->flags & DPAA2_NO_PREFETCH_RX) + eth_dev->rx_pkt_burst = dpaa2_dev_rx; + else + eth_dev->rx_pkt_burst = dpaa2_dev_prefetch_rx; + eth_dev->tx_pkt_burst = dpaa2_dev_tx; /* For secondary processes, the primary has done all the work */ if (rte_eal_process_type() != RTE_PROC_PRIMARY) { - /* In case of secondary, only burst and ops API need to be - * plugged. + /* In case of secondary, everything is done here. */ - eth_dev->dev_ops = &dpaa2_ethdev_ops; - eth_dev->rx_queue_count = dpaa2_dev_rx_queue_count; - if (priv->flags & DPAA2_RX_LOOPBACK_MODE) - eth_dev->rx_pkt_burst = dpaa2_dev_loopback_rx; - else if (priv->flags & DPAA2_NO_PREFETCH_RX) - eth_dev->rx_pkt_burst = dpaa2_dev_rx; - else - eth_dev->rx_pkt_burst = dpaa2_dev_prefetch_rx; - eth_dev->tx_pkt_burst = dpaa2_dev_tx; return 0; } @@ -3564,30 +3521,10 @@ dpaa2_dev_init(struct rte_eth_dev *eth_dev) priv->options = attr.options; priv->max_mac_filters = attr.mac_filter_entries; priv->max_vlan_filters = attr.vlan_filter_entries; - priv->flags = 0; #if defined(RTE_LIBRTE_IEEE1588) DPAA2_PMD_INFO("DPDK IEEE1588 is enabled"); priv->flags |= DPAA2_TX_CONF_ENABLE; #endif - /* Used with ``fslmc:dpni.1,drv_tx_conf=1`` */ - if (dpaa2_get_devargs(dev->devargs, DRIVER_TX_CONF)) { - priv->flags |= DPAA2_TX_CONF_ENABLE; - DPAA2_PMD_INFO("TX_CONF Enabled"); - } - - if (dpaa2_get_devargs(dev->devargs, DRIVER_ERROR_QUEUE)) { - priv->flags |= DPAA2_RX_ERROR_QUEUE_FLAG; - DPAA2_PMD_INFO("Enable error queue"); - } - - /* Packets with parse error to be dropped in hw */ - if (dpaa2_get_devargs(dev->devargs, DRIVER_RX_PARSE_ERR_DROP)) { - priv->flags |= DPAA2_PARSE_ERR_DROP; - DPAA2_PMD_INFO("Drop parse error packets in hw"); - } - - if (dpaa2_get_devargs(dev->devargs, DRIVER_RX_PRINT_PARSER_RESULT)) - priv->flags |= DPAA2_RX_PRINT_PSR_RESULT_FLAG; /* Allocate memory for hardware structure for queues */ ret = dpaa2_alloc_rx_tx_queues(eth_dev); @@ -3666,20 +3603,6 @@ dpaa2_dev_init(struct rte_eth_dev *eth_dev) goto init_err; } - eth_dev->dev_ops = &dpaa2_ethdev_ops; - eth_dev->rx_queue_count = dpaa2_dev_rx_queue_count; - - if (dpaa2_get_devargs(dev->devargs, DRIVER_LOOPBACK_MODE)) { - eth_dev->rx_pkt_burst = dpaa2_dev_loopback_rx; - DPAA2_PMD_INFO("Loopback mode"); - } else if (dpaa2_get_devargs(dev->devargs, DRIVER_NO_PREFETCH_MODE)) { - eth_dev->rx_pkt_burst = dpaa2_dev_rx; - DPAA2_PMD_INFO("No Prefetch mode"); - } else { - eth_dev->rx_pkt_burst = dpaa2_dev_prefetch_rx; - } - eth_dev->tx_pkt_burst = dpaa2_dev_tx; - /* Init fields w.r.t. classification */ memset(&priv->extract.qos_key_extract, 0, sizeof(struct dpaa2_key_extract)); @@ -3704,8 +3627,7 @@ dpaa2_dev_init(struct rte_eth_dev *eth_dev) } ret = dpni_set_max_frame_length(dpni_dev, CMD_PRI_LOW, priv->token, - RTE_ETHER_MAX_LEN - RTE_ETHER_CRC_LEN - + VLAN_TAG_SIZE); + RTE_ETHER_MAX_LEN - RTE_ETHER_CRC_LEN + VLAN_TAG_SIZE); if (ret) { DPAA2_PMD_ERR("Unable to set mtu. check config"); goto init_err; @@ -3985,12 +3907,6 @@ static struct rte_dpaa2_driver rte_dpaa2_pmd = { RTE_PMD_REGISTER_DPAA2(NET_DPAA2_PMD_DRIVER_NAME, rte_dpaa2_pmd); RTE_PMD_REGISTER_PARAM_STRING(NET_DPAA2_PMD_DRIVER_NAME, - DRIVER_LOOPBACK_MODE "= " - DRIVER_NO_PREFETCH_MODE "=" - DRIVER_TX_CONF "=" - DRIVER_RX_PARSE_ERR_DROP "=" - DRIVER_RX_PRINT_PARSER_RESULT "=" - DRIVER_ERROR_QUEUE "=" - DRIVER_NO_TAILDROP "=" - DRIVER_NO_DATA_STASHING "="); + DPAA2_DEVARGS(DPAAX_DEVARG_STRING_ENTRY) +); RTE_LOG_REGISTER_DEFAULT(dpaa2_logtype_pmd, NOTICE); -- 2.43.0