Re: [Xen-devel] [PATCH 1/2] VMX: also use proper instruction mnemonic for VMREAD

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 16.09.13 at 16:38, "Nakajima, Jun" <jun.nakajima@xxxxxxxxx> wrote:
>> @@ -667,74 +669,78 @@ static void vmx_ctxt_switch_to(struct vc
>>  void vmx_get_segment_register(struct vcpu *v, enum x86_segment seg,
>>                                struct segment_register *reg)
>>  {
>> -    uint32_t attr = 0;
>> +    unsigned long attr = 0, sel = 0, limit;
> 
> This is a minor thing, but why do you initialize 'attr' and 'sel', but
> not 'limit'?

Because limit gets initialized in all the case statements, but attr
and sel don't (for the GDTR and IDTR cases).

Jan


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