* [PATCH V2] SKL intel_pstate update MSR values when changing governors [not found] <SKL intel_pstate update MSR values when changing governors> @ 2015-11-18 1:58 ` Alexandra Yates 2015-11-18 2:15 ` Srinivas Pandruvada 2015-11-18 22:58 ` [PATCH V3] " Alexandra Yates 1 sibling, 1 reply; 10+ messages in thread From: Alexandra Yates @ 2015-11-18 1:58 UTC (permalink / raw) To: rjw, lenb, srinivas.pandruvada, viresh.kumar, linux-pm, linux-kernel; +Cc: root From: root <root@skl-s-fedora22.jf.intel.com> When changing from powersave to performance governors Intel_pstate fails to update the MSR values that reflect the max_perf_pct to 100%. For instance in SKL reading rdmsr 0x774: Governor MSR max_perf_pct ========= ======== ============ Powersave 80002808 100% Powersave 80002008 80% Performance 80002028 [error] 100% Performance 80002828 [expected] 100% The line label [error] shows the culprit. At this point the MSR should reflect the max_perf_pct that is 100%, that corresponds to MSR 80002828 as shown on the next line of the example. Which is the maximum performance for the Performance governor. Instead it holds back the MSR value previously set by the Powersave, in this case 80002028. This patch allows the system to print the correct MSR value 80002828 that corresponds to the 100% max_perf_pct when changing from powersave to performance governors. For more information on the MSR values for SKL please visit ISDM under Managing HWP. Signed-off-by: root <root@skl-s-fedora22.jf.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(); return 0; } -- 1.9.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH V2] SKL intel_pstate update MSR values when changing governors 2015-11-18 1:58 ` [PATCH V2] SKL intel_pstate update MSR values when changing governors Alexandra Yates @ 2015-11-18 2:15 ` Srinivas Pandruvada 0 siblings, 0 replies; 10+ messages in thread From: Srinivas Pandruvada @ 2015-11-18 2:15 UTC (permalink / raw) To: Alexandra Yates; +Cc: rjw, lenb, viresh.kumar, linux-pm, linux-kernel, root On Tue, 2015-11-17 at 17:58 -0800, Alexandra Yates wrote: > From: root <root@skl-s-fedora22.jf.intel.com> You need to fix the git config ... The change itself is OK. Thanks, Srinivas > > When changing from powersave to performance governors > Intel_pstate fails to update the MSR values that reflect the > max_perf_pct to 100%. For instance in SKL reading rdmsr 0x774: > > Governor MSR max_perf_pct > ========= ======== ============ > Powersave 80002808 100% > Powersave 80002008 80% > Performance 80002028 [error] 100% > Performance 80002828 [expected] 100% > > The line label [error] shows the culprit. At this point the MSR > should reflect the max_perf_pct that is 100%, that corresponds > to MSR 80002828 as shown on the next line of the example. Which > is the maximum performance for the Performance governor. > Instead it holds back the MSR value previously set by the > Powersave, in this case 80002028. > > This patch allows the system to print the correct MSR value > 80002828 that corresponds to the 100% max_perf_pct when changing > from powersave to performance governors. > > For more information on the MSR values for SKL please visit > ISDM under Managing HWP. > > Signed-off-by: root <root@skl-s-fedora22.jf.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(); > return 0; > } > ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH V3] SKL intel_pstate update MSR values when changing governors [not found] <SKL intel_pstate update MSR values when changing governors> 2015-11-18 1:58 ` [PATCH V2] SKL intel_pstate update MSR values when changing governors Alexandra Yates @ 2015-11-18 22:58 ` Alexandra Yates 2015-11-21 0:16 ` Srinivas Pandruvada 1 sibling, 1 reply; 10+ messages in thread From: Alexandra Yates @ 2015-11-18 22:58 UTC (permalink / raw) To: srinivas.pandruvada, lenb, viresh.kumar, linux-pm, linux-kernel, rjw Cc: Alexandra Yates When changing from powersave to performance governors Intel_pstate fails to update the MSR values that reflect the max_perf_pct to 100%. For instance in SKL reading rdmsr 0x774: Governor MSR max_perf_pct ========= ======== ============ Powersave 80002808 100% Powersave 80002008 80% Performance 80002028 [error] 100% Performance 80002828 [expected] 100% The line label [error] shows the culprit. At this point the MSR should reflect the max_perf_pct that is 100%, that corresponds to MSR 80002828 as shown on the next line of the example. Which is the maximum performance for the Performance governor. Instead it holds back the MSR value previously set by the Powersave, in this case 80002028. This patch allows the system to print the correct MSR value 80002828 that corresponds to the 100% max_perf_pct when changing from powersave to performance governors. 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> --- 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(); return 0; } -- 1.9.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH V3] SKL intel_pstate update MSR values when changing governors 2015-11-18 22:58 ` [PATCH V3] " Alexandra Yates @ 2015-11-21 0:16 ` Srinivas Pandruvada 2015-11-24 1:09 ` Rafael J. Wysocki 0 siblings, 1 reply; 10+ messages in thread From: Srinivas Pandruvada @ 2015-11-21 0:16 UTC (permalink / raw) To: Alexandra Yates, lenb, viresh.kumar, linux-pm, linux-kernel, rjw On 11/18/2015 02:58 PM, Alexandra Yates wrote: > When changing from powersave to performance governors > Intel_pstate fails to update the MSR values that reflect the > max_perf_pct to 100%. For instance in SKL reading rdmsr 0x774: > > Governor MSR max_perf_pct > ========= ======== ============ > Powersave 80002808 100% > Powersave 80002008 80% > Performance 80002028 [error] 100% > Performance 80002828 [expected] 100% > > The line label [error] shows the culprit. At this point the MSR > should reflect the max_perf_pct that is 100%, that corresponds > to MSR 80002828 as shown on the next line of the example. Which > is the maximum performance for the Performance governor. > Instead it holds back the MSR value previously set by the > Powersave, in this case 80002028. > > This patch allows the system to print the correct MSR value > 80002828 that corresponds to the 100% max_perf_pct when changing > from powersave to performance governors. > > 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(); > return 0; > } > > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH V3] SKL intel_pstate update MSR values when changing governors 2015-11-21 0:16 ` Srinivas Pandruvada @ 2015-11-24 1:09 ` Rafael J. Wysocki 2015-11-24 1:17 ` Srinivas Pandruvada 0 siblings, 1 reply; 10+ messages in thread From: Rafael J. Wysocki @ 2015-11-24 1:09 UTC (permalink / raw) To: Srinivas Pandruvada, Alexandra Yates Cc: Len Brown, Viresh Kumar, linux-pm@vger.kernel.org, Linux Kernel Mailing List, Rafael J. Wysocki Hi, On Sat, Nov 21, 2015 at 1:16 AM, Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> wrote: > > > On 11/18/2015 02:58 PM, Alexandra Yates wrote: >> >> When changing from powersave to performance governors >> Intel_pstate fails to update the MSR values that reflect the >> max_perf_pct to 100%. For instance in SKL reading rdmsr 0x774: >> >> Governor MSR max_perf_pct >> ========= ======== ============ >> Powersave 80002808 100% >> Powersave 80002008 80% >> Performance 80002028 [error] 100% >> Performance 80002828 [expected] 100% >> >> The line label [error] shows the culprit. At this point the MSR >> should reflect the max_perf_pct that is 100%, that corresponds >> to MSR 80002828 as shown on the next line of the example. Which >> is the maximum performance for the Performance governor. >> Instead it holds back the MSR value previously set by the >> Powersave, in this case 80002028. >> >> This patch allows the system to print the correct MSR value >> 80002828 that corresponds to the 100% max_perf_pct when changing >> from powersave to performance governors. Is this only about what is printed or does it mean that the driver actually uses an incorrect MSR value? >> 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? >> return 0; >> } >> >> > -- Thanks, Rafael ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH V3] SKL intel_pstate update MSR values when changing governors 2015-11-24 1:09 ` Rafael J. Wysocki @ 2015-11-24 1:17 ` Srinivas Pandruvada 2015-11-24 14:28 ` Rafael J. Wysocki 0 siblings, 1 reply; 10+ messages in thread From: Srinivas Pandruvada @ 2015-11-24 1:17 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Alexandra Yates, Len Brown, Viresh Kumar, linux-pm@vger.kernel.org, Linux Kernel Mailing List, Rafael J. Wysocki 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 > <srinivas.pandruvada@linux.intel.com> wrote: > > > > > > On 11/18/2015 02:58 PM, Alexandra Yates wrote: > >> > >> When changing from powersave to performance governors > >> Intel_pstate fails to update the MSR values that reflect the > >> max_perf_pct to 100%. For instance in SKL reading rdmsr 0x774: > >> > >> Governor MSR max_perf_pct > >> ========= ======== ============ > >> Powersave 80002808 100% > >> Powersave 80002008 80% > >> Performance 80002028 [error] 100% > >> Performance 80002828 [expected] 100% > >> > >> The line label [error] shows the culprit. At this point the MSR > >> should reflect the max_perf_pct that is 100%, that corresponds > >> to MSR 80002828 as shown on the next line of the example. Which > >> is the maximum performance for the Performance governor. > >> Instead it holds back the MSR value previously set by the > >> Powersave, in this case 80002028. > >> > >> This patch allows the system to print the correct MSR value > >> 80002828 that corresponds to the 100% max_perf_pct when changing > >> from powersave to performance governors. > > Is this only about what is printed or does it mean that the driver > actually uses an incorrect MSR value? Driver was not configuring correct value in HWP MSR, which sets min and max limits, so the HWP is not using correct limits. In HWP case driver doesn't do anything other than setting this MSR. When she means print correct MSR, is that when changing the mode from powersave to peformance, when she prints MSRs (rdmsr 0x774), this doesn't have correct max limit. > > >> 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. > > >> return 0; > >> } > >> > >> > > -- > > Thanks, > Rafael > -- > 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH V3] SKL intel_pstate update MSR values when changing governors 2015-11-24 1:17 ` Srinivas Pandruvada @ 2015-11-24 14:28 ` Rafael J. Wysocki 2015-11-24 15:57 ` Srinivas Pandruvada 0 siblings, 1 reply; 10+ messages in thread From: Rafael J. Wysocki @ 2015-11-24 14:28 UTC (permalink / raw) To: Srinivas Pandruvada Cc: Rafael J. Wysocki, Alexandra Yates, Len Brown, Viresh Kumar, linux-pm@vger.kernel.org, Linux Kernel Mailing List 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 > > <srinivas.pandruvada@linux.intel.com> 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? Thanks, Rafael ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH V3] SKL intel_pstate update MSR values when changing governors 2015-11-24 14:28 ` Rafael J. Wysocki @ 2015-11-24 15:57 ` Srinivas Pandruvada 2015-11-24 16:36 ` Rafael J. Wysocki 0 siblings, 1 reply; 10+ messages in thread From: Srinivas Pandruvada @ 2015-11-24 15:57 UTC (permalink / raw) 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 > > > <srinivas.pandruvada@linux.intel.com> 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 > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH V3] SKL intel_pstate update MSR values when changing governors 2015-11-24 15:57 ` Srinivas Pandruvada @ 2015-11-24 16:36 ` Rafael J. Wysocki 2015-11-24 16:27 ` Srinivas Pandruvada 0 siblings, 1 reply; 10+ messages in thread From: Rafael J. Wysocki @ 2015-11-24 16:36 UTC (permalink / raw) To: Srinivas Pandruvada Cc: Rafael J. Wysocki, Alexandra Yates, Len Brown, Viresh Kumar, linux-pm@vger.kernel.org, Linux Kernel Mailing List On Tuesday, November 24, 2015 07:57:42 AM Srinivas Pandruvada wrote: > 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 > > > > <srinivas.pandruvada@linux.intel.com> 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%. OK, so that's the intended behavior of "performance" in the HWP case, right? I'll apply this patch, but I'm going to write a new changelog for it, then. Thanks, Rafael ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH V3] SKL intel_pstate update MSR values when changing governors 2015-11-24 16:36 ` Rafael J. Wysocki @ 2015-11-24 16:27 ` Srinivas Pandruvada 0 siblings, 0 replies; 10+ messages in thread From: Srinivas Pandruvada @ 2015-11-24 16:27 UTC (permalink / raw) 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 17:36 +0100, Rafael J. Wysocki wrote: > On Tuesday, November 24, 2015 07:57:42 AM Srinivas Pandruvada wrote: > > 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 > > > > > <srinivas.pandruvada@linux.intel.com> 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%. > > OK, so that's the intended behavior of "performance" in the HWP case, > right? > Yes. > I'll apply this patch, but I'm going to write a new changelog for it, > then. Sure. Thanks, Srinivas > > Thanks, > Rafael > ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2015-11-24 16:27 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <SKL intel_pstate update MSR values when changing governors>
2015-11-18 1:58 ` [PATCH V2] SKL intel_pstate update MSR values when changing governors Alexandra Yates
2015-11-18 2:15 ` Srinivas Pandruvada
2015-11-18 22:58 ` [PATCH V3] " Alexandra Yates
2015-11-21 0:16 ` Srinivas Pandruvada
2015-11-24 1:09 ` Rafael J. Wysocki
2015-11-24 1:17 ` Srinivas Pandruvada
2015-11-24 14:28 ` Rafael J. Wysocki
2015-11-24 15:57 ` Srinivas Pandruvada
2015-11-24 16:36 ` Rafael J. Wysocki
2015-11-24 16:27 ` Srinivas Pandruvada
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).