Messages in this thread Patch in this message | | | Date | Tue, 19 May 2009 16:09:54 +0800 | From | Shaohua Li <> | Subject | [PATCH 2/2]cpuidle: makes AMD C1E works in suspend/resume path |
| |
When AMD C1E is enabled, local APIC timer will stop even in C1. To avoid suspend/resume hang, this patch removes C1 and replace it with a cpu_relax() in suspend/resume path. This hasn't any impact in runtime path.
http://bugzilla.kernel.org/show_bug.cgi?id=13233
[ impact: avoid suspend/resume hang in AMD CPU with C1E enabled ]
Tested-by: Dmitry Lyzhyn <thisistempbox@yahoo.com> Signed-off-by: Shaohua Li <shaohua.li@intel.com> --- drivers/acpi/processor_idle.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux/drivers/acpi/processor_idle.c =================================================================== --- linux.orig/drivers/acpi/processor_idle.c 2009-05-19 09:56:59.000000000 +0800 +++ linux/drivers/acpi/processor_idle.c 2009-05-19 10:01:23.000000000 +0800 @@ -834,8 +834,8 @@ static int acpi_idle_enter_c1(struct cpu /* Do not access any ACPI IO ports in suspend path */ if (acpi_idle_suspend) { - acpi_safe_halt(); local_irq_enable(); + cpu_relax(); return 0; }
| |