From: Venkat Duvvuru <[email protected]> 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 <[email protected]> Signed-off-by: Somnath Kotur <[email protected]> --- 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 9642796..7d9459f 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.10.1.613.g2cc2e70
