RE: [Xen-devel] [pvops-dom0] Adding MCA logging support in pv_ops

["Ke, Liping" <liping.ke@xxxxxxxxx>]

Hi, Jan

I also noticed that in .31, several important missing field such as apic_id, 
socket id is added. Those field is better to be filled too.
Since now pv_ops is based on .30, those changes haven't happen yet.

So what's your suggestion? Seems we have to keep the old code and make
a patch to it when .31 is supported in pv_ops?

Thanks& Regards,
Criping



Ke, Liping wrote:
> Hi,  Jan& Jeremy
> 
> Thanks for the feedback. This is back port from 2.6.18 old DOM0
> Kernel. 
> 
> And also, I got some feedbacks from Andi Kleen. I will modify all of
> them and resend it tomorrow. 
> 
> Thanks a lot!
> Criping
> 
> Jan Beulich wrote:
>>>>> "Ke, Liping" <liping.ke@xxxxxxxxx> 30.07.09 07:15 >>>
>>> This patch is backport from DOM0 cs902
>> 
>> What base kernel version is this against? I noticed that in .31
>> struct mce doesn't have a res1 field anymore, and hence there's no
>> place to (compatibly) store mc_bank->mc_ctrl2. Do you have any plans
>> on how to address this? Additionally, the new use of the space
>> previously consumed by the res1 field seems to put under question
>> whether that was actually forward compatible (namely, whether it
>> might confuse newer user space tools). 
>> 
>> Thanks, Jan
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel


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