On 12 September 2013 12:10, Srivatsa S. Bhat <srivatsa.bhat@xxxxxxxxxxxxxxxxxx> wrote: > That said, your fix doesn't look correct. See below. I thought I was perfect !! :( > ... and change this suitably (from 1 to 0 etc..) ? To add to it, it will look more > clear as well: > > if (cpus == 0) { > /* No cpus in policy, so free it */ > } else { > /* Restart governor */ > } Currently cpus never become zero as we clear mask only while there are more than one cpu in a policy... Wait let me see what's the cleanest way to get this fixed.. -- 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