diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2008-02-08 07:19:12 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-02-08 12:22:27 -0500 |
commit | d12619b5ff5664623524aef796514d1946ea3b4a (patch) | |
tree | 23140ca0148f9154440d3056e2040d8df468a24b /kernel/exit.c | |
parent | 430c623121ea88ca80595c99fdc63b7f8a803ae5 (diff) |
fix group stop with exit race
do_signal_stop() counts all sub-thread and sets ->group_stop_count
accordingly. Every thread should decrement ->group_stop_count and stop,
the last one should notify the parent.
However a sub-thread can exit before it notices the signal_pending(), or it
may be somewhere in do_exit() already. In that case the group stop never
finishes properly.
Note: this is a minimal fix, we can add some optimizations later. Say we
can return quickly if thread_group_empty(). Also, we can move some signal
related code from exit_notify() to exit_signals().
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Acked-by: Davide Libenzi <davidel@xmailserver.org>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Roland McGrath <roland@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'kernel/exit.c')
-rw-r--r-- | kernel/exit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/exit.c b/kernel/exit.c index d7815f570882..8f3bf53a5b4d 100644 --- a/kernel/exit.c +++ b/kernel/exit.c | |||
@@ -947,7 +947,7 @@ fastcall NORET_TYPE void do_exit(long code) | |||
947 | schedule(); | 947 | schedule(); |
948 | } | 948 | } |
949 | 949 | ||
950 | tsk->flags |= PF_EXITING; | 950 | exit_signals(tsk); /* sets PF_EXITING */ |
951 | /* | 951 | /* |
952 | * tsk->flags are checked in the futex code to protect against | 952 | * tsk->flags are checked in the futex code to protect against |
953 | * an exiting task cleaning up the robust pi futexes. | 953 | * an exiting task cleaning up the robust pi futexes. |