When unmapping VM_HUGETLB mappings, vm flags need to be updated. Since
the vmas have been detached, so it sounds safe to update vm flags with
read mmap_sem.

Cc: Michal Hocko <mho...@kernel.org>
Cc: Vlastimil Babka <vba...@suse.cz>
Signed-off-by: Yang Shi <yang....@linux.alibaba.com>
---
 mm/mmap.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/mm/mmap.c b/mm/mmap.c
index e92f680..3b9f734 100644
--- a/mm/mmap.c
+++ b/mm/mmap.c
@@ -2812,7 +2812,6 @@ static int do_munmap_zap_rlock(struct mm_struct *mm, 
unsigned long start,
 
        /*
         * Unmapping vmas, which have:
-        *   VM_HUGETLB or
         *   VM_PFNMAP or
         *   uprobes
         * need get done with write mmap_sem held since they may update
@@ -2821,7 +2820,7 @@ static int do_munmap_zap_rlock(struct mm_struct *mm, 
unsigned long start,
        for (vma = start_vma; vma && vma->vm_start < end; vma = vma->vm_next) {
                if ((vma->vm_file &&
                    has_uprobes(vma, vma->vm_start, vma->vm_end)) ||
-                   (vma->vm_flags & (VM_HUGETLB | VM_PFNMAP)))
+                   (vma->vm_flags & VM_PFNMAP))
                        goto regular_path;
        }
 
-- 
1.8.3.1

Reply via email to