lkml.org 
[lkml]   [2015]   [Jul]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] cpufreq: Drop unnecessary arguments from two functions
On 22-07-15, 04:09, Rafael J. Wysocki wrote:
> static int cpufreq_remove_dev(struct device *dev, struct subsys_interface *sif)
> {
> unsigned int cpu = dev->id;
> + struct cpufreq_policy *policy = per_cpu(cpufreq_cpu_data, cpu);
> int ret;
>
> /*
> @@ -1530,7 +1525,6 @@ static int cpufreq_remove_dev(struct dev
> * link or free policy here.
> */
> if (cpu_is_offline(cpu)) {
> - struct cpufreq_policy *policy = per_cpu(cpufreq_cpu_data, cpu);
> struct cpumask mask;
>
> if (!policy)
> @@ -1547,17 +1541,18 @@ static int cpufreq_remove_dev(struct dev
> remove_cpu_dev_symlink(policy, cpu);
> return 0;
> }
> + } else {
> + ret = __cpufreq_remove_dev_prepare(dev);
> + if (!ret)
> + ret = __cpufreq_remove_dev_finish(dev);
>
> - cpufreq_policy_free(policy, true);
> - return 0;
> + /* The CPU is online, so the policy cannot be inactive here. */
> + if (ret)
> + return ret;
> }
>
> - ret = __cpufreq_remove_dev_prepare(dev, sif);
> -
> - if (!ret)
> - ret = __cpufreq_remove_dev_finish(dev, sif);
> -
> - return ret;
> + cpufreq_policy_free(policy, true);

You need to check for policy_is_inactive(policy) before freeing
policy. cpufreq_remove_dev() will be called one by one for all CPUs of
a policy ..

--
viresh


\
 
 \ /
  Last update: 2015-07-22 09:01    [W:0.068 / U:0.204 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site