diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2008-11-17 15:40:08 +0100 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2008-11-17 16:55:55 +0100 |
| commit | 2b5fe6de58276d0b5a7c884d5dbfc300ca47db78 (patch) | |
| tree | 02cc368e46a795bd2c0162d278f9c5549f4e1e5a /kernel/exit.c | |
| parent | thread_group_cputime: kill the bogus ->signal != NULL check (diff) | |
| download | linux-2b5fe6de58276d0b5a7c884d5dbfc300ca47db78.tar.xz linux-2b5fe6de58276d0b5a7c884d5dbfc300ca47db78.zip | |
thread_group_cputime: move a couple of callsites outside of ->siglock
Impact: relax the locking of cpu-time accounting calls
->siglock buys nothing for thread_group_cputime() in do_sys_times() and
wait_task_zombie() (which btw takes the unrelated parent's ->siglock).
Actually I think do_sys_times() doesn't need ->siglock at all.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/exit.c')
| -rw-r--r-- | kernel/exit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/exit.c b/kernel/exit.c index ae2b92be5fae..b9c4d8bb72e5 100644 --- a/kernel/exit.c +++ b/kernel/exit.c @@ -1330,10 +1330,10 @@ static int wait_task_zombie(struct task_struct *p, int options, * group, which consolidates times for all threads in the * group including the group leader. */ + thread_group_cputime(p, &cputime); spin_lock_irq(&p->parent->sighand->siglock); psig = p->parent->signal; sig = p->signal; - thread_group_cputime(p, &cputime); psig->cutime = cputime_add(psig->cutime, cputime_add(cputime.utime, |