On Tue, Mar 3, 2015 at 3:09 PM, Kapileshwar Singh <[email protected]> wrote: > On 03/03/15 13:07, Viresh Kumar wrote:
[...] >> Please goto the depth of this thing, as I don't think it should happen. >> >> Over that I was asking you if you have tested the solution Javi gave, >> because OPPs >> wouldn't have been initialized for other CPUs once policy->cpu goes down. > I did test this but we were working with the assumption that OPPs should be > populated for all the CPUs and also that OPPs are lost for a hotplugged CPU > which I see is not the case. > > We have looked at this more closely and found that problem lies in: > > freq = cpufreq_quick_get(cpumask_any(&cpufreq_device->allowed_cpus)); > > which returns a NULL frequency as we are not checking for online CPUs here. > We shall come up with a fix for this. Many thanks for helping us with the > investigation. > You can use any_online_cpu(..) instead of cpumask_any IMO Regards, Sudeep -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
