RE: [PATCH] mptfusion: print Doorbell register in a case of hard reset and timeout

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----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


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux