12 Sep
2013
12 Sep
'13
6:47 a.m.
On 12 September 2013 12:10, Srivatsa S. Bhat srivatsa.bhat@linux.vnet.ibm.com 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..