diff mbox

intel_pstate: Remove periodic P state boost

Message ID 1390495707-6297-1-git-send-email-dirk.j.brandewie@intel.com (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

dirk.brandewie@gmail.com Jan. 23, 2014, 4:48 p.m. UTC
From: Dirk Brandewie <dirk.j.brandewie@intel.com>

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 resolved 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

reference:
  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>
---
 drivers/cpufreq/intel_pstate.c | 11 -----------
 1 file changed, 11 deletions(-)

Comments

dirk.brandewie@gmail.com Jan. 23, 2014, 4:51 p.m. UTC | #1
Please ignore. Sorry for the noise I found this in the 3.14 material.

--Dirk
On 01/23/2014 08:48 AM, dirk.brandewie@gmail.com wrote:
> From: Dirk Brandewie <dirk.j.brandewie@intel.com>
>
> 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 resolved 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
>
> reference:
>    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>
> ---
>   drivers/cpufreq/intel_pstate.c | 11 -----------
>   1 file changed, 11 deletions(-)
>
> diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
> index d51f17ed..5a060eb 100644
> --- a/drivers/cpufreq/intel_pstate.c
> +++ b/drivers/cpufreq/intel_pstate.c
> @@ -84,8 +84,6 @@ struct cpudata {
>   	struct pstate_data pstate;
>   	struct _pid pid;
>
> -	int min_pstate_count;
> -
>   	u64	prev_aperf;
>   	u64	prev_mperf;
>   	int	sample_ptr;
> @@ -568,15 +566,6 @@ static void intel_pstate_timer_func(unsigned long __data)
>
>   	intel_pstate_sample(cpu);
>   	intel_pstate_adjust_busy_pstate(cpu);
> -
> -	if (cpu->pstate.current_pstate == cpu->pstate.min_pstate) {
> -		cpu->min_pstate_count++;
> -		if (!(cpu->min_pstate_count % 5)) {
> -			intel_pstate_set_pstate(cpu, cpu->pstate.max_pstate);
> -		}
> -	} else
> -		cpu->min_pstate_count = 0;
> -
>   	intel_pstate_set_sample_time(cpu);
>   }
>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
index d51f17ed..5a060eb 100644
--- a/drivers/cpufreq/intel_pstate.c
+++ b/drivers/cpufreq/intel_pstate.c
@@ -84,8 +84,6 @@  struct cpudata {
 	struct pstate_data pstate;
 	struct _pid pid;
 
-	int min_pstate_count;
-
 	u64	prev_aperf;
 	u64	prev_mperf;
 	int	sample_ptr;
@@ -568,15 +566,6 @@  static void intel_pstate_timer_func(unsigned long __data)
 
 	intel_pstate_sample(cpu);
 	intel_pstate_adjust_busy_pstate(cpu);
-
-	if (cpu->pstate.current_pstate == cpu->pstate.min_pstate) {
-		cpu->min_pstate_count++;
-		if (!(cpu->min_pstate_count % 5)) {
-			intel_pstate_set_pstate(cpu, cpu->pstate.max_pstate);
-		}
-	} else
-		cpu->min_pstate_count = 0;
-
 	intel_pstate_set_sample_time(cpu);
 }