diff options
author | Tejun Heo <tj@kernel.org> | 2010-12-18 10:30:05 -0500 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2010-12-30 06:22:03 -0500 |
commit | 7b543a5334ff4ea2e3ad3b777fc23cdb8072a988 (patch) | |
tree | 8fbdf5275411190f78f3bdee3c40e87285bf8e30 /arch/x86/kernel/smpboot.c | |
parent | 0a3aee0da4402aa19b66e458038533c896fb80c6 (diff) |
x86: Replace uses of current_cpu_data with this_cpu ops
Replace all uses of current_cpu_data with this_cpu operations on the
per cpu structure cpu_info. The scala accesses are replaced with the
matching this_cpu ops which results in smaller and more efficient
code.
In the long run, it might be a good idea to remove cpu_data() macro
too and use per_cpu macro directly.
tj: updated description
Cc: Yinghai Lu <yinghai@kernel.org>
Cc: Ingo Molnar <mingo@elte.hu>
Acked-by: H. Peter Anvin <hpa@zytor.com>
Acked-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'arch/x86/kernel/smpboot.c')
-rw-r--r-- | arch/x86/kernel/smpboot.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/arch/x86/kernel/smpboot.c b/arch/x86/kernel/smpboot.c index ff4e5a113a5b..0720071086d1 100644 --- a/arch/x86/kernel/smpboot.c +++ b/arch/x86/kernel/smpboot.c | |||
@@ -430,7 +430,7 @@ void __cpuinit set_cpu_sibling_map(int cpu) | |||
430 | 430 | ||
431 | cpumask_set_cpu(cpu, c->llc_shared_map); | 431 | cpumask_set_cpu(cpu, c->llc_shared_map); |
432 | 432 | ||
433 | if (current_cpu_data.x86_max_cores == 1) { | 433 | if (__this_cpu_read(cpu_info.x86_max_cores) == 1) { |
434 | cpumask_copy(cpu_core_mask(cpu), cpu_sibling_mask(cpu)); | 434 | cpumask_copy(cpu_core_mask(cpu), cpu_sibling_mask(cpu)); |
435 | c->booted_cores = 1; | 435 | c->booted_cores = 1; |
436 | return; | 436 | return; |
@@ -1094,7 +1094,7 @@ void __init native_smp_prepare_cpus(unsigned int max_cpus) | |||
1094 | 1094 | ||
1095 | preempt_disable(); | 1095 | preempt_disable(); |
1096 | smp_cpu_index_default(); | 1096 | smp_cpu_index_default(); |
1097 | current_cpu_data = boot_cpu_data; | 1097 | memcpy(__this_cpu_ptr(&cpu_info), &boot_cpu_data, sizeof(cpu_info)); |
1098 | cpumask_copy(cpu_callin_mask, cpumask_of(0)); | 1098 | cpumask_copy(cpu_callin_mask, cpumask_of(0)); |
1099 | mb(); | 1099 | mb(); |
1100 | /* | 1100 | /* |
@@ -1397,11 +1397,11 @@ static inline void mwait_play_dead(void) | |||
1397 | int i; | 1397 | int i; |
1398 | void *mwait_ptr; | 1398 | void *mwait_ptr; |
1399 | 1399 | ||
1400 | if (!cpu_has(¤t_cpu_data, X86_FEATURE_MWAIT)) | 1400 | if (!cpu_has(__this_cpu_ptr(&cpu_info), X86_FEATURE_MWAIT)) |
1401 | return; | 1401 | return; |
1402 | if (!cpu_has(¤t_cpu_data, X86_FEATURE_CLFLSH)) | 1402 | if (!cpu_has(__this_cpu_ptr(&cpu_info), X86_FEATURE_CLFLSH)) |
1403 | return; | 1403 | return; |
1404 | if (current_cpu_data.cpuid_level < CPUID_MWAIT_LEAF) | 1404 | if (__this_cpu_read(cpu_info.cpuid_level) < CPUID_MWAIT_LEAF) |
1405 | return; | 1405 | return; |
1406 | 1406 | ||
1407 | eax = CPUID_MWAIT_LEAF; | 1407 | eax = CPUID_MWAIT_LEAF; |
@@ -1452,7 +1452,7 @@ static inline void mwait_play_dead(void) | |||
1452 | 1452 | ||
1453 | static inline void hlt_play_dead(void) | 1453 | static inline void hlt_play_dead(void) |
1454 | { | 1454 | { |
1455 | if (current_cpu_data.x86 >= 4) | 1455 | if (__this_cpu_read(cpu_info.x86) >= 4) |
1456 | wbinvd(); | 1456 | wbinvd(); |
1457 | 1457 | ||
1458 | while (1) { | 1458 | while (1) { |