sched/cpuacct: No need to check subsys active state
authorLi Zefan <lizefan@huawei.com>
Fri, 29 Mar 2013 06:44:28 +0000 (14:44 +0800)
committerIngo Molnar <mingo@kernel.org>
Wed, 10 Apr 2013 11:54:22 +0000 (13:54 +0200)
Now we're guaranteed when cpuacct_charge() and
cpuacct_account_field() are called, cpuacct has already been
properly initialized, so we no longer need those checks.

Signed-off-by: Li Zefan <lizefan@huawei.com>
Cc: Tejun Heo <tj@kernel.org>
Acked-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/5155384C.7000508@huawei.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
kernel/sched/cpuacct.c

index 75e46d2..ef57ab6 100644 (file)
@@ -247,9 +247,6 @@ void cpuacct_charge(struct task_struct *tsk, u64 cputime)
        struct cpuacct *ca;
        int cpu;
 
-       if (unlikely(!cpuacct_subsys.active))
-               return;
-
        cpu = task_cpu(tsk);
 
        rcu_read_lock();
@@ -278,9 +275,6 @@ void cpuacct_account_field(struct task_struct *p, int index, u64 val)
        struct kernel_cpustat *kcpustat;
        struct cpuacct *ca;
 
-       if (unlikely(!cpuacct_subsys.active))
-               return;
-
        rcu_read_lock();
        ca = task_ca(p);
        while (ca != &root_cpuacct) {