Re: [Xen-devel] [PATCH 5/5] X86: enable MPX feature to HVM guest

["Liu, Jinsong" <jinsong.liu@xxxxxxxxx>]

Jan Beulich wrote:
>>>> On 19.11.13 at 11:54, "Liu, Jinsong" <jinsong.liu@xxxxxxxxx> wrote:
>> From 9d0248fddf86c96efcb16963f0842927cb5c23c0 Mon Sep 17 00:00:00
>> 2001 From: Liu Jinsong <jinsong.liu@xxxxxxxxx>
>> Date: Tue, 19 Nov 2013 18:48:20 +0800
>> Subject: [PATCH 5/5] X86: enable MPX feature to HVM guest
>> 
>> Signed-off-by: Xudong Hao <xudong.hao@xxxxxxxxx>
>> Reviewed-by: Liu Jinsong <jinsong.liu@xxxxxxxxx>
>> ---
>>  xen/arch/x86/xstate.c |    1 +
>>  1 files changed, 1 insertions(+), 0 deletions(-)
>> 
>> diff --git a/xen/arch/x86/xstate.c b/xen/arch/x86/xstate.c
>> index 1fd43c9..2bc9b60 100644
>> --- a/xen/arch/x86/xstate.c
>> +++ b/xen/arch/x86/xstate.c
>> @@ -373,6 +373,7 @@ int handle_xsetbv(u32 index, u64 new_bv)
>>      mask = new_bv & ~curr->arch.xcr0_accum;
>>      curr->arch.xcr0 = new_bv;
>>      curr->arch.xcr0_accum |= new_bv;
>> +    curr->arch.nonlazy_xstate_used = !!(curr->arch.xcr0 &
>> XSTATE_NONLAZY); 
> 
> See v2 of patch 2. The patch is wrong this way anyway, and
> was be mis-ordered (it's a prereq to patch 4).
> 

Yes, this patch in fact could be dropped per patch 2 V2.

Thanks Jan and Andrew!
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel