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

From: Peter Zijlstra
Date: Fri Mar 29 2013 - 06:36:09 EST


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@xxxxxxxxx>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/