Commit 5224b961 authored by Linus Torvalds's avatar Linus Torvalds

smp: Fix error case handling in smp_call_function_*()

Commit 8053871d ("smp: Fix smp_call_function_single_async()
locking") fixed the locking for the asynchronous smp-call case, but in
the process of moving the lock handling around, one of the error cases
ended up not unlocking the call data at all.

This went unnoticed on x86, because this is a "caller is buggy" case,
where the caller is trying to call a non-existent CPU.  But apparently
ARM does that (at least under qemu-arm).  Bindly doing cross-cpu calls
to random CPU's that aren't even online seems a bit fishy, but the error
handling was clearly not correct.

Simply add the missing "csd_unlock()" to the error path.
Reported-and-tested-by: default avatarGuenter Roeck <linux@roeck-us.net>
Analyzed-by: default avatarRabin Vincent <rabin@rab.in>
Acked-by: default avatarIngo Molnar <mingo@kernel.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 64fb1d0e
...@@ -159,8 +159,10 @@ static int generic_exec_single(int cpu, struct call_single_data *csd, ...@@ -159,8 +159,10 @@ static int generic_exec_single(int cpu, struct call_single_data *csd,
} }
if ((unsigned)cpu >= nr_cpu_ids || !cpu_online(cpu)) if ((unsigned)cpu >= nr_cpu_ids || !cpu_online(cpu)) {
csd_unlock(csd);
return -ENXIO; return -ENXIO;
}
csd->func = func; csd->func = func;
csd->info = info; csd->info = info;
......
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