WARNING - OLD ARCHIVES

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/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [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.
> 
i have tried the patch on bare metal (works) and under 32bit
3.4.2-rc1-pre (panic)...i dont get netconsole working, so i have no
output...hope that helps...

> Best Regards
> Ke
> 
>> -----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 
>> please
>> have a try? For the 32bit, I am still trying to reproduce it, will make you 
>> updated
>> later.
>>
>> Best Regards
>> Ke
>>
>> ===============
>>
>> 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?
>> E.g.
>>> 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
>>>
>>> -
>>> daniel
> 


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