linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Gabriele Mazzotta <gabriele.mzt@gmail.com>
To: Tejun Heo <tj@kernel.org>
Cc: linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org
Subject: Re: SATA link power management issues
Date: Fri, 09 Jan 2015 23:50:05 +0100	[thread overview]
Message-ID: <1437237.yiFxI2gUSM@xps13> (raw)
In-Reply-To: <20150109220253.GI2785@htj.dyndns.org>

On Friday 09 January 2015 17:02:53 Tejun Heo wrote:
> On Fri, Jan 09, 2015 at 05:00:18PM -0500, Tejun Heo wrote:
> > It's not normal.  It definitely doesn't happen with all devices.  I
> > guess it could be a quirk from the device side.  What's on the other
> > side of the connector?  Can you post the full kernel log after such
> > incidence?
> 
> What happens if you comment out the DIPM part of ata_eh_set_lpm() -
> the first ata_for_each_dev() loop in the function?

I commented the first loop and something seems to have changed.
I was able to get back to max_performance with no errors once, but
most of the times I get errors.
The last policy change here below went fine for instance:

[   32.368603] setting min_power
[   32.368670] ahci 0000:00:1f.2: port does not support device sleep
[   38.550840] setting max_performance
[   38.564932] ahci 0000:00:1f.2: port does not support device sleep
[   38.579716] ata3: exception Emask 0x10 SAct 0x0 SErr 0x50000 action 0xe frozen
[   38.579722] ata3: irq_stat 0x00400000, PHY RDY changed
[   38.579726] ata3: SError: { PHYRdyChg CommWake }
[   38.579731] ata3: hard resetting link
[   39.301543] ata3: SATA link up 6.0 Gbps (SStatus 133 SControl 300)
[   39.304783] ata3.00: configured for UDMA/133
[   39.317499] ahci 0000:00:1f.2: port does not support device sleep
[   39.317509] ata3: EH complete
[   46.412652] setting min_power
[   46.412769] ahci 0000:00:1f.2: port does not support device sleep
[   53.869869] setting max_performance
[   53.885083] ahci 0000:00:1f.2: port does not support device sleep
[   53.898857] ata3: exception Emask 0x10 SAct 0x0 SErr 0x50000 action 0xe frozen
[   53.898865] ata3: irq_stat 0x00400000, PHY RDY changed
[   53.898870] ata3: SError: { PHYRdyChg CommWake }
[   53.898877] ata3: hard resetting link
[   54.621720] ata3: SATA link up 6.0 Gbps (SStatus 133 SControl 300)
[   54.624864] ata3.00: configured for UDMA/133
[   54.637674] ahci 0000:00:1f.2: port does not support device sleep
[   54.637680] ata3: EH complete
[   58.837307] setting min_power
[   58.837427] ahci 0000:00:1f.2: port does not support device sleep
[   63.269078] setting max_performance
[   63.284571] ahci 0000:00:1f.2: port does not support device sleep
[   63.297954] ata3: exception Emask 0x10 SAct 0x0 SErr 0x50000 action 0xe frozen
[   63.297958] ata3: irq_stat 0x00400000, PHY RDY changed
[   63.297961] ata3: SError: { PHYRdyChg CommWake }
[   63.297966] ata3: hard resetting link
[   64.021185] ata3: SATA link up 6.0 Gbps (SStatus 133 SControl 300)
[   64.024291] ata3.00: configured for UDMA/133
[   64.037206] ahci 0000:00:1f.2: port does not support device sleep
[   64.037214] ata3: EH complete
[   69.282264] setting min_power
[   69.282320] ahci 0000:00:1f.2: port does not support device sleep
[   74.933589] setting max_performance
[   74.945847] ahci 0000:00:1f.2: port does not support device sleep

  parent reply	other threads:[~2015-01-09 22:50 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-08 19:37 SATA link power management issues Gabriele Mazzotta
2015-01-09 22:00 ` Tejun Heo
2015-01-09 22:02   ` Tejun Heo
2015-01-09 22:05     ` Tejun Heo
2015-01-09 22:50     ` Gabriele Mazzotta [this message]
2015-01-12 13:16       ` Tejun Heo
2015-01-12 17:03         ` Gabriele Mazzotta
2015-01-12 17:07           ` Tejun Heo
2015-01-12 17:15             ` Gabriele Mazzotta
2015-01-09 22:40   ` Gabriele Mazzotta
     [not found] ` <CAOHikRDTgVWL7_WUS3Yvce01GyP5TcyRm8ojfAyTHLTZ7xiR7A@mail.gmail.com>
2015-01-12 17:16   ` Suman Tripathi
2015-01-12 20:26     ` Gabriele Mazzotta
     [not found]       ` <CAOHikRCp=m6tgKEmdg2pLj=XB2ugCGhj0rkeqCzF2+DPR8O9sw@mail.gmail.com>
2015-01-12 20:36         ` Gabriele Mazzotta
     [not found]           ` <CAOHikRC43JBMwCahBqT1znJLBnbr3Ui+szKhVQoK8cpfapTQyg@mail.gmail.com>
2015-01-12 20:54             ` Gabriele Mazzotta
2015-01-12 23:05 ` Gabriele Mazzotta
2015-02-22 20:53   ` Gabriele Mazzotta
2015-04-20 20:02     ` Gabriele Mazzotta
2015-04-21 15:31       ` Tejun Heo
2015-04-21 20:29         ` Gabriele Mazzotta
2015-04-21 20:56           ` Tejun Heo
2015-04-21 21:44             ` Gabriele Mazzotta
2015-04-24 21:44               ` Tejun Heo
2015-04-25  9:34                 ` [PATCH] libata: Ignore spurious PHY events on LPM policy change Gabriele Mazzotta
2015-04-25 15:37                   ` Tejun Heo
2015-04-25 17:57                     ` Gabriele Mazzotta

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=1437237.yiFxI2gUSM@xps13 \
    --to=gabriele.mzt@gmail.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).