|
 |
f95c89 |
From ef7f935ad4295dbc775456efd99c583cd51eaa74 Mon Sep 17 00:00:00 2001
|
|
|
f95c89 |
From: Jonathan Toppins <jtoppins@redhat.com>
|
|
|
f95c89 |
Date: Fri, 6 Dec 2019 20:12:43 -0500
|
|
|
f95c89 |
Subject: [PATCH 87/96] [netdrv] bnxt_en: Adjust the time to wait before
|
|
|
f95c89 |
polling firmware readiness
|
|
|
f95c89 |
|
|
|
f95c89 |
Message-id: <b727698b6103985a26da3cd6e0c57fc1dca21cd7.1575651771.git.jtoppins@redhat.com>
|
|
|
f95c89 |
Patchwork-id: 291376
|
|
|
f95c89 |
O-Subject: [PATCH rhel8 03/13] bnxt_en: Adjust the time to wait before polling firmware readiness.
|
|
|
f95c89 |
Bugzilla: 1773724
|
|
|
f95c89 |
RH-Acked-by: Steve Best <sbest@redhat.com>
|
|
|
f95c89 |
RH-Acked-by: David Arcari <darcari@redhat.com>
|
|
|
f95c89 |
RH-Acked-by: Jarod Wilson <jarod@redhat.com>
|
|
|
f95c89 |
RH-Acked-by: John Linville <linville@redhat.com>
|
|
|
f95c89 |
|
|
|
f95c89 |
When firmware indicates that driver needs to invoke firmware reset
|
|
|
f95c89 |
which is common for both error recovery and live firmware reset path,
|
|
|
f95c89 |
driver needs a different time to wait before polling for firmware
|
|
|
f95c89 |
readiness.
|
|
|
f95c89 |
|
|
|
f95c89 |
Modify the wait time to fw_reset_min_dsecs, which is initialised to
|
|
|
f95c89 |
correct timeout for error recovery and firmware reset.
|
|
|
f95c89 |
|
|
|
f95c89 |
Fixes: 4037eb715680 ("bnxt_en: Add a new BNXT_FW_RESET_STATE_POLL_FW_DOWN state.")
|
|
|
f95c89 |
Signed-off-by: Vasundhara Volam <vasundhara-v.volam@broadcom.com>
|
|
|
f95c89 |
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
|
|
|
f95c89 |
Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
|
|
|
f95c89 |
(cherry picked from commit c6a9e7aa2e8b15402022a15625284069d4fd6df0)
|
|
|
f95c89 |
Bugzilla: 1773724
|
|
|
f95c89 |
Build Info: https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=25154853
|
|
|
f95c89 |
Tested: simple boot test
|
|
|
f95c89 |
Signed-off-by: Jonathan Toppins <jtoppins@redhat.com>
|
|
|
f95c89 |
Signed-off-by: Bruno Meneguele <bmeneg@redhat.com>
|
|
|
f95c89 |
---
|
|
|
f95c89 |
drivers/net/ethernet/broadcom/bnxt/bnxt.c | 7 ++-----
|
|
|
f95c89 |
1 file changed, 2 insertions(+), 5 deletions(-)
|
|
|
f95c89 |
|
|
|
f95c89 |
Index: src/drivers/net/ethernet/broadcom/bnxt/bnxt.c
|
|
|
f95c89 |
===================================================================
|
|
|
f95c89 |
--- src.orig/drivers/net/ethernet/broadcom/bnxt/bnxt.c 2020-02-06 16:23:21.672458426 +0100
|
|
|
f95c89 |
+++ src/drivers/net/ethernet/broadcom/bnxt/bnxt.c 2020-02-06 16:23:22.071454764 +0100
|
|
|
f95c89 |
@@ -10674,14 +10674,11 @@
|
|
|
f95c89 |
bp->fw_reset_state = BNXT_FW_RESET_STATE_RESET_FW;
|
|
|
f95c89 |
}
|
|
|
f95c89 |
/* fall through */
|
|
|
f95c89 |
- case BNXT_FW_RESET_STATE_RESET_FW: {
|
|
|
f95c89 |
- u32 wait_dsecs = bp->fw_health->post_reset_wait_dsecs;
|
|
|
f95c89 |
-
|
|
|
f95c89 |
+ case BNXT_FW_RESET_STATE_RESET_FW:
|
|
|
f95c89 |
bnxt_reset_all(bp);
|
|
|
f95c89 |
bp->fw_reset_state = BNXT_FW_RESET_STATE_ENABLE_DEV;
|
|
|
f95c89 |
- bnxt_queue_fw_reset_work(bp, wait_dsecs * HZ / 10);
|
|
|
f95c89 |
+ bnxt_queue_fw_reset_work(bp, bp->fw_reset_min_dsecs * HZ / 10);
|
|
|
f95c89 |
return;
|
|
|
f95c89 |
- }
|
|
|
f95c89 |
case BNXT_FW_RESET_STATE_ENABLE_DEV:
|
|
|
f95c89 |
if (test_bit(BNXT_STATE_FW_FATAL_COND, &bp->state) &&
|
|
|
f95c89 |
bp->fw_health) {
|