All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jeff@garzik.org>
To: Tejun Heo <tj@kernel.org>
Cc: Arjan van de Ven <arjan@linux.intel.com>,
	t.artem@mailcity.com,
	"linux-ide@vger.kernel.org" <linux-ide@vger.kernel.org>
Subject: Re: ahci: CAP_SSS and parallel scan
Date: Sat, 08 May 2010 17:53:30 -0400	[thread overview]
Message-ID: <4BE5DD5A.8030701@garzik.org> (raw)
In-Reply-To: <4BE52383.9080908@kernel.org>

On 05/08/2010 04:40 AM, Tejun Heo wrote:
> On 05/07/2010 11:33 PM, Jeff Garzik wrote:
>> I'm not sure the usefulness is limited, as it definitely avoids power
>> spikes on server BIOSen that care.
>
> So, are there actually machines which get affected by this?  In that
> case, sure, the current behavior is the right one.

The current behavior is the safe, conservative interpretation.  The 
alternative is risking unwanted power spikes for the sake of boot speed.

I would set a high barrier for taking that risk:  polling multiple 
hardware vendors about in-the-field SSS usage.


>> Also, it seems unwise for the Linux SATA driver to do the exact
>> -opposite- of what the SSS bit intends, by default.
>
> Yeah, well, the thing is that SSS in itself doesn't really indicate
> need for staggered spin up.  It just says it knows how to.

A valid point, I agree.  But absent any other method of communicating 
that SSS behavior is desired, activating staggered spin-up based on SSS 
is the only logical interpretation, AFAICS.  SSS bit set implies SSS 
-might- be needed.  If SSS might be needed, then one must stagger 
spin-up or risk failing to meet the requirement.  !SSS is the only case 
where you are -guaranteed- not to need staggered spin-up.

	Jeff




  reply	other threads:[~2010-05-08 21:53 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-06  8:22 ahci: CAP_SSS and parallel scan Tejun Heo
2010-05-06  9:30 ` Arjan van de Ven
2010-05-06 14:44   ` Tejun Heo
2010-05-06 19:49     ` Jeff Garzik
2010-05-07  5:14       ` Tejun Heo
2010-05-07 21:33         ` Jeff Garzik
2010-05-08  8:40           ` Tejun Heo
2010-05-08 21:53             ` Jeff Garzik [this message]
2010-05-06 20:19     ` Mark Lord
2010-05-07  5:16       ` Tejun Heo

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=4BE5DD5A.8030701@garzik.org \
    --to=jeff@garzik.org \
    --cc=arjan@linux.intel.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=t.artem@mailcity.com \
    --cc=tj@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 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.