From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH 2/2] Register native mce handler as vMCE bounce back point Date: Mon, 16 Apr 2012 16:28:21 -0400 Message-ID: <20120416202821.GC14982@phenom.dumpdata.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: "Liu, Jinsong" Cc: "xen-devel@lists.xensource.com" , "linux-kernel@vger.kernel.org" List-Id: xen-devel@lists.xenproject.org On Mon, Apr 16, 2012 at 01:07:35AM +0000, Liu, Jinsong wrote: > >From 76e40a60878ff72986fd8d92611400195ae0f997 Mon Sep 17 00:00:00 2001 > From: Liu, Jinsong > Date: Mon, 16 Apr 2012 00:16:58 +0800 > Subject: [PATCH 2/2] Register native mce handler as vMCE bounce back point > > When xen hyeprvisor inject vMCE to guest, use native mce handler to handle it hypervisor > > Signed-off-by: Liu, Jinsong > Signed-off-by: Ke, Liping > Signed-off-by: Jiang, Yunhong > Signed-off-by: Jeremy Fitzhardinge > --- > arch/x86/xen/enlighten.c | 10 +++++++--- > 1 files changed, 7 insertions(+), 3 deletions(-) > > diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c > index 15628d4..346ba64 100644 > --- a/arch/x86/xen/enlighten.c > +++ b/arch/x86/xen/enlighten.c > @@ -614,8 +614,8 @@ static int cvt_gate_to_trap(int vector, const gate_desc *val, > /* > * Look for known traps using IST, and substitute them > * appropriately. The debugger ones are the only ones we care > - * about. Xen will handle faults like double_fault and > - * machine_check, so we should never see them. Warn if > + * about. Xen will handle faults like double_fault, > + * so we should never see them. Warn if > * there's an unexpected IST-using fault handler. > */ > if (addr == (unsigned long)debug) > @@ -630,7 +630,11 @@ static int cvt_gate_to_trap(int vector, const gate_desc *val, > return 0; > #ifdef CONFIG_X86_MCE > } else if (addr == (unsigned long)machine_check) { > - return 0; > + /* > + * when xen hyeprvisor inject vMCE to guest, > + * use native mce handler to handle it > + */ > + ; Can you just take the check out? > #endif > } else { > /* Some other trap using IST? */ > -- > 1.7.1