[folded-merged] mm-vmscan-use-vma-iterator-instead-of-vm_next-fix.patch removed from -mm tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The quilt patch titled
     Subject: mm/vmscan: use the proper VMA iterator
has been removed from the -mm tree.  Its filename was
     mm-vmscan-use-vma-iterator-instead-of-vm_next-fix.patch

This patch was dropped because it was folded into mm-vmscan-use-vma-iterator-instead-of-vm_next.patch

------------------------------------------------------
From: Yu Zhao <yuzhao@xxxxxxxxxx>
Subject: mm/vmscan: use the proper VMA iterator
Date: Mon, 12 Sep 2022 14:01:28 -0600

When get_next_vma() finishes iterating VMAs within a range [start, end),
it expects args->vma to point the first VMA out of that range, if such a
VMA exists.  This allows its callers to continue the iteration with a new
range above the previous one, if those callers choose to.

for_each_vma_range() always sets args->vma to NULL after it's done.  This
may mislead those callers to conclude that there are no more VMAs, and in
turn they terminate their iterations prematurely.

This fix replaces for_each_vma_range() with for_each_vma() and explicitly
checks whether the next VMA is still within range, and if not, returns
false to indicate the current range has ended.  The callers may continue
with the next range if args->vma is not NULL.

Link: https://lkml.kernel.org/r/Yx+QGOgHg1Wk8tGK@xxxxxxxxxx
Signed-off-by: Yu Zhao <yuzhao@xxxxxxxxxx>
Cc: Liam Howlett <liam.howlett@xxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---

 mm/vmscan.c |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

--- a/mm/vmscan.c~mm-vmscan-use-vma-iterator-instead-of-vm_next-fix
+++ a/mm/vmscan.c
@@ -3779,7 +3779,10 @@ static bool get_next_vma(unsigned long m
 	VM_WARN_ON_ONCE(mask & size);
 	VM_WARN_ON_ONCE((start & mask) != (*vm_start & mask));
 
-	for_each_vma_range(vmi, args->vma, end) {
+	for_each_vma(vmi, args->vma) {
+		if (end && end <= args->vma->vm_start)
+			return false;
+
 		if (should_skip_vma(args->vma->vm_start, args->vma->vm_end, args))
 			continue;
 
_

Patches currently in -mm which might be from yuzhao@xxxxxxxxxx are

mm-x86-arm64-add-arch_has_hw_pte_young.patch
mm-x86-add-config_arch_has_nonleaf_pmd_young.patch
mm-vmscanc-refactor-shrink_node.patch
revert-include-linux-mm_inlineh-fold-__update_lru_size-into-its-sole-caller.patch
mm-multi-gen-lru-groundwork.patch
mm-multi-gen-lru-minimal-implementation.patch
mm-multi-gen-lru-exploit-locality-in-rmap.patch
mm-multi-gen-lru-support-page-table-walks.patch
mm-multi-gen-lru-optimize-multiple-memcgs.patch
mm-multi-gen-lru-kill-switch.patch
mm-multi-gen-lru-thrashing-prevention.patch
mm-multi-gen-lru-debugfs-interface.patch
mm-multi-gen-lru-admin-guide.patch
mm-multi-gen-lru-design-doc.patch
mm-vmscan-use-vma-iterator-instead-of-vm_next.patch




[Index of Archives]     [Kernel Archive]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux