On Tue, Aug 30, 2022 at 4:45 PM Isaku Yamahata <isaku.yamahata@xxxxxxxxx> wrote: > > On Fri, Aug 26, 2022 at 04:12:21PM -0700, > David Matlack <dmatlack@xxxxxxxxxx> wrote: > > > > @@ -4185,15 +4181,25 @@ static int __kvm_faultin_pfn(struct kvm_vcpu *vcpu, struct kvm_page_fault *fault > > fault->pfn = __gfn_to_pfn_memslot(slot, fault->gfn, false, NULL, > > fault->write, &fault->map_writable, > > &fault->hva); > > + > > return RET_PF_CONTINUE; > > } > > nit: unnecessary code churn. > Otherwise, code looks good to me. My mistake, thanks for the catch.