diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2006-03-29 02:11:27 +0200 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-29 04:36:44 +0200 |
commit | a7e5328a06a2beee3a2bbfaf87ce2a7bbe937de1 (patch) | |
tree | 168102a80e9b98d19e8bf39156d35dacdb253c3d /kernel/exit.c | |
parent | [PATCH] make fork() atomic wrt pgrp/session signals (diff) | |
download | linux-a7e5328a06a2beee3a2bbfaf87ce2a7bbe937de1.tar.xz linux-a7e5328a06a2beee3a2bbfaf87ce2a7bbe937de1.zip |
[PATCH] cleanup __exit_signal->cleanup_sighand path
Move 'tsk->sighand = NULL' from cleanup_sighand() to __exit_signal(). This
makes the exit path more understandable and allows us to do
cleanup_sighand() outside of ->siglock protected section.
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'kernel/exit.c')
-rw-r--r-- | kernel/exit.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/kernel/exit.c b/kernel/exit.c index 22399caf7574..bc0ec674d3f4 100644 --- a/kernel/exit.c +++ b/kernel/exit.c @@ -114,10 +114,11 @@ static void __exit_signal(struct task_struct *tsk) __unhash_process(tsk); tsk->signal = NULL; - cleanup_sighand(tsk); + tsk->sighand = NULL; spin_unlock(&sighand->siglock); rcu_read_unlock(); + __cleanup_sighand(sighand); clear_tsk_thread_flag(tsk,TIF_SIGPENDING); flush_sigqueue(&tsk->pending); if (sig) { |