aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOleg Nesterov <oleg@tv-sign.ru>2007-10-17 02:27:23 -0400
committerLinus Torvalds <torvalds@woody.linux-foundation.org>2007-10-17 11:42:53 -0400
commit356d6d5058c8082b9e811838ab2fa27825c947e4 (patch)
treeb75accbc155f490f148a989e355dbdeb59c8607f
parentb2c903b8790467ae400f6992ac01bc8913b49e19 (diff)
exec: consolidate 2 fast-paths
Now that we don't pre-allocate the new ->sighand, we can kill the first fast path, it doesn't make sense any longer. At best, it can save one "list_empty()" check but leads to the code duplication. Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru> Cc: Roland McGrath <roland@redhat.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--fs/exec.c9
1 files changed, 0 insertions, 9 deletions
diff --git a/fs/exec.c b/fs/exec.c
index c6e91c3dc2a1..ab5a4a3ece46 100644
--- a/fs/exec.c
+++ b/fs/exec.c
@@ -752,15 +752,6 @@ static int de_thread(struct task_struct *tsk)
752 struct task_struct *leader = NULL; 752 struct task_struct *leader = NULL;
753 int count; 753 int count;
754 754
755 /*
756 * If we don't share sighandlers, then we aren't sharing anything
757 * and we can just re-use it all.
758 */
759 if (atomic_read(&oldsighand->count) <= 1) {
760 exit_itimers(sig);
761 return 0;
762 }
763
764 if (thread_group_empty(tsk)) 755 if (thread_group_empty(tsk))
765 goto no_thread_group; 756 goto no_thread_group;
766 757