Skip to content

Commit 1eb8f69

Browse files
committed
x86/topology: Make __max_die_per_package available unconditionally
Move it outside of CONFIG_SMP in order to avoid ifdeffery at the usage sites. Fixes: 76e2fc6 ("x86/cpu/amd: Set __max_die_per_package on AMD") Reported-by: Stephen Rothwell <[email protected]> Reported-by: kernel test robot <[email protected]> Signed-off-by: Borislav Petkov <[email protected]> Link: https://lkml.kernel.org/r/[email protected]
1 parent 66a4250 commit 1eb8f69

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

arch/x86/include/asm/topology.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,8 @@ extern const struct cpumask *cpu_coregroup_mask(int cpu);
110110
#define topology_die_id(cpu) (cpu_data(cpu).cpu_die_id)
111111
#define topology_core_id(cpu) (cpu_data(cpu).cpu_core_id)
112112

113+
extern unsigned int __max_die_per_package;
114+
113115
#ifdef CONFIG_SMP
114116
#define topology_die_cpumask(cpu) (per_cpu(cpu_die_map, cpu))
115117
#define topology_core_cpumask(cpu) (per_cpu(cpu_core_map, cpu))
@@ -118,8 +120,6 @@ extern const struct cpumask *cpu_coregroup_mask(int cpu);
118120
extern unsigned int __max_logical_packages;
119121
#define topology_max_packages() (__max_logical_packages)
120122

121-
extern unsigned int __max_die_per_package;
122-
123123
static inline int topology_max_die_per_package(void)
124124
{
125125
return __max_die_per_package;

arch/x86/kernel/cpu/topology.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,10 @@
2525
#define BITS_SHIFT_NEXT_LEVEL(eax) ((eax) & 0x1f)
2626
#define LEVEL_MAX_SIBLINGS(ebx) ((ebx) & 0xffff)
2727

28-
#ifdef CONFIG_SMP
2928
unsigned int __max_die_per_package __read_mostly = 1;
3029
EXPORT_SYMBOL(__max_die_per_package);
3130

31+
#ifdef CONFIG_SMP
3232
/*
3333
* Check if given CPUID extended toplogy "leaf" is implemented
3434
*/

0 commit comments

Comments
 (0)