From: Olof Johansson <olof@lixom.net>
To: Tejun Heo <tj@kernel.org>
Cc: linux-ide@vger.kernel.org, jgarzik@pobox.com
Subject: Re: link resets with SSD on AHCI
Date: Wed, 12 May 2010 13:49:06 -0500 [thread overview]
Message-ID: <20100512184906.GA1893@lixom.net> (raw)
In-Reply-To: <4BE1452A.6040607@kernel.org>
On Wed, May 05, 2010 at 12:15:06PM +0200, Tejun Heo wrote:
> Hello,
>
> On 04/29/2010 11:59 PM, Olof Johansson wrote:
> > I did notice that ALPM is enabled at boot, and doesn't seem to be
> > re-enabled after the error reset. Based on this, I experimented with
> > disabling it (just returning -EINVAL in ahci_enable_alpm). That did make
> > the problem not happen after a significant test run (overnight vs 4.5
> > minutes above).
>
> It could be that libata's ALPM enable sequence isn't liked by the
> controller. libata first resets the link disabling all powersave
> transitions, then turn on ALPM then allows powersave transitions.
> It's possible that the controller or device somehow gets upset by this
> (ie. the device is told to go to powersave mode only to find out that
> the host side isn't allowing it).
>
> Does the attached patch make any difference? Can you please post the
> kernel boot log with the patch applied?
The patch didn't make a difference for me, but I got sidetracked looking
at some other things and didn't get a chance to collect data just yet;
it's coming.
I also got second thoughts about what actually makes it happen; and
I am now of the suspicion that it does indeed take an invocation of
the laptop-mode tools setting power settings for battery operation
(i.e. taking the link off of max_performance) for the problem to
show. Earlier I had been of the impression that it happened even with
it being left untouched, that does not seem to be the case.
As for ALPM overall: I have also noticed that the link doesn't actually
seem to be put in low-power mode with these SSD devices, since I see
now power consumption difference between having it on or off.
-Olof
prev parent reply other threads:[~2010-05-12 18:32 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-29 21:59 link resets with SSD on AHCI Olof Johansson
2010-05-05 10:15 ` Tejun Heo
2010-05-12 18:49 ` Olof Johansson [this message]
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=20100512184906.GA1893@lixom.net \
--to=olof@lixom.net \
--cc=jgarzik@pobox.com \
--cc=linux-ide@vger.kernel.org \
--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.