]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
printk: Fix wake_up_klogd() vs cpu hotplug
authorHeiko Carstens <heiko.carstens@de.ibm.com>
Fri, 26 Nov 2010 12:42:47 +0000 (13:42 +0100)
committerAK <andi@firstfloor.org>
Sun, 6 Feb 2011 19:03:40 +0000 (11:03 -0800)
commit78dd4b02b5d68c4d64557152a35941150f90ff82
treeaae1bfb3712deba7efba605ca8cd33e07856f22e
parent89c24ec0eee40272f0387eeb6a6cc31e6bbc1544
printk: Fix wake_up_klogd() vs cpu hotplug

commit 49f4138346b3cec2706adff02658fe27ceb1e46f upstream.

wake_up_klogd() may get called from preemptible context but uses
__raw_get_cpu_var() to write to a per cpu variable. If it gets preempted
between getting the address and writing to it, the cpu in question could be
offline if the process gets scheduled back and hence writes to the per cpu data
of an offline cpu.

This buggy behaviour was introduced with fa33507a "printk: robustify
printk, fix #2" which was supposed to fix a "using smp_processor_id() in
preemptible" warning.

Let's use this_cpu_write() instead which disables preemption and makes sure
that the outlined scenario cannot happen.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Acked-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
LKML-Reference: <20101126124247.GC7023@osiris.boeblingen.de.ibm.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
kernel/printk.c