]> git.baikalelectronics.ru Git - kernel.git/commit
x86: fix HT cpu booting on 32-bit
authorHugh Dickins <hugh@veritas.com>
Wed, 30 Apr 2008 15:17:46 +0000 (16:17 +0100)
committerIngo Molnar <mingo@elte.hu>
Wed, 30 Apr 2008 21:15:35 +0000 (23:15 +0200)
commitd9bc5352ff7d3e3618b47f31bf9ca6349a1abe01
tree6f7c9031a7d5ff01baf9686118e1af7a44f1e894
parentfc51bf71fb15e1d88379220289d48f697e52b571
x86: fix HT cpu booting on 32-bit

Since recent smpboot 32/64-bit merge, my dual Xeon with HT has been
booting only 2 of its 4 cpus (when running an i386 kernel; but x86_64
is okay).  J.A. Magallón reports the same.

 native_cpu_up: bad cpu 2
 native_cpu_up: bad cpu 3

The mach-default cpu_present_to_apicid() was just returning cpu number
(2, 3) instead of apicid (6, 7): looks like we now need the x86_64 code
even for the i386 case.

Comparing with other versions of cpu_present_to_apicid(), it seems a
good idea to include an NR_CPUS test too, since cpu_present() doesn't
include that; but that wasn't a problem here, and may no problem at all.

Prior to that smpboot merge, my Xeon booted the two HT siblings on one
physical first, then the two siblings on the other physical after - when
i386, but alternated them when x86_64.  Since the merge, the x86_64
sequence is unchanged, but the i386 sequence is now like x86_64.

I prefer this consistency, and I prefer the new sequence: booting with
maxcpus=2 then uses the independent physicals without HT sharing.

Signed-off-by: Hugh Dickins <hugh@veritas.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
include/asm-x86/mach-default/mach_apic.h