On 14 April 2018 at 12:05, Peter Zijlstra <[email protected]> wrote: > On Fri, Mar 16, 2018 at 12:25:39PM +0100, Vincent Guittot wrote: >> diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h >> index 783eacf..a8003a9 100644 >> --- a/kernel/sched/sched.h >> +++ b/kernel/sched/sched.h >> @@ -592,6 +592,8 @@ struct rt_rq { >> unsigned long rt_nr_total; >> int overloaded; >> struct plist_head pushable_tasks; >> + >> + struct sched_avg avg; > > We only want this for the root cgroup, right? So why is this per cgroup?
Yes it's only for root cgroup. I have put it there for consistency with the CFS' PELT but it's only waste Bytes > > That is, I was expecting it to be rq::rt_avg or something.

