summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTodd Poynor <toddpoynor@google.com>2012-04-26 21:41:40 -0700
committerAnton Vorontsov <anton.vorontsov@linaro.org>2012-11-05 11:47:13 -0800
commit5f5bc6eeefe8fed8ebd837c49933ee06e6195850 (patch)
tree75cd2904eb86b991b08875c4bb242a945ea012d8
parent90b4630a59734161ad7d9af761f7a0640bfef6e6 (diff)
downloadlinux-topics-5f5bc6eeefe8fed8ebd837c49933ee06e6195850.tar.gz
cpufreq: interactive: set floor for boosted speed
Allow speed to drop to flooor frequency but not below, don't pin to speed at last boost. Change-Id: I0147c2b7a2e61ba16820605af6baaf09570be787 Signed-off-by: Todd Poynor <toddpoynor@google.com>
-rw-r--r--drivers/cpufreq/cpufreq_interactive.c23
1 files changed, 13 insertions, 10 deletions
diff --git a/drivers/cpufreq/cpufreq_interactive.c b/drivers/cpufreq/cpufreq_interactive.c
index c4390b27579..e1310c449dc 100644
--- a/drivers/cpufreq/cpufreq_interactive.c
+++ b/drivers/cpufreq/cpufreq_interactive.c
@@ -46,10 +46,11 @@ struct cpufreq_interactive_cpuinfo {
int idling;
u64 target_set_time;
u64 target_set_time_in_idle;
- u64 target_validate_time;
struct cpufreq_policy *policy;
struct cpufreq_frequency_table *freq_table;
unsigned int target_freq;
+ unsigned int floor_freq;
+ u64 floor_validate_time;
int governor_enabled;
};
@@ -227,10 +228,10 @@ static void cpufreq_interactive_timer(unsigned long data)
new_freq = pcpu->freq_table[index].frequency;
/*
- * Do not scale down unless we have been at this frequency for the
- * minimum sample time since last validated.
+ * Do not scale below floor_freq unless we have been at or above the
+ * floor frequency for the minimum sample time since last validated.
*/
- if (new_freq < pcpu->target_freq) {
+ if (new_freq < pcpu->floor_freq) {
if (pcpu->timer_run_time - pcpu->target_validate_time
< min_sample_time) {
trace_cpufreq_interactive_notyet(data, cpu_load,
@@ -239,7 +240,8 @@ static void cpufreq_interactive_timer(unsigned long data)
}
}
- pcpu->target_validate_time = pcpu->timer_run_time;
+ pcpu->floor_freq = new_freq;
+ pcpu->floor_validate_time = pcpu->timer_run_time;
if (pcpu->target_freq == new_freq) {
trace_cpufreq_interactive_already(data, cpu_load,
@@ -507,12 +509,12 @@ static void cpufreq_interactive_boost(void)
}
/*
- * Refresh time at which current (possibly being
- * boosted) speed last validated (reset timer for
- * allowing speed to drop).
+ * Set floor freq and (re)start timer for when last
+ * validated.
*/
- pcpu->target_validate_time = ktime_to_us(ktime_get());
+ pcpu->floor_freq = hispeed_freq;
+ pcpu->floor_validate_time = ktime_to_us(ktime_get());
}
spin_unlock_irqrestore(&up_cpumask_lock, flags);
@@ -810,7 +812,8 @@ static int cpufreq_governor_interactive(struct cpufreq_policy *policy,
pcpu->target_set_time_in_idle =
get_cpu_idle_time_us(j,
&pcpu->target_set_time);
- pcpu->target_validate_time =
+ pcpu->floor_freq = pcpu->target_freq;
+ pcpu->floor_validate_time =
pcpu->target_set_time;
pcpu->governor_enabled = 1;
smp_wmb();