diff options
| author | Oleg Nesterov <oleg@tv-sign.ru> | 2006-04-11 22:18:58 +0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-04-11 07:34:01 -0700 |
| commit | a145410dccdb44f81d3b56763ef9b6f721f4e47c (patch) | |
| tree | a2e6635223890acb3191332eb7ba7835fbd24586 /kernel | |
| parent | 2514395ef88b46e895726a8d40966cb83de7940c (diff) | |
| download | lwn-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>
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) 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); |