]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
mm/mmap: Change do_brk_munmap() to use do_mas_align_munmap()
authorLiam R. Howlett <Liam.Howlett@Oracle.com>
Tue, 1 Dec 2020 02:30:04 +0000 (21:30 -0500)
committerLiam R. Howlett <Liam.Howlett@oracle.com>
Wed, 20 Oct 2021 19:22:34 +0000 (15:22 -0400)
do_brk_munmap() has already aligned the address and has a maple tree
state to be used.  Use the new do_mas_align_munmap() to avoid
unnecessary alignment and error checks.

Signed-off-by: Liam R. Howlett <Liam.Howlett@Oracle.com>
mm/mmap.c

index 1319098c43b97b376330f4fd7fc692c23674e5b5..45721a492ef64e6e64c4b92eab6d452118201b60 100644 (file)
--- a/mm/mmap.c
+++ b/mm/mmap.c
@@ -3013,13 +3013,16 @@ static int do_brk_munmap(struct ma_state *mas, struct vm_area_struct *vma,
        struct mm_struct *mm = vma->vm_mm;
        struct vm_area_struct unmap;
        unsigned long unmap_pages;
-       int ret = 1;
+       int ret;
 
        arch_unmap(mm, newbrk, oldbrk);
 
-       if (likely(vma->vm_start >= newbrk)) { // remove entire mapping(s)
+       if (likely((vma->vm_end < oldbrk) ||
+                  ((vma->vm_start == newbrk) && (vma->vm_end == oldbrk)))) {
+               // remove entire mapping(s)
                mas_lock(mas);
-               ret = do_mas_munmap(mas, mm, newbrk, oldbrk-newbrk, uf, true);
+               ret = do_mas_align_munmap(mas, vma, mm, newbrk, oldbrk, uf,
+                                         true);
                if (ret != 1)
                        mas_unlock(mas);
                goto munmap_full_vma;