lkml.org 
[lkml]   [2007]   [Feb]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: 2.6.20-git10 (and -git13): BUG at drivers/pci/pci.c:817 during resume from disk
Adrian Bunk wrote:
> On Sun, Feb 25, 2007 at 01:28:42PM +0900, Tejun Heo wrote:
>> Hello, Adrian.
>>
>> Adrian Bunk wrote:
>>> On Tue, Feb 20, 2007 at 12:43:41PM +0900, Tejun Heo wrote:
>>>> Rafael J. Wysocki wrote:
>>>>> Update:
>>>>>
>>>>> I get the same BUG with 2.6.20-git13 100% of the time during the resume.
>>>>> The system seems to be fully functional nonetheless.
>>>> Known bug, will be fixed soon.
>>> Is this a variation of
>>>
>>> Subject : HP nx6325: usb mouse stops working after resume from ram
>>> References : http://lkml.org/lkml/2007/2/21/413
>>> Submitter : Arkadiusz Miskiewicz <arekm@maven.pl>
>>> Caused-By : Konstantin Karasyov <konstantin.a.karasyov@intel.com>
>>> commit 0a6139027f3986162233adc17285151e78b39cac
>>> Status : unknown
>>>
>>> or one of the other ACPI problems in 2.6.21-rc1?
>> No, it's not related to ACPI. It's caused by devres changes and fixed
>> in libata-dev tree.
>
> I'm currently a bit lost with all the ATA/ACPI/suspend regression
> reports...
>
> Which patch fixes this regression?

What I was talking about was the following one. It's a libata bug
discovered by devres sanity check and unrelated to ACPI.

Author: Tejun Heo <htejun@gmail.com> 2007-02-20 18:14:48
Committer: Jeff Garzik <jeff@garzik.org> 2007-02-21 18:58:18
Parent: 5e5c74a5e11d1e2a99d03132cc6c4455016db6c2 (sata_nv: delay on switching between NCQ and non-NCQ commands)
Child: 6d1245bf294e9ea65b3717be9fa0338bfb6ff6c9 (libata: separate out ata_ncq_enabled())
Branches: upstream-fixes, upstream, ALL
Follows: v2.6.21-rc1
Precedes:

libata: disable pdev on all suspend events

libata used disable pdev only on PM_EVENT_SUSPEND while re-enable pdev
unconditionally. This was okay before ref-counted pdev enable update
but it now makes the pdev pinned after swsusp cycle (enabled twice but
disabled only once) and devres sanity check whines about it.

Fix it by unconditionally disabling pdev on all suspend events.

Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>

--
tejun
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-02-25 17:39    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site