copy_process(): remove the unneeded clear_tsk_thread_flag(TIF_SIGPENDING)

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linux Kernel Mailing List
Date: Thursday, June 18, 2009 - 3:00 pm

Gitweb:     http://git.kernel.org/linus/72a1de39f89325a834a8c70b2a0d8f71d919f640
Commit:     72a1de39f89325a834a8c70b2a0d8f71d919f640
Parent:     77d1ef79568b337f599b75795acc8f78a87ba9ba
Author:     Oleg Nesterov <oleg@redhat.com>
AuthorDate: Wed Jun 17 16:27:37 2009 -0700
Committer:  Linus Torvalds <torvalds@linux-foundation.org>
CommitDate: Thu Jun 18 13:03:52 2009 -0700

    copy_process(): remove the unneeded clear_tsk_thread_flag(TIF_SIGPENDING)
    
    The forked child can have TIF_SIGPENDING if it was copied from parent's
    ti->flags.  But this is harmless and actually almost never happens,
    because copy_process() can't succeed if signal_pending() == T.
    
    Signed-off-by: Oleg Nesterov <oleg@redhat.com>
    Acked-by: Roland McGrath <roland@redhat.com>
    Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
    Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
---
 kernel/fork.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/kernel/fork.c b/kernel/fork.c
index be022c2..467746b 100644
--- a/kernel/fork.c
+++ b/kernel/fork.c
@@ -1029,7 +1029,6 @@ static struct task_struct *copy_process(unsigned long clone_flags,
 	p->vfork_done = NULL;
 	spin_lock_init(&p->alloc_lock);
 
-	clear_tsk_thread_flag(p, TIF_SIGPENDING);
 	init_sigpending(&p->pending);
 
 	p->utime = cputime_zero;
--
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
copy_process(): remove the unneeded clear_tsk_thread_flag( ..., Linux Kernel Mailing ..., (Thu Jun 18, 3:00 pm)