From: Andrew Cooper <andrew.cooper3@citrix.com>
Date: Thu, 1 Nov 2018 17:37:48 +0000 (+0000)
Subject: x86/vvmx: Don't call vmsucceed() at the end of virtual_vmexit()
X-Git-Tag: archive/raspbian/4.14.0+80-gd101b417b7-1+rpi1^2~63^2~2836
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=fc3637e9af9a301d92695999299a3e9a8458c3c1;p=xen.git

x86/vvmx: Don't call vmsucceed() at the end of virtual_vmexit()

The correct value for RFLAGS is established earlier in the function, and a
successful vmexit logically discards the previous executing context.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Roger Pau Monné <roger.pau@citrix.com>
Reviewed-by: Sergey Dyasli <sergey.dyasli@citrix.com>
Acked-by: Kevin Tian <kevin.tian@intel.com>
---

diff --git a/xen/arch/x86/hvm/vmx/vvmx.c b/xen/arch/x86/hvm/vmx/vvmx.c
index 41c4e2fb82..a72b519ce0 100644
--- a/xen/arch/x86/hvm/vmx/vvmx.c
+++ b/xen/arch/x86/hvm/vmx/vvmx.c
@@ -1371,7 +1371,6 @@ static void virtual_vmexit(struct cpu_user_regs *regs)
         nvmx_update_apicv(v);
 
     nvcpu->nv_vmswitch_in_progress = 0;
-    vmsucceed(regs);
 }
 
 static void nvmx_eptp_update(void)