mirror of
https://github.com/followmsi/android_kernel_google_msm.git
synced 2024-11-06 23:17:41 +00:00
KVM: Fix user memslot overlap check
commit 5419369ed6
upstream.
Prior to memory slot sorting this loop compared all of the user memory
slots for overlap with new entries. With memory slot sorting, we're
just checking some number of entries in the array that may or may not
be user slots. Instead, walk all the slots with kvm_for_each_memslot,
which has the added benefit of terminating early when we hit the first
empty slot, and skip comparison to private slots.
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
91534f7365
commit
9d4c74b862
1 changed files with 5 additions and 8 deletions
|
@ -693,8 +693,7 @@ int __kvm_set_memory_region(struct kvm *kvm,
|
|||
int r;
|
||||
gfn_t base_gfn;
|
||||
unsigned long npages;
|
||||
unsigned long i;
|
||||
struct kvm_memory_slot *memslot;
|
||||
struct kvm_memory_slot *memslot, *slot;
|
||||
struct kvm_memory_slot old, new;
|
||||
struct kvm_memslots *slots, *old_memslots;
|
||||
|
||||
|
@ -741,13 +740,11 @@ int __kvm_set_memory_region(struct kvm *kvm,
|
|||
|
||||
/* Check for overlaps */
|
||||
r = -EEXIST;
|
||||
for (i = 0; i < KVM_MEMORY_SLOTS; ++i) {
|
||||
struct kvm_memory_slot *s = &kvm->memslots->memslots[i];
|
||||
|
||||
if (s == memslot || !s->npages)
|
||||
kvm_for_each_memslot(slot, kvm->memslots) {
|
||||
if (slot->id >= KVM_MEMORY_SLOTS || slot == memslot)
|
||||
continue;
|
||||
if (!((base_gfn + npages <= s->base_gfn) ||
|
||||
(base_gfn >= s->base_gfn + s->npages)))
|
||||
if (!((base_gfn + npages <= slot->base_gfn) ||
|
||||
(base_gfn >= slot->base_gfn + slot->npages)))
|
||||
goto out_free;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue