On Friday, March 22, 2013 01:17:18 PM Rafael J. Wysocki wrote: > [Adding Boris and Thomas to the CC.] > ... > > I don't have enough knowledge about this driver and how is it used for all > > x86 systems and so want somebody else (who has some prior experience with > > it) to check how policy->cpus and policy->related_cpus must be set from > > driver->init(). > > OK, so what exactly do you need to now? > > This has to be addressed before final 3.9 this way or another - and the > sooner the better. Is this all to try to fix "cpufreq driver gets loaded while some cores were set offline before"? I wonder how you run into "cpufreq is initialized with offlined cpus" case. I remember that there were problems, but it's nearly impossible to run into this if the cpufreq driver is loaded early at boot. cpufreq_add_dev() and friends are complicated. Those init functions got split some time ago and there also slipped in a bug even it was simple splitting of functions. I do not have time to look at fcf8058296edbc3de43adf095824fc32b067b9f8 right now. Don't know how much other stuff depends on it and how sever it is on ARM that cpufreq does not correctly initialize with offlined cpus..., I would revert this patch. There are other cpu related drivers (at least on x86) which cannot initialize correctly if the CPUs got offlined before driver init. Obviously this is not a clever thing to do. Thomas -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
