diff options
author | Shaohua Li <shaohua.li@intel.com> | 2009-05-19 04:09:54 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2009-05-26 23:45:33 -0400 |
commit | 7d60e8ab0d5507229dfbdf456501cc378610fa01 (patch) | |
tree | 008fe5442f841fd555b7fa43253c166f2b5c8099 | |
parent | 87ad57bacb25c3f24c54f142ef445f68277705f0 (diff) |
cpuidle: fix AMD C1E suspend hang
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>
Signed-off-by: Len Brown <len.brown@intel.com>
-rw-r--r-- | drivers/acpi/processor_idle.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c index 6b7bcc7e3e1d..10a2d913635a 100644 --- a/drivers/acpi/processor_idle.c +++ b/drivers/acpi/processor_idle.c | |||
@@ -834,8 +834,8 @@ static int acpi_idle_enter_c1(struct cpuidle_device *dev, | |||
834 | 834 | ||
835 | /* Do not access any ACPI IO ports in suspend path */ | 835 | /* Do not access any ACPI IO ports in suspend path */ |
836 | if (acpi_idle_suspend) { | 836 | if (acpi_idle_suspend) { |
837 | acpi_safe_halt(); | ||
838 | local_irq_enable(); | 837 | local_irq_enable(); |
838 | cpu_relax(); | ||
839 | return 0; | 839 | return 0; |
840 | } | 840 | } |
841 | 841 | ||