diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2006-03-28 19:11:27 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-28 21:36:44 -0500 |
commit | a7e5328a06a2beee3a2bbfaf87ce2a7bbe937de1 (patch) | |
tree | 168102a80e9b98d19e8bf39156d35dacdb253c3d /kernel/exit.c | |
parent | 4a2c7a7837da1b91468e50426066d988050e4d56 (diff) |
[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) | |||
114 | __unhash_process(tsk); | 114 | __unhash_process(tsk); |
115 | 115 | ||
116 | tsk->signal = NULL; | 116 | tsk->signal = NULL; |
117 | cleanup_sighand(tsk); | 117 | tsk->sighand = NULL; |
118 | spin_unlock(&sighand->siglock); | 118 | spin_unlock(&sighand->siglock); |
119 | rcu_read_unlock(); | 119 | rcu_read_unlock(); |
120 | 120 | ||
121 | __cleanup_sighand(sighand); | ||
121 | clear_tsk_thread_flag(tsk,TIF_SIGPENDING); | 122 | clear_tsk_thread_flag(tsk,TIF_SIGPENDING); |
122 | flush_sigqueue(&tsk->pending); | 123 | flush_sigqueue(&tsk->pending); |
123 | if (sig) { | 124 | if (sig) { |