Re: [PATCH V3 5/5] cpufreq: postfix policy directory with the first CPU in related_cpus

2015-10-16 Thread Viresh Kumar
On 16-10-15, 12:50, Saravana Kannan wrote: > In the lines that follow this code we set the > per_cpu(cpufreq_cpu_data) to point to the new policy. But if the > subsequent cpu->get() fails, we goto out_exit_policy. But that label > doesn't clean up the per_cpu(cpufreq_cpu_data). So, I think we need

Re: [PATCH V3 5/5] cpufreq: postfix policy directory with the first CPU in related_cpus

2015-10-16 Thread Saravana Kannan
On 10/16/2015 12:11 AM, Viresh Kumar wrote: The sysfs policy directory is postfixed currently with the CPU number for which the policy was created, which isn't necessarily the first CPU in related_cpus mask. To make it more consistent and predictable, lets postfix the policy with the first cpu i

[PATCH V3 5/5] cpufreq: postfix policy directory with the first CPU in related_cpus

2015-10-16 Thread Viresh Kumar
The sysfs policy directory is postfixed currently with the CPU number for which the policy was created, which isn't necessarily the first CPU in related_cpus mask. To make it more consistent and predictable, lets postfix the policy with the first cpu in related-cpus mask. Suggested-by: Saravana K