Skip to content

Commit 0f2dcee

Browse files
Vasundhara VolamSomasundaram Krishnasamy
authored andcommitted
bnxt_en: Increase timeout for HWRM_DBG_COREDUMP_XX commands
Firmware coredump messages take much longer than standard messages, so increase the timeout accordingly. Fixes: 6c5657d ("bnxt_en: Add support for ethtool get dump.") Signed-off-by: Vasundhara Volam <[email protected]> Signed-off-by: Michael Chan <[email protected]> Signed-off-by: David S. Miller <[email protected]> (cherry picked from commit 57a8730) Orabug: 30431202 Signed-off-by: Somasundaram Krishnasamy <[email protected]> Reviewed-by: John Donnelly <[email protected]>
1 parent fcdf0c7 commit 0f2dcee

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

drivers/net/ethernet/broadcom/bnxt/bnxt.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -563,6 +563,7 @@ struct nqe_cn {
563563
#define DFLT_HWRM_CMD_TIMEOUT 500
564564
#define HWRM_CMD_TIMEOUT (bp->hwrm_cmd_timeout)
565565
#define HWRM_RESET_TIMEOUT ((HWRM_CMD_TIMEOUT) * 4)
566+
#define HWRM_COREDUMP_TIMEOUT ((HWRM_CMD_TIMEOUT) * 12)
566567
#define HWRM_RESP_ERR_CODE_MASK 0xffff
567568
#define HWRM_RESP_LEN_OFFSET 4
568569
#define HWRM_RESP_LEN_MASK 0xffff0000

drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3068,7 +3068,7 @@ static int bnxt_hwrm_dbg_coredump_initiate(struct bnxt *bp, u16 component_id,
30683068
req.component_id = cpu_to_le16(component_id);
30693069
req.segment_id = cpu_to_le16(segment_id);
30703070

3071-
return hwrm_send_message(bp, &req, sizeof(req), HWRM_CMD_TIMEOUT);
3071+
return hwrm_send_message(bp, &req, sizeof(req), HWRM_COREDUMP_TIMEOUT);
30723072
}
30733073

30743074
static int bnxt_hwrm_dbg_coredump_retrieve(struct bnxt *bp, u16 component_id,

0 commit comments

Comments
 (0)