On Tue, Sep 24, 2019 at 10:47:50AM -0400, Igor Mammedov wrote: [...]
> @@ -2877,6 +2912,7 @@ static bool kvm_accel_has_memory(MachineState *ms, > AddressSpace *as, > > for (i = 0; i < kvm->nr_as; ++i) { > if (kvm->as[i].as == as && kvm->as[i].ml) { > + size = MIN(kvm_max_slot_size, size); > return NULL != kvm_lookup_matching_slot(kvm->as[i].ml, > start_addr, size); > } Ideally we could also check that the whole (start_addr, size) region is covered by KVM memslots here, but with current code I can't think of a case where the result doesn't match with only checking the 1st memslot. So I assume it's fine. Reviewed-by: Peter Xu <pet...@redhat.com> -- Peter Xu