Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1249264

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

Show all headers | View raw

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
> another label to jump to if ->get() fails

We call cpufreq_policy_free() in that case and that does the cleanup
you are talking about.

-- 
viresh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH V2 5/5] cpufreq: postfix policy directory with the first CPU in related_cpus Viresh Kumar <viresh.kumar@linaro.org> - 2015-10-15 18:10 +0200
  Re: [PATCH V2 5/5] cpufreq: postfix policy directory with the first  CPU in related_cpus Saravana Kannan <skannan@codeaurora.org> - 2015-10-15 21:20 +0200
    Re: [PATCH V2 5/5] cpufreq: postfix policy directory with the first  CPU in related_cpus Viresh Kumar <viresh.kumar@linaro.org> - 2015-10-16 09:10 +0200
      [PATCH V3 5/5] cpufreq: postfix policy directory with the first CPU in related_cpus Viresh Kumar <viresh.kumar@linaro.org> - 2015-10-16 09:20 +0200
        Re: [PATCH V3 5/5] cpufreq: postfix policy directory with the first  CPU in related_cpus Saravana Kannan <skannan@codeaurora.org> - 2015-10-16 22:00 +0200
          Re: [PATCH V3 5/5] cpufreq: postfix policy directory with the first  CPU in related_cpus Viresh Kumar <viresh.kumar@linaro.org> - 2015-10-17 06:30 +0200

csiph-web