[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [PATCH 2/4] ACPI: Backport to get correct throtting info after evaluating _PDC


  • To: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: "Wei, Gang" <gang.wei@xxxxxxxxx>
  • Date: Thu, 26 Feb 2009 21:03:27 +0800
  • Accept-language: en-US
  • Acceptlanguage: en-US
  • Cc: 'Keir Fraser' <keir.fraser@xxxxxxxxxxxxx>
  • Delivery-date: Thu, 26 Feb 2009 05:04:55 -0800
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcmYEp2viwmQa35oQcyR5IlC/jtgYA==
  • Thread-topic: [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@xxxxxxxxx>
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@xxxxxxxxx>
    Signed-off-by: Li Shaohua <shaohua.li@xxxxxxxxx>
    Signed-off-by: Len Brown <len.brown@xxxxxxxxx>

Signed-off-by: Wei Gang <gang.wei@xxxxxxxxx>

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);
 

Attachment: 2-backport-get-t-info-af-pdc.patch
Description: 2-backport-get-t-info-af-pdc.patch

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.