diff options
author | Dirk Brandewie <dirk.j.brandewie@intel.com> | 2013-12-17 12:42:07 -0500 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-12-21 19:04:55 -0500 |
commit | 91a4cd4f3d8169d7398f9123683f64575927c682 (patch) | |
tree | a3aba0366e7543d496c7c28daf1546a51a2a3b34 /drivers/cpufreq/intel_pstate.c | |
parent | 007bea098b869945a462420a1f9d442ff169f722 (diff) |
intel_pstate: Remove periodic P state boost
Remove the periodic P state boost. This code required for some corner
case benchmark tests. The calculation of the required P state was
incorrect/inaccurate and would not allow P state increase.
This was fixed by a combination of commits:
2134ed4 cpufreq / intel_pstate: Change to scale off of max P-state
d253d2a intel_pstate: Improve accuracy by not truncating until final result
References: https://bugzilla.kernel.org/show_bug.cgi?id=64271
Reported-by: Doug Smythies <dsmythies@telus.net>
Signed-off-by: Dirk Brandewie <dirk.j.brandewie@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/cpufreq/intel_pstate.c')
-rw-r--r-- | drivers/cpufreq/intel_pstate.c | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c index c84b280238a4..41e21bb01c9c 100644 --- a/drivers/cpufreq/intel_pstate.c +++ b/drivers/cpufreq/intel_pstate.c | |||
@@ -92,8 +92,6 @@ struct cpudata { | |||
92 | struct vid_data vid; | 92 | struct vid_data vid; |
93 | struct _pid pid; | 93 | struct _pid pid; |
94 | 94 | ||
95 | int min_pstate_count; | ||
96 | |||
97 | u64 prev_aperf; | 95 | u64 prev_aperf; |
98 | u64 prev_mperf; | 96 | u64 prev_mperf; |
99 | int sample_ptr; | 97 | int sample_ptr; |
@@ -617,15 +615,6 @@ static void intel_pstate_timer_func(unsigned long __data) | |||
617 | 615 | ||
618 | intel_pstate_sample(cpu); | 616 | intel_pstate_sample(cpu); |
619 | intel_pstate_adjust_busy_pstate(cpu); | 617 | intel_pstate_adjust_busy_pstate(cpu); |
620 | |||
621 | if (cpu->pstate.current_pstate == cpu->pstate.min_pstate) { | ||
622 | cpu->min_pstate_count++; | ||
623 | if (!(cpu->min_pstate_count % 5)) { | ||
624 | intel_pstate_set_pstate(cpu, cpu->pstate.max_pstate); | ||
625 | } | ||
626 | } else | ||
627 | cpu->min_pstate_count = 0; | ||
628 | |||
629 | intel_pstate_set_sample_time(cpu); | 618 | intel_pstate_set_sample_time(cpu); |
630 | } | 619 | } |
631 | 620 | ||