cpufreq: fix jiffies/cputime mixup in conservative/ondemand governors
authorAndreas Schwab <schwab@linux-m68k.org>
Wed, 24 Oct 2012 20:16:34 +0000 (22:16 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 14 Nov 2012 23:33:07 +0000 (00:33 +0100)
The function get_cpu_idle_time_jiffy in both the conservative and
ondemand governors use jiffies_to_usecs to convert a cputime value to
usecs which gives the wrong value on architectures where cputime and
jiffies use different units.  Only matters if NO_HZ is disabled, since
otherwise get_cpu_idle_time_us should already return a valid value, and
get_cpu_idle_time_jiffy isn't actually called.

Signed-off-by: Andreas Schwab <schwab@linux-m68k.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/cpufreq_governor.c

index 0001071cdcdb992167023151a270a874abb80bce..679842a8d34a817417a474bc8c02209e93173df9 100644 (file)
@@ -33,9 +33,9 @@ static inline u64 get_cpu_idle_time_jiffy(unsigned int cpu, u64 *wall)
 
        idle_time = cur_wall_time - busy_time;
        if (wall)
-               *wall = jiffies_to_usecs(cur_wall_time);
+               *wall = cputime_to_usecs(cur_wall_time);
 
-       return jiffies_to_usecs(idle_time);
+       return cputime_to_usecs(idle_time);
 }
 
 cputime64_t get_cpu_idle_time(unsigned int cpu, cputime64_t *wall)