From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.lore.kernel.org (Postfix) with ESMTP id ECB0BE73165 for ; Mon, 2 Feb 2026 19:34:13 +0000 (UTC) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8874B40671; Mon, 2 Feb 2026 20:33:42 +0100 (CET) Received: from office2.cesnet.cz (office2.cesnet.cz [78.128.248.237]) by mails.dpdk.org (Postfix) with ESMTP id 64F764027D; Mon, 2 Feb 2026 20:33:36 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cesnet.cz; s=office2-2020; t=1770060816; bh=qqBMGSThhEwQCpIk9HId/Ef2lI1hZuPj3zxZwt46Bz0=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=RbJYpQOQSaa20hZF1bYgsvCx31uU0UK8MjpDyXhSA4NylYbFaC9CE+dsSPIrgJNz5 O+WF6N9De7uOL1x46M/K+5n/Va70u0NjlDOHscaJvJFtJ/fJplKJjPfeo2Bjx+Zn4j VKMANDzMD/jwk3K8/BlpLDf66WG3T6zX/EhomFUVVovcxDMpqpUxZN9nc7WPwQ5e8n b69+ElRst+k7eRYyxa0bck7unO02txE7MzY2UVylDBLY2cp2Vt4smc8eokku+uuoN4 7TklqHhtjXJ//xctI+yi4HBeskFBkYRDIP3o3F3zAyMNwVeTNyljLR2/IPz0AjuISa fDPjCvU7/R4vQ== Received: from emil.cesnet.cz (gtx107.cesnet.cz [IPv6:2001:718:812:27::107]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by office2.cesnet.cz (Postfix) with ESMTPSA id 34EA71180072; Mon, 2 Feb 2026 20:33:36 +0100 (CET) From: spinler@cesnet.cz To: dev@dpdk.org Cc: Martin Spinler , stable@dpdk.org Subject: [PATCH v5 5/6] net/nfb: release allocated resources correctly Date: Mon, 2 Feb 2026 20:33:29 +0100 Message-ID: <20260202193330.3324681-6-spinler@cesnet.cz> X-Mailer: git-send-email 2.52.0 In-Reply-To: <20260202193330.3324681-1-spinler@cesnet.cz> References: <20260115140134.235877-1-spinler@cesnet.cz> <20260202193330.3324681-1-spinler@cesnet.cz> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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: Martin Spinler Internal handles are initialized in eth_dev_init, so the cleanup should be done in eth_dev_uninit. Fixes: 6435f9a0ac22 ("net/nfb: add new netcope driver") Cc: stable@dpdk.org Signed-off-by: Martin Spinler --- drivers/net/nfb/nfb_ethdev.c | 74 ++++++++++++++++++++---------------- 1 file changed, 42 insertions(+), 32 deletions(-) diff --git a/drivers/net/nfb/nfb_ethdev.c b/drivers/net/nfb/nfb_ethdev.c index 34949d6b1f..5197642f61 100644 --- a/drivers/net/nfb/nfb_ethdev.c +++ b/drivers/net/nfb/nfb_ethdev.c @@ -18,6 +18,8 @@ #include "nfb_rxmode.h" #include "nfb.h" +static int nfb_eth_dev_uninit(struct rte_eth_dev *dev); + /** * Default MAC addr */ @@ -160,8 +162,6 @@ nfb_eth_dev_stop(struct rte_eth_dev *dev) uint16_t nb_rx = dev->data->nb_rx_queues; uint16_t nb_tx = dev->data->nb_tx_queues; - dev->data->dev_started = 0; - for (i = 0; i < nb_tx; i++) nfb_eth_tx_queue_stop(dev, i); @@ -184,7 +184,6 @@ static int nfb_eth_dev_configure(struct rte_eth_dev *dev __rte_unused) { int ret; - struct pmd_internals *internals = dev->process_private; struct rte_eth_conf *dev_conf = &dev->data->dev_conf; if (dev_conf->rxmode.offloads & RTE_ETH_RX_OFFLOAD_TIMESTAMP) { @@ -193,12 +192,16 @@ nfb_eth_dev_configure(struct rte_eth_dev *dev __rte_unused) &nfb_timestamp_rx_dynflag); if (ret != 0) { NFB_LOG(ERR, "Cannot register Rx timestamp field/flag %d", ret); - nfb_close(internals->nfb); - return -rte_errno; + ret = -ENOMEM; + goto err_ts_register; } } return 0; + +err_ts_register: + nfb_eth_dev_uninit(dev); + return ret; } static uint32_t @@ -262,32 +265,28 @@ nfb_eth_dev_info(struct rte_eth_dev *dev, static int nfb_eth_dev_close(struct rte_eth_dev *dev) { - struct pmd_internals *internals = dev->process_private; uint16_t i; uint16_t nb_rx = dev->data->nb_rx_queues; uint16_t nb_tx = dev->data->nb_tx_queues; - int ret; - nfb_nc_rxmac_deinit(internals->rxmac, internals->max_rxmac); - nfb_nc_txmac_deinit(internals->txmac, internals->max_txmac); - - if (rte_eal_process_type() != RTE_PROC_PRIMARY) - return 0; - - ret = nfb_eth_dev_stop(dev); - - for (i = 0; i < nb_rx; i++) { - nfb_eth_rx_queue_release(dev, i); - dev->data->rx_queues[i] = NULL; - } - dev->data->nb_rx_queues = 0; - for (i = 0; i < nb_tx; i++) { - nfb_eth_tx_queue_release(dev, i); - dev->data->tx_queues[i] = NULL; + if (rte_eal_process_type() == RTE_PROC_PRIMARY) { + for (i = 0; i < nb_rx; i++) { + if (dev->data->rx_queues[i]) { + nfb_eth_rx_queue_release(dev, i); + dev->data->rx_queues[i] = NULL; + } + } + for (i = 0; i < nb_tx; i++) { + if (dev->data->tx_queues[i]) { + nfb_eth_tx_queue_release(dev, i); + dev->data->tx_queues[i] = NULL; + } + } } - dev->data->nb_tx_queues = 0; - return ret; + nfb_eth_dev_uninit(dev); + + return 0; } /** @@ -529,7 +528,7 @@ nfb_eth_dev_init(struct rte_eth_dev *dev) dev->device->numa_node); if (internals == NULL) { ret = -ENOMEM; - return ret; + goto err_alloc_internals; } dev->process_private = internals; @@ -547,8 +546,8 @@ nfb_eth_dev_init(struct rte_eth_dev *dev) internals->nfb = nfb_open(nfb_dev); if (internals->nfb == NULL) { NFB_LOG(ERR, "nfb_open(): failed to open %s", nfb_dev); - rte_free(internals); - return -EINVAL; + ret = -EINVAL; + goto err_nfb_open; } max_rx_queues = ndp_get_rx_queue_available_count(internals->nfb); max_tx_queues = ndp_get_tx_queue_available_count(internals->nfb); @@ -583,9 +582,8 @@ nfb_eth_dev_init(struct rte_eth_dev *dev) sizeof(struct rte_ether_addr) * mac_count, RTE_CACHE_LINE_SIZE); if (data->mac_addrs == NULL) { NFB_LOG(ERR, "Could not alloc space for MAC address"); - nfb_close(internals->nfb); - rte_free(internals); - return -EINVAL; + ret = -ENOMEM; + goto err_malloc_mac_addrs; } rte_eth_random_addr(eth_addr_init.addr_bytes); @@ -607,6 +605,16 @@ nfb_eth_dev_init(struct rte_eth_dev *dev) pci_addr->function); return 0; + +err_malloc_mac_addrs: + nfb_nc_rxmac_deinit(internals->rxmac, internals->max_rxmac); + nfb_nc_txmac_deinit(internals->txmac, internals->max_txmac); + nfb_close(internals->nfb); + +err_nfb_open: + rte_free(internals); +err_alloc_internals: + return ret; } /** @@ -625,7 +633,9 @@ nfb_eth_dev_uninit(struct rte_eth_dev *dev) struct rte_pci_addr *pci_addr = &pci_dev->addr; struct pmd_internals *internals = dev->process_private; - nfb_eth_dev_close(dev); + nfb_nc_rxmac_deinit(internals->rxmac, internals->max_rxmac); + nfb_nc_txmac_deinit(internals->txmac, internals->max_txmac); + nfb_close(internals->nfb); rte_free(internals); -- 2.52.0