From e9ccabda04111afbc6b20655cce3474272d3ad90 Mon Sep 17 00:00:00 2001 From: Rahul Gupta Date: Wed, 9 Oct 2019 18:41:48 -0700 Subject: [PATCH] net/bnxt: reduce cleanup time during reset recovery In stop_op(), don't wait for link down event's ASYNC response from FW. Fixes: c023cd5b2192 ("net/bnxt: fix async link handling and update") Cc: stable@dpdk.org Signed-off-by: Rahul Gupta Reviewed-by: Kalesh AP Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c index 84f7b87f31..69dafc41ff 100644 --- a/drivers/net/bnxt/bnxt_ethdev.c +++ b/drivers/net/bnxt/bnxt_ethdev.c @@ -967,8 +967,12 @@ static void bnxt_dev_stop_op(struct rte_eth_dev *eth_dev) eth_dev->data->dev_link.link_status = 0; } bnxt_dev_set_link_down_op(eth_dev); - /* Wait for link to be reset and the async notification to process. */ - rte_delay_ms(BNXT_LINK_WAIT_INTERVAL * 2); + + /* Wait for link to be reset and the async notification to process. + * During reset recovery, there is no need to wait + */ + if (!is_bnxt_in_error(bp)) + rte_delay_ms(BNXT_LINK_WAIT_INTERVAL * 2); /* Clean queue intr-vector mapping */ rte_intr_efd_disable(intr_handle); -- 2.20.1