diff options
author | Andi Kleen <ak@suse.de> | 2006-02-16 17:42:04 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-02-17 11:00:40 -0500 |
commit | ab68805955ee3dd84a6aa76cd70e61fde996968d (patch) | |
tree | abeaadcc28a6d60ca31fe85a732ace6a6df1232e /arch | |
parent | 2391c4b594eb28abd58102de8f4e5d7a4fa39f4c (diff) |
[PATCH] x86_64: Don't enable ATI apicmaintimer workaround when the machine has C2 or C3
Many laptops have problems with ticking the local APIC timer in C2/C3.
The code added earlier to use it by default on ATI didn't really work
for them. Don't enable it when the system supports C2/C3.
This doesn't fix the problem fully, but at least it's not worse than before.
Signed-off-by: Andi Kleen <ak@suse.de>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/x86_64/kernel/io_apic.c | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/arch/x86_64/kernel/io_apic.c b/arch/x86_64/kernel/io_apic.c index 4282d72b2a26..2585c1d92b26 100644 --- a/arch/x86_64/kernel/io_apic.c +++ b/arch/x86_64/kernel/io_apic.c | |||
@@ -30,6 +30,9 @@ | |||
30 | #include <linux/mc146818rtc.h> | 30 | #include <linux/mc146818rtc.h> |
31 | #include <linux/acpi.h> | 31 | #include <linux/acpi.h> |
32 | #include <linux/sysdev.h> | 32 | #include <linux/sysdev.h> |
33 | #ifdef CONFIG_ACPI | ||
34 | #include <acpi/acpi_bus.h> | ||
35 | #endif | ||
33 | 36 | ||
34 | #include <asm/io.h> | 37 | #include <asm/io.h> |
35 | #include <asm/smp.h> | 38 | #include <asm/smp.h> |
@@ -260,6 +263,8 @@ __setup("apic", enable_ioapic_setup); | |||
260 | 263 | ||
261 | And another hack to disable the IOMMU on VIA chipsets. | 264 | And another hack to disable the IOMMU on VIA chipsets. |
262 | 265 | ||
266 | ... and others. Really should move this somewhere else. | ||
267 | |||
263 | Kludge-O-Rama. */ | 268 | Kludge-O-Rama. */ |
264 | void __init check_ioapic(void) | 269 | void __init check_ioapic(void) |
265 | { | 270 | { |
@@ -307,6 +312,17 @@ void __init check_ioapic(void) | |||
307 | case PCI_VENDOR_ID_ATI: | 312 | case PCI_VENDOR_ID_ATI: |
308 | if (apic_runs_main_timer != 0) | 313 | if (apic_runs_main_timer != 0) |
309 | break; | 314 | break; |
315 | #ifdef CONFIG_ACPI | ||
316 | /* Don't do this for laptops right | ||
317 | right now because their timer | ||
318 | doesn't necessarily tick in C2/3 */ | ||
319 | if (acpi_fadt.revision >= 3 && | ||
320 | (acpi_fadt.plvl2_lat + acpi_fadt.plvl3_lat) < 1100) { | ||
321 | printk(KERN_INFO | ||
322 | "ATI board detected, but seems to be a laptop. Timer might be shakey, sorry\n"); | ||
323 | break; | ||
324 | } | ||
325 | #endif | ||
310 | printk(KERN_INFO | 326 | printk(KERN_INFO |
311 | "ATI board detected. Using APIC/PM timer.\n"); | 327 | "ATI board detected. Using APIC/PM timer.\n"); |
312 | apic_runs_main_timer = 1; | 328 | apic_runs_main_timer = 1; |