The patch titled
Subject: mglru: mm/vmscan.c: fix imprecise comments
has been added to the -mm mm-unstable branch. Its filename is
mglru-mm-vmscanc-fix-imprecise-comments.patch
This patch will shortly appear at
https://git.kernel.org/pub/scm/linux/kernel/git/akpm/25-new.git/tree/patches/mglru-mm-vmscanc-fix-imprecise-comments.patch
This patch will later appear in the mm-unstable branch at
git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm
Before you just go and hit "reply", please:
a) Consider who else should be cc'ed
b) Prefer to cc a suitable mailing list as well
c) Ideally: find the original patch on the mailing list and do a
reply-to-all to that, adding suitable additional cc's
*** Remember to use Documentation/process/submit-checklist.rst when testing your code ***
The -mm tree is included into linux-next via the mm-everything
branch at git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm
and is updated there every 2-3 working days
------------------------------------------------------
From: Yu Zhao <yuzhao@xxxxxxxxxx>
Subject: mglru: mm/vmscan.c: fix imprecise comments
Date: Wed, 28 Sep 2022 12:46:20 -0600
Link: https://lkml.kernel.org/r/YzSWfFI+MOeb1ils@xxxxxxxxxx
Signed-off-by: Yu Zhao <yuzhao@xxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
mm/vmscan.c | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
--- a/mm/vmscan.c~mglru-mm-vmscanc-fix-imprecise-comments
+++ a/mm/vmscan.c
@@ -5076,7 +5076,7 @@ static bool should_abort_scan(struct lru
DEFINE_MAX_SEQ(lruvec);
if (!current_is_kswapd()) {
- /* age each memcg once to ensure fairness */
+ /* age each memcg at most once to ensure fairness */
if (max_seq - seq > 1)
return true;
@@ -5101,10 +5101,9 @@ static bool should_abort_scan(struct lru
/*
* A minimum amount of work was done under global memory pressure. For
- * kswapd, it may be overshooting. For direct reclaim, the target isn't
- * met, and yet the allocation may still succeed, since kswapd may have
- * caught up. In either case, it's better to stop now, and restart if
- * necessary.
+ * kswapd, it may be overshooting. For direct reclaim, the allocation
+ * may succeed if all suitable zones are somewhat safe. In either case,
+ * it's better to stop now, and restart later if necessary.
*/
for (i = 0; i <= sc->reclaim_idx; i++) {
unsigned long wmark;
_
Patches currently in -mm which might be from yuzhao@xxxxxxxxxx are
mm-mglru-dont-sync-disk-for-each-aging-cycle.patch
mglru-mm-vmscanc-fix-imprecise-comments.patch
