bnxt_en: Increase timeout for HWRM_DBG_COREDUMP_XX commands
authorVasundhara Volam <vasundhara-v.volam@broadcom.com>
Sat, 14 Sep 2019 04:01:39 +0000 (00:01 -0400)
committerDavid S. Miller <davem@davemloft.net>
Mon, 16 Sep 2019 14:44:27 +0000 (16:44 +0200)
Firmware coredump messages take much longer than standard messages,
so increase the timeout accordingly.

Fixes: 6c5657d085ae ("bnxt_en: Add support for ethtool get dump.")
Signed-off-by: Vasundhara Volam <vasundhara-v.volam@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnxt/bnxt.h
drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c

index 333b0a8..42a8a75 100644 (file)
@@ -648,6 +648,7 @@ struct nqe_cn {
 #define SHORT_HWRM_CMD_TIMEOUT         20
 #define HWRM_CMD_TIMEOUT               (bp->hwrm_cmd_timeout)
 #define HWRM_RESET_TIMEOUT             ((HWRM_CMD_TIMEOUT) * 4)
+#define HWRM_COREDUMP_TIMEOUT          ((HWRM_CMD_TIMEOUT) * 12)
 #define HWRM_RESP_ERR_CODE_MASK                0xffff
 #define HWRM_RESP_LEN_OFFSET           4
 #define HWRM_RESP_LEN_MASK             0xffff0000
index 235265e..51c1404 100644 (file)
@@ -3112,7 +3112,7 @@ static int bnxt_hwrm_dbg_coredump_initiate(struct bnxt *bp, u16 component_id,
        req.component_id = cpu_to_le16(component_id);
        req.segment_id = cpu_to_le16(segment_id);
 
-       return hwrm_send_message(bp, &req, sizeof(req), HWRM_CMD_TIMEOUT);
+       return hwrm_send_message(bp, &req, sizeof(req), HWRM_COREDUMP_TIMEOUT);
 }
 
 static int bnxt_hwrm_dbg_coredump_retrieve(struct bnxt *bp, u16 component_id,