summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOleg Nesterov <oleg@tv-sign.ru>2006-04-11 22:18:58 +0400
committerLinus Torvalds <torvalds@g5.osdl.org>2006-04-11 07:34:01 -0700
commita145410dccdb44f81d3b56763ef9b6f721f4e47c (patch)
treea2e6635223890acb3191332eb7ba7835fbd24586
parent2514395ef88b46e895726a8d40966cb83de7940c (diff)
downloadlwn-a145410dccdb44f81d3b56763ef9b6f721f4e47c.tar.gz
lwn-a145410dccdb44f81d3b56763ef9b6f721f4e47c.zip
[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>
-rw-r--r--kernel/signal.c1
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)
if (t == NULL)
/* restart balancing at this thread */
t = p->signal->curr_target = p;
- BUG_ON(t->tgid != p->tgid);
while (!wants_signal(sig, t)) {
t = next_thread(t);