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

RE: [Xen-devel] [PATCH] DOM0: Adding MCA Loging support in DOM0


  • To: Jan Beulich <JBeulich@xxxxxxxxxx>, "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
  • From: "Ke, Liping" <liping.ke@xxxxxxxxx>
  • Date: Mon, 15 Jun 2009 15:49:20 +0800
  • Accept-language: en-US
  • Acceptlanguage: en-US
  • Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 15 Jun 2009 00:50:24 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcntiN2ZXVU7dySSQWuDdXlMY65TXQAAdssg
  • Thread-topic: [Xen-devel] [PATCH] DOM0: Adding MCA Loging support in DOM0

Hi, Jan

Thanks a lot! I modified them and resent it.

Regards,
Criping



>> config X86_MCE
>> -    bool "Machine check support" if EMBEDDED
>> -    depends on !X86_64_XEN
>> -    default y
>> +    bool "Machine check support"
>> +    depends on (!XEN_UNPRIVILEGED_GUEST)
>> +    default n
>>      help
>>         Include a machine check error handler to report hardware errors.
>>         This version will require the mcelog utility to decode some
> 
> Once again you change a default value here for no apparent reason.

I originally plan not enable this option by default. But yes, it's fine to 
enable it.

>> +#ifndef CONFIG_X86_64_XEN
>>      mce_cpu_features(c);
>> +#endif
>> }
>> 
>> /*
> 
> This conditional can be avoided if the preceding one would be moved
> inside the function (to cover just the entire function body).

Yes. I will change it.

> 
> g_mi appears to be no longer used outside of
> arch/x86_64/kernel/mce_dom0.c, so it could be (statically) defined
> there instead of here. 
> 

Yes. I need to clean this code after changes.

> Jan

Attachment: intel_mce_dom0.patch
Description: intel_mce_dom0.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®.