This is a supplemental update that includes
"whpx: i386: x2apic emulation for kernel-irqchip=off, feature probing"
v4 unmodified.
This might be too big for QEMU 11.0 at this point though...
"whpx: i386: disable TbFlushHypercalls for emulated LAPIC" is a
bugfix, and "target/i386: emulate: include name of unhandled instruction"
is a debugging aid.
"whpx: i386: x2apic emulation" makes things slightly better for Windows
10 users. But I strongly recommend *not* relying on it when possible and
using kernel-irqchip=on instead. On Windows 10 however that's more murky
because PIC interrupt injection is broken (interrupts don't wake the vCPU
from HLT) in that case.
"whpx: i386: wire up feature probing" is yet another commit adding a code path
not used on Windows 10. It'll tell the user today which CPU features they set
are incompatible with the hardware but it does not sync that to the CPUID view
that the guest has.
And then another commit to enable x2apic emulation by default even for
kernel-irqchip=off + re-introducing provided by QEMU enlightenments in a more
functional form to signal that the x2apic can be used. I'm not aware of the
actual vmware freq leaf being used though.
"whpx: x2apic emulation for kernel-irqchip=off follow-up" is rolled into this series.
"whpx: i386: reintroduce enlightenments for Windows 10" is a bugfix to make
x2APIC work as intended on Windows 10 without emulating an IOMMU.
And dependent on this series so included, CPUID intercepts finally...
However that's only supported starting from Windows 11/Server 2022.
Also ended up switching over Windows 10 to kernel-irqchip=off
by default due to PIC interrupt injection being broken.
Note: on Windows 10, pic=off is still not very useful on this
release on AMD CPUs as Linux will fail to find a calibration
source for the timer, as it doesn't understand the VMware TSC
frequency leaf. On Intel, Linux will get the TSC frequency from
CPUID instead and boot up fine.
On Windows 11, Hyper-V enlightenments expose the necessary clock
info, so the scenario only affects hyperv=off.
That can be solved at a later time.
Some performance (or lack thereof...) numbers:
On a Ryzen 7 8700GE with a Windows 10 VM running with KVM in nested virt, with
kernel-irqchip=off for the virt Alpine Linux x86_64 ISO (3.23.3) with -smp cores=2,
boot times as reported through dmesg:
- QEMU 10.2: 83 seconds
- QEMU 10.2 with a single core: 18.1 seconds
- as of this series, x2apic forced off: 29 seconds
- as of this series, out of the box: 18 seconds
- with kernel-irqchip=on and EDK2: 16.5 seconds
- with kernel-irqchip=off and EDK2: 9.5 seconds
- and with 1 core instead of two: 12.6 seconds
And with this series on a Windows 11 VM on the same hardware:
- kernel-irqchip=on: 6.5 seconds
- kernel-irqchip=on, x2apic forced off: 7.6 seconds
- kernel-irqchip=off: 8.3 seconds
- hyperv=off,kernel-irqchip=off: 7.6 seconds... which is faster,
so the absence of enlightenment support on Windows 10 doesn't explain things...
With kernel-irqchip=on on Windows 10, when booting with SeaBIOS, it gets stuck in
syslinux due to PIC interrupt injection being broken there. That can be counted
as an infinite boot time (?).
checkpatch false positives:
ERROR: spaces required around that '*' (ctx:WxV)
+ UINT32 Ecx, WHV_CPUID_OUTPUT *CpuidOutput))
Not a multiplication but a pointer reference.
ERROR: space prohibited after that '&' (ctx:ExW)
+ & CPUID_7_0_EDX_CET_IBT) {
^
ERROR: space prohibited after that '&' (ctx:ExW)
+ & CPUID_7_0_ECX_CET_SHSTK) {
^
Because it's multiline.
Changes in v6:
- kernel-irqchip=off fix: re-registering the interrupt window
when the existing one has too low of a TPR value.
Folded into "whpx: i386: kernel-irqchip=off fixes".
- I/O port access fast path cleanup commit added at the end
The path relied on a side effect of whpx_get_reg instead of
something cleaner.
- Use the CR8 register provided by the hypervisor only when
kernel-irqchip=off. Rely on the APIC state synchronisation
otherwise.
This fixes some register sync errors that surfaced when setting
the TPR values properly.
Changes in v5/v1 with rename:
- kernel-irqchip=off fixes, notably making 64-bit Windows bootable
- switching over Windows 10 to kernel-irqchip=off by default
Changes in v4:
- Ugh for a revision sent quickly, unbreaking the arm64 build...
- and making checkpatch happier to some extent
Changes in v3:
- Fixing CPUID intercepts so that QEMU CPU models work fine now, instead
of the partial intercept that was present in QEMU 10.2
- cleanups
Changes in v2:
- GCC warned when a variable name was re-used within a different (but overlapping)
scope in the same function. It also warned with a -Werror=maybe-uninitialized for
the MSR write case. Address those
- make the in-KVM enlightenments path available on Windows 11 too when -M hyperv=off.
Mohamed Mediouni (11):
target/i386: emulate: include name of unhandled instruction
whpx: i386: x2apic emulation
whpx: i386: wire up feature probing
whpx: i386: disable TbFlushHypercalls for emulated LAPIC
whpx: i386: enable x2apic by default for user-mode LAPIC
whpx: i386: reintroduce enlightenments for Windows 10
whpx: i386: introduce proper cpuid support
whpx: i386: kernel-irqchip=off fixes
whpx: i386: use WHvX64RegisterCr8 only when kernel-irqchip=off
whpx: i386: disable kernel-irqchip on Windows 10 when PIC enabled
whpx: i386: IO port fast path cleanup
accel/whpx/whpx-common.c | 2 +
include/system/whpx-common.h | 1 +
include/system/whpx-internal.h | 10 +
target/arm/whpx/whpx-all.c | 1 +
target/i386/cpu.c | 25 ++
target/i386/emulate/x86_emu.c | 4 +-
target/i386/whpx/whpx-all.c | 467 ++++++++++++++++++++++++++-------
target/i386/whpx/whpx-i386.h | 4 +
8 files changed, 418 insertions(+), 96 deletions(-)
create mode 100644 target/i386/whpx/whpx-i386.h
--
2.50.1 (Apple Git-155)