Re: [Xen-devel] [PATCH V2] Use #defines instead of magic constants in vmx_vmcs_save()

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 05.08.14 at 14:24, <rcojocaru@xxxxxxxxxxxxxxx> wrote:
> --- a/xen/arch/x86/hvm/vmx/vmx.c
> +++ b/xen/arch/x86/hvm/vmx/vmx.c
> @@ -434,7 +434,8 @@ static void vmx_vmcs_save(struct vcpu *v, struct 
> hvm_hw_cpu *c)
>      c->error_code = 0;
>      __vmread(VM_ENTRY_INTR_INFO, &ev);
>      if ( (ev & INTR_INFO_VALID_MASK) &&
> -         hvm_event_needs_reinjection((ev >> 8) & 7, ev & 0xff) )
> +         hvm_event_needs_reinjection(MASK_EXTR(ev, INTR_INFO_INTR_TYPE_MASK),
> +                                     ev & INTR_INFO_VECTOR_MASK) )
>      {
>          c->pending_event = ev;
>          __vmread(VM_ENTRY_EXCEPTION_ERROR_CODE, &ev);

There is another instance in vmx_idtv_reinject() which you'll also
want to fix.

Jan


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