diff options
author | Andi Kleen <ak@suse.de> | 2005-09-12 12:49:24 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-09-12 13:50:56 -0400 |
commit | 5bf97e01192f0fcdff2f0e9eae063e7785f6f915 (patch) | |
tree | ef7139d95cfa67c2a80884cb60382342ad5bf79c /arch/x86_64/kernel | |
parent | 27183ebd330a676fe29f7eb9b7ff582492246ec8 (diff) |
[PATCH] x86-64: Use physflat on Intel for < 8 CPUs with CPU hotplug
This avoids races with the APIC broadcast/mask modes.
Signed-off-by: Andi Kleen <ak@suse.de>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/x86_64/kernel')
-rw-r--r-- | arch/x86_64/kernel/genapic.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/arch/x86_64/kernel/genapic.c b/arch/x86_64/kernel/genapic.c index fe416fd4dbe0..7a64ea181788 100644 --- a/arch/x86_64/kernel/genapic.c +++ b/arch/x86_64/kernel/genapic.c | |||
@@ -104,9 +104,14 @@ void __init clustered_apic_check(void) | |||
104 | * (We don't use lowest priority delivery + HW APIC IRQ steering, so | 104 | * (We don't use lowest priority delivery + HW APIC IRQ steering, so |
105 | * can ignore the clustered logical case and go straight to physical.) | 105 | * can ignore the clustered logical case and go straight to physical.) |
106 | */ | 106 | */ |
107 | if (clusters <= 1 && max_cluster <= 8 && cluster_cnt[0] == max_cluster) | 107 | if (clusters <= 1 && max_cluster <= 8 && cluster_cnt[0] == max_cluster) { |
108 | #ifdef CONFIG_HOTPLUG_CPU | ||
109 | /* Don't use APIC shortcuts in CPU hotplug to avoid races */ | ||
110 | genapic = &apic_physflat; | ||
111 | #else | ||
108 | genapic = &apic_flat; | 112 | genapic = &apic_flat; |
109 | else | 113 | #endif |
114 | } else | ||
110 | genapic = &apic_cluster; | 115 | genapic = &apic_cluster; |
111 | 116 | ||
112 | print: | 117 | print: |