>>> On 03.06.16 at 00:52, <ta...@tklengyel.com> wrote:
> --- a/xen/arch/x86/hvm/vmx/vmx.c
> +++ b/xen/arch/x86/hvm/vmx/vmx.c
> @@ -3392,11 +3392,11 @@ void vmx_vmexit_handler(struct cpu_user_regs *regs)
> break;
> }
> else {
> - int handled =
> + int rc =
> hvm_event_breakpoint(regs->eip,
> HVM_EVENT_SOFTWARE_BREAKPOINT);
>
> - if ( handled < 0 )
> + if ( !rc )
> {
> struct hvm_trap trap = {
> .vector = TRAP_int3,
> @@ -3410,7 +3410,7 @@ void vmx_vmexit_handler(struct cpu_user_regs *regs)
> hvm_inject_trap(&trap);
> break;
> }
> - else if ( handled )
> + else if ( rc > 0 )
> break;
> }
>
Ah, I guess that's what you were referring to on the other thread.
There's indeed quite a bit of cleanup potential here. The minimal
thing I'd like to ask for is to drop the pointless "else", as you're
touching that line anyway.
With that (also doable upon commit of course)
Reviewed-by: Jan Beulich <jbeul...@suse.com>
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
