From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 001A746ADE; Thu, 3 Jul 2025 07:26:42 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 124394064E; Thu, 3 Jul 2025 07:26:08 +0200 (CEST) Received: from inva020.nxp.com (inva020.nxp.com [92.121.34.13]) by mails.dpdk.org (Postfix) with ESMTP id 93B0D402A3 for ; Thu, 3 Jul 2025 07:26:02 +0200 (CEST) Received: from inva020.nxp.com (localhost [127.0.0.1]) by inva020.eu-rdc02.nxp.com (Postfix) with ESMTP id 675CC1A258F; Thu, 3 Jul 2025 07:26:02 +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 0B9CE1A2569; Thu, 3 Jul 2025 07:26:02 +0200 (CEST) Received: from lsv03379.swis.in-blr01.nxp.com (lsv03379.swis.in-blr01.nxp.com [92.120.147.188]) by aprdc01srsp001v.ap-rdc01.nxp.com (Postfix) with ESMTP id 7A7C61800241; Thu, 3 Jul 2025 13:26:01 +0800 (+08) From: vanshika.shukla@nxp.com To: dev@dpdk.org, Hemant Agrawal , Sachin Saxena Cc: Vanshika Shukla Subject: [v7 6/8] net/dpaa: add devargs for enabling err packets on main queue Date: Thu, 3 Jul 2025 10:55:56 +0530 Message-Id: <20250703052558.3700573-7-vanshika.shukla@nxp.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20250703052558.3700573-1-vanshika.shukla@nxp.com> References: <20250702101026.3301762-1-vanshika.shukla@nxp.com> <20250703052558.3700573-1-vanshika.shukla@nxp.com> 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-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org From: Vanshika Shukla Currently, error queue is mapped to the Rx queue and enabled by default. This patch adds the devargs to control the err packets on main queue. Also, in VSP mode the error queue should be disabled because the error packets from kernel are diverted to the Rx queue/err queue causing crash. Signed-off-by: Vanshika Shukla --- doc/guides/nics/dpaa.rst | 4 ++++ drivers/net/dpaa/dpaa_ethdev.c | 29 +++++++++++++++++++++-------- 2 files changed, 25 insertions(+), 8 deletions(-) diff --git a/doc/guides/nics/dpaa.rst b/doc/guides/nics/dpaa.rst index de3ae96e07..19d7d00da4 100644 --- a/doc/guides/nics/dpaa.rst +++ b/doc/guides/nics/dpaa.rst @@ -278,6 +278,10 @@ for details. * Use dev arg option ``drv_ieee1588=1`` to enable IEEE 1588 support at driver level, e.g. ``dpaa:fm1-mac3,drv_ieee1588=1``. +* Use dev arg option ``recv_err_pkts=1`` to receive all packets including + error packets and thus disabling hardware based packet handing + at driver level, e.g. ``dpaa:fm1-mac3,recv_err_pkts=1``. + FMAN Config ----------- diff --git a/drivers/net/dpaa/dpaa_ethdev.c b/drivers/net/dpaa/dpaa_ethdev.c index 7d0f830204..62cafb7073 100644 --- a/drivers/net/dpaa/dpaa_ethdev.c +++ b/drivers/net/dpaa/dpaa_ethdev.c @@ -52,9 +52,10 @@ #include #include -#define DRIVER_IEEE1588 "drv_ieee1588" -#define CHECK_INTERVAL 100 /* 100ms */ -#define MAX_REPEAT_TIME 90 /* 9s (90 * 100ms) in total */ +#define DRIVER_IEEE1588 "drv_ieee1588" +#define CHECK_INTERVAL 100 /* 100ms */ +#define MAX_REPEAT_TIME 90 /* 9s (90 * 100ms) in total */ +#define DRIVER_RECV_ERR_PKTS "recv_err_pkts" /* Supported Rx offloads */ static uint64_t dev_rx_offloads_sup = @@ -87,6 +88,8 @@ static int is_global_init; static int fmc_q = 1; /* Indicates the use of static fmc for distribution */ static int default_q; /* use default queue - FMC is not executed*/ int dpaa_ieee_1588; /* use to indicate if IEEE 1588 is enabled for the driver */ +bool dpaa_enable_recv_err_pkts; /* Enable main queue to receive error packets */ + /* At present we only allow up to 4 push mode queues as default - as each of * this queue need dedicated portal and we are short of portals. */ @@ -1273,10 +1276,12 @@ int dpaa_eth_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, } } - /* Enable main queue to receive error packets also by default */ + /* Enable main queue to receive error packets */ if (fif->mac_type != fman_offline_internal && - fif->mac_type != fman_onic) + fif->mac_type != fman_onic && + dpaa_enable_recv_err_pkts && !fif->is_shared_mac) { fman_if_set_err_fqid(fif, rxq->fqid); + } return 0; } @@ -2191,6 +2196,9 @@ dpaa_dev_init(struct rte_eth_dev *eth_dev) if (dpaa_get_devargs(dev->devargs, DRIVER_IEEE1588)) dpaa_ieee_1588 = 1; + if (dpaa_get_devargs(dev->devargs, DRIVER_RECV_ERR_PKTS)) + dpaa_enable_recv_err_pkts = 1; + memset((char *)dev_rx_fqids, 0, sizeof(uint32_t) * DPAA_MAX_NUM_PCD_QUEUES); @@ -2418,8 +2426,12 @@ dpaa_dev_init(struct rte_eth_dev *eth_dev) fman_intf->mac_type != fman_offline_internal && fman_intf->mac_type != fman_onic) { /* Configure error packet handling */ - fman_if_receive_rx_errors(fman_intf, - FM_FD_RX_STATUS_ERR_MASK); +#ifndef RTE_LIBRTE_DPAA_DEBUG_DRIVER + if (dpaa_enable_recv_err_pkts) +#endif + fman_if_receive_rx_errors(fman_intf, + FM_FD_RX_STATUS_ERR_MASK); + /* Disable RX mode */ fman_if_disable_rx(fman_intf); /* Disable promiscuous mode */ @@ -2619,5 +2631,6 @@ static struct rte_dpaa_driver rte_dpaa_pmd = { RTE_PMD_REGISTER_DPAA(net_dpaa, rte_dpaa_pmd); RTE_PMD_REGISTER_PARAM_STRING(net_dpaa, - DRIVER_IEEE1588 "="); + DRIVER_IEEE1588 "=" + DRIVER_RECV_ERR_PKTS "="); RTE_LOG_REGISTER_DEFAULT(dpaa_logtype_pmd, NOTICE); -- 2.25.1