diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2007-10-17 02:27:23 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-17 11:42:53 -0400 |
commit | 356d6d5058c8082b9e811838ab2fa27825c947e4 (patch) | |
tree | b75accbc155f490f148a989e355dbdeb59c8607f | |
parent | b2c903b8790467ae400f6992ac01bc8913b49e19 (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.c | 9 |
1 files changed, 0 insertions, 9 deletions
@@ -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 | ||