From: Linus Torvalds <torvalds@woody.linux-foundation.org>
Date: Wed, 26 Dec 2007 04:16:16 +0000 (-0800)
Subject: Revert "x86: fix show cpuinfo cpu number always zero"
X-Git-Tag: v2.6.24-rc7~37
X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=00684418707c7a1e36ebdedc4b30fbba5d5860b1;p=~emulex%2Finfiniband.git

Revert "x86: fix show cpuinfo cpu number always zero"

This reverts commit fbdcf18df73758b2e187ab94678b30cd5f6ff9f9.

As pointed out by Yanmin Zhang, the problem was already fixed
differently (and correctly), and rather than fix anything, it actually
causes us to create a sub-optimal sched-domains hierarchy (not setting
up the domain belonging to the core) when CONFIG_X86_HT=y.

Requested-by: Yanmin Zhang <yanmin_zhang@linux.intel.com>
Acked-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
---

diff --git a/arch/x86/kernel/smpboot_64.c b/arch/x86/kernel/smpboot_64.c
index 594889521da..500670c93d8 100644
--- a/arch/x86/kernel/smpboot_64.c
+++ b/arch/x86/kernel/smpboot_64.c
@@ -141,8 +141,8 @@ static void __cpuinit smp_store_cpu_info(int id)
 	struct cpuinfo_x86 *c = &cpu_data(id);
 
 	*c = boot_cpu_data;
-	identify_cpu(c);
 	c->cpu_index = id;
+	identify_cpu(c);
 	print_cpu_info(c);
 }