Re: [PATCH v2 0/5] cpuacct, cgroup: Kill cgroup_subsys.active

Peter Zijlstra
2013-03-29T10:36:00Z

On Fri, 2013-03-29 at 14:43 +0800, Li Zefan wrote:
> cpuacct is the only user of cgroup_subsys.active flag.
>
> The flag is needed because cpuacct_charge() and
> cpuacct_account_field() can
> be called when cpuacct hasn't been initialized during system bootup.
>
> This patch initializes cpuacct earlier, and the result is we don't
> have
> to check the flag in scheduler hot path.
>
> Note, this patchset is based on "[PATCH v2 0/7] sched: Split cpuacct"
>
> http://lkml.org/lkml/2013/3/28/1
>
> 0001-cpuacct-allocate-per_cpu-cpuusage-for-root-cpuacct-s.patch
> 0002-cpuacct-Initialize-root-cpuacct-earlier.patch
> 0003-cpuacct-Initialize-cpuacct-subsystem-earlier.patch
> 0004-cpuacct-No-need-to-check-subsys-active-state.patch
> 0005-cgroup-Remove-subsys.active-flag.patch

Seems sane enough

Acked-by: Peter Zijlstra <a.p.zijlstra-/NLkJaSkS4VmR6Xm/wNWPw@public.gmane.org>