From: Isaku Yamahata <isaku.yamahata@xxxxxxxxx> I've updated the patch series based on Marc's request and Yuan review and now eliminated kvm_arch_check_processor_compat(). Those two patch are compile only tested. - "KVM: arm64: Simplify the CPUHP logic" - "RFC: KVM: powerpc: Move processor compatibility check to hardware setup" - "RFC: KVM: Remove cpus_hardware_enabled and related sanity check" Changes from v3: - Updated "KVM: arm64: Simplify the CPUHP logic". - add preempt_disable/enable() around hardware_enable/disable() to keep the assumption of the arch callback. - Eliminated arch compat check callback, kvm_arch_check_processor_compat(). This patch series is to implement the suggestion by Sean Christopherson [1] to reorganize enable/disable cpu virtualization feature by replacing the arch-generic current enable/disable logic with PM related hooks. And convert kvm/x86 to use new hooks. - Untable x86 hardware enable logic, snapshot MSRs for user return notifier, enabling cpu virtualization on cpu online and platform resume. and real enabling of CPU virtualization feature - Introduce hooks related to PM. - Convert kvm/x86 code to user those hooks. - Split out hardware enabling/disabling logic into a separate file. Compile it for non-x86 code. Once conversion of other KVM archs is done, this file can be dropped. - Delete kvm_arch_check_processor_compat() - Delete cpus_hardware_enabled. 17/18 and 18/18 [1] https://lore.kernel.org/kvm/YvU+6fdkHaqQiKxp@xxxxxxxxxx/ [2] https://lore.kernel.org/all/20220216031528.92558-1-chao.gao@xxxxxxxxx/ Changes from v2: - Replace the first patch("KVM: x86: Drop kvm_user_return_msr_cpu_online()") with Sean's implementation - Included all patches of "Improve KVM's interaction with CPU hotplug" [2] Until v2, Tried to cherry-pick the least patches of it. It turned out that all the patches are desirable. Changes from v1: - Add a patch "KVM: Rename and move CPUHP_AP_KVM_STARTING to ONLINE section" to make online/offline callback to run thread context to use mutex instead of spin lock - fixes pointed by Chao Gao Chao Gao (4): KVM: x86: Move check_processor_compatibility from init ops to runtime ops Partially revert "KVM: Pass kvm_init()'s opaque param to additional arch funcs" KVM: Rename and move CPUHP_AP_KVM_STARTING to ONLINE section KVM: Do compatibility checks on hotplugged CPUs Isaku Yamahata (19): KVM: x86: Use this_cpu_ptr() instead of per_cpu_ptr(smp_processor_id()) KVM: Do processor compatibility check on resume KVM: Drop kvm_count_lock and instead protect kvm_usage_count with kvm_lock KVM: Add arch hooks for PM events with empty stub KVM: x86: Move TSC fixup logic to KVM arch resume callback KVM: Add arch hook when VM is added/deleted KVM: Move out KVM arch PM hooks and hardware enable/disable logic KVM: kvm_arch.c: Remove _nolock post fix KVM: kvm_arch.c: Remove a global variable, hardware_enable_failed KVM: Introduce a arch wrapper to check all processor compatibility KVM: x86: Duplicate arch callbacks related to pm events and compat check KVM: Eliminate kvm_arch_post_init_vm() KVM: Add config to not compile kvm_arch.c KVM: x86: Delete kvm_arch_hardware_enable/disable() KVM: x86: Make x86 processor compat check callback empty RFC: KVM: powerpc: Move processor compatibility check to hardware setup KVM: Eliminate kvm_arch_check_processor_compat() RFC: KVM: x86: Remove cpus_hardware_enabled and related sanity check RFC: KVM: Remove cpus_hardware_enabled and related sanity check Marc Zyngier (1): KVM: arm64: Simplify the CPUHP logic Sean Christopherson (2): KVM: x86: Drop kvm_user_return_msr_cpu_online() KVM: Provide more information in kernel log if hardware enabling fails Documentation/virt/kvm/locking.rst | 14 +- arch/arm64/kvm/arch_timer.c | 27 ++-- arch/arm64/kvm/arm.c | 18 ++- arch/arm64/kvm/vgic/vgic-init.c | 19 +-- arch/mips/kvm/mips.c | 5 - arch/powerpc/kvm/powerpc.c | 14 +- arch/riscv/kvm/main.c | 5 - arch/s390/kvm/kvm-s390.c | 5 - arch/x86/include/asm/kvm-x86-ops.h | 1 + arch/x86/include/asm/kvm_host.h | 2 +- arch/x86/kvm/Kconfig | 1 + arch/x86/kvm/svm/svm.c | 4 +- arch/x86/kvm/vmx/vmx.c | 24 ++-- arch/x86/kvm/x86.c | 221 ++++++++++++++++++++++++----- include/kvm/arm_arch_timer.h | 4 + include/kvm/arm_vgic.h | 4 + include/linux/cpuhotplug.h | 5 +- include/linux/kvm_host.h | 16 ++- virt/kvm/Kconfig | 3 + virt/kvm/Makefile.kvm | 3 + virt/kvm/kvm_arch.c | 140 ++++++++++++++++++ virt/kvm/kvm_main.c | 208 +++++++++------------------ 22 files changed, 481 insertions(+), 262 deletions(-) create mode 100644 virt/kvm/kvm_arch.c base-commit: b90cb1053190353cc30f0fef0ef1f378ccc063c5 -- 2.25.1
