linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL] First batch of KVM changes for 5.6 merge window
@ 2020-01-30 18:20 Paolo Bonzini
  2020-01-31 18:01 ` Linus Torvalds
  2020-01-31 19:35 ` pr-tracker-bot
  0 siblings, 2 replies; 8+ messages in thread
From: Paolo Bonzini @ 2020-01-30 18:20 UTC (permalink / raw)
  To: torvalds; +Cc: linux-kernel, kvm

Linus,

The following changes since commit ae6088216ce4b99b3a4aaaccd2eb2dd40d473d42:

  Merge tag 'trace-v5.5-rc5' of git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace (2020-01-06 15:38:38 -0800)

are available in the git repository at:


  https://git.kernel.org/pub/scm/virt/kvm/kvm.git tags/kvm-5.6-1

for you to fetch changes up to 4cbc418a44d5067133271bb6eeac2382f2bf94f7:

  Merge branch 'cve-2019-3016' into kvm-next-5.6 (2020-01-30 18:47:59 +0100)

----------------------------------------------------------------

ARM: Cleanups and corner case fixes

PPC: Bugfixes

x86:
* Support for mapping DAX areas with large nested page table entries.
* Cleanups and bugfixes here too.  A particularly important one is
a fix for FPU load when the thread has TIF_NEED_FPU_LOAD.  There is
also a race condition which could be used in guest userspace to exploit
the guest kernel, for which the embargo expired today.
* Fast path for IPI delivery vmexits, shaving about 200 clock cycles
from IPI latency.
* Protect against "Spectre-v1/L1TF" (bring data in the cache via
speculative out of bound accesses, use L1TF on the sibling hyperthread
to read it), which unfortunately is an even bigger whack-a-mole game
than SpectreV1.

Sean continues his mission to rewrite KVM.  In addition to a sizable
number of x86 patches, this time he contributed a pretty large refactoring
of vCPU creation that affects all architectures but should not have any
visible effect.

s390 will come next week together with some more x86 patches.

----------------------------------------------------------------
Alex Shi (1):
      KVM: remove unused guest_enter

Alexandru Elisei (1):
      KVM: arm64: Treat emulated TVAL TimerValue as a signed 32-bit integer

Andrew Jones (1):
      arm64: KVM: Add UAPI notes for swapped registers

Bharata B Rao (1):
      KVM: PPC: Book3S HV: Release lock on page-out failure path

Boris Ostrovsky (5):
      x86/kvm: Be careful not to clear KVM_VCPU_FLUSH_TLB bit
      x86/kvm: Introduce kvm_(un)map_gfn()
      x86/kvm: Cache gfn to pfn translation
      x86/KVM: Make sure KVM_VCPU_FLUSH_TLB flag is not missed
      x86/KVM: Clean up host's steal time structure

Christoffer Dall (1):
      KVM: arm64: Only sign-extend MMIO up to register width

David Michael (1):
      KVM: PPC: Book3S PR: Fix -Werror=return-type build failure

Eric Auger (5):
      KVM: arm/arm64: vgic-its: Fix restoration of unmapped collections
      KVM: arm64: pmu: Don't increment SW_INCR if PMCR.E is unset
      KVM: arm64: pmu: Don't mark a counter as chained if the odd one is disabled
      KVM: arm64: pmu: Fix chained SW_INCR counters
      KVM: arm64: pmu: Only handle supported event counters

Gavin Shan (2):
      tools/kvm_stat: Fix kvm_exit filter name
      KVM: arm/arm64: Fix young bit from mmu notifier

Haiwei Li (1):
      Adding 'else' to reduce checking.

James Morse (3):
      KVM: arm/arm64: Re-check VMA on detecting a poisoned page
      KVM: arm: Fix DFSR setting for non-LPAE aarch32 guests
      KVM: arm: Make inject_abt32() inject an external abort instead

Jim Mattson (3):
      kvm: nVMX: VMWRITE checks VMCS-link pointer before VMCS field
      kvm: nVMX: VMWRITE checks unsupported field before read-only field
      kvm: nVMX: Aesthetic cleanup of handle_vmread and handle_vmwrite

John Allen (1):
      kvm/svm: PKU not currently supported

Krish Sadhukhan (1):
      KVM: nVMX: Check GUEST_DR7 on vmentry of nested guests

Leonardo Bras (2):
      KVM: PPC: Book3S: Replace current->mm by kvm->mm
      KVM: PPC: Book3E: Replace current->mm by kvm->mm

Marc Zyngier (1):
      KVM: arm/arm64: Cleanup MMIO handling

Marios Pomonis (13):
      KVM: x86: Protect x86_decode_insn from Spectre-v1/L1TF attacks
      KVM: x86: Protect kvm_hv_msr_[get|set]_crash_data() from Spectre-v1/L1TF attacks
      KVM: x86: Refactor picdev_write() to prevent Spectre-v1/L1TF attacks
      KVM: x86: Protect ioapic_read_indirect() from Spectre-v1/L1TF attacks
      KVM: x86: Protect ioapic_write_indirect() from Spectre-v1/L1TF attacks
      KVM: x86: Protect kvm_lapic_reg_write() from Spectre-v1/L1TF attacks
      KVM: x86: Protect MSR-based index computations in fixed_msr_to_seg_unit() from Spectre-v1/L1TF attacks
      KVM: x86: Protect MSR-based index computations in pmu.h from Spectre-v1/L1TF attacks
      KVM: x86: Protect MSR-based index computations from Spectre-v1/L1TF attacks in x86.c
      KVM: x86: Refactor prefix decoding to prevent Spectre-v1/L1TF attacks
      KVM: x86: Protect exit_reason from being used in Spectre-v1/L1TF attacks
      KVM: x86: Protect DR-based index computations from Spectre-v1/L1TF attacks
      KVM: x86: Protect pmu_intel.c from Spectre-v1/L1TF attacks

Mark Brown (1):
      arm64: KVM: Annotate guest entry/exit as a single function

Mark Rutland (3):
      KVM: arm64: Correct PSTATE on exception entry
      KVM: arm/arm64: Correct CPSR on exception entry
      KVM: arm/arm64: Correct AArch32 SPSR on exception entry

Miaohe Lin (19):
      KVM: vmx: remove unreachable statement in vmx_get_msr_feature()
      KVM: get rid of var page in kvm_set_pfn_dirty()
      KVM: explicitly set rmap_head->val to 0 in pte_list_desc_remove_entry()
      KVM: x86: Fix some comment typos
      KVM: lib: use jump label to handle resource release in irq_bypass_register_consumer()
      KVM: lib: use jump label to handle resource release in irq_bypass_register_producer()
      KVM: x86: check kvm_pit outside kvm_vm_ioctl_reinject()
      KVM: Fix some wrong function names in comment
      KVM: Fix some out-dated function names in comment
      KVM: Fix some comment typos and missing parentheses
      KVM: Fix some grammar mistakes
      KVM: hyperv: Fix some typos in vcpu unimpl info
      KVM: Fix some writing mistakes
      KVM: vmx: delete meaningless nested_vmx_prepare_msr_bitmap() declaration
      KVM: nVMX: vmread should not set rflags to specify success in case of #PF
      KVM: x86: avoid clearing pending exception event twice
      KVM: apic: short-circuit kvm_apic_accept_pic_intr() when pic intr is accepted
      KVM: VMX: remove duplicated segment cache clear
      KVM: X86: Add 'else' to unify fastop and execute call path

Milan Pandurov (1):
      kvm: Refactor handling of VM debugfs files

Oliver Upton (1):
      KVM: nVMX: WARN on failure to set IA32_PERF_GLOBAL_CTRL

Paolo Bonzini (11):
      KVM: x86: use CPUID to locate host page table reserved bits
      KVM: x86: fix overlap between SPTE_MMIO_MASK and generation
      KVM: x86: list MSR_IA32_UCODE_REV as an emulated MSR
      KVM: async_pf: drop kvm_arch_async_page_present wrappers
      KVM: x86: avoid incorrect writes to host MSR_IA32_SPEC_CTRL
      Revert "KVM: x86: Add a WARN on TIF_NEED_FPU_LOAD in kvm_load_guest_fpu()"
      KVM: Move running VCPU from ARM to common code
      KVM: x86: inline memslot_valid_for_gpte
      Merge tag 'kvmarm-5.6' of git://git.kernel.org/.../kvmarm/kvmarm into HEAD
      Merge tag 'kvm-ppc-next-5.6-2' of git://git.kernel.org/.../paulus/powerpc into HEAD
      Merge branch 'cve-2019-3016' into kvm-next-5.6

Peng Hao (1):
      kvm/x86: export kvm_vector_hashing_enabled() is unnecessary

Peter Xu (10):
      KVM: X86: Fix kvm_bitmap_or_dest_vcpus() to use irq shorthand
      KVM: X86: Move irrelevant declarations out of ioapic.h
      KVM: X86: Use APIC_DEST_* macros properly in kvm_lapic_irq.dest_mode
      KVM: X86: Drop KVM_APIC_SHORT_MASK and KVM_APIC_DEST_MASK
      KVM: X86: Fix callers of kvm_apic_match_dest() to use correct macros
      KVM: X86: Convert the last users of "shorthand = 0" to use macros
      KVM: Remove kvm_read_guest_atomic()
      KVM: Add build-time error check on kvm_run size
      KVM: X86: Don't take srcu lock in init_rmode_identity_map()
      KVM: X86: Drop x86_set_memory_region()

Russell King (1):
      arm64: kvm: Fix IDMAP overlap with HYP VA

Sean Christopherson (100):
      KVM: x86: Fix potential put_fpu() w/o load_fpu() on MPX platform
      KVM: x86: Add a WARN on TIF_NEED_FPU_LOAD in kvm_load_guest_fpu()
      KVM: x86: Use gpa_t for cr2/gpa to fix TDP support on 32-bit KVM
      KVM: x86/mmu: Move definition of make_mmu_pages_available() up
      KVM: x86/mmu: Fold nonpaging_map() into nonpaging_page_fault()
      KVM: x86/mmu: Move nonpaging_page_fault() below try_async_pf()
      KVM: x86/mmu: Refactor handling of cache consistency with TDP
      KVM: x86/mmu: Refactor the per-slot level calculation in mapping_level()
      KVM: x86/mmu: Refactor handling of forced 4k pages in page faults
      KVM: x86/mmu: Incorporate guest's page level into max level for shadow MMU
      KVM: x86/mmu: Persist gfn_lpage_is_disallowed() to max_level
      KVM: x86/mmu: Rename lpage_disallowed to account_disallowed_nx_lpage
      KVM: x86/mmu: Consolidate tdp_page_fault() and nonpaging_page_fault()
      KVM: x86/mmu: Move transparent_hugepage_adjust() above __direct_map()
      KVM: x86/mmu: Move calls to thp_adjust() down a level
      KVM: x86/mmu: Move root_hpa validity checks to top of page fault handler
      KVM: x86/mmu: WARN on an invalid root_hpa
      KVM: x86/mmu: WARN if root_hpa is invalid when handling a page fault
      KVM: VMX: Add non-canonical check on writes to RTIT address MSRs
      KVM: VMX: Add helper to consolidate up PT/RTIT WRMSR fault logic
      KVM: x86: Don't let userspace set host-reserved cr4 bits
      KVM: x86: Ensure all logical CPUs have consistent reserved cr4 bits
      KVM: x86: Drop special XSAVE handling from guest_cpuid_has()
      KVM: x86: Add macro to ensure reserved cr4 bits checks stay in sync
      KVM: x86: Add dedicated emulator helpers for querying CPUID features
      KVM: x86: Move bit() helper to cpuid.h
      KVM: x86: Add CPUID_7_1_EAX to the reverse CPUID table
      KVM: x86: Expand build-time assertion on reverse CPUID usage
      KVM: x86: Refactor and rename bit() to feature_bit() macro
      KVM: x86/mmu: Reorder the reserved bit check in prefetch_invalid_gpte()
      KVM: x86/mmu: Micro-optimize nEPT's bad memptype/XWR checks
      KVM: x86/mmu: Apply max PA check for MMIO sptes to 32-bit KVM
      KVM: PPC: Book3S HV: Uninit vCPU if vcore creation fails
      KVM: PPC: Book3S PR: Free shared page if mmu initialization fails
      KVM: x86: Free wbinvd_dirty_mask if vCPU creation fails
      KVM: VMX: Allocate VPID after initializing VCPU
      KVM: VMX: Use direct vcpu pointer during vCPU create/free
      KVM: SVM: Use direct vcpu pointer during vCPU create/free
      KVM: x86: Allocate vcpu struct in common x86 code
      KVM: x86: Move FPU allocation to common x86 code
      KVM: x86: Move allocation of pio_data page down a few lines
      KVM: x86: Move kvm_vcpu_init() invocation to common code
      KVM: PPC: e500mc: Add build-time assert that vcpu is at offset 0
      KVM: PPC: Allocate vcpu struct in common PPC code
      KVM: PPC: Book3S PR: Allocate book3s and shadow vcpu after common init
      KVM: PPC: e500mc: Move reset of oldpir below call to kvm_vcpu_init()
      KVM: PPC: Move kvm_vcpu_init() invocation to common code
      KVM: MIPS: Use kvm_vcpu_cache to allocate vCPUs
      KVM: MIPS: Drop kvm_arch_vcpu_free()
      KVM: PPC: Drop kvm_arch_vcpu_free()
      KVM: arm: Drop kvm_arch_vcpu_free()
      KVM: x86: Remove spurious kvm_mmu_unload() from vcpu destruction path
      KVM: x86: Remove spurious clearing of async #PF MSR
      KVM: x86: Drop kvm_arch_vcpu_free()
      KVM: Remove kvm_arch_vcpu_free() declaration
      KVM: Add kvm_arch_vcpu_precreate() to handle pre-allocation issues
      KVM: s390: Move guts of kvm_arch_vcpu_init() into kvm_arch_vcpu_create()
      KVM: s390: Invoke kvm_vcpu_init() before allocating sie_page
      KVM: MIPS: Invoke kvm_vcpu_uninit() immediately prior to freeing vcpu
      KVM: x86: Invoke kvm_vcpu_uninit() immediately prior to freeing vcpu
      KVM: Introduce kvm_vcpu_destroy()
      KVM: Move vcpu alloc and init invocation to common code
      KVM: Unexport kvm_vcpu_cache and kvm_vcpu_{un}init()
      KVM: Move initialization of preempt notifier to kvm_vcpu_init()
      KVM: x86: Move guts of kvm_arch_vcpu_setup() into kvm_arch_vcpu_create()
      KVM: MIPS: Move .vcpu_setup() call to kvm_arch_vcpu_create()
      KVM: s390: Manually invoke vcpu setup during kvm_arch_vcpu_create()
      KVM: PPC: BookE: Setup vcpu during kvmppc_core_vcpu_create()
      KVM: Drop kvm_arch_vcpu_setup()
      KVM: x86: Move all vcpu init code into kvm_arch_vcpu_create()
      KVM: MIPS: Move all vcpu init code into kvm_arch_vcpu_create()
      KVM: ARM: Move all vcpu init code into kvm_arch_vcpu_create()
      KVM: PPC: Move all vcpu init code into kvm_arch_vcpu_create()
      KVM: arm64: Free sve_state via arm specific hook
      KVM: Drop kvm_arch_vcpu_init() and kvm_arch_vcpu_uninit()
      KVM: Move putting of vcpu->pid to kvm_vcpu_destroy()
      KVM: Move vcpu->run page allocation out of kvm_vcpu_init()
      KVM: x86: Handle TIF_NEED_FPU_LOAD in kvm_{load,put}_guest_fpu()
      KVM: x86: Ensure guest's FPU state is loaded when accessing for emulation
      KVM: x86: Revert "KVM: X86: Fix fpu state crash in kvm guest"
      KVM: x86: Remove unused ctxt param from emulator's FPU accessors
      KVM: x86: Perform non-canonical checks in 32-bit KVM
      KVM: Check for a bad hva before dropping into the ghc slow path
      KVM: Clean up __kvm_gfn_to_hva_cache_init() and its callers
      KVM: Return immediately if __kvm_gfn_to_hva_cache_init() fails
      KVM: x86/mmu: Enforce max_level on HugeTLB mappings
      mm: thp: KVM: Explicitly check for THP when populating secondary MMU
      KVM: Use vcpu-specific gva->hva translation when querying host page size
      KVM: Play nice with read-only memslots when querying host page size
      x86/mm: Introduce lookup_address_in_mm()
      KVM: x86/mmu: Refactor THP adjust to prep for changing query
      KVM: x86/mmu: Walk host page tables to find THP mappings
      KVM: x86/mmu: Drop level optimization from fast_page_fault()
      KVM: x86/mmu: Rely on host page tables to find HugeTLB mappings
      KVM: x86/mmu: Remove obsolete gfn restoration in FNAME(fetch)
      KVM: x86/mmu: Zap any compound page when collapsing sptes
      KVM: x86/mmu: Fold max_mapping_level() into kvm_mmu_hugepage_adjust()
      KVM: x86/mmu: Remove lpage_is_disallowed() check from set_spte()
      KVM: x86/mmu: Use huge pages for DAX-backed files
      KVM: x86: Use a typedef for fastop functions

Shannon Zhao (1):
      KVM: ARM: Call hyp_cpu_pm_exit at the right place

Sukadev Bhattiprolu (2):
      KVM: PPC: Add skip_page_out parameter to uvmem functions
      KVM: PPC: Book3S HV: Implement H_SVM_INIT_ABORT hcall

Tom Lendacky (1):
      KVM: SVM: Override default MMIO mask if memory encryption is enabled

Vitaly Kuznetsov (1):
      x86/kvm/hyper-v: remove stale evmcs_already_enabled check from nested_enable_evmcs()

Wanpeng Li (2):
      KVM: VMX: FIXED+PHYSICAL mode single target IPI fastpath
      KVM: LAPIC: micro-optimize fixed mode ipi delivery

Xiaoyao Li (3):
      KVM: VMX: Rename INTERRUPT_PENDING to INTERRUPT_WINDOW
      KVM: VMX: Rename NMI_PENDING to NMI_WINDOW
      KVM: VMX: Fix the spelling of CPU_BASED_USE_TSC_OFFSETTING

YueHaibing (1):
      KVM: arm: Remove duplicate include

Zenghui Yu (4):
      KVM: Remove duplicated declaration of kvm_vcpu_kick
      KVM: arm/arm64: vgic: Handle GICR_PENDBASER.PTZ filed as RAZ
      KVM: arm/arm64: vgic-its: Properly check the unmapped coll in DISCARD handler
      KVM: arm/arm64: vgic: Drop the kvm_vgic_register_mmio_region()

zhengbin (1):
      KVM: PPC: Remove set but not used variable 'ra', 'rs', 'rt'

 Documentation/powerpc/ultravisor.rst               |  60 ++
 Documentation/virt/kvm/api.txt                     |   9 +
 arch/arm/include/asm/kvm_emulate.h                 |  27 +-
 arch/arm/include/asm/kvm_host.h                    |  16 +-
 arch/arm/include/asm/kvm_hyp.h                     |   1 +
 arch/arm/include/asm/kvm_mmio.h                    |  26 -
 arch/arm/kvm/guest.c                               |   5 -
 arch/arm64/include/asm/kvm_emulate.h               |  40 +-
 arch/arm64/include/asm/kvm_host.h                  |  16 +-
 arch/arm64/include/asm/kvm_mmio.h                  |  29 -
 arch/arm64/include/asm/ptrace.h                    |   1 +
 arch/arm64/include/uapi/asm/kvm.h                  |  12 +-
 arch/arm64/include/uapi/asm/ptrace.h               |   1 +
 arch/arm64/kvm/guest.c                             |   5 -
 arch/arm64/kvm/hyp/entry.S                         |   7 +-
 arch/arm64/kvm/inject_fault.c                      |  70 ++-
 arch/arm64/kvm/reset.c                             |   2 +-
 arch/arm64/kvm/va_layout.c                         |  56 +-
 arch/mips/kvm/mips.c                               |  84 +--
 arch/powerpc/include/asm/hvcall.h                  |   1 +
 arch/powerpc/include/asm/kvm_book3s_uvmem.h        |  10 +-
 arch/powerpc/include/asm/kvm_host.h                |   1 +
 arch/powerpc/include/asm/kvm_ppc.h                 |   5 +-
 arch/powerpc/kvm/book3s.c                          |   9 +-
 arch/powerpc/kvm/book3s_64_mmu_hv.c                |   4 +-
 arch/powerpc/kvm/book3s_64_mmu_radix.c             |   2 +-
 arch/powerpc/kvm/book3s_64_vio.c                   |  10 +-
 arch/powerpc/kvm/book3s_hv.c                       |  42 +-
 arch/powerpc/kvm/book3s_hv_uvmem.c                 |  34 +-
 arch/powerpc/kvm/book3s_pr.c                       |  34 +-
 arch/powerpc/kvm/book3s_xive_native.c              |   2 +-
 arch/powerpc/kvm/booke.c                           |  67 +--
 arch/powerpc/kvm/e500.c                            |  36 +-
 arch/powerpc/kvm/e500mc.c                          |  30 +-
 arch/powerpc/kvm/emulate_loadstore.c               |   5 -
 arch/powerpc/kvm/powerpc.c                         |  88 +--
 arch/s390/include/asm/kvm_host.h                   |   1 -
 arch/s390/kvm/kvm-s390.c                           | 118 ++--
 arch/x86/include/asm/kvm_emulate.h                 |   4 +
 arch/x86/include/asm/kvm_host.h                    |  34 +-
 arch/x86/include/asm/pgtable_types.h               |   4 +
 arch/x86/include/asm/vmx.h                         |   6 +-
 arch/x86/include/uapi/asm/vmx.h                    |   4 +-
 arch/x86/kvm/cpuid.c                               |   9 +-
 arch/x86/kvm/cpuid.h                               |  45 +-
 arch/x86/kvm/emulate.c                             | 133 +++--
 arch/x86/kvm/hyperv.c                              |  17 +-
 arch/x86/kvm/i8259.c                               |   6 +-
 arch/x86/kvm/ioapic.c                              |  41 +-
 arch/x86/kvm/ioapic.h                              |   6 -
 arch/x86/kvm/irq.h                                 |   3 +
 arch/x86/kvm/irq_comm.c                            |  18 +-
 arch/x86/kvm/lapic.c                               |  37 +-
 arch/x86/kvm/lapic.h                               |   9 +-
 arch/x86/kvm/mmu/mmu.c                             | 605 +++++++++------------
 arch/x86/kvm/mmu/paging_tmpl.h                     |  88 +--
 arch/x86/kvm/mmutrace.h                            |  12 +-
 arch/x86/kvm/mtrr.c                                |   8 +-
 arch/x86/kvm/pmu.h                                 |  18 +-
 arch/x86/kvm/svm.c                                 | 134 +++--
 arch/x86/kvm/vmx/capabilities.h                    |   5 +
 arch/x86/kvm/vmx/evmcs.c                           |   5 -
 arch/x86/kvm/vmx/nested.c                          | 189 +++----
 arch/x86/kvm/vmx/pmu_intel.c                       |  24 +-
 arch/x86/kvm/vmx/vmcs_shadow_fields.h              |   4 +-
 arch/x86/kvm/vmx/vmx.c                             | 294 +++++-----
 arch/x86/kvm/x86.c                                 | 569 +++++++++++--------
 arch/x86/kvm/x86.h                                 |  23 +-
 arch/x86/mm/pageattr.c                             |  11 +
 include/linux/context_tracking.h                   |   9 -
 include/linux/huge_mm.h                            |   6 +
 include/linux/kvm_host.h                           |  40 +-
 include/linux/kvm_types.h                          |   9 +-
 mm/huge_memory.c                                   |  11 +
 tools/arch/x86/include/uapi/asm/vmx.h              |   4 +-
 tools/kvm/kvm_stat/kvm_stat                        |   8 +-
 tools/testing/selftests/kvm/include/x86_64/vmx.h   |   8 +-
 .../selftests/kvm/x86_64/vmx_tsc_adjust_test.c     |   2 +-
 virt/kvm/arm/aarch32.c                             | 131 ++++-
 virt/kvm/arm/arch_timer.c                          |   5 +-
 virt/kvm/arm/arm.c                                 | 113 +---
 virt/kvm/arm/mmio.c                                |  68 +--
 virt/kvm/arm/mmu.c                                 |  32 +-
 virt/kvm/arm/perf.c                                |   6 +-
 virt/kvm/arm/pmu.c                                 | 114 ++--
 virt/kvm/arm/vgic/vgic-its.c                       |   6 +-
 virt/kvm/arm/vgic/vgic-mmio-v3.c                   |   5 +-
 virt/kvm/arm/vgic/vgic-mmio.c                      |  15 +-
 virt/kvm/arm/vgic/vgic-mmio.h                      |   5 -
 virt/kvm/async_pf.c                                |  31 +-
 virt/kvm/kvm_main.c                                | 435 +++++++++------
 virt/lib/irqbypass.c                               |  38 +-
 92 files changed, 2387 insertions(+), 2045 deletions(-)
 delete mode 100644 arch/arm/include/asm/kvm_mmio.h
 delete mode 100644 arch/arm64/include/asm/kvm_mmio.h

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] First batch of KVM changes for 5.6 merge window
  2020-01-30 18:20 [GIT PULL] First batch of KVM changes for 5.6 merge window Paolo Bonzini
@ 2020-01-31 18:01 ` Linus Torvalds
  2020-01-31 18:53   ` Sean Christopherson
  2020-01-31 21:08   ` Paolo Bonzini
  2020-01-31 19:35 ` pr-tracker-bot
  1 sibling, 2 replies; 8+ messages in thread
From: Linus Torvalds @ 2020-01-31 18:01 UTC (permalink / raw)
  To: Paolo Bonzini, Sean Christopherson, Borislav Petkov, Xiaoyao Li
  Cc: Linux Kernel Mailing List, KVM list

On Thu, Jan 30, 2020 at 10:20 AM Paolo Bonzini <pbonzini@redhat.com> wrote:
>
> Xiaoyao Li (3):
>       KVM: VMX: Rename INTERRUPT_PENDING to INTERRUPT_WINDOW
>       KVM: VMX: Rename NMI_PENDING to NMI_WINDOW
>       KVM: VMX: Fix the spelling of CPU_BASED_USE_TSC_OFFSETTING

So in the meantime, on the x86 merge window side, we have this:

  b39033f504a7 ("KVM: VMX: Use VMX_FEATURE_* flags to define VMCS control bits")

and while the above results in a conflict, that's not a problem. The
conflict was trivial to fix up.

HOWEVER.

It most definitely shows that the above renaming now means that the
names don't match. It didn't match 100% before either, but now the
differences are even bigger. The VMX_FEATURE_xyz bits have different
names than the CPU_BASED_xyz bits, and that seems a bit questionable.

So I'm not convinced about the renaming. The spelling fix is good: it
actually now more closely resembles the VMCS_FEATURE bit that already
had OFFSETTING with two T's.

But even that one isn't really the same even then. The CPU_BASED_xyz
thing has "USE_TSC_OFFSETTING", while the VMCS_FEATURE_xyz bit doesn't
have the "USE" part.

And the actual renaming means that now we basically have

  CPU_BASED_INTR_WINDOW_EXITING
  VMX_FEATURE_VIRTUAL_INTR_PENDING

and

  CPU_BASED_NMI_WINDOW_EXITING
  VMX_FEATURE_VIRTUAL_NMI_PENDING

for the same bit definitions (yeah, the VMX_FEATURE bits obviously
have the offset in them, so it's not the same _value_, but it's a 1:1
relationship between them).

There are other (pre-existing) differences, but while fixing up the
merge conflict I really got the feeling that it's confusing and wrong
to basically use different naming for these things when they are about
the same bit.

I don't care much which way it goes (maybe the VMX_FATURE_xyz bits
should be renamed instead of the other way around?) and I wonder what
the official documentation names are? Is there some standard here or
are people just picking names at random?

The two commits both came from intel.com addresses, so hopefully there
can be some intel-sanctioned resolution on the naming? Please?

Hmm?

                  Linus

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] First batch of KVM changes for 5.6 merge window
  2020-01-31 18:01 ` Linus Torvalds
@ 2020-01-31 18:53   ` Sean Christopherson
  2020-01-31 19:03     ` Linus Torvalds
  2020-01-31 21:08   ` Paolo Bonzini
  1 sibling, 1 reply; 8+ messages in thread
From: Sean Christopherson @ 2020-01-31 18:53 UTC (permalink / raw)
  To: Linus Torvalds
  Cc: Paolo Bonzini, Borislav Petkov, Xiaoyao Li,
	Linux Kernel Mailing List, KVM list

On Fri, Jan 31, 2020 at 10:01:37AM -0800, Linus Torvalds wrote:
> On Thu, Jan 30, 2020 at 10:20 AM Paolo Bonzini <pbonzini@redhat.com> wrote:
> >
> > Xiaoyao Li (3):
> >       KVM: VMX: Rename INTERRUPT_PENDING to INTERRUPT_WINDOW
> >       KVM: VMX: Rename NMI_PENDING to NMI_WINDOW
> >       KVM: VMX: Fix the spelling of CPU_BASED_USE_TSC_OFFSETTING
> 
> So in the meantime, on the x86 merge window side, we have this:
> 
>   b39033f504a7 ("KVM: VMX: Use VMX_FEATURE_* flags to define VMCS control bits")
> 
> and while the above results in a conflict, that's not a problem. The
> conflict was trivial to fix up.
> 
> HOWEVER.
> 
> It most definitely shows that the above renaming now means that the
> names don't match. It didn't match 100% before either, but now the
> differences are even bigger. The VMX_FEATURE_xyz bits have different
> names than the CPU_BASED_xyz bits, and that seems a bit questionable.
> 
> So I'm not convinced about the renaming. The spelling fix is good: it
> actually now more closely resembles the VMCS_FEATURE bit that already
> had OFFSETTING with two T's.
> 
> But even that one isn't really the same even then. The CPU_BASED_xyz
> thing has "USE_TSC_OFFSETTING", while the VMCS_FEATURE_xyz bit doesn't
> have the "USE" part.
> 
> And the actual renaming means that now we basically have
> 
>   CPU_BASED_INTR_WINDOW_EXITING
>   VMX_FEATURE_VIRTUAL_INTR_PENDING
> 
> and
> 
>   CPU_BASED_NMI_WINDOW_EXITING
>   VMX_FEATURE_VIRTUAL_NMI_PENDING
> 
> for the same bit definitions (yeah, the VMX_FEATURE bits obviously
> have the offset in them, so it's not the same _value_, but it's a 1:1
> relationship between them).
> 
> There are other (pre-existing) differences, but while fixing up the
> merge conflict I really got the feeling that it's confusing and wrong
> to basically use different naming for these things when they are about
> the same bit.
> 
> I don't care much which way it goes (maybe the VMX_FATURE_xyz bits
> should be renamed instead of the other way around?) and I wonder what
> the official documentation names are? Is there some standard here or
> are people just picking names at random?
> 
> The two commits both came from intel.com addresses, so hopefully there
> can be some intel-sanctioned resolution on the naming? Please?

Hrm.

For *_WINDOW_EXITING versus VIRTUAL_*_PENDING, VMX_FEATURE_* should be
renamed to use *_WINDOW_EXITING, as that's the nomenclature used by the
SDM.  I added the VMX_FEATURE_* names while KVM was still using
VIRTUAL_*_PENDING, and neglected to go back and update the series, probably
because I was in denial after lobbying to keep the non-SDM names[1] and
getting overruled[2] :-).

As for USE_TSC_OFFSETTING vs TSC_OFFSETTING, I'd like to keep the minor
differences.  VMX_FEATURES is intended to reflect the capabilities of the
CPU, whereas the CPU_BASED/EXEC masks are effectively "commands" from
software to hardware, e.g. "CPU has TSC offsetting" vs. "CPU, use TSC
offsetting".

Re-reading vmxfeatures.h, I botched a few names:

  USE_IO_BITMAPS and USE_MSR_BITMAPS shouldn't have the USE_ prefix, by my
  own capability vs. command argument.

  PAGE_MOD_LOGGING should simply be PML.  I have no idea why I chose to
  (partially) expand the acronym.

I assume the easiest thing would be send a cleanup patch for vmxfeatures.h
and route it through the KVM tree?

[1] https://lkml.kernel.org/r/20191206204747.GD5433@linux.intel.com/
[2] https://lkml.kernel.org/r/2beeb1fb-7d3a-d829-38e0-ddf76b65bd3c@redhat.com/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] First batch of KVM changes for 5.6 merge window
  2020-01-31 18:53   ` Sean Christopherson
@ 2020-01-31 19:03     ` Linus Torvalds
  0 siblings, 0 replies; 8+ messages in thread
From: Linus Torvalds @ 2020-01-31 19:03 UTC (permalink / raw)
  To: Sean Christopherson
  Cc: Paolo Bonzini, Borislav Petkov, Xiaoyao Li,
	Linux Kernel Mailing List, KVM list

On Fri, Jan 31, 2020 at 10:53 AM Sean Christopherson
<sean.j.christopherson@intel.com> wrote:
>
> I assume the easiest thing would be send a cleanup patch for vmxfeatures.h
> and route it through the KVM tree?

Probably. The KVM side is the only thing that seems to use the
defines, so any names changes should impact only them (we do have that
mkcapflags.sh script, but that should react automatically to any
changes in the #define names)

And this is obviously not a big deal, I just noticed the discrepancies
when doing that resolution.

                    Linus

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] First batch of KVM changes for 5.6 merge window
  2020-01-30 18:20 [GIT PULL] First batch of KVM changes for 5.6 merge window Paolo Bonzini
  2020-01-31 18:01 ` Linus Torvalds
@ 2020-01-31 19:35 ` pr-tracker-bot
  1 sibling, 0 replies; 8+ messages in thread
From: pr-tracker-bot @ 2020-01-31 19:35 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: torvalds, linux-kernel, kvm

The pull request you sent on Thu, 30 Jan 2020 19:20:42 +0100:

> https://git.kernel.org/pub/scm/virt/kvm/kvm.git tags/kvm-5.6-1

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/e813e65038389b66d2f8dd87588694caf8dc2923

Thank you!

-- 
Deet-doot-dot, I am a bot.
https://korg.wiki.kernel.org/userdoc/prtracker

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] First batch of KVM changes for 5.6 merge window
  2020-01-31 18:01 ` Linus Torvalds
  2020-01-31 18:53   ` Sean Christopherson
@ 2020-01-31 21:08   ` Paolo Bonzini
  2020-01-31 21:24     ` Borislav Petkov
  1 sibling, 1 reply; 8+ messages in thread
From: Paolo Bonzini @ 2020-01-31 21:08 UTC (permalink / raw)
  To: Linus Torvalds, Sean Christopherson, Borislav Petkov, Xiaoyao Li
  Cc: Linux Kernel Mailing List, KVM list

On 31/01/20 19:01, Linus Torvalds wrote:
> There are other (pre-existing) differences, but while fixing up the
> merge conflict I really got the feeling that it's confusing and wrong
> to basically use different naming for these things when they are about
> the same bit.

I was supposed to get a topic branch and fix everything up so that both
CPU_BASED_ and VMX_FEATURE_ constants would get the new naming.  When
Boris alerted me of the conflict and I said "thanks I'll sort it out",
he probably interpreted it as me not needing the topic branch anymore.
I then forgot to remind him, and here we are.

> I don't care much which way it goes (maybe the VMX_FATURE_xyz bits
> should be renamed instead of the other way around?) and I wonder what
> the official documentation names are? Is there some standard here or
> are people just picking names at random?

The official documentation names are the ones introduced by the KVM pull
request ("Table 24-6. Definitions of Primary Processor-Based
VM-Execution Controls").  In fact consistency with the documentation was
why we changed them.  On the other hand Sean wanted VMX_FEATURE_* to be
consistent with CPU_BASED_*, which made sense when he wrote the patch.

I'll change the names to match for next week's second batch of KVM changes.

Paolo


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] First batch of KVM changes for 5.6 merge window
  2020-01-31 21:08   ` Paolo Bonzini
@ 2020-01-31 21:24     ` Borislav Petkov
  2020-01-31 21:27       ` Paolo Bonzini
  0 siblings, 1 reply; 8+ messages in thread
From: Borislav Petkov @ 2020-01-31 21:24 UTC (permalink / raw)
  To: Paolo Bonzini
  Cc: Linus Torvalds, Sean Christopherson, Xiaoyao Li,
	Linux Kernel Mailing List, KVM list

On Fri, Jan 31, 2020 at 10:08:10PM +0100, Paolo Bonzini wrote:
> I was supposed to get a topic branch and fix everything up so that both
> CPU_BASED_ and VMX_FEATURE_ constants would get the new naming.  When
> Boris alerted me of the conflict and I said "thanks I'll sort it out",
> he probably interpreted it as me not needing the topic branch anymore.
> I then forgot to remind him, and here we are.

Damn, that was a misunderstanding. I actually had a topic branch:
tip:x86/cpu and was assuming that you'll see it from the tip-bot
notifications. But they went to lkml and you weren't CCed. And
regardless, I should've told you explicitly which one it is, sorry about
that. I'll be more explicit next time.

Thx.

-- 
Regards/Gruss,
    Boris.

SUSE Software Solutions Germany GmbH, GF: Felix Imendörffer, HRB 36809, AG Nürnberg

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] First batch of KVM changes for 5.6 merge window
  2020-01-31 21:24     ` Borislav Petkov
@ 2020-01-31 21:27       ` Paolo Bonzini
  0 siblings, 0 replies; 8+ messages in thread
From: Paolo Bonzini @ 2020-01-31 21:27 UTC (permalink / raw)
  To: Borislav Petkov
  Cc: Linus Torvalds, Sean Christopherson, Xiaoyao Li,
	Linux Kernel Mailing List, KVM list

On 31/01/20 22:24, Borislav Petkov wrote:
>> I was supposed to get a topic branch and fix everything up so that both
>> CPU_BASED_ and VMX_FEATURE_ constants would get the new naming.  When
>> Boris alerted me of the conflict and I said "thanks I'll sort it out",
>> he probably interpreted it as me not needing the topic branch anymore.
>> I then forgot to remind him, and here we are.
>
> Damn, that was a misunderstanding. I actually had a topic branch:
> tip:x86/cpu and was assuming that you'll see it from the tip-bot
> notifications. But they went to lkml and you weren't CCed. And
> regardless, I should've told you explicitly which one it is, sorry about
> that. I'll be more explicit next time.

No problem, it's fair to say that we both did our best to mess this up. :)

Paolo


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2020-01-31 21:27 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-30 18:20 [GIT PULL] First batch of KVM changes for 5.6 merge window Paolo Bonzini
2020-01-31 18:01 ` Linus Torvalds
2020-01-31 18:53   ` Sean Christopherson
2020-01-31 19:03     ` Linus Torvalds
2020-01-31 21:08   ` Paolo Bonzini
2020-01-31 21:24     ` Borislav Petkov
2020-01-31 21:27       ` Paolo Bonzini
2020-01-31 19:35 ` pr-tracker-bot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).