From mboxrd@z Thu Jan 1 00:00:00 1970 From: Srinivas Pandruvada Subject: Re: [PATCH V3] SKL intel_pstate update MSR values when changing governors Date: Tue, 24 Nov 2015 07:57:42 -0800 Message-ID: <1448380662.3070.3.camel@linux.intel.com> References: <1447887520-10399-1-git-send-email-alexandra.yates@linux.intel.com> <1448327856.24744.6.camel@spandruv-desk3.jf.intel.com> <8282537.nh6Mqkv7tq@vostro.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from mga01.intel.com ([192.55.52.88]:51644 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753246AbbKXP7m (ORCPT ); Tue, 24 Nov 2015 10:59:42 -0500 In-Reply-To: <8282537.nh6Mqkv7tq@vostro.rjw.lan> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: "Rafael J. Wysocki" Cc: "Rafael J. Wysocki" , Alexandra Yates , Len Brown , Viresh Kumar , "linux-pm@vger.kernel.org" , Linux Kernel Mailing List On Tue, 2015-11-24 at 15:28 +0100, Rafael J. Wysocki wrote: > On Monday, November 23, 2015 05:17:36 PM Srinivas Pandruvada wrote: > > On Tue, 2015-11-24 at 02:09 +0100, Rafael J. Wysocki wrote: > > > Hi, > > > > > > On Sat, Nov 21, 2015 at 1:16 AM, Srinivas Pandruvada > > > wrote: > > > > > > [cut] > > > > > > > > > For more information on the MSR values for SKL please visit > > > > > ISDM under Managing HWP. > > > > > > > > > > Signed-off-by: Alexandra Yates < > > > > > alexandra.yates@linux.intel.com> > > > > > > > > Acked-by: Srinivas Pandruvada < > > > > srinivas.pandruvada@linux.intel.com> > > > > > > > > > > --- > > > > > drivers/cpufreq/intel_pstate.c | 2 ++ > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > diff --git a/drivers/cpufreq/intel_pstate.c > > > > > b/drivers/cpufreq/intel_pstate.c > > > > > index 2e31d09..0eeb7da 100644 > > > > > --- a/drivers/cpufreq/intel_pstate.c > > > > > +++ b/drivers/cpufreq/intel_pstate.c > > > > > @@ -1242,6 +1242,8 @@ static int > > > > > intel_pstate_set_policy(struct > > > > > cpufreq_policy *policy) > > > > > policy->max >= policy->cpuinfo.max_freq) { > > > > > pr_debug("intel_pstate: set performance\n"); > > > > > limits = &performance_limits; > > > > > + if (hwp_active) > > > > > + intel_pstate_hwp_set(); > > > > > > Honestly, I'm not really sure how this is matching the changelog. > > > > > > What it does is to ensure that the correct limits are used when > > > in the > > > HWP mode too as far as I can say. Is my understanding correct > > > here? > > Yes. > > OK > > But if we make the "performance" limits take effect here, it will > effectively > make HWP use min = max = 100%, right? Is that what we want to happen > here? Yes. intel_pstate_hwp_set() uses limits->min_perf_pct and limits ->max_perf_pct to set HWP limits. Here these will be set to 100%. Thanks, Srinivas > > Thanks, > Rafael >