[PATCH v10 0/9] KVM: Add virtualization support of split lock detection

From: Xiaoyao Li
Date: Wed Aug 19 2020 - 02:47:18 EST


Hi maintainers,

Please help review this new version and give comments. There is only one
minor change from previous v9, i.e., adding patch 8.

---

This series aims to add the virtualization of split lock detection in
KVM.

Due to the fact that split lock detection is tightly coupled with CPU
model and CPU model is configurable by host VMM, we elect to use
paravirt method to expose and enumerate it for guest.

Changes in v10
- rebase to v5.9-rc1;
- Add one patch to move the initialization of cpu_model_supports_sld to
split_lock_setup();

Changes in v9
https://lkml.kernel.org/r/20200509110542.8159-1-xiaoyao.li@xxxxxxxxx
- rebase to v5.7-rc4
- Add one patch to rename TIF_SLD to TIF_SLD_DISABLED;
- Add one patch to remove bogus case in handle_guest_split_lock;
- Introduce flag X86_FEATURE_SPLIT_LOCK_DETECT_FATAL and thus drop
sld_state;
- Use X86_FEATURE_SPLIT_LOCK_DETECT and X86_FEATURE_SPLIT_LOCK_DETECT_FATAL
to determine the SLD state of host;
- Introduce split_lock_virt_switch() and two wrappers for KVM instead
of sld_update_to();
- Use paravirt to expose and enumerate split lock detection for guest;
- Split lock detection can be exposed to guest when host is sld_fatal,
even though host is SMT available.

Changes in v8:
https://lkml.kernel.org/r/20200414063129.133630-1-xiaoyao.li@xxxxxxxxx
- rebase to v5.7-rc1.
- basic enabling of split lock detection already merged.
- When host is sld_warn and nosmt, load guest's sld bit when in KVM
context, i.e., between vmx_prepare_switch_to_guest() and before
vmx_prepare_switch_to_host(), KVM uses guest sld setting.

Changes in v7:
https://lkml.kernel.org/r/20200325030924.132881-1-xiaoyao.li@xxxxxxxxx
- only pick patch 1 and patch 2, and hold all the left.
- Update SLD bit on each processor based on sld_state.

Changes in v6:
https://lkml.kernel.org/r/20200324151859.31068-1-xiaoyao.li@xxxxxxxxx
- Drop the sld_not_exist flag and use X86_FEATURE_SPLIT_LOCK_DETECT to
check whether need to init split lock detection. [tglx]
- Use tglx's method to verify the existence of split lock detectoin.
- small optimization of sld_update_msr() that the default value of
msr_test_ctrl_cache has split_lock_detect bit cleared.
- Drop the patch3 in v5 that introducing kvm_only option. [tglx]
- Rebase patch4-8 to kvm/queue.
- use the new kvm-cpu-cap to expose X86_FEATURE_CORE_CAPABILITIES in
Patch 6.

Changes in v5:
https://lkml.kernel.org/r/20200315050517.127446-1-xiaoyao.li@xxxxxxxxx
- Use X86_FEATURE_SPLIT_LOCK_DETECT flag in kvm to ensure split lock
detection is really supported.
- Add and export sld related helper functions in their related usecase
kvm patches.

Xiaoyao Li (9):
x86/split_lock: Rename TIF_SLD to TIF_SLD_DISABLED
x86/split_lock: Remove bogus case in handle_guest_split_lock()
x86/split_lock: Introduce flag X86_FEATURE_SLD_FATAL and drop
sld_state
x86/split_lock: Introduce split_lock_virt_switch() and two wrappers
x86/kvm: Introduce paravirt split lock detection enumeration
KVM: VMX: Enable MSR TEST_CTRL for guest
KVM: VMX: virtualize split lock detection
x86/split_lock: Set cpu_model_supports_sld to true after the existence
of split lock detection is verified BSP
x86/split_lock: Enable split lock detection initialization when
running as a guest on KVM

Documentation/virt/kvm/cpuid.rst | 29 +++++++----
arch/x86/include/asm/cpu.h | 36 ++++++++++++++
arch/x86/include/asm/cpufeatures.h | 1 +
arch/x86/include/asm/thread_info.h | 6 +--
arch/x86/include/uapi/asm/kvm_para.h | 8 ++--
arch/x86/kernel/cpu/intel.c | 61 +++++++++++++++--------
arch/x86/kernel/kvm.c | 3 ++
arch/x86/kernel/process.c | 2 +-
arch/x86/kvm/cpuid.c | 6 +++
arch/x86/kvm/vmx/vmx.c | 72 +++++++++++++++++++++++++---
arch/x86/kvm/vmx/vmx.h | 3 ++
arch/x86/kvm/x86.c | 6 ++-
arch/x86/kvm/x86.h | 16 +++++++
13 files changed, 207 insertions(+), 42 deletions(-)

--
2.18.4