[kvm-commits] [PATCH] KVM: Unmap kernel-allocated memory on slot destruction

[Avi Kivity]

From: Izik Eidus <[EMAIL PROTECTED]>

kvm_vm_ioctl_set_memory_region() is able to remove memory in addition to
adding it.  Therefore when using kernel swapping support for old userspaces,
we need to munmap the memory if the user request to remove it

Signed-off-by: Izik Eidus <[EMAIL PROTECTED]>
Signed-off-by: Avi Kivity <[EMAIL PROTECTED]>

diff --git a/drivers/kvm/kvm.h b/drivers/kvm/kvm.h
index f7181a4..12de42c 100644
--- a/drivers/kvm/kvm.h
+++ b/drivers/kvm/kvm.h
@@ -353,6 +353,7 @@ struct kvm_memory_slot {
        unsigned long *rmap;
        unsigned long *dirty_bitmap;
        unsigned long userspace_addr;
+       int user_alloc;
 };
 
 struct kvm {
diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c
index e67bc7c..df2feab 100644
--- a/drivers/kvm/kvm_main.c
+++ b/drivers/kvm/kvm_main.c
@@ -713,6 +713,7 @@ static int kvm_vm_ioctl_set_memory_region(struct kvm *kvm,
 
                memset(new.rmap, 0, npages * sizeof(*new.rmap));
 
+               new.user_alloc = user_alloc;
                if (user_alloc)
                        new.userspace_addr = mem->userspace_addr;
                else {
@@ -727,6 +728,19 @@ static int kvm_vm_ioctl_set_memory_region(struct kvm *kvm,
                        if (IS_ERR((void *)new.userspace_addr))
                                goto out_unlock;
                }
+       } else {
+               if (!old.user_alloc && old.rmap) {
+                       int ret;
+
+                       down_write(&current->mm->mmap_sem);
+                       ret = do_munmap(current->mm, old.userspace_addr,
+                                       old.npages * PAGE_SIZE);
+                       up_write(&current->mm->mmap_sem);
+                       if (ret < 0)
+                               printk(KERN_WARNING
+                                      "kvm_vm_ioctl_set_memory_region: "
+                                      "failed to munmap memory\n");
+               }
        }
 
        /* Allocate page dirty bitmap if needed */

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
kvm-commits mailing list
kvm-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-commits