[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 3/3] Nested VMX: Fix IA32_VMX_CR4_FIXED1 msr emulation



Jan Beulich wrote on 2013-09-05:
>>>> On 05.09.13 at 11:05, "Zhang, Yang Z" <yang.z.zhang@xxxxxxxxx> wrote:
>> Jan Beulich wrote on 2013-09-05:
>>>>>> On 05.09.13 at 04:57, Yang Zhang <yang.z.zhang@xxxxxxxxx> wrote:
>>>>      case MSR_IA32_VMX_CR4_FIXED1:
>>>> -        /* allow 0-settings except SMXE */
>>>> -        data = 0x267ff & ~X86_CR4_SMXE;
>>>> +        data |= (edx & cpufeat_mask(X86_FEATURE_VME) ?
>>> 
>>> Did you perhaps send a stale patch? I can't see how this would even
>>> have
>>> compiled: edx is uninitialized at this point afaict.
>> 
>> It is already filled. See the first patch.
> 
> Then the first patch fails to declare all four variables, and hence
> won't build. It just caught my eye that you declare the variables here
> and don't initialize them between declaration and use.

You are right. I adjust the order of patches but forget tweaking the code 
accordingly.

Best regards,
Yang



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.