]> www.infradead.org Git - users/jedix/linux-maple.git/commit
KVM: VMX: Drop support for forcing UC memory when guest CR0.CD=1
authorSean Christopherson <seanjc@google.com>
Sat, 9 Mar 2024 01:09:26 +0000 (17:09 -0800)
committerSean Christopherson <seanjc@google.com>
Wed, 5 Jun 2024 15:13:14 +0000 (08:13 -0700)
commite1548088ff544e4a4ae2c960bccf0eeff3a46297
tree44eed91ec9a6107f900f487e95288456fcd86e0b
parent0a7b73559b39497fae1b7b3d4bab69895097c37e
KVM: VMX: Drop support for forcing UC memory when guest CR0.CD=1

Drop KVM's emulation of CR0.CD=1 on Intel CPUs now that KVM no longer
honors guest MTRR memtypes, as forcing UC memory for VMs with
non-coherent DMA only makes sense if the guest is using something other
than PAT to configure the memtype for the DMA region.

Furthermore, KVM has forced WB memory for CR0.CD=1 since commit
fb279950ba02 ("KVM: vmx: obey KVM_QUIRK_CD_NW_CLEARED"), and no known
VMM in existence disables KVM_X86_QUIRK_CD_NW_CLEARED, let alone does
so with non-coherent DMA.

Lastly, commit fb279950ba02 ("KVM: vmx: obey KVM_QUIRK_CD_NW_CLEARED") was
from the same author as commit b18d5431acc7 ("KVM: x86: fix CR0.CD
virtualization"), and followed by a mere month.  I.e. forcing UC memory
was likely the result of code inspection or perhaps misdiagnosed failures,
and not the necessitate by a concrete use case.

Update KVM's documentation to note that KVM_X86_QUIRK_CD_NW_CLEARED is now
AMD-only, and to take an erratum for lack of CR0.CD virtualization on
Intel.

Tested-by: Xiangfei Ma <xiangfeix.ma@intel.com>
Tested-by: Yongwei Ma <yongwei.ma@intel.com>
Link: https://lore.kernel.org/r/20240309010929.1403984-3-seanjc@google.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
Documentation/virt/kvm/api.rst
Documentation/virt/kvm/x86/errata.rst
arch/x86/kvm/vmx/vmx.c
arch/x86/kvm/x86.c