Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1610140

[PATCH 06/16] cpufreq: intel_pstate: Set HWP sampling interval once

From "Rafael J. Wysocki" <rjw@rjwysocki.net>
Newsgroups linux.kernel
Subject [PATCH 06/16] cpufreq: intel_pstate: Set HWP sampling interval once
Date 2017-03-28 00:50 +0200
Message-ID <tpLCy-6Cv-13@gated-at.bofh.it> (permalink)
References <tpLsR-6yQ-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

In the HWP enabled case pid_params.sample_rate_ns only needs to be
updated once, because it is global, so do that when setting hwp_active
instead of doing it during the initialization of every CPU.

Moreover, pid_params.sample_rate_ms is never used if HWP is enabled,
so do not update it at all then.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---
 drivers/cpufreq/intel_pstate.c |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Index: linux-pm/drivers/cpufreq/intel_pstate.c
===================================================================
--- linux-pm.orig/drivers/cpufreq/intel_pstate.c
+++ linux-pm/drivers/cpufreq/intel_pstate.c
@@ -1911,8 +1911,6 @@ static int intel_pstate_init_cpu(unsigne
 			intel_pstate_disable_ee(cpunum);
 
 		intel_pstate_hwp_enable(cpu);
-		pid_params.sample_rate_ms = 50;
-		pid_params.sample_rate_ns = 50 * NSEC_PER_MSEC;
 	}
 
 	intel_pstate_get_cpu_pstates(cpu);
@@ -2563,6 +2561,7 @@ static int __init intel_pstate_init(void
 		} else {
 			hwp_active++;
 			intel_pstate.attr = hwp_cpufreq_attrs;
+			pid_params.sample_rate_ns = 50 * NSEC_PER_MSEC;
 			goto hwp_cpu_matched;
 		}
 	} else {

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 00/16] cpufreq: intel_pstate: Cleanups and optimizations "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 02/16] cpufreq: intel_pstate: Drop pointless initialization of PID parameters "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 13/16] cpufreq: intel_pstate: Drop struct cpu_defaults "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 10/16] cpufreq: intel_pstate: Use different utilization update callbacks "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 14/16] cpufreq: intel_pstate: Introduce pid_in_use() "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 08/16] cpufreq: intel_pstate: Drop driver_registered variable "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 03/16] cpufreq: intel_pstate: Initialize pid_params statically "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 06/16] cpufreq: intel_pstate: Set HWP sampling interval once "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 05/16] cpufreq: intel_pstate: Clean up intel_pstate_busy_pid_reset() "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 11/16] cpufreq: intel_pstate: Add update_util callback to pstate_funcs "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 09/16] cpufreq: intel_pstate: Modify check in intel_pstate_update_status() "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200
  [PATCH 01/16] cpufreq: intel_pstate: Eliminate struct perf_limits "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-28 00:50 +0200

csiph-web