Re: [Xen-devel] [PATCH v2] x86/monitor: add support for descriptor access events

[Razvan Cojocaru <rcojocaru@xxxxxxxxxxxxxxx>]

On 04/04/2017 12:57 PM, Adrian Pop wrote:
> Adds monitor support for descriptor access events (reads & writes of
> IDTR/GDTR/LDTR/TR) for the x86 architecture (VMX and SVM).
> 
> Signed-off-by: Adrian Pop <apop@xxxxxxxxxxxxxxx>
> ---
> changes in v2:
> - use two sub-structs (vmx/svm) for vm_event_desc_access (Tamas K
>   Lengyel)
> - more compact version of the descriptor VM-Exit handling on svm (Boris
>   Ostrovsky)
> - use bool instead of bool_t (Jan Beulich)
> - use curr, currd for the struct vcpu and struct domain local variables
>   (Jan Beulich)
> - move hvm_emulate_ctxt into a narrower scope (Jan Beulich)
> - remove leftover dead code (Jan Beulich)
> - order the monitor events numerically (Jan Beulich)
> - remove VM_EVENT_DESC_INVALID (Jan Beulich)
> - crash the domain if the descriptor access instruction can't be
>   emulated (Jan Beulich)
> - call hvm_inject_event without checking for pending events (Andrew
>   Cooper)
> - introduce structures for the VM-Exit instruction information used for
>   the descriptor instructions and use fewer magic numbers (Andrew
>   Cooper, Jan Beulich)

You've forgotten Wei Liu's ack for the first version of the patch.

For the vm_event bits:

Acked-by: Razvan Cojocaru <rcojocaru@xxxxxxxxxxxxxxx>


Thanks,
Razvan

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