On Tue, Jul 09, 2019 at 10:24:01AM -0700, Sean Christopherson wrote: > diff --git a/arch/x86/kvm/vmx/nested.c b/arch/x86/kvm/vmx/nested.c > index 6dd20e0ad2fa..3bfce3f507c7 100644 > --- a/arch/x86/kvm/vmx/nested.c > +++ b/arch/x86/kvm/vmx/nested.c > @@ -950,9 +960,9 @@ static int nested_vmx_load_cr3(struct kvm_vcpu *vcpu, unsigned long cr3, bool ne > u32 *entry_failure_code) > { > if (cr3 != kvm_read_cr3(vcpu) || (!nested_ept && pdptrs_changed(vcpu))) { > - if (!nested_cr3_valid(vcpu, cr3)) { > + if (CC(!nested_cr3_valid(vcpu, cr3))) { > *entry_failure_code = ENTRY_FAIL_DEFAULT; > - return -EINVAL; > + return 1; Gah, botched the rebase, I'll send a v3.