summaryrefslogtreecommitdiffstats
path: root/kernel/exit.c
diff options
context:
space:
mode:
authorOleg Nesterov <oleg@redhat.com>2008-11-17 15:40:08 +0100
committerIngo Molnar <mingo@elte.hu>2008-11-17 16:55:55 +0100
commit2b5fe6de58276d0b5a7c884d5dbfc300ca47db78 (patch)
tree02cc368e46a795bd2c0162d278f9c5549f4e1e5a /kernel/exit.c
parentce394471d13bf071939a9a0b48c64c297676d233 (diff)
downloadlinux-2b5fe6de58276d0b5a7c884d5dbfc300ca47db78.tar.gz
linux-2b5fe6de58276d0b5a7c884d5dbfc300ca47db78.tar.bz2
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.c2
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,