diff options
author | James Bottomley <James.Bottomley@SteelEye.com> | 2006-02-24 16:04:14 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-02-24 17:31:38 -0500 |
commit | 2b932f6cf052920fb3a6281499e08209b08f5086 (patch) | |
tree | c2710e09dd40ee9733bcd77234d6373acec741d2 /arch/i386/kernel/smpboot.c | |
parent | 1e275d406bf6b88e4de6925cf594b64bb2ec49bc (diff) |
[PATCH] x86: fix broken SMP boot sequence
Recent GDT changes broke the SMP boot sequence if the booting CPU is
numbered anything other than zero. There's also a subtle source of error
in that the boot time CPU now uses cpu_gdt_table (which is actually the GDT
for booting CPUs in head.S). This patch fixes both problems by making GDT
descriptors themselves allocated from a per_cpu area and switching to them
in cpu_init(), which now means that cpu_gdt_table is exclusively used for
booting CPUs again.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Cc: Zachary Amsden <zach@vmware.com>
Cc: Matt Tolentino <metolent@snoqualmie.dp.intel.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/i386/kernel/smpboot.c')
-rw-r--r-- | arch/i386/kernel/smpboot.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/arch/i386/kernel/smpboot.c b/arch/i386/kernel/smpboot.c index fb00ab7b7612..eba7f53f8b4a 100644 --- a/arch/i386/kernel/smpboot.c +++ b/arch/i386/kernel/smpboot.c | |||
@@ -898,12 +898,6 @@ static int __devinit do_boot_cpu(int apicid, int cpu) | |||
898 | unsigned long start_eip; | 898 | unsigned long start_eip; |
899 | unsigned short nmi_high = 0, nmi_low = 0; | 899 | unsigned short nmi_high = 0, nmi_low = 0; |
900 | 900 | ||
901 | if (!cpu_gdt_descr[cpu].address && | ||
902 | !(cpu_gdt_descr[cpu].address = get_zeroed_page(GFP_KERNEL))) { | ||
903 | printk("Failed to allocate GDT for CPU %d\n", cpu); | ||
904 | return 1; | ||
905 | } | ||
906 | |||
907 | ++cpucount; | 901 | ++cpucount; |
908 | 902 | ||
909 | /* | 903 | /* |