x86/cpu/amd: Hide unused legacy_fixup_core_id() function
authorArnd Bergmann <[email protected]>
Fri, 11 Aug 2017 11:19:11 +0000 (13:19 +0200)
committerIngo Molnar <[email protected]>
Fri, 11 Aug 2017 12:58:34 +0000 (14:58 +0200)
The newly introduced function is only used when CONFIG_SMP is set:

  arch/x86/kernel/cpu/amd.c:305:13: warning: 'legacy_fixup_core_id' defined but not used

This moves the existing #ifdef around the caller so it covers
legacy_fixup_core_id() as well.

Signed-off-by: Arnd Bergmann <[email protected]>
Cc: Borislav Petkov <[email protected]>
Cc: Emanuel Czirai <[email protected]>
Cc: Linus Torvalds <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Suravee Suthikulpanit <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Cc: Tom Lendacky <[email protected]>
Cc: Yazen Ghannam <[email protected]>
Fixes: b89b41d0b841 ("x86/cpu/amd: Limit cpu_core_id fixup to families older than F17h")
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Ingo Molnar <[email protected]>
arch/x86/kernel/cpu/amd.c

index 82d571c9923ec29cc0411d60323048318c6345f5..e44338dd62dd8fe26707ce39b98223fdbf2fa1c2 100644 (file)
@@ -297,6 +297,7 @@ static int nearby_node(int apicid)
 }
 #endif
 
+#ifdef CONFIG_SMP
 /*
  * Fix up cpu_core_id for pre-F17h systems to be in the
  * [0 .. cores_per_node - 1] range. Not really needed but
@@ -319,7 +320,6 @@ static void legacy_fixup_core_id(struct cpuinfo_x86 *c)
  *     Assumption: Number of cores in each internal node is the same.
  * (2) AMD processors supporting compute units
  */
-#ifdef CONFIG_SMP
 static void amd_get_topology(struct cpuinfo_x86 *c)
 {
        u8 node_id;