Wei, Gang
2009-Feb-26 13:03 UTC
[Xen-devel] [PATCH 2/4] ACPI: Backport to get correct throtting info after evaluating _PDC
ACPI: Backport to get correct throtting info after evaluating _PDC
commit 0ac3c571315a53c14d2733564f14ebdb911fe903
Author: Zhao Yakui <yakui.zhao@intel.com>
Date: Thu Nov 15 17:05:46 2007 +0800
ACPI: Get throttling info from BIOS only after evaluating _PDC
Previously _PDC was evaluated later, and thus we''d not get
the chance to tell the BIOS that we can suport FixedHW registers (MSRs)
and the BIOS would always ask us to use System I/O access
for throttling.
Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
Signed-off-by: Li Shaohua <shaohua.li@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Signed-off-by: Wei Gang <gang.wei@intel.com>
diff -r 179c0c9b98bc drivers/acpi/processor_core.c
--- a/drivers/acpi/processor_core.c Tue Feb 24 19:45:27 2009 +0800
+++ b/drivers/acpi/processor_core.c Tue Feb 24 19:54:08 2009 +0800
@@ -510,21 +510,6 @@ static int acpi_processor_get_info(struc
request_region(pr->throttling.address, 6, "ACPI CPU throttle");
}
-#if defined(CONFIG_CPU_FREQ) || defined(CONFIG_PROCESSOR_EXTERNAL_CONTROL)
- acpi_processor_ppc_has_changed(pr);
-#endif
-
- /*
- * pr->id may equal to -1 while processor_cntl_external enabled.
- * throttle and thermal module don''t support this case.
- * Tx only works when dom0 vcpu == pcpu num by far, as we give
- * control to dom0.
- */
- if (pr->id != -1) {
- acpi_processor_get_throttling_info(pr);
- acpi_processor_get_limit_info(pr);
- }
-
return 0;
}
@@ -587,6 +572,21 @@ static int acpi_processor_start(struct a
/* _PDC call should be done before doing anything else (if reqd.). */
arch_acpi_processor_init_pdc(pr);
acpi_processor_set_pdc(pr);
+
+#if defined(CONFIG_CPU_FREQ) || defined(CONFIG_PROCESSOR_EXTERNAL_CONTROL)
+ acpi_processor_ppc_has_changed(pr);
+#endif
+
+ /*
+ * pr->id may equal to -1 while processor_cntl_external enabled.
+ * throttle and thermal module don''t support this case.
+ * Tx only works when dom0 vcpu == pcpu num by far, as we give
+ * control to dom0.
+ */
+ if (pr->id != -1) {
+ acpi_processor_get_throttling_info(pr);
+ acpi_processor_get_limit_info(pr);
+ }
acpi_processor_power_init(pr, device);
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel