Messages in this thread Patch in this message | | | Date | Sat, 10 Mar 2007 01:18:54 -0500 | From | Ray Lee <> | Subject | Re: Odd suspend regression in 2.6.21-rc[123] |
| |
Ray Lee wrote: > In 2.6.21-rc1,2,3, my laptop will fully suspend to ram, but then > *immediately* resumes back from suspension. (It resumes just fine, as well.) [...] > HP/Compaq NX6125 system, AMD64, dmesg attached.
hg bisect found the below patch as the culprit, and reverting it does fix the regression. It's supposed to address "sometime ac/battery update stops after resume from disk." This thread: http://lkml.org/lkml/2007/2/24/111 appears to talk about the same issue, and therefore it may be solved without the below patch, so perhaps we can all be happy.
Regardless, I think my laptop no longer being able to go into S3 sleep is a bit more important than someone else's laptop merely not showing the correct AC status :-).
Please revert. (git patch id ed41dab90eb40ac4911e60406bc653661f0e4ce1)
Ray
ACPI: Disable GPEs in preparation for sleep.
http://bugzilla.kernel.org/show_bug.cgi?id=7887
Signed-off-by: Alexey Starikovskiy <alexey.y.starikovskiy@linux.intel.com> Signed-off-by: Vladimir Lebedev <vladimir.p.lebedev@intel.com> Signed-off-by: Len Brown <len.brown@intel.com>
committer: Len Brown <len.brown@intel.com>
diff -r 31db4f8c1b77 -r 2a00d393c882 drivers/acpi/hardware/hwsleep.c --- a/drivers/acpi/hardware/hwsleep.c Fri Feb 09 10:45:33 2007 -0800 +++ b/drivers/acpi/hardware/hwsleep.c Sat Feb 10 01:30:35 2007 -0500 @@ -235,6 +235,14 @@ acpi_status acpi_enter_sleep_state_prep( "While executing method _SST")); }
+ /* + * 1) Disable/Clear all GPEs + */ + status = acpi_hw_disable_all_gpes(); + if (ACPI_FAILURE(status)) { + return_ACPI_STATUS(status); + } + return_ACPI_STATUS(AE_OK); }
@@ -290,13 +298,8 @@ acpi_status asmlinkage acpi_enter_sleep_ }
/* - * 1) Disable/Clear all GPEs * 2) Enable all wakeup GPEs */ - status = acpi_hw_disable_all_gpes(); - if (ACPI_FAILURE(status)) { - return_ACPI_STATUS(status); - } acpi_gbl_system_awake_and_running = FALSE;
status = acpi_hw_enable_all_wakeup_gpes(); - 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/
| |