diff options
| author | Jan Beulich <jbeulich@suse.com> | 2018-11-09 11:42:10 +0100 |
|---|---|---|
| committer | Jan Beulich <jbeulich@suse.com> | 2018-11-09 11:42:10 +0100 |
| commit | afe08486589347eb523038abda5ad4a0a0289ce1 (patch) | |
| tree | aa492f1500244aa4daabee206522edddc4588603 /xen/arch/x86/acpi/cpufreq | |
| parent | d78e52da2dff699e4ac0142901adaba41031509c (diff) | |
cpufreq: convert to a single post-init driver (hooks) instance
This reduces the post-init memory footprint, eliminates a pointless
level of indirection at the use sites, and allows for subsequent
alternatives call patching.
Take the opportunity and also add a name to the PowerNow! instance.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Wei Liu <wei.liu2@citrix.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
Diffstat (limited to 'xen/arch/x86/acpi/cpufreq')
| -rw-r--r-- | xen/arch/x86/acpi/cpufreq/cpufreq.c | 10 | ||||
| -rw-r--r-- | xen/arch/x86/acpi/cpufreq/powernow.c | 7 |
2 files changed, 7 insertions, 10 deletions
diff --git a/xen/arch/x86/acpi/cpufreq/cpufreq.c b/xen/arch/x86/acpi/cpufreq/cpufreq.c index 1f8d02aab9..844ab85cd3 100644 --- a/xen/arch/x86/acpi/cpufreq/cpufreq.c +++ b/xen/arch/x86/acpi/cpufreq/cpufreq.c @@ -53,8 +53,6 @@ enum { struct acpi_cpufreq_data *cpufreq_drv_data[NR_CPUS]; -static struct cpufreq_driver acpi_cpufreq_driver; - static bool __read_mostly acpi_pstate_strict; boolean_param("acpi_pstate_strict", acpi_pstate_strict); @@ -355,7 +353,7 @@ static void feature_detect(void *info) if ( cpu_has_aperfmperf ) { policy->aperf_mperf = 1; - acpi_cpufreq_driver.getavg = get_measured_perf; + cpufreq_driver.getavg = get_measured_perf; } eax = cpuid_eax(6); @@ -593,7 +591,7 @@ acpi_cpufreq_cpu_init(struct cpufreq_policy *policy) policy->cur = acpi_cpufreq_guess_freq(data, policy->cpu); break; case ACPI_ADR_SPACE_FIXED_HARDWARE: - acpi_cpufreq_driver.get = get_cur_freq_on_cpu; + cpufreq_driver.get = get_cur_freq_on_cpu; policy->cur = get_cur_freq_on_cpu(cpu); break; default: @@ -635,7 +633,7 @@ static int acpi_cpufreq_cpu_exit(struct cpufreq_policy *policy) return 0; } -static struct cpufreq_driver acpi_cpufreq_driver = { +static const struct cpufreq_driver __initconstrel acpi_cpufreq_driver = { .name = "acpi-cpufreq", .verify = acpi_cpufreq_verify, .target = acpi_cpufreq_target, @@ -656,7 +654,7 @@ static int __init cpufreq_driver_init(void) return ret; } -__initcall(cpufreq_driver_init); +presmp_initcall(cpufreq_driver_init); int cpufreq_cpu_init(unsigned int cpuid) { diff --git a/xen/arch/x86/acpi/cpufreq/powernow.c b/xen/arch/x86/acpi/cpufreq/powernow.c index 8f1ac74f0f..025b37da03 100644 --- a/xen/arch/x86/acpi/cpufreq/powernow.c +++ b/xen/arch/x86/acpi/cpufreq/powernow.c @@ -52,8 +52,6 @@ #define ARCH_CPU_FLAG_RESUME 1 -static struct cpufreq_driver powernow_cpufreq_driver; - static void transition_pstate(void *pstate) { wrmsrl(MSR_PSTATE_CTRL, *(unsigned int *)pstate); @@ -215,7 +213,7 @@ static void feature_detect(void *info) if ( cpu_has_aperfmperf ) { policy->aperf_mperf = 1; - powernow_cpufreq_driver.getavg = get_measured_perf; + cpufreq_driver.getavg = get_measured_perf; } edx = cpuid_edx(CPUID_FREQ_VOLT_CAPABILITIES); @@ -347,7 +345,8 @@ static int powernow_cpufreq_cpu_exit(struct cpufreq_policy *policy) return 0; } -static struct cpufreq_driver powernow_cpufreq_driver = { +static const struct cpufreq_driver __initconstrel powernow_cpufreq_driver = { + .name = "powernow", .verify = powernow_cpufreq_verify, .target = powernow_cpufreq_target, .init = powernow_cpufreq_cpu_init, |