diff options
author | Li Zefan <lizf@cn.fujitsu.com> | 2009-03-11 21:23:01 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-03-12 23:54:58 -0400 |
commit | 7a46c594bf7f1f2eeb1e12d4b857d5f581957a92 (patch) | |
tree | 35731095fd8f01938a36d2647995ee86747e3bbf | |
parent | 65a37b29a8d7492f081d4f8da2227467af1c0c1d (diff) |
cpuacct: reduce one NULL check in fast-path
Impact: micro-optimization
In cpuacct_charge(), task_ca() will never return NULL, so change
for(...) to do { } while(...) to save one NULL check.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Menage <menage@google.com>
Cc: Balbir Singh <balbir@linux.vnet.ibm.com>
Cc: Bharata B Rao <bharata@linux.vnet.ibm.com>
LKML-Reference: <49B863F5.2060400@cn.fujitsu.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/sched.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index 0a76d0b6f215..61e63562f273 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
@@ -9599,10 +9599,11 @@ static void cpuacct_charge(struct task_struct *tsk, u64 cputime) | |||
9599 | cpu = task_cpu(tsk); | 9599 | cpu = task_cpu(tsk); |
9600 | ca = task_ca(tsk); | 9600 | ca = task_ca(tsk); |
9601 | 9601 | ||
9602 | for (; ca; ca = ca->parent) { | 9602 | do { |
9603 | u64 *cpuusage = per_cpu_ptr(ca->cpuusage, cpu); | 9603 | u64 *cpuusage = per_cpu_ptr(ca->cpuusage, cpu); |
9604 | *cpuusage += cputime; | 9604 | *cpuusage += cputime; |
9605 | } | 9605 | ca = ca->parent; |
9606 | } while (ca); | ||
9606 | } | 9607 | } |
9607 | 9608 | ||
9608 | struct cgroup_subsys cpuacct_subsys = { | 9609 | struct cgroup_subsys cpuacct_subsys = { |