Messages in this thread | | | Subject | Re: [PATCH v9 0/8] KVM: Add virtualization support of split lock detection | From | Xiaoyao Li <> | Date | Wed, 1 Jul 2020 10:46:59 +0800 |
| |
Ping for comments.
On 5/9/2020 7:05 PM, Xiaoyao Li wrote: > 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 v9 > - 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@intel.com > - 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@intel.com > - 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@intel.com > - 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@intel.com > - 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 (8): > 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: Enable split lock detection initialization when > running as an guest on KVM > > Documentation/virt/kvm/cpuid.rst | 29 +++++++---- > arch/x86/include/asm/cpu.h | 35 ++++++++++++++ > 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 | 59 ++++++++++++++++------- > 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 | 7 +++ > 13 files changed, 196 insertions(+), 41 deletions(-) >
| |