From: Oleg Nesterov Date: Wed, 17 Oct 2007 06:27:00 +0000 (-0700) Subject: wait_task_stopped/continued: remove unneeded p->signal != NULL check X-Git-Tag: v2.6.24-rc1~703 X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=715015e8da37c4d13e234def054bcbea116297e9;p=linux-2.6 wait_task_stopped/continued: remove unneeded p->signal != NULL check The child was found on ->children list under tasklist_lock, it must have a valid ->signal. __exit_signal() both removes the task from parent->children and clears ->signal "atomically" under write_lock(tasklist). Remove unneeded checks. Signed-off-by: Oleg Nesterov Acked-by: Roland McGrath Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- diff --git a/kernel/exit.c b/kernel/exit.c index b27a3dcde6..25f6805be5 100644 --- a/kernel/exit.c +++ b/kernel/exit.c @@ -1322,7 +1322,7 @@ static int wait_task_stopped(struct task_struct *p, int delayed_group_leader, if (!p->exit_code) return 0; if (delayed_group_leader && !(p->ptrace & PT_PTRACED) && - p->signal && p->signal->group_stop_count > 0) + p->signal->group_stop_count > 0) /* * A group stop is in progress and this is the group leader. * We won't report until all threads have stopped. @@ -1436,9 +1436,6 @@ static int wait_task_continued(struct task_struct *p, int noreap, pid_t pid; uid_t uid; - if (unlikely(!p->signal)) - return 0; - if (!(p->signal->flags & SIGNAL_STOP_CONTINUED)) return 0;