* [PATCH] cpufreq: intel_pstate: Set EPP/EPB to 0 in performance mode
@ 2016-11-18 3:15 Srinivas Pandruvada
2016-11-18 23:26 ` Rafael J. Wysocki
0 siblings, 1 reply; 4+ messages in thread
From: Srinivas Pandruvada @ 2016-11-18 3:15 UTC (permalink / raw)
To: lenb, rjw; +Cc: linux-pm, Srinivas Pandruvada
When user has selected performance policy, then set the EPP (Energy
Performance Preference) or EPB (Energy Performance Bias) to maximum
performance mode.
Also when user switch back to powersave, then restore EPP/EPB to power on
default value.
It is also possible to change EPP/EPB using direct MSR writes or using
x86_energy_perf_policy utility. When user changes the default EPP/EPB
value using the above methods in powersave policy, then this change
preserves those settings, whenever intel_pstate_hwp_set() is called.
Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
---
drivers/cpufreq/intel_pstate.c | 104 +++++++++++++++++++++++++++++++++++++++++
1 file changed, 104 insertions(+)
diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
index 4737520..2dade36 100644
--- a/drivers/cpufreq/intel_pstate.c
+++ b/drivers/cpufreq/intel_pstate.c
@@ -196,6 +196,8 @@ struct _pid {
* @sample: Storage for storing last Sample data
* @acpi_perf_data: Stores ACPI perf information read from _PSS
* @valid_pss_table: Set to true for valid ACPI _PSS entries found
+ * @epp_default: Power on default HWP energy performance preference
+ * or energy performance bias
*
* This structure stores per CPU instance data for all CPUs.
*/
@@ -222,6 +224,7 @@ struct cpudata {
bool valid_pss_table;
#endif
unsigned int iowait_boost;
+ int epp_default;
};
static struct cpudata **all_cpu_data;
@@ -559,12 +562,76 @@ static inline void update_turbo_state(void)
cpu->pstate.max_pstate == cpu->pstate.turbo_pstate);
}
+static int intel_pstate_get_epb(struct cpudata *cpu_data)
+{
+ u64 epb;
+ int ret;
+
+ if (!static_cpu_has(X86_FEATURE_EPB))
+ return -ENXIO;
+
+ ret = rdmsrl_on_cpu(cpu_data->cpu, MSR_IA32_ENERGY_PERF_BIAS, &epb);
+ if (ret)
+ return ret;
+
+ return (int)(epb & 0x0f);
+}
+
+static int intel_pstate_get_epp(struct cpudata *cpu_data, u64 hwp_req_data)
+{
+ int epp;
+
+ if (static_cpu_has(X86_FEATURE_HWP_EPP)) {
+ if (!hwp_req_data) {
+ int ret;
+
+ ret = rdmsrl_on_cpu(cpu_data->cpu,
+ MSR_HWP_REQUEST,
+ &hwp_req_data);
+ if (ret)
+ return ret;
+ }
+ epp = (hwp_req_data >> 24) & 0xff;
+ } else {
+ /* When there is no EPP present, HWP uses EPB settings */
+ epp = intel_pstate_get_epb(cpu_data);
+ }
+
+ return epp;
+}
+
+static void intel_pstate_update_epp(struct cpudata *cpu_data)
+{
+ int epp;
+
+ epp = intel_pstate_get_epp(cpu_data, 0);
+
+ if (!cpu_data->epp_default)
+ cpu_data->epp_default = epp;
+}
+
+static void intel_pstate_set_epb(int cpu, int pref)
+{
+ u64 epb;
+
+ if (!static_cpu_has(X86_FEATURE_EPB))
+ return;
+
+ if (rdmsrl_on_cpu(cpu, MSR_IA32_ENERGY_PERF_BIAS, &epb))
+ return;
+
+ epb = (epb & ~0x0f) | pref;
+ wrmsrl_on_cpu(cpu, MSR_IA32_ENERGY_PERF_BIAS, epb);
+}
+
static void intel_pstate_hwp_set(const struct cpumask *cpumask)
{
int min, hw_min, max, hw_max, cpu, range, adj_range;
u64 value, cap;
for_each_cpu(cpu, cpumask) {
+ struct cpudata *cpu_data = all_cpu_data[cpu];
+
rdmsrl_on_cpu(cpu, MSR_HWP_CAPABILITIES, &cap);
hw_min = HWP_LOWEST_PERF(cap);
hw_max = HWP_HIGHEST_PERF(cap);
@@ -586,6 +653,42 @@ static void intel_pstate_hwp_set(const struct cpumask *cpumask)
value &= ~HWP_MAX_PERF(~0L);
value |= HWP_MAX_PERF(max);
+ /*
+ * If there was error reading during epp/epb, then don't
+ * set EPP/EPB values. In this case
+ * cpu_data->epp_default will be set to -errno
+ */
+ if (cpu_data->epp_default >= 0) {
+ int epp;
+
+ if (cpu_data->policy == CPUFREQ_POLICY_PERFORMANCE) {
+ epp = 0;
+ } else {
+ /*
+ * If epp was non zero after policy switch to
+ * powersave policy or already in powersave
+ * policy, then EPP was never changed from
+ * power on value or user manually changed via
+ * some utilities. In this case don't set
+ * EPP/EPB in this case.
+ * Otherwise restore power on default EPP/EPB
+ * after policy switch from performance.
+ */
+ epp = intel_pstate_get_epp(cpu_data, value);
+ if (epp)
+ goto skip_epp;
+ else
+ epp = cpu_data->epp_default;
+ }
+
+ if (static_cpu_has(X86_FEATURE_HWP_EPP)) {
+ value &= ~GENMASK_ULL(31, 24);
+ value |= (u64)epp << 24;
+ } else {
+ intel_pstate_set_epb(cpu, epp);
+ }
+ }
+skip_epp:
wrmsrl_on_cpu(cpu, MSR_HWP_REQUEST, value);
}
}
@@ -818,6 +921,7 @@ static void intel_pstate_hwp_enable(struct cpudata *cpudata)
wrmsrl_on_cpu(cpudata->cpu, MSR_HWP_INTERRUPT, 0x00);
wrmsrl_on_cpu(cpudata->cpu, MSR_PM_ENABLE, 0x1);
+ intel_pstate_update_epp(cpudata);
}
static int atom_get_min_pstate(void)
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] cpufreq: intel_pstate: Set EPP/EPB to 0 in performance mode
2016-11-18 3:15 [PATCH] cpufreq: intel_pstate: Set EPP/EPB to 0 in performance mode Srinivas Pandruvada
@ 2016-11-18 23:26 ` Rafael J. Wysocki
2016-11-21 17:27 ` Srinivas Pandruvada
0 siblings, 1 reply; 4+ messages in thread
From: Rafael J. Wysocki @ 2016-11-18 23:26 UTC (permalink / raw)
To: Srinivas Pandruvada; +Cc: Len Brown, Rafael J. Wysocki, Linux PM
On Fri, Nov 18, 2016 at 4:15 AM, Srinivas Pandruvada
<srinivas.pandruvada@linux.intel.com> wrote:
> When user has selected performance policy, then set the EPP (Energy
> Performance Preference) or EPB (Energy Performance Bias) to maximum
> performance mode.
> Also when user switch back to powersave, then restore EPP/EPB to power on
> default value.
> It is also possible to change EPP/EPB using direct MSR writes or using
> x86_energy_perf_policy utility. When user changes the default EPP/EPB
> value using the above methods in powersave policy, then this change
> preserves those settings, whenever intel_pstate_hwp_set() is called.
>
> Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> ---
> drivers/cpufreq/intel_pstate.c | 104 +++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 104 insertions(+)
>
> diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
> index 4737520..2dade36 100644
> --- a/drivers/cpufreq/intel_pstate.c
> +++ b/drivers/cpufreq/intel_pstate.c
> @@ -196,6 +196,8 @@ struct _pid {
> * @sample: Storage for storing last Sample data
> * @acpi_perf_data: Stores ACPI perf information read from _PSS
> * @valid_pss_table: Set to true for valid ACPI _PSS entries found
> + * @epp_default: Power on default HWP energy performance preference
> + * or energy performance bias
> *
> * This structure stores per CPU instance data for all CPUs.
> */
> @@ -222,6 +224,7 @@ struct cpudata {
> bool valid_pss_table;
> #endif
> unsigned int iowait_boost;
> + int epp_default;
> };
>
> static struct cpudata **all_cpu_data;
> @@ -559,12 +562,76 @@ static inline void update_turbo_state(void)
> cpu->pstate.max_pstate == cpu->pstate.turbo_pstate);
> }
>
> +static int intel_pstate_get_epb(struct cpudata *cpu_data)
> +{
> + u64 epb;
> + int ret;
> +
> + if (!static_cpu_has(X86_FEATURE_EPB))
> + return -ENXIO;
> +
> + ret = rdmsrl_on_cpu(cpu_data->cpu, MSR_IA32_ENERGY_PERF_BIAS, &epb);
> + if (ret)
> + return ret;
> +
> + return (int)(epb & 0x0f);
> +}
> +
> +static int intel_pstate_get_epp(struct cpudata *cpu_data, u64 hwp_req_data)
> +{
> + int epp;
> +
> + if (static_cpu_has(X86_FEATURE_HWP_EPP)) {
> + if (!hwp_req_data) {
> + int ret;
> +
> + ret = rdmsrl_on_cpu(cpu_data->cpu,
> + MSR_HWP_REQUEST,
> + &hwp_req_data);
> + if (ret)
> + return ret;
> + }
> + epp = (hwp_req_data >> 24) & 0xff;
> + } else {
> + /* When there is no EPP present, HWP uses EPB settings */
> + epp = intel_pstate_get_epb(cpu_data);
> + }
> +
> + return epp;
> +}
> +
> +static void intel_pstate_update_epp(struct cpudata *cpu_data)
> +{
> + int epp;
> +
> + epp = intel_pstate_get_epp(cpu_data, 0);
> +
> + if (!cpu_data->epp_default)
> + cpu_data->epp_default = epp;
> +}
> +
> +static void intel_pstate_set_epb(int cpu, int pref)
> +{
> + u64 epb;
> +
> + if (!static_cpu_has(X86_FEATURE_EPB))
> + return;
> +
> + if (rdmsrl_on_cpu(cpu, MSR_IA32_ENERGY_PERF_BIAS, &epb))
> + return;
> +
> + epb = (epb & ~0x0f) | pref;
> + wrmsrl_on_cpu(cpu, MSR_IA32_ENERGY_PERF_BIAS, epb);
> +}
> +
> static void intel_pstate_hwp_set(const struct cpumask *cpumask)
> {
> int min, hw_min, max, hw_max, cpu, range, adj_range;
> u64 value, cap;
>
> for_each_cpu(cpu, cpumask) {
> + struct cpudata *cpu_data = all_cpu_data[cpu];
> +
> rdmsrl_on_cpu(cpu, MSR_HWP_CAPABILITIES, &cap);
> hw_min = HWP_LOWEST_PERF(cap);
> hw_max = HWP_HIGHEST_PERF(cap);
> @@ -586,6 +653,42 @@ static void intel_pstate_hwp_set(const struct cpumask *cpumask)
>
> value &= ~HWP_MAX_PERF(~0L);
> value |= HWP_MAX_PERF(max);
> + /*
> + * If there was error reading during epp/epb, then don't
> + * set EPP/EPB values. In this case
> + * cpu_data->epp_default will be set to -errno
> + */
> + if (cpu_data->epp_default >= 0) {
> + int epp;
> +
> + if (cpu_data->policy == CPUFREQ_POLICY_PERFORMANCE) {
Why don't we simply save the EPP/EPB here, ->
> + epp = 0;
> + } else {
> + /*
> + * If epp was non zero after policy switch to
> + * powersave policy or already in powersave
> + * policy, then EPP was never changed from
> + * power on value or user manually changed via
> + * some utilities. In this case don't set
> + * EPP/EPB in this case.
-> and restore it here unless the current value is nonzero (in which
case we will discard the saved value)?
This way we would avoid some ugly scenarios in which users might not
get what they expected (if I'm not mistaken).
> + * Otherwise restore power on default EPP/EPB
> + * after policy switch from performance.
> + */
> + epp = intel_pstate_get_epp(cpu_data, value);
> + if (epp)
> + goto skip_epp;
> + else
> + epp = cpu_data->epp_default;
> + }
> +
> + if (static_cpu_has(X86_FEATURE_HWP_EPP)) {
> + value &= ~GENMASK_ULL(31, 24);
> + value |= (u64)epp << 24;
> + } else {
> + intel_pstate_set_epb(cpu, epp);
> + }
> + }
> +skip_epp:
> wrmsrl_on_cpu(cpu, MSR_HWP_REQUEST, value);
> }
> }
> @@ -818,6 +921,7 @@ static void intel_pstate_hwp_enable(struct cpudata *cpudata)
> wrmsrl_on_cpu(cpudata->cpu, MSR_HWP_INTERRUPT, 0x00);
>
> wrmsrl_on_cpu(cpudata->cpu, MSR_PM_ENABLE, 0x1);
> + intel_pstate_update_epp(cpudata);
> }
>
> static int atom_get_min_pstate(void)
> --
Thanks,
Rafael
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] cpufreq: intel_pstate: Set EPP/EPB to 0 in performance mode
2016-11-18 23:26 ` Rafael J. Wysocki
@ 2016-11-21 17:27 ` Srinivas Pandruvada
2016-11-21 20:57 ` Rafael J. Wysocki
0 siblings, 1 reply; 4+ messages in thread
From: Srinivas Pandruvada @ 2016-11-21 17:27 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: Len Brown, Rafael J. Wysocki, Linux PM
On Sat, 2016-11-19 at 00:26 +0100, Rafael J. Wysocki wrote:
> On Fri, Nov 18, 2016 at 4:15 AM, Srinivas Pandruvada
> <srinivas.pandruvada@linux.intel.com> wrote:
[...]
> > + if (cpu_data->policy ==
> > CPUFREQ_POLICY_PERFORMANCE) {
>
> Why don't we simply save the EPP/EPB here, ->
We can, but we loose the ability to restore to the default power on
configuration. For other control limits, users have ability to look at
the sysfs and set the limits back to power on default. Since we don't
have any other way to look at go back to default EPP, once save and
restore. So if this behavior is not important, then this can be done.
Thanks,
Srinivas
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] cpufreq: intel_pstate: Set EPP/EPB to 0 in performance mode
2016-11-21 17:27 ` Srinivas Pandruvada
@ 2016-11-21 20:57 ` Rafael J. Wysocki
0 siblings, 0 replies; 4+ messages in thread
From: Rafael J. Wysocki @ 2016-11-21 20:57 UTC (permalink / raw)
To: Srinivas Pandruvada
Cc: Rafael J. Wysocki, Len Brown, Rafael J. Wysocki, Linux PM
On Mon, Nov 21, 2016 at 6:27 PM, Srinivas Pandruvada
<srinivas.pandruvada@linux.intel.com> wrote:
> On Sat, 2016-11-19 at 00:26 +0100, Rafael J. Wysocki wrote:
>> On Fri, Nov 18, 2016 at 4:15 AM, Srinivas Pandruvada
>> <srinivas.pandruvada@linux.intel.com> wrote:
>
> [...]
>
>> > + if (cpu_data->policy ==
>> > CPUFREQ_POLICY_PERFORMANCE) {
>>
>> Why don't we simply save the EPP/EPB here, ->
>
> We can, but we loose the ability to restore to the default power on
> configuration. For other control limits, users have ability to look at
> the sysfs and set the limits back to power on default. Since we don't
> have any other way to look at go back to default EPP, once save and
> restore. So if this behavior is not important, then this can be done.
I don't think we need to provide a way to go back to the power-on
default (no matter what the used did to the EPP in the meantime) in
this patch.
If it does what I suggested, it actually will go back to the power-on
default when switching the policy to "powersave" unless some changes
have been made to the setting while using the "performance" policy.
Thanks,
Rafael
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-11-21 21:09 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-18 3:15 [PATCH] cpufreq: intel_pstate: Set EPP/EPB to 0 in performance mode Srinivas Pandruvada
2016-11-18 23:26 ` Rafael J. Wysocki
2016-11-21 17:27 ` Srinivas Pandruvada
2016-11-21 20:57 ` Rafael J. Wysocki
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).