public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
* [patch 01/17] mptbase: reset ioc initiator during PCI resume
@ 2008-03-28 21:48 akpm
  2008-03-30 16:22 ` James Bottomley
  0 siblings, 1 reply; 2+ messages in thread
From: akpm @ 2008-03-28 21:48 UTC (permalink / raw)
  To: James.Bottomley; +Cc: linux-scsi, akpm, djwong, Eric.Moore, Sathya.Prakash

From: "Darrick J. Wong" <djwong@us.ibm.com>

It appears that the LSI SAS 1064E chip needs to be reset after a
suspend/resume cycle before the driver attempts further communications with
the chip.  Without this patch, resuming the chip results in this error
message being printed repeatedly and no more disk I/O.

mptbase: ioc0: ERROR - Invalid IOC facts reply, msgLength=0 offsetof=6!

So far it seems to fix suspend/resume on all the MPT Fusion cards I have
(SAS and U320 SCSI) but since I don't know the internals of that chip I
can't say for sure if this is a proper fix.

Signed-off-by: Darrick J. Wong <djwong@us.ibm.com>
Cc: "Moore, Eric" <Eric.Moore@lsi.com>
Cc: <Sathya.Prakash@lsi.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 drivers/message/fusion/mptbase.c |    6 ++++++
 1 file changed, 6 insertions(+)

diff -puN drivers/message/fusion/mptbase.c~mptbase-reset-ioc-initiator-during-pci-resume drivers/message/fusion/mptbase.c
--- a/drivers/message/fusion/mptbase.c~mptbase-reset-ioc-initiator-during-pci-resume
+++ a/drivers/message/fusion/mptbase.c
@@ -1892,6 +1892,12 @@ mpt_resume(struct pci_dev *pdev)
 		}
 	}
 
+	/* put ioc into READY_STATE */
+	if(SendIocReset(ioc, MPI_FUNCTION_IOC_MESSAGE_UNIT_RESET, CAN_SLEEP)) {
+		printk(MYIOC_s_ERR_FMT
+		"pci-resume:  IOC msg unit reset failed!\n", ioc->name);
+	}
+
 	/* bring ioc to operational state */
 	printk(MYIOC_s_INFO_FMT "Sending mpt_do_ioc_recovery\n", ioc->name);
 	recovery_state = mpt_do_ioc_recovery(ioc, MPT_HOSTEVENT_IOC_BRINGUP,
_

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [patch 01/17] mptbase: reset ioc initiator during PCI resume
  2008-03-28 21:48 [patch 01/17] mptbase: reset ioc initiator during PCI resume akpm
@ 2008-03-30 16:22 ` James Bottomley
  0 siblings, 0 replies; 2+ messages in thread
From: James Bottomley @ 2008-03-30 16:22 UTC (permalink / raw)
  To: akpm; +Cc: linux-scsi, djwong, Eric.Moore, Sathya.Prakash

On Fri, 2008-03-28 at 14:48 -0700, akpm@linux-foundation.org wrote:
> From: "Darrick J. Wong" <djwong@us.ibm.com>
> 
> It appears that the LSI SAS 1064E chip needs to be reset after a
> suspend/resume cycle before the driver attempts further communications with
> the chip.  Without this patch, resuming the chip results in this error
> message being printed repeatedly and no more disk I/O.
> 
> mptbase: ioc0: ERROR - Invalid IOC facts reply, msgLength=0 offsetof=6!
> 
> So far it seems to fix suspend/resume on all the MPT Fusion cards I have
> (SAS and U320 SCSI) but since I don't know the internals of that chip I
> can't say for sure if this is a proper fix.

After a huge amount of wrangling, a fix for this which everyone agreed
to is upstream here:

commit 4d4109d0eb6930e59942d019980b83af679f187b
Author: Prakash, Sathya <sathya.prakash@lsi.com>
Date:   Fri Mar 7 16:19:50 2008 +0530

    [SCSI] mpt fusion: Power Management fixes for MPT SAS PCI-E controllers

James



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-03-30 16:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-03-28 21:48 [patch 01/17] mptbase: reset ioc initiator during PCI resume akpm
2008-03-30 16:22 ` James Bottomley

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox