From: Venkat Duvvuru <venkatkumar.duvv...@broadcom.com> 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 <venkatkumar.duvv...@broadcom.com> Signed-off-by: Somnath Kotur <somnath.ko...@broadcom.com> --- 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 e7ec99e..744f715 100644 --- a/drivers/net/bnxt/bnxt_ethdev.c +++ b/drivers/net/bnxt/bnxt_ethdev.c @@ -4772,6 +4772,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; @@ -4813,8 +4814,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.7.4