lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20250627140932.1278169c50ba873937444599@linux-foundation.org>
Date: Fri, 27 Jun 2025 14:09:32 -0700
From: Andrew Morton <akpm@...ux-foundation.org>
To: <zhongjinji@...or.com>
Cc: <linux-mm@...ck.org>, <yuzhao@...gle.com>,
 <linux-kernel@...r.kernel.org>, <yipengxiang@...or.com>,
 <liulu.liu@...or.com>, <feng.han@...or.com>, Axel Rasmussen
 <axelrasmussen@...gle.com>, Yuanchu Xie <yuanchu@...gle.com>, Greg Thelen
 <gthelen@...gle.com>
Subject: Re: [PATCH] mm: vmscan: Page scanning depends on swappiness and
 refault

On Sat, 28 Jun 2025 00:26:06 +0800 <zhongjinji@...or.com> wrote:

> From: z00025326 <z00025326@...onor.com>
> 
> The current MGLRU aging strategy isn’t flexible enough. For example,
> when the system load and pressure are low, reclaiming more anonymous
> pages might be better. But when the system is under heavy pressure,
> enough file pages are needed for quick reclaim. Right now, when MGLRU
> is on, changing the swappiness value doesn’t really let you prioritize
> reclaiming certain types of pages in different situations.
> 
> This patch changes the aging strategy to adjust the reclaim ratio based
> on swappiness and refault values, allowing anonymous and file pages to
> age separately. and it can prioritize reclaiming certain types of pages
> and doesn’t have to wait until all the oldest pages are reclaimed before
> moving on to the next aging generation.
> 
> ...
>
>  include/linux/mm_inline.h |  19 +-
>  include/linux/mmzone.h    |  13 +-
>  include/linux/swap.h      |   1 +
>  mm/vmscan.c               | 797 ++++++++++++++++++++++----------------
>  mm/workingset.c           |  10 +-
>  5 files changed, 494 insertions(+), 346 deletions(-)
> 

(replying to https://lkml.kernel.org/r/20250627162606.30609-1-zhongjinji@honor.com)

That is one big patch!  I'll avoid it at this time, see what the
reviewers say.

Have you reviewed Documentation/admin-guide/mm/multigen_lru.rst to see
if any updates are appropriate?

I expect people are going to want to see a lot of detail about the
runtime effects of this change.  What sort of workloads are being
targeted, what were the effects on testing results.  What sort of
workloads might be harmed by the change and what were the effects on
testing results for those, etcetera.


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ