Re: [PATCH v4 7/7] cpufreq: schedutil: New governor based on scheduler utilization data

From: Peter Zijlstra
Date: Wed Mar 16 2016 - 13:54:13 EST


On Wed, Mar 16, 2016 at 03:59:18PM +0100, Rafael J. Wysocki wrote:
> +static void sugov_update_commit(struct sugov_policy *sg_policy, u64 time,
> + unsigned int next_freq)
> +{
> + struct cpufreq_policy *policy = sg_policy->policy;
> +
> + if (next_freq > policy->max)
> + next_freq = policy->max;
> + else if (next_freq < policy->min)
> + next_freq = policy->min;

I'm still very much undecided on these policy min/max thresholds. I
don't particularly like them.