Commit cbb01c2f authored by Arun Easi's avatar Arun Easi Committed by Martin K. Petersen

scsi: qla2xxx: Fix MPI failure AEN (8200) handling

Today, upon an MPI failure AEN, on top of collecting an MPI dump, a regular
firmware dump is also taken and then chip reset. This is disruptive to IOs
and not required. Make the firmware dump collection, followed by chip
reset, optional (not done by default).

Firmware dump buffer and MPI dump buffer are independent of each
other with this change and each can have dump that was taken at two
different times for two different issues. The MPI dump is saved in a
separate buffer and is retrieved differently from firmware dump.

To collect full dump on MPI failure AEN, a module parameter is
introduced:
    ql2xfulldump_on_mpifail (default: 0)

Link: https://lore.kernel.org/r/20200331104015.24868-2-njavali@marvell.comReported-by: default avatarkbuild test robot <lkp@intel.com>
Reviewed-by: default avatarHimanshu Madhani <himanshu.madhani@oracle.com>
Signed-off-by: default avatarArun Easi <aeasi@marvell.com>
Signed-off-by: default avatarNilesh Javali <njavali@marvell.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent e304142c
...@@ -26,7 +26,8 @@ qla2x00_sysfs_read_fw_dump(struct file *filp, struct kobject *kobj, ...@@ -26,7 +26,8 @@ qla2x00_sysfs_read_fw_dump(struct file *filp, struct kobject *kobj,
struct qla_hw_data *ha = vha->hw; struct qla_hw_data *ha = vha->hw;
int rval = 0; int rval = 0;
if (!(ha->fw_dump_reading || ha->mctp_dump_reading)) if (!(ha->fw_dump_reading || ha->mctp_dump_reading ||
ha->mpi_fw_dump_reading))
return 0; return 0;
mutex_lock(&ha->optrom_mutex); mutex_lock(&ha->optrom_mutex);
...@@ -42,6 +43,10 @@ qla2x00_sysfs_read_fw_dump(struct file *filp, struct kobject *kobj, ...@@ -42,6 +43,10 @@ qla2x00_sysfs_read_fw_dump(struct file *filp, struct kobject *kobj,
} else if (ha->mctp_dumped && ha->mctp_dump_reading) { } else if (ha->mctp_dumped && ha->mctp_dump_reading) {
rval = memory_read_from_buffer(buf, count, &off, ha->mctp_dump, rval = memory_read_from_buffer(buf, count, &off, ha->mctp_dump,
MCTP_DUMP_SIZE); MCTP_DUMP_SIZE);
} else if (ha->mpi_fw_dumped && ha->mpi_fw_dump_reading) {
rval = memory_read_from_buffer(buf, count, &off,
ha->mpi_fw_dump,
ha->mpi_fw_dump_len);
} else if (ha->fw_dump_reading) { } else if (ha->fw_dump_reading) {
rval = memory_read_from_buffer(buf, count, &off, ha->fw_dump, rval = memory_read_from_buffer(buf, count, &off, ha->fw_dump,
ha->fw_dump_len); ha->fw_dump_len);
...@@ -103,7 +108,6 @@ qla2x00_sysfs_write_fw_dump(struct file *filp, struct kobject *kobj, ...@@ -103,7 +108,6 @@ qla2x00_sysfs_write_fw_dump(struct file *filp, struct kobject *kobj,
qla82xx_set_reset_owner(vha); qla82xx_set_reset_owner(vha);
qla8044_idc_unlock(ha); qla8044_idc_unlock(ha);
} else { } else {
ha->fw_dump_mpi = 1;
qla2x00_system_error(vha); qla2x00_system_error(vha);
} }
break; break;
...@@ -137,6 +141,22 @@ qla2x00_sysfs_write_fw_dump(struct file *filp, struct kobject *kobj, ...@@ -137,6 +141,22 @@ qla2x00_sysfs_write_fw_dump(struct file *filp, struct kobject *kobj,
vha->host_no); vha->host_no);
} }
break; break;
case 8:
if (!ha->mpi_fw_dump_reading)
break;
ql_log(ql_log_info, vha, 0x70e7,
"MPI firmware dump cleared on (%ld).\n", vha->host_no);
ha->mpi_fw_dump_reading = 0;
ha->mpi_fw_dumped = 0;
break;
case 9:
if (ha->mpi_fw_dumped && !ha->mpi_fw_dump_reading) {
ha->mpi_fw_dump_reading = 1;
ql_log(ql_log_info, vha, 0x70e8,
"Raw MPI firmware dump ready for read on (%ld).\n",
vha->host_no);
}
break;
} }
return count; return count;
} }
...@@ -706,7 +726,8 @@ qla2x00_sysfs_write_reset(struct file *filp, struct kobject *kobj, ...@@ -706,7 +726,8 @@ qla2x00_sysfs_write_reset(struct file *filp, struct kobject *kobj,
scsi_unblock_requests(vha->host); scsi_unblock_requests(vha->host);
break; break;
case 0x2025d: case 0x2025d:
if (!IS_QLA81XX(ha) && !IS_QLA83XX(ha)) if (!IS_QLA81XX(ha) && !IS_QLA83XX(ha) &&
!IS_QLA27XX(ha) && !IS_QLA28XX(ha))
return -EPERM; return -EPERM;
ql_log(ql_log_info, vha, 0x706f, ql_log(ql_log_info, vha, 0x706f,
...@@ -724,6 +745,8 @@ qla2x00_sysfs_write_reset(struct file *filp, struct kobject *kobj, ...@@ -724,6 +745,8 @@ qla2x00_sysfs_write_reset(struct file *filp, struct kobject *kobj,
qla83xx_idc_audit(vha, IDC_AUDIT_TIMESTAMP); qla83xx_idc_audit(vha, IDC_AUDIT_TIMESTAMP);
qla83xx_idc_unlock(vha, 0); qla83xx_idc_unlock(vha, 0);
break; break;
} else if (IS_QLA27XX(ha) || IS_QLA28XX(ha)) {
qla27xx_reset_mpi(vha);
} else { } else {
/* Make sure FC side is not in reset */ /* Make sure FC side is not in reset */
WARN_ON_ONCE(qla2x00_wait_for_hba_online(vha) != WARN_ON_ONCE(qla2x00_wait_for_hba_online(vha) !=
...@@ -737,6 +760,7 @@ qla2x00_sysfs_write_reset(struct file *filp, struct kobject *kobj, ...@@ -737,6 +760,7 @@ qla2x00_sysfs_write_reset(struct file *filp, struct kobject *kobj,
scsi_unblock_requests(vha->host); scsi_unblock_requests(vha->host);
break; break;
} }
break;
case 0x2025e: case 0x2025e:
if (!IS_P3P_TYPE(ha) || vha != base_vha) { if (!IS_P3P_TYPE(ha) || vha != base_vha) {
ql_log(ql_log_info, vha, 0x7071, ql_log(ql_log_info, vha, 0x7071,
......
...@@ -3223,6 +3223,7 @@ struct isp_operations { ...@@ -3223,6 +3223,7 @@ struct isp_operations {
uint32_t); uint32_t);
void (*fw_dump) (struct scsi_qla_host *, int); void (*fw_dump) (struct scsi_qla_host *, int);
void (*mpi_fw_dump)(struct scsi_qla_host *, int);
int (*beacon_on) (struct scsi_qla_host *); int (*beacon_on) (struct scsi_qla_host *);
int (*beacon_off) (struct scsi_qla_host *); int (*beacon_off) (struct scsi_qla_host *);
...@@ -3748,6 +3749,11 @@ struct qlt_hw_data { ...@@ -3748,6 +3749,11 @@ struct qlt_hw_data {
#define LEAK_EXCHG_THRESH_HOLD_PERCENT 75 /* 75 percent */ #define LEAK_EXCHG_THRESH_HOLD_PERCENT 75 /* 75 percent */
struct qla_hw_data_stat {
u32 num_fw_dump;
u32 num_mpi_reset;
};
/* /*
* Qlogic host adapter specific data structure. * Qlogic host adapter specific data structure.
*/ */
...@@ -4230,7 +4236,6 @@ struct qla_hw_data { ...@@ -4230,7 +4236,6 @@ struct qla_hw_data {
uint32_t fw_dump_len; uint32_t fw_dump_len;
u32 fw_dump_alloc_len; u32 fw_dump_alloc_len;
bool fw_dumped; bool fw_dumped;
bool fw_dump_mpi;
unsigned long fw_dump_cap_flags; unsigned long fw_dump_cap_flags;
#define RISC_PAUSE_CMPL 0 #define RISC_PAUSE_CMPL 0
#define DMA_SHUTDOWN_CMPL 1 #define DMA_SHUTDOWN_CMPL 1
...@@ -4241,6 +4246,10 @@ struct qla_hw_data { ...@@ -4241,6 +4246,10 @@ struct qla_hw_data {
#define ISP_MBX_RDY 6 #define ISP_MBX_RDY 6
#define ISP_SOFT_RESET_CMPL 7 #define ISP_SOFT_RESET_CMPL 7
int fw_dump_reading; int fw_dump_reading;
void *mpi_fw_dump;
u32 mpi_fw_dump_len;
int mpi_fw_dump_reading:1;
int mpi_fw_dumped:1;
int prev_minidump_failed; int prev_minidump_failed;
dma_addr_t eft_dma; dma_addr_t eft_dma;
void *eft; void *eft;
...@@ -4454,6 +4463,8 @@ struct qla_hw_data { ...@@ -4454,6 +4463,8 @@ struct qla_hw_data {
uint16_t last_zio_threshold; uint16_t last_zio_threshold;
#define DEFAULT_ZIO_THRESHOLD 5 #define DEFAULT_ZIO_THRESHOLD 5
struct qla_hw_data_stat stat;
}; };
struct active_regions { struct active_regions {
......
...@@ -173,6 +173,7 @@ extern int ql2xenablemsix; ...@@ -173,6 +173,7 @@ extern int ql2xenablemsix;
extern int qla2xuseresexchforels; extern int qla2xuseresexchforels;
extern int ql2xexlogins; extern int ql2xexlogins;
extern int ql2xdifbundlinginternalbuffers; extern int ql2xdifbundlinginternalbuffers;
extern int ql2xfulldump_on_mpifail;
extern int qla2x00_loop_reset(scsi_qla_host_t *); extern int qla2x00_loop_reset(scsi_qla_host_t *);
extern void qla2x00_abort_all_cmds(scsi_qla_host_t *, int); extern void qla2x00_abort_all_cmds(scsi_qla_host_t *, int);
...@@ -645,6 +646,7 @@ extern void qla82xx_fw_dump(scsi_qla_host_t *, int); ...@@ -645,6 +646,7 @@ extern void qla82xx_fw_dump(scsi_qla_host_t *, int);
extern void qla8044_fw_dump(scsi_qla_host_t *, int); extern void qla8044_fw_dump(scsi_qla_host_t *, int);
extern void qla27xx_fwdump(scsi_qla_host_t *, int); extern void qla27xx_fwdump(scsi_qla_host_t *, int);
extern void qla27xx_mpi_fwdump(scsi_qla_host_t *, int);
extern ulong qla27xx_fwdt_calculate_dump_size(struct scsi_qla_host *, void *); extern ulong qla27xx_fwdt_calculate_dump_size(struct scsi_qla_host *, void *);
extern int qla27xx_fwdt_template_valid(void *); extern int qla27xx_fwdt_template_valid(void *);
extern ulong qla27xx_fwdt_template_size(void *); extern ulong qla27xx_fwdt_template_size(void *);
...@@ -933,5 +935,6 @@ extern void qla24xx_process_purex_list(struct purex_list *); ...@@ -933,5 +935,6 @@ extern void qla24xx_process_purex_list(struct purex_list *);
/* nvme.c */ /* nvme.c */
void qla_nvme_unregister_remote_port(struct fc_port *fcport); void qla_nvme_unregister_remote_port(struct fc_port *fcport);
void qla27xx_reset_mpi(scsi_qla_host_t *vha);
void qla_handle_els_plogi_done(scsi_qla_host_t *vha, struct event_arg *ea); void qla_handle_els_plogi_done(scsi_qla_host_t *vha, struct event_arg *ea);
#endif /* _QLA_GBL_H */ #endif /* _QLA_GBL_H */
...@@ -3339,6 +3339,8 @@ qla2x00_alloc_fw_dump(scsi_qla_host_t *vha) ...@@ -3339,6 +3339,8 @@ qla2x00_alloc_fw_dump(scsi_qla_host_t *vha)
dump_size / 1024); dump_size / 1024);
if (IS_QLA27XX(ha) || IS_QLA28XX(ha)) { if (IS_QLA27XX(ha) || IS_QLA28XX(ha)) {
ha->mpi_fw_dump = (char *)fw_dump +
ha->fwdt[1].dump_size;
mutex_unlock(&ha->optrom_mutex); mutex_unlock(&ha->optrom_mutex);
return; return;
} }
......
...@@ -757,6 +757,39 @@ qla2x00_find_fcport_by_nportid(scsi_qla_host_t *vha, port_id_t *id, ...@@ -757,6 +757,39 @@ qla2x00_find_fcport_by_nportid(scsi_qla_host_t *vha, port_id_t *id,
return NULL; return NULL;
} }
/* Shall be called only on supported adapters. */
static void
qla27xx_handle_8200_aen(scsi_qla_host_t *vha, uint16_t *mb)
{
struct qla_hw_data *ha = vha->hw;
bool reset_isp_needed = 0;
ql_log(ql_log_warn, vha, 0x02f0,
"MPI Heartbeat stop. MPI reset is%s needed. "
"MB0[%xh] MB1[%xh] MB2[%xh] MB3[%xh]\n",
mb[0] & BIT_8 ? "" : " not",
mb[0], mb[1], mb[2], mb[3]);
if ((mb[1] & BIT_8) == 0)
return;
ql_log(ql_log_warn, vha, 0x02f1,
"MPI Heartbeat stop. FW dump needed\n");
if (ql2xfulldump_on_mpifail) {
ha->isp_ops->fw_dump(vha, 1);
reset_isp_needed = 1;
}
ha->isp_ops->mpi_fw_dump(vha, 1);
if (reset_isp_needed) {
vha->hw->flags.fw_init_done = 0;
set_bit(ISP_ABORT_NEEDED, &vha->dpc_flags);
qla2xxx_wake_dpc(vha);
}
}
/** /**
* qla2x00_async_event() - Process aynchronous events. * qla2x00_async_event() - Process aynchronous events.
* @vha: SCSI driver HA context * @vha: SCSI driver HA context
...@@ -872,9 +905,9 @@ qla2x00_async_event(scsi_qla_host_t *vha, struct rsp_que *rsp, uint16_t *mb) ...@@ -872,9 +905,9 @@ qla2x00_async_event(scsi_qla_host_t *vha, struct rsp_que *rsp, uint16_t *mb)
"ISP System Error - mbx1=%xh mbx2=%xh mbx3=%xh.\n ", "ISP System Error - mbx1=%xh mbx2=%xh mbx3=%xh.\n ",
mb[1], mb[2], mb[3]); mb[1], mb[2], mb[3]);
ha->fw_dump_mpi = if ((IS_QLA27XX(ha) || IS_QLA28XX(ha)) &&
(IS_QLA27XX(ha) || IS_QLA28XX(ha)) && RD_REG_WORD(&reg24->mailbox7) & BIT_8)
RD_REG_WORD(&reg24->mailbox7) & BIT_8; ha->isp_ops->mpi_fw_dump(vha, 1);
ha->isp_ops->fw_dump(vha, 1); ha->isp_ops->fw_dump(vha, 1);
ha->flags.fw_init_done = 0; ha->flags.fw_init_done = 0;
QLA_FW_STOPPED(ha); QLA_FW_STOPPED(ha);
...@@ -1375,20 +1408,7 @@ qla2x00_async_event(scsi_qla_host_t *vha, struct rsp_que *rsp, uint16_t *mb) ...@@ -1375,20 +1408,7 @@ qla2x00_async_event(scsi_qla_host_t *vha, struct rsp_que *rsp, uint16_t *mb)
case MBA_IDC_AEN: case MBA_IDC_AEN:
if (IS_QLA27XX(ha) || IS_QLA28XX(ha)) { if (IS_QLA27XX(ha) || IS_QLA28XX(ha)) {
ha->flags.fw_init_done = 0; qla27xx_handle_8200_aen(vha, mb);
ql_log(ql_log_warn, vha, 0xffff,
"MPI Heartbeat stop. Chip reset needed. MB0[%xh] MB1[%xh] MB2[%xh] MB3[%xh]\n",
mb[0], mb[1], mb[2], mb[3]);
if ((mb[1] & BIT_8) ||
(mb[2] & BIT_8)) {
ql_log(ql_log_warn, vha, 0xd013,
"MPI Heartbeat stop. FW dump needed\n");
ha->fw_dump_mpi = 1;
ha->isp_ops->fw_dump(vha, 1);
}
set_bit(ISP_ABORT_NEEDED, &vha->dpc_flags);
qla2xxx_wake_dpc(vha);
} else if (IS_QLA83XX(ha)) { } else if (IS_QLA83XX(ha)) {
mb[4] = RD_REG_WORD(&reg24->mailbox4); mb[4] = RD_REG_WORD(&reg24->mailbox4);
mb[5] = RD_REG_WORD(&reg24->mailbox5); mb[5] = RD_REG_WORD(&reg24->mailbox5);
......
...@@ -35,6 +35,11 @@ static int apidev_major; ...@@ -35,6 +35,11 @@ static int apidev_major;
*/ */
struct kmem_cache *srb_cachep; struct kmem_cache *srb_cachep;
int ql2xfulldump_on_mpifail;
module_param(ql2xfulldump_on_mpifail, int, S_IRUGO | S_IWUSR);
MODULE_PARM_DESC(ql2xfulldump_on_mpifail,
"Set this to take full dump on MPI hang.");
/* /*
* CT6 CTX allocation cache * CT6 CTX allocation cache
*/ */
...@@ -2518,6 +2523,7 @@ static struct isp_operations qla27xx_isp_ops = { ...@@ -2518,6 +2523,7 @@ static struct isp_operations qla27xx_isp_ops = {
.read_nvram = NULL, .read_nvram = NULL,
.write_nvram = NULL, .write_nvram = NULL,
.fw_dump = qla27xx_fwdump, .fw_dump = qla27xx_fwdump,
.mpi_fw_dump = qla27xx_mpi_fwdump,
.beacon_on = qla24xx_beacon_on, .beacon_on = qla24xx_beacon_on,
.beacon_off = qla24xx_beacon_off, .beacon_off = qla24xx_beacon_off,
.beacon_blink = qla83xx_beacon_blink, .beacon_blink = qla83xx_beacon_blink,
......
...@@ -12,6 +12,33 @@ ...@@ -12,6 +12,33 @@
#define IOBASE(vha) IOBAR(ISPREG(vha)) #define IOBASE(vha) IOBAR(ISPREG(vha))
#define INVALID_ENTRY ((struct qla27xx_fwdt_entry *)0xffffffffffffffffUL) #define INVALID_ENTRY ((struct qla27xx_fwdt_entry *)0xffffffffffffffffUL)
/* hardware_lock assumed held. */
static void
qla27xx_write_remote_reg(struct scsi_qla_host *vha,
u32 addr, u32 data)
{
char *reg = (char *)ISPREG(vha);
ql_dbg(ql_dbg_misc, vha, 0xd300,
"%s: addr/data = %xh/%xh\n", __func__, addr, data);
WRT_REG_DWORD(reg + IOBASE(vha), 0x40);
WRT_REG_DWORD(reg + 0xc4, data);
WRT_REG_DWORD(reg + 0xc0, addr);
}
void
qla27xx_reset_mpi(scsi_qla_host_t *vha)
{
ql_dbg(ql_dbg_misc + ql_dbg_verbose, vha, 0xd301,
"Entered %s.\n", __func__);
qla27xx_write_remote_reg(vha, 0x104050, 0x40004);
qla27xx_write_remote_reg(vha, 0x10405c, 0x4);
vha->hw->stat.num_mpi_reset++;
}
static inline void static inline void
qla27xx_insert16(uint16_t value, void *buf, ulong *len) qla27xx_insert16(uint16_t value, void *buf, ulong *len)
{ {
...@@ -997,6 +1024,62 @@ qla27xx_fwdt_template_valid(void *p) ...@@ -997,6 +1024,62 @@ qla27xx_fwdt_template_valid(void *p)
return true; return true;
} }
void
qla27xx_mpi_fwdump(scsi_qla_host_t *vha, int hardware_locked)
{
ulong flags = 0;
bool need_mpi_reset = 1;
#ifndef __CHECKER__
if (!hardware_locked)
spin_lock_irqsave(&vha->hw->hardware_lock, flags);
#endif
if (!vha->hw->mpi_fw_dump) {
ql_log(ql_log_warn, vha, 0x02f3, "-> mpi_fwdump no buffer\n");
} else if (vha->hw->mpi_fw_dumped) {
ql_log(ql_log_warn, vha, 0x02f4,
"-> MPI firmware already dumped (%p) -- ignoring request\n",
vha->hw->mpi_fw_dump);
} else {
struct fwdt *fwdt = &vha->hw->fwdt[1];
ulong len;
void *buf = vha->hw->mpi_fw_dump;
ql_log(ql_log_warn, vha, 0x02f5, "-> fwdt1 running...\n");
if (!fwdt->template) {
ql_log(ql_log_warn, vha, 0x02f6,
"-> fwdt1 no template\n");
goto bailout;
}
len = qla27xx_execute_fwdt_template(vha, fwdt->template, buf);
if (len == 0) {
goto bailout;
} else if (len != fwdt->dump_size) {
ql_log(ql_log_warn, vha, 0x02f7,
"-> fwdt1 fwdump residual=%+ld\n",
fwdt->dump_size - len);
} else {
need_mpi_reset = 0;
}
vha->hw->mpi_fw_dump_len = len;
vha->hw->mpi_fw_dumped = 1;
ql_log(ql_log_warn, vha, 0x02f8,
"-> MPI firmware dump saved to buffer (%lu/%p)\n",
vha->host_no, vha->hw->mpi_fw_dump);
qla2x00_post_uevent_work(vha, QLA_UEVENT_CODE_FW_DUMP);
}
bailout:
if (need_mpi_reset)
qla27xx_reset_mpi(vha);
#ifndef __CHECKER__
if (!hardware_locked)
spin_unlock_irqrestore(&vha->hw->hardware_lock, flags);
#endif
}
void void
qla27xx_fwdump(scsi_qla_host_t *vha, int hardware_locked) qla27xx_fwdump(scsi_qla_host_t *vha, int hardware_locked)
{ {
...@@ -1015,30 +1098,25 @@ qla27xx_fwdump(scsi_qla_host_t *vha, int hardware_locked) ...@@ -1015,30 +1098,25 @@ qla27xx_fwdump(scsi_qla_host_t *vha, int hardware_locked)
vha->hw->fw_dump); vha->hw->fw_dump);
} else { } else {
struct fwdt *fwdt = vha->hw->fwdt; struct fwdt *fwdt = vha->hw->fwdt;
uint j;
ulong len; ulong len;
void *buf = vha->hw->fw_dump; void *buf = vha->hw->fw_dump;
uint count = vha->hw->fw_dump_mpi ? 2 : 1;
for (j = 0; j < count; j++, fwdt++, buf += len) { ql_log(ql_log_warn, vha, 0xd011, "-> fwdt0 running...\n");
ql_log(ql_log_warn, vha, 0xd011,
"-> fwdt%u running...\n", j);
if (!fwdt->template) { if (!fwdt->template) {
ql_log(ql_log_warn, vha, 0xd012, ql_log(ql_log_warn, vha, 0xd012,
"-> fwdt%u no template\n", j); "-> fwdt0 no template\n");
break; goto bailout;
} }
len = qla27xx_execute_fwdt_template(vha, len = qla27xx_execute_fwdt_template(vha, fwdt->template, buf);
fwdt->template, buf);
if (len == 0) { if (len == 0) {
goto bailout; goto bailout;
} else if (len != fwdt->dump_size) { } else if (len != fwdt->dump_size) {
ql_log(ql_log_warn, vha, 0xd013, ql_log(ql_log_warn, vha, 0xd013,
"-> fwdt%u fwdump residual=%+ld\n", "-> fwdt0 fwdump residual=%+ld\n",
j, fwdt->dump_size - len); fwdt->dump_size - len);
} }
}
vha->hw->fw_dump_len = buf - (void *)vha->hw->fw_dump; vha->hw->fw_dump_len = len;
vha->hw->fw_dumped = 1; vha->hw->fw_dumped = 1;
ql_log(ql_log_warn, vha, 0xd015, ql_log(ql_log_warn, vha, 0xd015,
...@@ -1048,7 +1126,6 @@ qla27xx_fwdump(scsi_qla_host_t *vha, int hardware_locked) ...@@ -1048,7 +1126,6 @@ qla27xx_fwdump(scsi_qla_host_t *vha, int hardware_locked)
} }
bailout: bailout:
vha->hw->fw_dump_mpi = 0;
#ifndef __CHECKER__ #ifndef __CHECKER__
if (!hardware_locked) if (!hardware_locked)
spin_unlock_irqrestore(&vha->hw->hardware_lock, flags); spin_unlock_irqrestore(&vha->hw->hardware_lock, flags);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment