]> pilppa.com Git - linux-2.6-omap-h63xx.git/commit
[PATCH] cpufreq_conservative: make for_each_cpu() safe
authorAlexander Clouter <alex@digriz.org.uk>
Wed, 22 Mar 2006 09:59:16 +0000 (09:59 +0000)
committerDominik Brodowski <linux@dominikbrodowski.net>
Sun, 26 Mar 2006 08:14:54 +0000 (10:14 +0200)
commit08a28e2e98aa821cf6f15f8a267beb2f33377bb9
treec1bc076d41c50e76cca7e1af023d1d1ace1b8326
parente8a02572252f9115c2b8296c40fd8b985f06f872
[PATCH] cpufreq_conservative: make for_each_cpu() safe

All these changes should make cpufreq_conservative safe in regards to the x86
for_each_cpu cpumask.h changes and whatnot.

Whilst making it safe a number of pointless for loops related to the cpu
mask's were removed.  I was never comfortable with all those for loops,
especially as the iteration is over the same data again and again for each
CPU you had in a single poll, an O(n^2) outcome to frequency scaling.

The approach I use is to assume by default no CPU's exist and it sets the
requested_freq to zero as a kind of flag, the reasoning is in the source ;)
If the CPU is queried and requested_freq is zero then it initialises the
variable to current_freq and then continues as if nothing happened which
should be the same net effect as before?

Signed-off-by: Alexander Clouter <alex-kernel@digriz.org.uk>
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
drivers/cpufreq/cpufreq_conservative.c