From: James Bottomley <James.Bottomley@SteelEye.com>
To: brking@us.ibm.com
Cc: SCSI Mailing List <linux-scsi@vger.kernel.org>, haren@us.ibm.com
Subject: Re: [PATCH 1/1] ipr: Fix for adapter shutdown issue
Date: Wed, 15 Jun 2005 10:47:38 -0500 [thread overview]
Message-ID: <1118850458.5045.34.camel@mulgrave> (raw)
In-Reply-To: <42B04A9C.7050503@us.ibm.com>
On Wed, 2005-06-15 at 10:34 -0500, Brian King wrote:
> For scsi disks attached to an ipr adapter, the SYNCHRONIZE_CACHE command
> gets sent to the disk and works just like when attached to any other
> HBA. The ipr disk array devices, however, do not support the SYNC_CACHE command,
> nor do they support the caching mode page, so SYNC_CACHE never gets sent.
> So, the shutdown hook is needed to flush the adapter's battery backed write
> cache for all attached disk arrays on system shutdown.
Well, that means you have a whole lot more trouble in 2.6.12 than simply
failing to flush a cache on shutdown. The barrier code now uses cache
synchronization commands, so if you crash the on-disk image will not be
what a journalling filesystem expects. As long as the battery keeps the
information alive in the cache, I assume this corrects itself when ipr
next powers up, but if you trusted this, you wouldn't be fussing about
the shutdown cache flush, now would you?
James
next prev parent reply other threads:[~2005-06-15 15:47 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-06-15 15:15 [PATCH 1/1] ipr: Fix for adapter shutdown issue brking
2005-06-15 15:17 ` Christoph Hellwig
2005-06-15 16:01 ` Greg KH
2005-06-15 16:14 ` Brian King
2005-06-15 15:28 ` James Bottomley
2005-06-15 15:34 ` Brian King
2005-06-15 15:47 ` James Bottomley [this message]
2005-06-15 16:11 ` Brian King
2005-06-15 16:44 ` James Bottomley
2005-06-15 17:05 ` Patrick Mansfield
2005-06-15 17:23 ` James Bottomley
2005-06-16 12:00 ` Jens Axboe
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1118850458.5045.34.camel@mulgrave \
--to=james.bottomley@steeleye.com \
--cc=brking@us.ibm.com \
--cc=haren@us.ibm.com \
--cc=linux-scsi@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox