[Xen-devel] RE: [pvops-dom0]Update MCE LOG data

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

Hi, Jeremy
It's compatible with Andi's future newer mce structure.
And also it should be also compatible with old mcelog version, I noticed mcgcap 
is put at the tail of new mce structure in kernel. 
It should have same behavior as kernel.

Re-Paste the patch for adding sign off info.

Thanks& Regards,
criping

-----Original Message-----
From: Jeremy Fitzhardinge [mailto:jeremy@xxxxxxxx] 
Sent: Tuesday, April 20, 2010 3:54 AM
To: Ke, Liping
Cc: Jiang, Yunhong; xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [pvops-dom0]Update MCE LOG data

On 04/19/2010 01:55 AM, Ke, Liping wrote:
> Hi, Jeremy and all
> This small patch is to pass mcg_cap information from XEN to DOM0 since now 
> new pv-ops kernel's mce data structure is changed and need this mcg_cap 
> information.
>   

Is this forwards and backwards compatible?

    J

> Thanks& Regards,
> Criping
>
>
>
>
> From 475b637e8dabb32446ee28e6b1235729e4b0f4bb Mon Sep 17 00:00:00 2001
> From: Liping Ke <liping.ke@xxxxxxxxx>
> Date: Mon, 19 Apr 2010 16:42:48 +0800
> Subject: [PATCH] Update MCE LOG data
> More mce data is provided in new DOM0 kernel, we need to pass
> mcg_cap information from XEN to DOM0 in vIRQ accordingly.
>
> ---
>  drivers/xen/mce.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/xen/mce.c b/drivers/xen/mce.c
> index b354dc8..8758e75 100644
> --- a/drivers/xen/mce.c
> +++ b/drivers/xen/mce.c
> @@ -72,6 +72,7 @@ static int convert_log(struct mc_info *mi)
>       m.socketid = g_physinfo[i].mc_chipid;
>       m.cpu = m.extcpu = g_physinfo[i].mc_cpunr;
>       m.cpuvendor = (__u8)g_physinfo[i].mc_vendor;
> +     m.mcgcap = g_physinfo[i].mc_msrvalues[0].value;
>       x86_mcinfo_lookup(&mic, mi, MC_TYPE_BANK);
>       do {
>               if (mic == NULL || mic->size == 0)
> @@ -83,6 +84,7 @@ static int convert_log(struct mc_info *mi)
>                       m.addr = mc_bank->mc_addr;
>                       m.tsc = mc_bank->mc_tsc;
>                       m.bank = mc_bank->mc_bank;
> +                     m.finished = 1;
>                       /*log this record*/
>                       mce_log(&m);
>               }
>

Attachment: 0001-Update-MCE-LOG-data.patch
Description: 0001-Update-MCE-LOG-data.patch

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