]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
arch/x86: Use maple tree iterators for vdso/vma
authorLiam R. Howlett <Liam.Howlett@Oracle.com>
Mon, 4 Jan 2021 19:30:25 +0000 (14:30 -0500)
committerLiam R. Howlett <Liam.Howlett@Oracle.com>
Thu, 24 Jun 2021 20:04:22 +0000 (16:04 -0400)
Signed-off-by: Liam R. Howlett <Liam.Howlett@Oracle.com>
arch/x86/entry/vdso/vma.c

index 235a5794296acbef611d65d2de8cbec7de9f4ed3..c0b160a9585d8264650129bd868b485986d56f80 100644 (file)
@@ -128,15 +128,19 @@ int vdso_join_timens(struct task_struct *task, struct time_namespace *ns)
        struct mm_struct *mm = task->mm;
        struct vm_area_struct *vma;
 
+       MA_STATE(mas, &mm->mm_mt, 0, 0);
+
        mmap_read_lock(mm);
+       rcu_read_lock();
 
-       for (vma = mm->mmap; vma; vma = vma->vm_next) {
+       mas_for_each(&mas, vma, ULONG_MAX) {
                unsigned long size = vma->vm_end - vma->vm_start;
 
                if (vma_is_special_mapping(vma, &vvar_mapping))
                        zap_page_range(vma, vma->vm_start, size);
        }
 
+       rcu_read_unlock();
        mmap_read_unlock(mm);
        return 0;
 }
@@ -354,6 +358,7 @@ int map_vdso_once(const struct vdso_image *image, unsigned long addr)
 {
        struct mm_struct *mm = current->mm;
        struct vm_area_struct *vma;
+       MA_STATE(mas, &mm->mm_mt, 0, 0);
 
        mmap_write_lock(mm);
        /*
@@ -363,13 +368,16 @@ int map_vdso_once(const struct vdso_image *image, unsigned long addr)
         * We could search vma near context.vdso, but it's a slowpath,
         * so let's explicitly check all VMAs to be completely sure.
         */
-       for (vma = mm->mmap; vma; vma = vma->vm_next) {
+       mas_lock(&mas);
+       mas_for_each(&mas, vma, ULONG_MAX) {
                if (vma_is_special_mapping(vma, &vdso_mapping) ||
                                vma_is_special_mapping(vma, &vvar_mapping)) {
+                       mas_unlock(&mas);
                        mmap_write_unlock(mm);
                        return -EEXIST;
                }
        }
+       mas_unlock(&mas);
        mmap_write_unlock(mm);
 
        return map_vdso(image, addr);