Commit ad4e680f authored by Alexey Dobriyan's avatar Alexey Dobriyan Committed by Andi Kleen

[PATCH] i386: use smp_call_function_single()

It will execure cpuid only on the cpu we need.
Signed-off-by: default avatarAlexey Dobriyan <adobriyan@openvz.org>
Signed-off-by: default avatarAndi Kleen <ak@suse.de>
parent d958f143
...@@ -48,7 +48,6 @@ static struct class *cpuid_class; ...@@ -48,7 +48,6 @@ static struct class *cpuid_class;
#ifdef CONFIG_SMP #ifdef CONFIG_SMP
struct cpuid_command { struct cpuid_command {
int cpu;
u32 reg; u32 reg;
u32 *data; u32 *data;
}; };
...@@ -57,7 +56,6 @@ static void cpuid_smp_cpuid(void *cmd_block) ...@@ -57,7 +56,6 @@ static void cpuid_smp_cpuid(void *cmd_block)
{ {
struct cpuid_command *cmd = (struct cpuid_command *)cmd_block; struct cpuid_command *cmd = (struct cpuid_command *)cmd_block;
if (cmd->cpu == smp_processor_id())
cpuid(cmd->reg, &cmd->data[0], &cmd->data[1], &cmd->data[2], cpuid(cmd->reg, &cmd->data[0], &cmd->data[1], &cmd->data[2],
&cmd->data[3]); &cmd->data[3]);
} }
...@@ -70,11 +68,10 @@ static inline void do_cpuid(int cpu, u32 reg, u32 * data) ...@@ -70,11 +68,10 @@ static inline void do_cpuid(int cpu, u32 reg, u32 * data)
if (cpu == smp_processor_id()) { if (cpu == smp_processor_id()) {
cpuid(reg, &data[0], &data[1], &data[2], &data[3]); cpuid(reg, &data[0], &data[1], &data[2], &data[3]);
} else { } else {
cmd.cpu = cpu;
cmd.reg = reg; cmd.reg = reg;
cmd.data = data; cmd.data = data;
smp_call_function(cpuid_smp_cpuid, &cmd, 1, 1); smp_call_function_single(cpu, cpuid_smp_cpuid, &cmd, 1, 1);
} }
preempt_enable(); preempt_enable();
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment