| Message ID | 20210827081752.54337-1-schspa@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | cpufreq_ondemand: fix bad auto calculated frequency. | expand |
On 27-08-21, 16:17, schspa wrote: > We can litmit cpufreq range by change min & max from cpufreq_policy. > So cpu frequency target should be in range [policy->min, policy->max]. > > Signed-off-by: schspa <schspa@gmail.com> > --- > drivers/cpufreq/cpufreq_ondemand.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/cpufreq/cpufreq_ondemand.c b/drivers/cpufreq/cpufreq_ondemand.c > index ac361a8b1d3b..8afb2c84c38c 100644 > --- a/drivers/cpufreq/cpufreq_ondemand.c > +++ b/drivers/cpufreq/cpufreq_ondemand.c > @@ -151,8 +151,8 @@ static void od_update(struct cpufreq_policy *policy) > /* Calculate the next frequency proportional to load */ > unsigned int freq_next, min_f, max_f; > > - min_f = policy->cpuinfo.min_freq; > - max_f = policy->cpuinfo.max_freq; > + min_f = policy->min; > + max_f = policy->max; > freq_next = min_f + load * (max_f - min_f) / 100; No. The current calculations are right. The new frequency is proportional to current load and it needs to take into account the entire freq range of the CPUs. Note that we will eventually try to get the resultant frequency within policy->min/max range in __cpufreq_driver_target(). -- viresh
diff --git a/drivers/cpufreq/cpufreq_ondemand.c b/drivers/cpufreq/cpufreq_ondemand.c index ac361a8b1d3b..8afb2c84c38c 100644 --- a/drivers/cpufreq/cpufreq_ondemand.c +++ b/drivers/cpufreq/cpufreq_ondemand.c @@ -151,8 +151,8 @@ static void od_update(struct cpufreq_policy *policy) /* Calculate the next frequency proportional to load */ unsigned int freq_next, min_f, max_f; - min_f = policy->cpuinfo.min_freq; - max_f = policy->cpuinfo.max_freq; + min_f = policy->min; + max_f = policy->max; freq_next = min_f + load * (max_f - min_f) / 100; /* No longer fully busy, reset rate_mult */
We can litmit cpufreq range by change min & max from cpufreq_policy. So cpu frequency target should be in range [policy->min, policy->max]. Signed-off-by: schspa <schspa@gmail.com> --- drivers/cpufreq/cpufreq_ondemand.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)