On 11/20/2011 01:26 PM, Avi Kivity wrote: > > > > int kvm_vcpu_init(struct kvm_vcpu *vcpu, struct kvm *kvm, unsigned id); > > void kvm_vcpu_uninit(struct kvm_vcpu *vcpu); > > @@ -335,7 +340,14 @@ static inline struct kvm_memslots *kvm_memslots(struct kvm *kvm) > > static inline struct kvm_memory_slot * > > id_to_memslot(struct kvm_memslots *slots, int id) > > { > > - return &slots->memslots[id]; > > + int i; > > + > > + for (i = 0; i < KVM_MEM_SLOTS_NUM; i++) > > + if (slots->memslots[i].id == id) > > + return &slots->memslots[i]; > > + > > Is that in any hot path? we could make an array for doing this translation. Never mind, I see patch 6. I should have known you wouldn't leave it like that. -- 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