leandrof/linux
1
0
Fork 0
forked from Minki/linux
Code Issues Pull Requests Packages Projects Releases Wiki Activity

cpufreq: intel_pstate: Free memory only when turning off

Browse Source 
When intel_pstate switches the operation mode from "active" to
"passive" or the other way around, freeing its data structures
representing CPUs and allocating them again from scratch is not
necessary and wasteful.  Moreover, if these data structures are
preserved, the cached HWP Request MSR value from there may be
written to the MSR to start with to reinitialize it and help to
restore the EPP value set previously (it is set to 0xFF when CPUs
go offline to allow their SMT siblings to use the full range of
EPP values and that also happens when the driver gets unregistered).

Accordingly, modify the driver to only do a full cleanup on driver
object registration errors and when its status is changed to "off"
via sysfs and to write the cached HWP Request MSR value back to
the MSR on CPU init if the data structure representing the given
CPU is still there.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
This commit is contained in:
Rafael J. Wysocki 2020-09-01 18:33:41 +02:00
parent 4adcf2e582
commit 55671ea325
1 changed files with 26 additions and 35 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

61
drivers/cpufreq/intel_pstate.c
View File

@ -2116,24 +2116,30 @@ static int intel_pstate_init_cpu(unsigned int cpunum)
all_cpu_data[cpunum] = cpu; all_cpu_data[cpunum] = cpu;
cpu->cpu = cpunum;
cpu->epp_default = -EINVAL; cpu->epp_default = -EINVAL;
cpu->epp_powersave = -EINVAL;
if (hwp_active) {
const struct x86_cpu_id *id;
intel_pstate_hwp_enable(cpu);
id = x86_match_cpu(intel_pstate_hwp_boost_ids);
if (id && intel_pstate_acpi_pm_profile_server())
hwp_boost = true;
}
} else if (hwp_active) {
/*
* Re-enable HWP in case this happens after a resume from ACPI
* S3 if the CPU was offline during the whole system/resume
* cycle.
*/
intel_pstate_hwp_reenable(cpu);
} }
cpu = all_cpu_data[cpunum]; cpu->epp_powersave = -EINVAL;
cpu->epp_policy = 0;
cpu->cpu = cpunum;
if (hwp_active) {
const struct x86_cpu_id *id;
intel_pstate_hwp_enable(cpu);
cpu->epp_policy = 0;
id = x86_match_cpu(intel_pstate_hwp_boost_ids);
if (id && intel_pstate_acpi_pm_profile_server())
hwp_boost = true;
}
intel_pstate_get_cpu_pstates(cpu); intel_pstate_get_cpu_pstates(cpu);
@ -2730,9 +2736,6 @@ static void intel_pstate_driver_cleanup(void)
} }
put_online_cpus(); put_online_cpus();
if (intel_pstate_driver == &intel_pstate)
intel_pstate_sysfs_hide_hwp_dynamic_boost();
intel_pstate_driver = NULL; intel_pstate_driver = NULL;
} }
@ -2758,14 +2761,6 @@ static int intel_pstate_register_driver(struct cpufreq_driver *driver)
return 0; return 0;
} }
static int intel_pstate_unregister_driver(void)
{
cpufreq_unregister_driver(intel_pstate_driver);
intel_pstate_driver_cleanup();
return 0;
}
static ssize_t intel_pstate_show_status(char *buf) static ssize_t intel_pstate_show_status(char *buf)
{ {
if (!intel_pstate_driver) if (!intel_pstate_driver)
@ -2777,8 +2772,6 @@ static ssize_t intel_pstate_show_status(char *buf)
static int intel_pstate_update_status(const char *buf, size_t size) static int intel_pstate_update_status(const char *buf, size_t size)
{ {
int ret;
if (size == 3 && !strncmp(buf, "off", size)) { if (size == 3 && !strncmp(buf, "off", size)) {
if (!intel_pstate_driver) if (!intel_pstate_driver)
return -EINVAL; return -EINVAL;
@ -2786,7 +2779,8 @@ static int intel_pstate_update_status(const char *buf, size_t size)
if (hwp_active) if (hwp_active)
return -EBUSY; return -EBUSY;
return intel_pstate_unregister_driver(); cpufreq_unregister_driver(intel_pstate_driver);
intel_pstate_driver_cleanup();
} }
if (size == 6 && !strncmp(buf, "active", size)) { if (size == 6 && !strncmp(buf, "active", size)) {
@ -2794,9 +2788,7 @@ static int intel_pstate_update_status(const char *buf, size_t size)
if (intel_pstate_driver == &intel_pstate) if (intel_pstate_driver == &intel_pstate)
return 0; return 0;
ret = intel_pstate_unregister_driver(); cpufreq_unregister_driver(intel_pstate_driver);
if (ret)
return ret;
} }
return intel_pstate_register_driver(&intel_pstate); return intel_pstate_register_driver(&intel_pstate);
@ -2807,9 +2799,8 @@ static int intel_pstate_update_status(const char *buf, size_t size)
if (intel_pstate_driver == &intel_cpufreq) if (intel_pstate_driver == &intel_cpufreq)
return 0; return 0;
ret = intel_pstate_unregister_driver(); cpufreq_unregister_driver(intel_pstate_driver);
if (ret) intel_pstate_sysfs_hide_hwp_dynamic_boost();
return ret;
} }
return intel_pstate_register_driver(&intel_cpufreq); return intel_pstate_register_driver(&intel_cpufreq);