diff options
author | Anton Vorontsov <anton.vorontsov@linaro.org> | 2012-02-07 01:49:51 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-02-09 12:03:30 -0500 |
commit | d3a532a9c617106a0169232d40164ee35d0440b5 (patch) | |
tree | 4204ef5560c353f91cd275fb8303d799869dd97c /drivers/tty/sysrq.c | |
parent | e502babe0a85226f2417b60a8710cf8192879180 (diff) |
sysrq: Properly check for kernel threads
There's a real possibility of killing kernel threads that might
have issued use_mm(), so kthread's mm might become non-NULL.
This patch fixes the issue by checking for PF_KTHREAD (just as
get_task_mm()).
Suggested-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Anton Vorontsov <anton.vorontsov@linaro.org>
Acked-by: David Rientjes <rientjes@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty/sysrq.c')
-rw-r--r-- | drivers/tty/sysrq.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/drivers/tty/sysrq.c b/drivers/tty/sysrq.c index a1bcad7ef739..8db9125133b8 100644 --- a/drivers/tty/sysrq.c +++ b/drivers/tty/sysrq.c | |||
@@ -324,9 +324,12 @@ static void send_sig_all(int sig) | |||
324 | 324 | ||
325 | read_lock(&tasklist_lock); | 325 | read_lock(&tasklist_lock); |
326 | for_each_process(p) { | 326 | for_each_process(p) { |
327 | if (p->mm && !is_global_init(p)) | 327 | if (p->flags & PF_KTHREAD) |
328 | /* Not swapper, init nor kernel thread */ | 328 | continue; |
329 | force_sig(sig, p); | 329 | if (is_global_init(p)) |
330 | continue; | ||
331 | |||
332 | force_sig(sig, p); | ||
330 | } | 333 | } |
331 | read_unlock(&tasklist_lock); | 334 | read_unlock(&tasklist_lock); |
332 | } | 335 | } |