Messages in this thread | | | Date | Thu, 21 Aug 2008 21:14:03 +0200 | From | "Vegard Nossum" <> | Subject | Re: latest -git: hibernate: possible circular locking dependency detected |
| |
On Thu, Aug 21, 2008 at 9:07 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > On Thursday, 21 of August 2008, Vegard Nossum wrote: >> On Thu, Aug 21, 2008 at 8:31 PM, Vegard Nossum <vegard.nossum@gmail.com> wrote: >> > On Thu, Aug 21, 2008 at 8:22 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: >> >> On Thursday, 21 of August 2008, Vegard Nossum wrote: >> >>> >> >>> I just got this on v2.6.27-rc4 (+ unrelated fix): >> >>> >> >>> ACPI: Preparing to enter system sleep state S5 >> >> >> >> This looks fishy. >> >> >> >> Please check what's there in /sys/power/disk . >> >> >> >> Sorry, forgot this one: >> >> # cat /sys/power/disk >> test testproc [shutdown] reboot > > Hm, I wonder why 'platform' isn't there. > > Perhaps the dmesg log will tell. > > [/me looks] > > No, it doesn't. The BIOS evidently supports S4. > > [/me looks into the source code] > > I have no idea whatsoever how it is possible to have > > "ACPI: (supports S0 S1 S3 S4 S5)" > > in the log and not to have "platform" in the output of "cat /sys/power/disk".
I'm sorry, I made a stupid mistake: That file was taken from the system when it was booted with acpi=off. The boot log had acpi enabled, of course. Here is the same file (now booted without acpi=off):
# cat /sys/power/disk [platform] test testproc shutdown reboot
> Can you please add > > printk(KERN_CRIT "PM: hibernation_ops = %p\n", hibernation_ops); > > at the end of kernel/power/disk.c:hibernation_set_ops() and see if it gets > printed during boot?
I guess this is now not necessary anymore?
Vegard
-- "The animistic metaphor of the bug that maliciously sneaked in while the programmer was not looking is intellectually dishonest as it disguises that the error is the programmer's own creation." -- E. W. Dijkstra, EWD1036
| |