[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4C7F6931.4050707@gmail.com>
Date: Thu, 02 Sep 2010 11:06:57 +0200
From: Tejun Heo <htejun@...il.com>
To: "Rafael J. Wysocki" <rjw@...k.pl>
CC: Stephan Diestelhorst <stephan.diestelhorst@....com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-ide@...r.kernel.org" <linux-ide@...r.kernel.org>,
"linux-pm@...ts.osdl.org" <linux-pm@...ts.osdl.org>,
Stephan Diestelhorst <stephan.diestelhorst@...il.com>
Subject: Re: [PATCH] SATA / AHCI: Do not play with the link PM during suspend
to RAM
Hello, Rafael.
On 08/27/2010 01:09 AM, Rafael J. Wysocki wrote:
> Well, no luck. I was able to reproduce the issue on my box with this patch
> applied on top of 2.6.32-rc2.
>
> Which probably means that the link power management is not really involved
> here and seems to turn up this statement:
>
> rc = ata_host_request_pm(host, mesg, 0, ATA_EHI_QUIET, 1);
>
> in ata_host_suspend() as the culprit.
>
> Does it make sense?
So, LPM doesn't have anything to do with the problem. The fact that
this only happens on specific machines is strange. Maybe the BIOS is
doing something fishy to the controller on disk spindown during
suspend. Can the BIOS tell that the system is going for suspend by
the time sd suspend is called? I'll prep another patch which will
make EH skip certain steps and ignore failures during suspend, which
basically mimics what your patch does but in safer way.
Thanks.
--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists