diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2008-04-30 03:52:45 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-30 11:29:34 -0400 |
commit | 6ca25b551309eb1b1b41f83414a92f7472e0b23d (patch) | |
tree | a2c5785ec484ec997ac6a03d3fceae04e4d21795 /include/linux/signal.h | |
parent | e442055193e4584218006e616c9bdce0c5e9ae5c (diff) |
kill_pid_info: don't take now unneeded tasklist_lock
Previously handle_stop_signal(SIGCONT) could drop ->siglock. That is why
kill_pid_info(SIGCONT) takes tasklist_lock to make sure the target task can't
go away after unlock. Not needed now.
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Cc: Roland McGrath <roland@redhat.com>
Cc: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include/linux/signal.h')
-rw-r--r-- | include/linux/signal.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/include/linux/signal.h b/include/linux/signal.h index 42d2e0a948f4..84f997f8aa53 100644 --- a/include/linux/signal.h +++ b/include/linux/signal.h | |||
@@ -362,8 +362,6 @@ int unhandled_signal(struct task_struct *tsk, int sig); | |||
362 | #define sig_kernel_stop(sig) \ | 362 | #define sig_kernel_stop(sig) \ |
363 | (((sig) < SIGRTMIN) && siginmask(sig, SIG_KERNEL_STOP_MASK)) | 363 | (((sig) < SIGRTMIN) && siginmask(sig, SIG_KERNEL_STOP_MASK)) |
364 | 364 | ||
365 | #define sig_needs_tasklist(sig) ((sig) == SIGCONT) | ||
366 | |||
367 | #define sig_user_defined(t, signr) \ | 365 | #define sig_user_defined(t, signr) \ |
368 | (((t)->sighand->action[(signr)-1].sa.sa_handler != SIG_DFL) && \ | 366 | (((t)->sighand->action[(signr)-1].sa.sa_handler != SIG_DFL) && \ |
369 | ((t)->sighand->action[(signr)-1].sa.sa_handler != SIG_IGN)) | 367 | ((t)->sighand->action[(signr)-1].sa.sa_handler != SIG_IGN)) |