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
