From 5f5ccaa05c9a4423fb2c7803023ed13468034549 Mon Sep 17 00:00:00 2001 From: Venkat Duvvuru Date: Thu, 24 Oct 2019 13:14:32 +0530 Subject: [PATCH] net/bnxt: uninit locks during resource cleanup Currently, bnxt_uninit_locks is called inside bnxt_dev_uninit. However, the more appropriate place to call it is inside bnxt_uninit_resources. This is more logical, as all the resources uninitialization is happening inside bnxt_uninit_resources. Signed-off-by: Venkat Duvvuru Signed-off-by: Somnath Kotur --- drivers/net/bnxt/bnxt_ethdev.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c index 9642796eeb..7d9459f0a1 100644 --- a/drivers/net/bnxt/bnxt_ethdev.c +++ b/drivers/net/bnxt/bnxt_ethdev.c @@ -4806,6 +4806,7 @@ bnxt_uninit_resources(struct bnxt *bp, bool reconfig_dev) } } + bnxt_uninit_locks(bp); rte_free(bp->ptp_cfg); bp->ptp_cfg = NULL; return rc; @@ -4842,8 +4843,6 @@ bnxt_dev_uninit(struct rte_eth_dev *eth_dev) eth_dev->rx_pkt_burst = NULL; eth_dev->tx_pkt_burst = NULL; - bnxt_uninit_locks(bp); - return rc; } -- 2.20.1