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 | ||
