KVM: VMX: Reorder some skip_emulated_instruction calls
authorKyle Huey <me@kylehuey.com>
Tue, 29 Nov 2016 20:40:38 +0000 (12:40 -0800)
committerPaolo Bonzini <pbonzini@redhat.com>
Thu, 8 Dec 2016 14:31:04 +0000 (15:31 +0100)
The functions being moved ahead of skip_emulated_instruction here don't
need updated IPs, and skipping the emulated instruction at the end will
make it easier to return its value.

Signed-off-by: Kyle Huey <khuey@kylehuey.com>
Signed-off-by: Radim Krčmář <rkrcmar@redhat.com>
arch/x86/kvm/vmx.c

index e4af9699d72600603ea63146eb81c0b39c3c6c29..f2f9cf595c07e8a443d4dde15a28d895bb068441 100644 (file)
@@ -5708,8 +5708,8 @@ static int handle_cr(struct kvm_vcpu *vcpu)
        case 2: /* clts */
                handle_clts(vcpu);
                trace_kvm_cr_write(0, kvm_read_cr0(vcpu));
-               skip_emulated_instruction(vcpu);
                vmx_fpu_activate(vcpu);
+               skip_emulated_instruction(vcpu);
                return 1;
        case 1: /*mov from cr*/
                switch (cr) {
@@ -6133,8 +6133,8 @@ static int handle_ept_misconfig(struct kvm_vcpu *vcpu)
 
        gpa = vmcs_read64(GUEST_PHYSICAL_ADDRESS);
        if (!kvm_io_bus_write(vcpu, KVM_FAST_MMIO_BUS, gpa, 0, NULL)) {
-               skip_emulated_instruction(vcpu);
                trace_kvm_fast_mmio(gpa);
+               skip_emulated_instruction(vcpu);
                return 1;
        }
 
@@ -6507,8 +6507,8 @@ static int handle_pause(struct kvm_vcpu *vcpu)
        if (ple_gap)
                grow_ple_window(vcpu);
 
-       skip_emulated_instruction(vcpu);
        kvm_vcpu_on_spin(vcpu);
+       skip_emulated_instruction(vcpu);
 
        return 1;
 }
@@ -6962,8 +6962,8 @@ static int handle_vmon(struct kvm_vcpu *vcpu)
 
        vmx->nested.vmxon = true;
 
-       skip_emulated_instruction(vcpu);
        nested_vmx_succeed(vcpu);
+       skip_emulated_instruction(vcpu);
        return 1;
 
 out_shadow_vmcs:
@@ -7083,8 +7083,8 @@ static int handle_vmoff(struct kvm_vcpu *vcpu)
        if (!nested_vmx_check_permission(vcpu))
                return 1;
        free_nested(to_vmx(vcpu));
-       skip_emulated_instruction(vcpu);
        nested_vmx_succeed(vcpu);
+       skip_emulated_instruction(vcpu);
        return 1;
 }
 
@@ -7124,8 +7124,8 @@ static int handle_vmclear(struct kvm_vcpu *vcpu)
 
        nested_free_vmcs02(vmx, vmptr);
 
-       skip_emulated_instruction(vcpu);
        nested_vmx_succeed(vcpu);
+       skip_emulated_instruction(vcpu);
        return 1;
 }