From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Fitzhardinge Subject: Re: [pvops-dom0]Update MCE LOG data Date: Mon, 19 Apr 2010 12:53:36 -0700 Message-ID: <4BCCB4C0.4040601@goop.org> References: <789F9655DD1B8F43B48D77C5D30659731D797FC3@shsmsx501.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <789F9655DD1B8F43B48D77C5D30659731D797FC3@shsmsx501.ccr.corp.intel.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: "Ke, Liping" Cc: "Jiang, Yunhong" , "xen-devel@lists.xensource.com" List-Id: xen-devel@lists.xenproject.org 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 > 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); > } >