All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matthew Wilcox <matthew@wil.cx>
To: James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: "Rengarajan, Narayanan (STSD)" <narayanan.rengarajan@hp.com>,
	"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>
Subject: Re: [PATCH 1/1] : Spinning up disk is observed on standby paths until timeout, resulting in longer path restoration time.
Date: Fri, 20 Feb 2009 09:13:34 -0700	[thread overview]
Message-ID: <20090220161334.GV16841@parisc-linux.org> (raw)
In-Reply-To: <1235145795.9025.4.camel@localhost.localdomain>

On Fri, Feb 20, 2009 at 10:03:15AM -0600, James Bottomley wrote:
> > The port isn't coming out of standby state.  We send it a TEST_UNIT_READY,
> > it replies with a 0x04/0x0b.  At that point, we currently decide to send
> > it a START_STOP and wait 100 seconds.  This is clearly a crappy decision
> > on our part, we should just bail.
> 
> So we should be bailing on manual intervention, TP standby and TP
> unavailable?  It looks like TP assymetric access transition is waitable.

I think that's correct (and I think my version of this patch makes that
clearer).

SPC 4r14 isn't clear on 'Asymmetric Access Transition' -- I can't tell
whether that state is entered on transition *to* active, or *from*
active, or both.

> It also looks like offline and notify (enable spinup) required are also
> not worth waiting for ... although the latter is a SAS power management
> state which it's not clear to me how to handle properly.

Offline is only applicable to M and V (Media Changer and Automation)
devices, neither of which should be attached to by sd.

I don't know what 'Enable Spinup' is for -- maybe Doug knows?  Sending a
START_STOP to the device might be exactly what they intend for us to do.
Under a 'First, Do No Harm' theory, perhaps we should leave well enough
alone and just add Standby and Unavailable?

-- 
Matthew Wilcox				Intel Open Source Technology Centre
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step."

  reply	other threads:[~2009-02-20 16:13 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-20 11:14 [PATCH 1/1] : Spinning up disk is observed on standby paths until timeout, resulting in longer path restoration time Rengarajan, Narayanan (STSD)
2009-02-20 15:36 ` James Bottomley
2009-02-20 15:52   ` Matthew Wilcox
2009-02-20 16:03     ` James Bottomley
2009-02-20 16:13       ` Matthew Wilcox [this message]
2009-02-20 16:24         ` James Bottomley
2009-02-20 17:04           ` Matthew Wilcox
2009-02-23 11:48             ` Rengarajan, Narayanan (STSD)
2009-02-23 14:52               ` James Bottomley
2009-02-28 21:26                 ` Matthew Wilcox
2009-02-28 23:56                   ` James Bottomley

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=20090220161334.GV16841@parisc-linux.org \
    --to=matthew@wil.cx \
    --cc=James.Bottomley@HansenPartnership.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=narayanan.rengarajan@hp.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.