[PATCH v2 0/7] KVM: x86: CPUID emulation and tracing fixes

Sean Christopherson
2020-03-05T01:34:41Z

Round two of trying to wrangle kvm_cpuid() into submission. Two more bug
fixes, both related to querying for an "AMD" vendor, in addition to the
fixes in v1 (tracing and Hypervisor/Centaur range checks).

In theory, everything up to the refactoring is non-controversial, i.e. we
can bikeshed the refactoring without delaying the bug fixes.

v2:
- Use Jan's patch to fix the trace bug. [Everyone]
- Rework Hypervisor/Centaur handling so that only the Hypervisor
sub-ranges get the restrictive 0xffffff00 mask, and so that Centaur's
range only gets recognized when the guest vendor is Centaur. [Jim]
- Add the aforementioned bug fixes.
- Add a patch to do build time assertions on the vendor string, which
are hand coded u32s in the emulator (for direct comparison against
CPUID register output).
- Drop the patch to add CPUID.maxphyaddr emulator helper. [Paolo]
- Redo refactoring patches to land them after all the bug fixes
and to do the refactoring without any semantic changes in the
emulator.

Jan Kiszka (1):
KVM: x86: Trace the original requested CPUID function in kvm_cpuid()

Sean Christopherson (6):
KVM: x86: Add helpers to perform CPUID-based guest vendor check
KVM x86: Extend AMD specific guest behavior to Hygon virtual CPUs
KVM: x86: Fix CPUID range checks for Hypervisor and Centaur classes
KVM: x86: Add build-time assertions on validity of vendor strings
KVM: x86: Refactor out-of-range logic to contain the madness
KVM: x86: Refactor kvm_cpuid() param that controls out-of-range logic

arch/x86/include/asm/kvm_emulate.h | 37 +++++++++-
arch/x86/kvm/cpuid.c | 111 +++++++++++++++++++++--------
arch/x86/kvm/cpuid.h | 8 ++-
arch/x86/kvm/emulate.c | 64 ++++++++---------
arch/x86/kvm/mmu/mmu.c | 3 +-
arch/x86/kvm/svm.c | 2 +-
arch/x86/kvm/x86.c | 7 +-
7 files changed, 162 insertions(+), 70 deletions(-)

--
2.24.1