mbox series

[v9,0/8] KVM: Add virtualization support of split lock detection

Message ID 20200509110542.8159-1-xiaoyao.li@intel.com (mailing list archive)
Headers show
Series KVM: Add virtualization support of split lock detection | expand

Message

Xiaoyao Li May 9, 2020, 11:05 a.m. UTC
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(-)

Comments

Xiaoyao Li May 18, 2020, 1:27 a.m. UTC | #1
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.

Thomas and Paolo,

Do you have time to have a look at this version?
Xiaoyao Li May 26, 2020, 6:19 a.m. UTC | #2
Hi Thomas,

On 5/18/2020 9:27 AM, Xiaoyao Li wrote:
> 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.
> 
> Thomas and Paolo,
> 
> Do you have time to have a look at this version?

Does this series have any chance to meet 5.8?

If not, do you plan to take a look at it after merge window?

Thanks,
-Xiaoyao
Xiaoyao Li July 1, 2020, 2:46 a.m. UTC | #3
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(-)
>