Commit 044d5c26 authored by Lan Tianyu's avatar Lan Tianyu Committed by Zhang Rui

Thermal/cpu_cooling: Return directly for the cpu out of allowed_cpus in the...

Thermal/cpu_cooling: Return directly for the cpu out of allowed_cpus in the cpufreq_thermal_notifier()

cpufreq_thermal_notifier() is to change the cpu's cpufreq in the allowed_cpus mask
when associated thermal-cpufreq cdev's cooling state is changed. It's a cpufreq policy
notifier handler and it will be triggered even if those cpus out of allowed_cpus has
changed freq policy.

cpufreq_thermal_notifier() checks the policy->cpu. If it belongs to allowed_cpus,
change max_freq(default to 0) to the desire cpufreq value and pass 0 and max_freq
to cpufreq_verify_within_limits() as cpufreq scope. But if not, do nothing and
max_freq will remain 0. This will cause the cpufreq scope to become 0~0. This
is not right. This patch is to return directly after finding cpu not belonging
to allowed_cpus.
Signed-off-by: default avatarLan Tianyu <tianyu.lan@intel.com>
Signed-off-by: default avatarZhang Rui <rui.zhang@intel.com>
parent 584d88b2
...@@ -322,6 +322,8 @@ static int cpufreq_thermal_notifier(struct notifier_block *nb, ...@@ -322,6 +322,8 @@ static int cpufreq_thermal_notifier(struct notifier_block *nb,
if (cpumask_test_cpu(policy->cpu, &notify_device->allowed_cpus)) if (cpumask_test_cpu(policy->cpu, &notify_device->allowed_cpus))
max_freq = notify_device->cpufreq_val; max_freq = notify_device->cpufreq_val;
else
return 0;
/* Never exceed user_policy.max */ /* Never exceed user_policy.max */
if (max_freq > policy->user_policy.max) if (max_freq > policy->user_policy.max)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment