cpufreq: interactive: don't skip waking up speedchange_task if target_freq > policy...
authorMinsung Kim <ms925.kim@samsung.com>
Sat, 29 Nov 2014 12:43:53 +0000 (21:43 +0900)
committerJohn Stultz <john.stultz@linaro.org>
Tue, 16 Feb 2016 21:53:04 +0000 (13:53 -0800)
When __cpufreq_driver_target() in speedchange_task failed for some reason, the
policy->cur could be lower than the target_freq. The governor misses to change
the target_freq if the target_freq is equal to the next_freq at the next sample
time.

Added a check to prevent the CPU to stay at the speed that is lower than the
target_freq for long duration.

Change-Id: Ibfdcd193b8280390b8f8374a63218aa31267f310
Signed-off-by: Minsung Kim <ms925.kim@samsung.com>
drivers/cpufreq/cpufreq_interactive.c

index a0a7c3aef859a4ac003dd8a91ace0fb96ab21169..5121976ade550206bb9a0ecce0929b76582b5637 100644 (file)
@@ -432,7 +432,8 @@ static void cpufreq_interactive_timer(unsigned long data)
                pcpu->floor_validate_time = now;
        }
 
-       if (pcpu->target_freq == new_freq) {
+       if (pcpu->target_freq == new_freq &&
+                       pcpu->target_freq <= pcpu->policy->cur) {
                trace_cpufreq_interactive_already(
                        data, cpu_load, pcpu->target_freq,
                        pcpu->policy->cur, new_freq);