aboutsummaryrefslogtreecommitdiffstats
path: root/arch
diff options
context:
space:
mode:
authorMatthew Garrett <mjg@redhat.com>2010-07-20 13:52:00 -0400
committerDave Jones <davej@redhat.com>2010-07-26 15:25:34 -0400
commit47f8bcf362410b631a4d99ff5c79ec6b9dd3ace6 (patch)
tree2c6486e767ec0007a01db589b8eec05d90d148a0 /arch
parent6f90388ac98e8cb2c63e307ffb13871a6b87f29b (diff)
[CPUFREQ] pcc driver should check for pcch method before calling _OSC
The pcc specification documents an _OSC method that's incompatible with the one defined as part of the ACPI spec. This shouldn't be a problem as both are supposed to be guarded with a UUID. Unfortunately approximately nobody (including HP, who wrote this spec) properly check the UUID on entry to the _OSC call. Right now this could result in surprising behaviour if the pcc driver performs an _OSC call on a machine that doesn't implement the pcc specification. Check whether the PCCH method exists first in order to reduce this probability. Signed-off-by: Matthew Garrett <mjg@redhat.com> Cc: Naga Chumbalkar <nagananda.chumbalkar@hp.com> Signed-off-by: Dave Jones <davej@redhat.com>
Diffstat (limited to 'arch')
-rw-r--r--arch/x86/kernel/cpu/cpufreq/pcc-cpufreq.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/arch/x86/kernel/cpu/cpufreq/pcc-cpufreq.c b/arch/x86/kernel/cpu/cpufreq/pcc-cpufreq.c
index ce7cde713e71..01bd25c3c7ca 100644
--- a/arch/x86/kernel/cpu/cpufreq/pcc-cpufreq.c
+++ b/arch/x86/kernel/cpu/cpufreq/pcc-cpufreq.c
@@ -397,13 +397,17 @@ static int __init pcc_cpufreq_probe(void)
397 struct pcc_memory_resource *mem_resource; 397 struct pcc_memory_resource *mem_resource;
398 struct pcc_register_resource *reg_resource; 398 struct pcc_register_resource *reg_resource;
399 union acpi_object *out_obj, *member; 399 union acpi_object *out_obj, *member;
400 acpi_handle handle, osc_handle; 400 acpi_handle handle, osc_handle, pcch_handle;
401 int ret = 0; 401 int ret = 0;
402 402
403 status = acpi_get_handle(NULL, "\\_SB", &handle); 403 status = acpi_get_handle(NULL, "\\_SB", &handle);
404 if (ACPI_FAILURE(status)) 404 if (ACPI_FAILURE(status))
405 return -ENODEV; 405 return -ENODEV;
406 406
407 status = acpi_get_handle(handle, "PCCH", &pcch_handle);
408 if (ACPI_FAILURE(status))
409 return -ENODEV;
410
407 status = acpi_get_handle(handle, "_OSC", &osc_handle); 411 status = acpi_get_handle(handle, "_OSC", &osc_handle);
408 if (ACPI_SUCCESS(status)) { 412 if (ACPI_SUCCESS(status)) {
409 ret = pcc_cpufreq_do_osc(&osc_handle); 413 ret = pcc_cpufreq_do_osc(&osc_handle);