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


RE: [Xen-devel] pvops dom0 20090825 xen_acpi panic

To: Daniel Schroeder <sec@xxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] pvops dom0 20090825 xen_acpi panic
From: "Yu, Ke" <ke.yu@xxxxxxxxx>
Date: Fri, 28 Aug 2009 16:45:59 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: Jeremy Fitzhardinge <jeremy@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 28 Aug 2009 01:46:40 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4D05DB80B95B23498C72C700BD6C2E0B339DBE9C@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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>
References: <4A942449.8020707@xxxxxxxxxxxxxxx> <4D05DB80B95B23498C72C700BD6C2E0B339DB926@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <4A96DE1C.8000003@xxxxxxxxxxxxxxx> <4D05DB80B95B23498C72C700BD6C2E0B339DBE9C@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AconTG+6ufVML44ySL6NA2OKAkyPIgAND2BwAA6h0mA=
Thread-topic: [Xen-devel] pvops dom0 20090825 xen_acpi panic
I have tried 32bit case, however I cannot reproduce the kernel panic, either in 
32bit dom0/32bit hypervisor case or 32bit dom0/64bit hypervisor case. if you 
still meet the kernel panic after applying the patch I posted, please let me 
know. I will dig it more.

Best Regards

>-----Original Message-----
>From: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
>[mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Yu, Ke
>Sent: Friday, August 28, 2009 11:06 AM
>To: Daniel Schroeder
>Cc: Jeremy Fitzhardinge; xen-devel@xxxxxxxxxxxxxxxxxxx
>Subject: RE: [Xen-devel] pvops dom0 20090825 xen_acpi panic
>Oh I see. The following patch should fix the panic in bare metal, could you 
>have a try? For the 32bit, I am still trying to reproduce it, will make you 
>Best Regards
>Fix xen_acpi_processor_extcntl_init xen_start_info NULL pointer issue
>when kernel runs in bare metal, the xen_start_info could be a NULL pointer,
>so add condition check to avoid the kernel panic.
>Signed-off-by: Yu Ke <ke.yu@xxxxxxxxx>
>diff --git a/drivers/xen/acpi_processor.c b/drivers/xen/acpi_processor.c
>index d2b76e9..6a4e8e4 100644
>--- a/drivers/xen/acpi_processor.c
>+++ b/drivers/xen/acpi_processor.c
>@@ -430,10 +430,14 @@ static int xen_hotplug_notifier(struct acpi_processor
>*pr, int event)
> static int __init xen_acpi_processor_extcntl_init(void)
> {
>-      unsigned int pmbits = (xen_start_info->flags & SIF_PM_MASK) >> 8;
>+      unsigned int pmbits;
>-      if (!pmbits)
>+      /* Only xen dom0 is allowed to handle ACPI processor info */
>+      if (!xen_initial_domain())
>               return 0;
>+      pmbits = (xen_start_info->flags & SIF_PM_MASK) >> 8;
>       if (pmbits & XEN_PROCESSOR_PM_CX)
>               xen_ops.pm_ops[PM_TYPE_IDLE] = xen_cx_notifier;
>       if (pmbits & XEN_PROCESSOR_PM_PX)
>>-----Original Message-----
>>From: Daniel Schroeder [mailto:sec@xxxxxxxxxxxxxxx]
>>Sent: Friday, August 28, 2009 3:27 AM
>>To: Yu, Ke
>>Cc: Jeremy Fitzhardinge; xen-devel@xxxxxxxxxxxxxxxxxxx
>>Subject: Re: [Xen-devel] pvops dom0 20090825 xen_acpi panic
>>Hi Ke,
>>> Could you share more info on your configuration, so that I can reproduce it?
>>is it 32bit dom0 under 64bit hypervisor? Also do you mind attach the
>>domi0 .config file?
>>this is under a 32bit dom0 but it even happens on bare metal...under kvm
>>  the same panic occurs...
>>.config is here http://x17.eu/xen_acpi_panic.config.gz

Xen-devel mailing list