Commit 7aeae027 authored by Ke Guo's avatar Ke Guo Committed by Sean Christopherson
Browse files

KVM: SVM: Use kvm_pat_valid() directly instead of kvm_mtrr_valid() 



Use kvm_pat_valid() directly instead of bouncing through kvm_mtrr_valid().
The PAT is not an MTRR, and kvm_mtrr_valid() just redirects to
kvm_pat_valid(), i.e. is exempt from KVM's "zap SPTEs" logic that's
needed to honor guest MTRRs when the VM has a passthrough device with
non-coherent DMA (KVM does NOT set "ignore guest PAT" in this case, and so
enables hardware virtualization of the guest's PAT, i.e. doesn't need to
manually emulate the PAT memtype).

Signed-off-by: default avatarKe Guo <guoke@uniontech.com>
[sean: massage changelog]
Reviewed-by: default avatarKai Huang <kai.huang@intel.com>
Link: https://lore.kernel.org/r/20230511233351.635053-3-seanjc@google.com


Signed-off-by: default avatarSean Christopherson <seanjc@google.com>
parent a33ba1bf

arch/x86/kvm/svm/svm.c

+1 −1
Original line number Diff line number Diff line
@@ -2939,7 +2939,7 @@ static int svm_set_msr(struct kvm_vcpu *vcpu, struct msr_data *msr) 


		break;

	case MSR_IA32_CR_PAT:

		if (!kvm_mtrr_valid(vcpu, MSR_IA32_CR_PAT, data))

		if (!kvm_pat_valid(data))

			return 1;

		vcpu->arch.pat = data;

		svm->vmcb01.ptr->save.g_pat = data;