diff options
author | Oleg Nesterov <oleg@redhat.com> | 2008-11-17 09:39:52 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-11-17 10:55:53 -0500 |
commit | 74fcd524e808975dd546dac847119f1995a7c622 (patch) | |
tree | 9b6d2aa46509926a7922183ba728bb0300bfbc1e | |
parent | ad133ba3dc283300e5b62b5b7211d2f39fbf6ee7 (diff) |
account_steal_time: kill the unneeded account_group_system_time()
Impact: remove unnecessary accounting call
I don't actually understand account_steal_time() and I failed to find the
commit which added account_group_system_time(), but this looks bogus.
In any case rq->idle must be single-threaded, so it can't have ->totals.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/sched.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index c94baf2969e7..b388c9b243e9 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
@@ -4202,7 +4202,6 @@ void account_steal_time(struct task_struct *p, cputime_t steal) | |||
4202 | 4202 | ||
4203 | if (p == rq->idle) { | 4203 | if (p == rq->idle) { |
4204 | p->stime = cputime_add(p->stime, steal); | 4204 | p->stime = cputime_add(p->stime, steal); |
4205 | account_group_system_time(p, steal); | ||
4206 | if (atomic_read(&rq->nr_iowait) > 0) | 4205 | if (atomic_read(&rq->nr_iowait) > 0) |
4207 | cpustat->iowait = cputime64_add(cpustat->iowait, tmp); | 4206 | cpustat->iowait = cputime64_add(cpustat->iowait, tmp); |
4208 | else | 4207 | else |