Commit a48ac1c9 authored Jan 20, 2020 by chenqiwu Committed by Viresh Kumar Jan 20, 2020

cpufreq: brcmstb-avs: fix imbalance of cpufreq policy refcount



brcm_avs_cpufreq_get() calls cpufreq_cpu_get() to get the cpufreq
policy, meanwhile, it also increments the kobject reference count
to mark it busy. However, a corresponding call of cpufreq_cpu_put()
is ignored to decrement the kobject reference count back, which may
lead to a potential stuck risk that the cpuhp thread deadly waits
for dropping of kobject refcount when cpufreq policy free.

With this patch, the cpuhp thread can be easily exercised by
attempting to force an unbind of the CPUfreq driver.

Signed-off-by: chenqiwu <chenqiwu@xiaomi.com>
[ Viresh: Dropped !policy check ]
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

parent 8ead819f

drivers/cpufreq/brcmstb-avs-cpufreq.c

+2 −0

Original line number	Diff line number	Diff line
		@@ -455,6 +455,8 @@ static unsigned int brcm_avs_cpufreq_get(unsigned int cpu)
		struct cpufreq_policy *policy = cpufreq_cpu_get(cpu);
		struct private_data *priv = policy->driver_data;

		cpufreq_cpu_put(policy);

		return brcm_avs_get_frequency(priv->base);
		}

Admin message