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] Add NMI Injection and Pending Support in VMX

To: "Keir Fraser" <keir@xxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] RE: [Patch] Add NMI Injection and Pending Support in VMX
From: "Shan, Haitao" <haitao.shan@xxxxxxxxx>
Date: Wed, 4 Jul 2007 10:42:03 +0800
Cc: "Dong, Eddie" <eddie.dong@xxxxxxxxx>, "Li, Susie" <susie.li@xxxxxxxxx>, "Li, Xin B" <xin.b.li@xxxxxxxxx>, "Cui, Dexuan" <dexuan.cui@xxxxxxxxx>
Delivery-date: Tue, 03 Jul 2007 19:40:00 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C2B04A84.11CCD%keir@xxxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <823A93EED437D048963A3697DB0E35DE6E59D3@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <C2B04A84.11CCD%keir@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcezKHO8GVeq3H5BSPWle/Skc4bxjQACFWLQAGC4rxACMsm0wAAG46WYABEyaXA=
Thread-topic: [Xen-devel] RE: [Patch] Add NMI Injection and Pending Support in VMX
Hi, Keir

Thanks! The patch is much more clean.
But I have a little questions.
1> If we do have IDT vectoring occurs, and at the same time, a NMI or irq is 
pending, should we better do a check to enable_intr_window instead of just 
return?
            if ( (idtv_info_field&INTR_INFO_INTR_TYPE_MASK) == INTR_TYPE_NMI )
                __vmwrite(GUEST_INTERRUPTIBILITY_INFO,
                          __vmread(GUEST_INTERRUPTIBILITY_INFO) &
                          ~VMX_INTR_SHADOW_NMI);
             HVM_DBG_LOG(DBG_LEVEL_1, "idtv_info_field=%x", idtv_info_field);
             return;

Best Regards
Haitao Shan

-----Original Message-----
From: Keir Fraser [mailto:keir@xxxxxxxxxxxxx] 
Sent: 2007年7月4日 1:49
To: Shan, Haitao; xen-devel@xxxxxxxxxxxxxxxxxxx
Cc: Li, Susie; Li, Xin B; Dong, Eddie; Cui, Dexuan
Subject: Re: [Xen-devel] RE: [Patch] Add NMI Injection and Pending Support in 
VMX

Hi Haiato,

Checked in as 15466:e6d5e4709466 (see
http://xenbits.xensource.com/staging/xen-unstable.hg).

But I 'fixed it' in a number of ways. :-) So please take a look and see if
you think I missed anything important. I particularly didn't like your
restructuring inside vmx_intr_assist() so I stripped that out.

 -- Keir

On 3/7/07 15:35, "Shan, Haitao" <haitao.shan@xxxxxxxxx> wrote:

> Hi, Keir,
> 
> I have modified my NMIP patch to follow your approach.
> Please review the attached patch. Thanks!
> 
> Best Regards
> Haitao Shan
> 
> -----Original Message-----
> From: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
> [mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Shan, Haitao
> Sent: 2007年6月22日 17:59
> To: Keir Fraser; xen-devel@xxxxxxxxxxxxxxxxxxx
> Cc: Dong, Eddie; Cui, Dexuan
> Subject: [Xen-devel] RE: [Patch] Add NMI Injection and Pending Support in VMX
> 
> Keir,
> 
> The attached is the synced patch.
> 
> Best Regards
> Haitao Shan
> 
> -----Original Message-----
> From: Keir Fraser [mailto:keir@xxxxxxxxxxxxx]
> Sent: 2007年6月20日 19:48
> To: Shan, Haitao; xen-devel@xxxxxxxxxxxxxxxxxxx
> Cc: Dong, Eddie; Cui, Dexuan
> Subject: Re: [Patch] Add NMI Injection and Pending Support in VMX
> 
> Please sync with 15388:50358c4b37f4, which adds NMI support (but doesn't
> support the new Intel virtual-NMIs feature).
> 
>  -- Keir
> 
> On 20/6/07 11:47, "Shan, Haitao" <haitao.shan@xxxxxxxxx> wrote:
> 
>> Currently, Xen does not support injecting an NMI to HVM guest OS. Adding
>> this 
>> feature is necessary for those softwares which depend on NMI to function
>> correctly,
>> such as KDB and oprofile.
>> 
>> The attached patch allows NMI to be injected to guest OS in NMIP capable
>> platforms.
>> It also enables to queue an NMI and then inject it as soon as possible.
>> 
>> Signed-off-by:      Haitao Shan <haitao.shan@xxxxxxxxx>
>> Signed-off-by:      Eddie Dong   <eddie.dong@xxxxxxxxx>
>> Signed-off-by:      Dexuan Cui    <dexuan.cui@xxxxxxxxx>
>> 
>> Please refer to the latest Intel SDM 3B (published in May, 2007):
>> http://www.intel.com/products/processor/manuals/index.htm
>> 
>> 
>> Best Regards
>> Haitao Shan
>> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel

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