On Fri, Feb 19, 2016 at 11:35 PM, Srinivas Pandruvada
[off-list ref] wrote:

On Fri, 2016-02-19 at 14:47 +0530, Viresh Kumar wrote:

quoted

Adding the maintainers of the driver in cc now.

[...]
Hi Viresh,

quoted

Hi Guys,

Joonas reported a lockdep between cpufreq and intel-pstate driver and
we are looking for probable solutions.

I failed to understand why should we run intel_pstate_hwp_set() for
each online CPU, while the frequency is changed only for a group of
CPUs that belong to a policy.

Don't need from set_policy interface.

quoted

Ofcourse intel_pstate_hwp_set() is
required to be run for all CPUs, while the sysfs files are touched.
And so, do we have a problem with below diff?

I gave a quick test, the change works fine.

Yes, the patch looks correct to me.

Thanks,
Rafael