This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


[Xen-devel] [PATCH][PV_OPS_DOM0] Fix domain0 hang when cpuidle is disabl

To: Jeremy Fitzhardinge <jeremy@xxxxxxxx>
Subject: [Xen-devel] [PATCH][PV_OPS_DOM0] Fix domain0 hang when cpuidle is disabled in Xen
From: "Yu, Ke" <ke.yu@xxxxxxxxx>
Date: Sun, 30 Aug 2009 10:23:31 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Sat, 29 Aug 2009 19:24:06 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcopGN3RFZhC0WWJSciPYMfo6jfMuw==
Thread-topic: [PATCH][PV_OPS_DOM0] Fix domain0 hang when cpuidle is disabled in Xen
    Fix the issue of domain0 hang when cpuidle is disabled in Xen
    when cpuidle is disabled in xen, domain0 will use native cpuidle driver to 
enter real C state, which is not allowed and will cause domain0 hang. this 
patch change the condition check to fix this issue.
    Signed-off-by: Yu Ke <ke.yu@xxxxxxxxx>

diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
index c2e24ac..a7308bb 100644
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -1146,7 +1146,7 @@ int acpi_processor_cst_has_changed(struct acpi_processor 
        if (!pr->flags.power_setup_done)
                return -ENODEV;
-    if (processor_cntl_xen_pm()) {
+    if (xen_initial_domain()) {
                        PROCESSOR_PM_CHANGE, PM_TYPE_IDLE);
@@ -1216,7 +1216,7 @@ int __cpuinit acpi_processor_power_init(struct 
acpi_processor *pr,
         * platforms that only support C1.
        if (pr->flags.power) {
-               if (processor_cntl_xen_pm())
+               if (xen_initial_domain())
                                        PROCESSOR_PM_INIT, PM_TYPE_IDLE);
                else {

Attachment: cpuidle_dom0_hang.patch
Description: cpuidle_dom0_hang.patch

Xen-devel mailing list
<Prev in Thread] Current Thread [Next in Thread>