Re: [PATCH cgroup/for-4.15] cgroup, sched: Move basic cpu stats from cgroup.stat to cpu.stat

From: Peter Zijlstra
Date: Thu Oct 26 2017 - 13:43:39 EST


On Mon, Oct 23, 2017 at 04:18:27PM -0700, Tejun Heo wrote:
> The basic cpu stat is currently shown with "cpu." prefix in
> cgroup.stat, and the same information is duplicated in cpu.stat when
> cpu controller is enabled. This is ugly and not very scalable as we
> want to expand the coverage of stat information which is always
> available.
>
> This patch makes cgroup core always create "cpu.stat" file and show
> the basic cpu stat there and calls the cpu controller to show the
> extra stats when enabled. This ensures that the same information
> isn't presented in multiple places and makes future expansion of basic
> stats easier.
>
> Signed-off-by: Tejun Heo <tj@xxxxxxxxxx>

> Documentation/cgroup-v2.txt | 15 +++-------
> include/linux/cgroup-defs.h | 2 +
> include/linux/cgroup.h | 2 -
> kernel/cgroup/cgroup-internal.h | 1
> kernel/cgroup/cgroup.c | 60 ++++++++++++++++++++++++++++++++++++++--
> kernel/cgroup/stat.c | 10 +++---
> kernel/sched/core.c | 13 ++------
> 7 files changed, 75 insertions(+), 28 deletions(-)

Acked-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx>