diff options
| author | Oleg Nesterov <oleg@tv-sign.ru> | 2007-10-16 23:27:23 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-17 08:42:53 -0700 |
| commit | 356d6d5058c8082b9e811838ab2fa27825c947e4 (patch) | |
| tree | b75accbc155f490f148a989e355dbdeb59c8607f | |
| parent | b2c903b8790467ae400f6992ac01bc8913b49e19 (diff) | |
| download | linux-stable-356d6d5058c8082b9e811838ab2fa27825c947e4.tar.gz linux-stable-356d6d5058c8082b9e811838ab2fa27825c947e4.tar.bz2 linux-stable-356d6d5058c8082b9e811838ab2fa27825c947e4.zip | |
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
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) struct task_struct *leader = NULL; int count; - /* - * If we don't share sighandlers, then we aren't sharing anything - * and we can just re-use it all. - */ - if (atomic_read(&oldsighand->count) <= 1) { - exit_itimers(sig); - return 0; - } - if (thread_group_empty(tsk)) goto no_thread_group; |