On 08/09/2012 01:34 PM, Takuya Yoshikawa wrote: > On Tue, 7 Aug 2012 12:57:13 +0200 > Alexander Graf <agraf@xxxxxxx> wrote: > >> +struct kvm_memory_slot *hva_to_memslot(struct kvm *kvm, hva_t hva) >> +{ >> + struct kvm_memslots *slots = kvm_memslots(kvm); >> + struct kvm_memory_slot *memslot; >> + >> + kvm_for_each_memslot(memslot, slots) >> + if (hva >= memslot->userspace_addr && >> + hva < memslot->userspace_addr + memslot->npages) >> + return memslot; >> + >> + return NULL; >> +} > > Can't we have two memory slots which contain that hva? > I thought that's why hva handler had to check all slots. We can and do. Good catch. -- error compiling committee.c: too many arguments to function _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/cucslists/listinfo/kvmarm