[PATCH v3 00/13] KVM: nVMX: Adjust VMPTRLD/VMPTRST behavior with active eVMCS

Sean Christopherson posted 13 patches 1 month ago
There is a newer version of this series
arch/x86/kvm/vmx/nested.c                     |  24 ++-
.../testing/selftests/kvm/include/x86/evmcs.h |   8 -
.../selftests/kvm/include/x86/processor.h     |   2 +-
tools/testing/selftests/kvm/include/x86/vmx.h | 173 +++++++-----------
.../testing/selftests/kvm/lib/x86/memstress.c |   6 +-
tools/testing/selftests/kvm/lib/x86/vmx.c     |  20 +-
.../selftests/kvm/x86/aperfmperf_test.c       |   6 +-
.../kvm/x86/evmcs_smm_controls_test.c         |   4 +-
.../testing/selftests/kvm/x86/hyperv_evmcs.c  |  57 +++---
.../selftests/kvm/x86/kvm_buslock_test.c      |   6 +-
.../selftests/kvm/x86/nested_close_kvm_test.c |   6 +-
.../selftests/kvm/x86/nested_dirty_log_test.c |   6 +-
.../selftests/kvm/x86/nested_emulation_test.c |   6 +-
.../kvm/x86/nested_exceptions_test.c          |   6 +-
.../kvm/x86/nested_invalid_cr3_test.c         |   8 +-
.../selftests/kvm/x86/nested_tdp_fault_test.c |   6 +-
.../kvm/x86/nested_tsc_adjust_test.c          |   6 +-
.../kvm/x86/nested_tsc_scaling_test.c         |   6 +-
.../kvm/x86/save_restore_pf_stress_test.c     |   8 +-
tools/testing/selftests/kvm/x86/smm_test.c    |   4 +-
tools/testing/selftests/kvm/x86/state_test.c  |  42 ++---
.../kvm/x86/triple_fault_event_test.c         |   6 +-
.../selftests/kvm/x86/vmx_apic_access_test.c  |   8 +-
.../kvm/x86/vmx_apicv_updates_test.c          |   8 +-
.../kvm/x86/vmx_invalid_nested_guest_state.c  |   6 +-
.../kvm/x86/vmx_nested_la57_state_test.c      |   6 +-
.../kvm/x86/vmx_preemption_timer_test.c       |  10 +-
27 files changed, 214 insertions(+), 240 deletions(-)
[PATCH v3 00/13] KVM: nVMX: Adjust VMPTRLD/VMPTRST behavior with active eVMCS
Posted by Sean Christopherson 1 month ago
Vitaly's series to adjust KVM's emulation of VMPTRLD/VMPTRST when the vCPU
has an active eVMCS to match Hyper-V, and more importantly to not put the
vCPU into an infinite loop.

The patches from Vitaly remain the same from v2.  The churn I piled on top
came from wanting to use KVM_ASM_SAFE() for more VMX instructions to verify
the RFLAGS bug+fix Sashiko pointed out, and I lack impulse control when it
comes to cleanups.

v3:
 - See above.

v2:
 - https://lore.kernel.org/all/20260818165258.2613603-1-vkuznets@redhat.com
 - Fix build when !CONFIG_KVM_HYPERV [Sashiko]
 - Drop now-unneeded evmcs_vmptrst() [Sashiko]
 - Avoid hardcoding VMPTRLD insn length in hyperv_evmcs [Sashiko], use
   KVM_ASM_SAFE framework instead [Sean]

v1: https://lore.kernel.org/all/20260814162221.2144473-1-vkuznets@redhat.com

Sean Christopherson (9):
  KVM: selftests: Don't clobber RFLAGS in happy path of __KVM_ASM_SAFE()
  KVM: selftests: Assert success in vmptrst(), kill off vmptrstz()
  KVM: selftests: Always assert that vmxon() and
    prepare_for_vmx_operation() succeed
  KVM: selftests: Always assert that vmclear() succeeds
  KVM: selftests: Always assert that vmptrld() succeeds
  KVM: selftests: Drop useless return code from load_vmcs()
  KVM: selftests: Add macros to handle simple VMX instructions
  KVM: selftests: Dedup assembly code for VMLAUNCH and VMRESUME
  KVM: selftests: Add and use double-underscore versions of vmlaunch()
    and vmresume()

Vitaly Kuznetsov (4):
  KVM: nVMX: Make VMPTRLD result in #UD when eVMCS is used
  KVM: nVMX: Make VMPTRST return eVMCS GPA when it is used
  KVM: selftests: Adapt to the updated VMPTRST behavior when eVMCS is
    used
  KVM: selftests: Check VMPTRLD with active eVMCS

 arch/x86/kvm/vmx/nested.c                     |  24 ++-
 .../testing/selftests/kvm/include/x86/evmcs.h |   8 -
 .../selftests/kvm/include/x86/processor.h     |   2 +-
 tools/testing/selftests/kvm/include/x86/vmx.h | 173 +++++++-----------
 .../testing/selftests/kvm/lib/x86/memstress.c |   6 +-
 tools/testing/selftests/kvm/lib/x86/vmx.c     |  20 +-
 .../selftests/kvm/x86/aperfmperf_test.c       |   6 +-
 .../kvm/x86/evmcs_smm_controls_test.c         |   4 +-
 .../testing/selftests/kvm/x86/hyperv_evmcs.c  |  57 +++---
 .../selftests/kvm/x86/kvm_buslock_test.c      |   6 +-
 .../selftests/kvm/x86/nested_close_kvm_test.c |   6 +-
 .../selftests/kvm/x86/nested_dirty_log_test.c |   6 +-
 .../selftests/kvm/x86/nested_emulation_test.c |   6 +-
 .../kvm/x86/nested_exceptions_test.c          |   6 +-
 .../kvm/x86/nested_invalid_cr3_test.c         |   8 +-
 .../selftests/kvm/x86/nested_tdp_fault_test.c |   6 +-
 .../kvm/x86/nested_tsc_adjust_test.c          |   6 +-
 .../kvm/x86/nested_tsc_scaling_test.c         |   6 +-
 .../kvm/x86/save_restore_pf_stress_test.c     |   8 +-
 tools/testing/selftests/kvm/x86/smm_test.c    |   4 +-
 tools/testing/selftests/kvm/x86/state_test.c  |  42 ++---
 .../kvm/x86/triple_fault_event_test.c         |   6 +-
 .../selftests/kvm/x86/vmx_apic_access_test.c  |   8 +-
 .../kvm/x86/vmx_apicv_updates_test.c          |   8 +-
 .../kvm/x86/vmx_invalid_nested_guest_state.c  |   6 +-
 .../kvm/x86/vmx_nested_la57_state_test.c      |   6 +-
 .../kvm/x86/vmx_preemption_timer_test.c       |  10 +-
 27 files changed, 214 insertions(+), 240 deletions(-)


base-commit: 76671054f9a1ff6abb976583cd8da37650acdc97
-- 
2.55.0.887.g758fc8c411-goog