lkml.org 
[lkml]   [2018]   [Nov]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v9 08/15] sched/topology: Make Energy Aware Scheduling depend on schedutil
On Thursday 22 Nov 2018 at 16:49:04 (+0100), Rafael J. Wysocki wrote:
> On Thu, Nov 22, 2018 at 3:05 PM Peter Zijlstra <peterz@infradead.org> wrote:
> >
> > On Mon, Nov 19, 2018 at 02:18:50PM +0000, Quentin Perret wrote:
> > > + if (rd->pd)
> > > + pr_warn("rd %*pbl: Disabling EAS, schedutil is mandatory\n",
> > > + cpumask_pr_args(cpu_map));
> > > + goto free;
> >
> > I would have preferred just not allowing the governor to change at all,
> > but if you want to allow that, this is, I suppose, the best you can do.
> >
> > Rafael, are you otherwise OK with the grubbing inside cpufreq?
>
> Sort of. :-)
>
> I would put the sched_cpufreq_governor_change() header (and the static
> inline stub of it) into linux/cpufreq.h (instead of
> linux/sched/cpufreq.h) so that cpufreq.c doesn't have to include
> linux/sched/cpufreq.h which is kind of confusing.

That works for me.

Thanks,
Quentin

\
 
 \ /
  Last update: 2018-11-22 16:56    [W:0.071 / U:3.496 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site