On 06/04/2015 12:12 PM, Viresh Kumar wrote:
On 04-06-15, 12:06, Preeti U Murthy wrote:
Your fix may not be complete and here is why. The reason we see the crash is because we have *only* attempted to serialize calls to cpufreq_governor_dbs() and not attempted to serialize *entire logical sequence of operations*. Let's take a look at what is happening as a consequence.
You missed my logs (For the first time in my life I wrote them so well). This is what I mentioned in 3/3:
" These two issues need to be solved separately and the responsibility should be properly divided between cpufreq and governor core.
The first problem is more about the governor core, as it needs to protect its structures properly. And the second problem should be fixed in cpufreq core instead of governor, as its all about sequence of events.
My point is do we really need to treat them as separate problems ? Will not serializing sequence of events help solve both issues ?
When we know the problem, why not fix it proper, rather than breaking it up ?
This patch is trying to solve only the first problem. "
I NEVER claimed that I solved all the issues.
That is true. My intention was to point out explicitly what still remains to be solved. It is true that you have mentioned that the problem in the cpufreq core is about sequencing of events. I intended to highlight what it was.
I would have restrained from pointing it out had the issues that I am seeing waned a wee bit, but it has not, which is why I did not see value in having the third patch as a stand alone patch, with more going in as series.
Regards Preeti U Murthy