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

Re: [Xen-devel] RE: [PATCH] Don't enable irq for machine check vmexit

To: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>, Tim Deegan <Tim.Deegan@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] RE: [PATCH] Don't enable irq for machine check vmexit
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Fri, 05 Feb 2010 14:59:05 +0000
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 05 Feb 2010 07:01:18 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C8EDE645B81E5141A8C6B2F73FD9265118FED93B52@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcqlpWnLbYBV1hx/SiKbBgRcbuXrewApwMQgAAV2gqIAAuP2YAABevi3
Thread-topic: [Xen-devel] RE: [PATCH] Don't enable irq for machine check vmexit
User-agent: Microsoft-Entourage/12.23.0.091001
On 05/02/2010 14:36, "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx> wrote:

>> How about the attached alternative, which avoids repeated reads of
>> VM_INTR_INFO? Also I'm not sure whether checking for
>> VM_EXIT_REASONS_FAILED_VMENTRY is useful, so I removed it. After all,
>> EXIT_REASON_MCE_DURING_VMENTRY should imply it anyway.
> 
> Thanks for your patch. Yes, it is much better to avoid the repeated read.
> 
> I'm not sure if it is ok if we don't check the VM_EXIT_REASONS_FAILED_VMENTRY.
> Checking the SDM and seems it is ok. In fact, I didn't find effective method
> to test this VMEntry MCE failed case, althgouh I can test MCE VMExit with
> EXIT_REASON_EXCEPTION_NMI case easily. (I will try to find a method to test
> this VMEntry failure case next week, maybe poison the VMCS range can trigger
> it, I'm not sure).

Well, we don't seem to know what bit 31 is for. Or, at least, we don't know
how it should affect our behaviour in the vmexit handler. So looking at it
does seem a bit pointless.

 -- Keir



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