diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2006-04-11 14:18:58 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-04-11 10:34:01 -0400 |
commit | a145410dccdb44f81d3b56763ef9b6f721f4e47c (patch) | |
tree | a2e6635223890acb3191332eb7ba7835fbd24586 /kernel | |
parent | 2514395ef88b46e895726a8d40966cb83de7940c (diff) |
[PATCH] __group_complete_signal: remove bogus BUG_ON
Commit e56d090310d7625ecb43a1eeebd479f04affb48b
[PATCH] RCU signal handling
made this BUG_ON() unsafe. This code runs under ->siglock,
while switch_exec_pids() takes tasklist_lock.
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/signal.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/kernel/signal.c b/kernel/signal.c index 5ccaac505e8d..b14f895027c3 100644 --- a/kernel/signal.c +++ b/kernel/signal.c | |||
@@ -868,7 +868,6 @@ __group_complete_signal(int sig, struct task_struct *p) | |||
868 | if (t == NULL) | 868 | if (t == NULL) |
869 | /* restart balancing at this thread */ | 869 | /* restart balancing at this thread */ |
870 | t = p->signal->curr_target = p; | 870 | t = p->signal->curr_target = p; |
871 | BUG_ON(t->tgid != p->tgid); | ||
872 | 871 | ||
873 | while (!wants_signal(sig, t)) { | 872 | while (!wants_signal(sig, t)) { |
874 | t = next_thread(t); | 873 | t = next_thread(t); |