WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] RE: [PATCH] X86 MCE: Add SRAR handler

To: "JBeulich@xxxxxxxx" <JBeulich@xxxxxxxx>, "Liu, Jinsong" <jinsong.liu@xxxxxxxxx>
Subject: [Xen-devel] RE: [PATCH] X86 MCE: Add SRAR handler
From: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Date: Wed, 9 Nov 2011 07:59:42 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, "keir.xen@xxxxxxxxx" <keir.xen@xxxxxxxxx>, "Shan, Haitao" <haitao.shan@xxxxxxxxx>, "tim.deegan@xxxxxxxxxx" <tim.deegan@xxxxxxxxxx>
Delivery-date: Tue, 08 Nov 2011 16:04:29 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4EB1221F020000780005E635@xxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <BC00F5384FCFC9499AF06F92E8B78A9E26A4F3C1DA@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <4EB1221F020000780005E635@xxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcyZRd2RaetyDoBdRh6fnn6oNntfrAFK8yFg
Thread-topic: [PATCH] X86 MCE: Add SRAR handler

> > Concern 1: for SRAR IFU error, since RIPV=EIPV=0, it maybe an async
error
> > which occur at guest but root from hypervisor.
> > [Jinsong]:
> >     Yes, but EIPV didn't tell us where the error root from (it's just a
> > hint, warning us async possibility).
> >     It no need to overkill xen at mce isr, instead, at mce softirq we
can
> > find out error root location and then handle accordingly:
> >     * at mce isr:
> >             /* a total insurance */
> >             /* if error is async, we delay handle it at mce softirq */
> >             if ( !(gstatus & MCG_STATUS_RIPV) && !guest_mode(regs))
> >                 return -1;
> 
> I continue to think that guest_mode() must not be used without
> EIPV, no matter what the purpose of the use.

Jan, as explained before, this is purely a sanity check to make sure we can
go to next level error handling, and the check for guest_mode have nothing
to do with where the error happen, but just make sure that if the context in
the stack is for hypervisor mode, and RIPV told us that we can't switch
context, then we can't do anything more but just panic since it's complexity
to pre-emption/ context switch in hypervisor mode.

Thanks
--jyh

Attachment: smime.p7s
Description: S/MIME cryptographic signature

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>