> -----Original Message----- > From: Kei Tokunaga [mailto:tokunaga.keiich@xxxxxxxxxxxxxx] > Sent: Tuesday, June 22, 2010 2:48 PM > To: Desai, Kashyap > Cc: linux-scsi@xxxxxxxxxxxxxxx; Kei Tokunaga > Subject: Re: [PATCH] mptfusion: print Doorbell register in a case of > hard reset and timeout > > Kei Tokunaga wrote: > > Hi Kashyap, > > > > Printing Doorbell register in a case of hard reset and timeout > > should be useful for figuring out the state of the system. > > Hi Kashyap, > > Can you give this patch an ACK? I confirmed it still applies to > 03.04.16. Kei, Patch looks OK to me. I have commented few things below. Can you please correct it and resend? Thanks, Kashyap > > Thanks, > Kei > > > Signed-off-by: Kei Tokunaga <tokunaga.keiich@xxxxxxxxxxxxxx> > > > > drivers/message/fusion/mptbase.c | 13 ++++++++----- > > drivers/message/fusion/mptctl.c | 20 ++++++++++++++++---- > > drivers/message/fusion/mptsas.c | 5 +++-- > > drivers/message/fusion/mptscsih.c | 12 ++++++++---- > > 4 files changed, 35 insertions(+), 15 deletions(-) > > > > diff --git a/drivers/message/fusion/mptbase.c > b/drivers/message/fusion/mptbase.c > > index a6a5701..50d5b23 100644 > > --- a/drivers/message/fusion/mptbase.c > > +++ b/drivers/message/fusion/mptbase.c > > @@ -5062,8 +5062,9 @@ mptbase_sas_persist_operation(MPT_ADAPTER *ioc, > u8 persist_opcode) > > if (ioc->mptbase_cmds.status & > MPT_MGMT_STATUS_DID_IOCRESET) > > goto out; > > if (!timeleft) { > > - printk(KERN_DEBUG "%s: Issuing Reset from %s!!\n", > > - ioc->name, __func__); > > + printk(MYIOC_s_WARN_FMT > > + "Issuing Reset from %s!!, doorbell=0x%08x\n", > > + ioc->name, __func__, mpt_GetIocState(ioc, 0)); > > mpt_Soft_Hard_ResetHandler(ioc, CAN_SLEEP); > > mpt_free_msg_frame(ioc, mf); > > } > > @@ -6454,8 +6455,9 @@ out: > > mutex_unlock(&ioc->mptbase_cmds.mutex); > > if (issue_hard_reset) { > > issue_hard_reset = 0; > > - printk(MYIOC_s_WARN_FMT "Issuing Reset from %s!!\n", > > - ioc->name, __func__); > > + printk(MYIOC_s_WARN_FMT > > + "Issuing Reset from %s!!, doorbell=0x%08x\n", > > + ioc->name, __func__, mpt_GetIocState(ioc, 0)); > > if (retry_count == 0) { > > if (mpt_Soft_Hard_ResetHandler(ioc, CAN_SLEEP) != 0) > > retry_count++; > > @@ -7144,7 +7146,8 @@ mpt_HardResetHandler(MPT_ADAPTER *ioc, int > sleepFlag) > > rc = mpt_do_ioc_recovery(ioc, MPT_HOSTEVENT_IOC_RECOVER, > sleepFlag); > > if (rc != 0) { > > printk(KERN_WARNING MYNAM > > - ": WARNING - (%d) Cannot recover %s\n", rc, ioc->name); > > + ": WARNING - (%d) Cannot recover %s, > doorbell=0x%08x\n", > > + rc, ioc->name, mpt_GetIocState(ioc, 0)); > > } else { > > if (ioc->hard_resets < -1) > > ioc->hard_resets++; > > diff --git a/drivers/message/fusion/mptctl.c > b/drivers/message/fusion/mptctl.c > > index f06b291..b8a925c 100644 > > --- a/drivers/message/fusion/mptctl.c > > +++ b/drivers/message/fusion/mptctl.c > > @@ -946,10 +946,14 @@ retry_wait: > > mpt_free_msg_frame(iocp, mf); > > goto fwdl_out; > > } > > - if (!timeleft) > > + if (!timeleft) { > > + printk(MYIOC_s_WARN_FMT > > + "FW download timeout, doorbell=0x%08x\n", > > + iocp->name, mpt_GetIocState(iocp, 0)); > > mptctl_timeout_expired(iocp, mf); > > - else > > + } else { > > goto retry_wait; > > + } This extra brackets are not required if it has single code line. > > goto fwdl_out; > > } > > > > @@ -2293,6 +2297,10 @@ retry_wait: > > goto done_free_mem; > > } > > if (!timeleft) { > > + printk(MYIOC_s_WARN_FMT > > + "mpt cmd timeout, doorbell=0x%08x" > > + " function=0x%x\n", > > + ioc->name, mpt_GetIocState(ioc, 0), function); > > if (function == MPI_FUNCTION_SCSI_TASK_MGMT) > > mutex_unlock(&ioc->taskmgmt_cmds.mutex); > > mptctl_timeout_expired(ioc, mf); > > @@ -2600,10 +2608,14 @@ retry_wait: > > mpt_free_msg_frame(ioc, mf); > > goto out; > > } > > - if (!timeleft) > > + if (!timeleft) { > > + printk(MYIOC_s_WARN_FMT > > + "HOST INFO command timeout, > doorbell=0x%08x\n", > > + ioc->name, mpt_GetIocState(ioc, 0)); > > mptctl_timeout_expired(ioc, mf); > > - else > > + } else { > > goto retry_wait; > > + } Same as above. > > goto out; > > } > > > > diff --git a/drivers/message/fusion/mptsas.c > b/drivers/message/fusion/mptsas.c > > index ac000e8..0a34e0c 100644 > > --- a/drivers/message/fusion/mptsas.c > > +++ b/drivers/message/fusion/mptsas.c > > @@ -4727,8 +4727,9 @@ mptsas_broadcast_primative_work(struct > fw_event_work *fw_event) > > mutex_unlock(&ioc->taskmgmt_cmds.mutex); > > > > if (issue_reset) { > > - printk(MYIOC_s_WARN_FMT "Issuing Reset from %s!!\n", > > - ioc->name, __func__); > > + printk(MYIOC_s_WARN_FMT > > + "Issuing Reset from %s!! doorbell=0x%08x\n", > > + ioc->name, __func__, mpt_GetIocState(ioc, 0)); > > mpt_Soft_Hard_ResetHandler(ioc, CAN_SLEEP); > > } > > mptsas_free_fw_event(ioc, fw_event); > > diff --git a/drivers/message/fusion/mptscsih.c > b/drivers/message/fusion/mptscsih.c > > index 7bd4c0f..693af3b 100644 > > --- a/drivers/message/fusion/mptscsih.c > > +++ b/drivers/message/fusion/mptscsih.c > > @@ -1704,8 +1704,9 @@ mptscsih_IssueTaskMgmt(MPT_SCSI_HOST *hd, u8 > type, u8 channel, u8 id, int lun, > > > > CLEAR_MGMT_STATUS(ioc->taskmgmt_cmds.status) > > if (issue_hard_reset) { > > - printk(MYIOC_s_WARN_FMT "Issuing Reset from %s!!\n", > > - ioc->name, __func__); > > + printk(MYIOC_s_WARN_FMT > > + "Issuing Reset from %s!! doorbell=0x%08x\n", > > + ioc->name, __func__, mpt_GetIocState(ioc, 0)); > > retval = mpt_Soft_Hard_ResetHandler(ioc, CAN_SLEEP); > > mpt_free_msg_frame(ioc, mf); > > } > > @@ -3047,8 +3048,11 @@ mptscsih_do_cmd(MPT_SCSI_HOST *hd, > INTERNAL_CMD *io) > > goto out; > > } > > if (!timeleft) { > > - printk(MYIOC_s_WARN_FMT "Issuing Reset from %s!!\n", > > - ioc->name, __func__); > > + printk(MYIOC_s_WARN_FMT > > + "Issuing Reset from %s!! doorbell=0x%08xh" > > + " cmd=0x%02x\n", > > + ioc->name, __func__, mpt_GetIocState(ioc, 0), > > + cmd); > > mpt_Soft_Hard_ResetHandler(ioc, CAN_SLEEP); > > mpt_free_msg_frame(ioc, mf); > > } > > > > > > > > > > > > -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html