mglru: mm/vmscan.c: fix imprecise comments
authorYu Zhao <yuzhao@google.com>
Wed, 28 Sep 2022 18:46:20 +0000 (12:46 -0600)
committerAndrew Morton <akpm@linux-foundation.org>
Fri, 7 Oct 2022 21:28:40 +0000 (14:28 -0700)
Link: https://lkml.kernel.org/r/YzSWfFI+MOeb1ils@google.com
Signed-off-by: Yu Zhao <yuzhao@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/vmscan.c

index 3240d5dd7784aa82feacdccad45f02abe03bf465..04d8b88e521648752342e3319d3136a02a838a5a 100644 (file)
@@ -5076,7 +5076,7 @@ static bool should_abort_scan(struct lruvec *lruvec, unsigned long seq,
        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 lruvec *lruvec, unsigned long seq,
 
        /*
         * 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;