On 03/29/2012 11:22 AM, Xiao Guangrong wrote: > Split FNAME(prefetch_invalid_gpte) to check gpte independently which will > be used in the later patch > > > -static bool FNAME(prefetch_invalid_gpte)(struct kvm_vcpu *vcpu, > - struct kvm_mmu_page *sp, u64 *spte, > - pt_element_t gpte) > +static bool FNAME(invalid_gpte)(struct kvm_vcpu *vcpu, pt_element_t gpte) > { > if (is_rsvd_bits_set(&vcpu->arch.mmu, gpte, PT_PAGE_TABLE_LEVEL)) > - goto no_present; > + return true; > > if (!is_present_gpte(gpte)) > - goto no_present; > + return true; > > if (!(gpte & PT_ACCESSED_MASK)) > - goto no_present; > + return true; > > return false; A comment (or a name change) is needed to indicate that we're not checking for an invalid gpte, just invalid for caching in an spte. > +} > > -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html