[PATCH v5 0/6] mm/mglru: clean up folio counters and flag usage
From: Kairui Song via B4 Relay
Date: Wed Sep 02 2026 - 05:59:59 EST
This is a cleanup series separated out from the MGLRU-FG series [1]. As
that series is getting too long in following updates, seperate out the
clean up part for easier review and merge.
No feature change is intended, except one bugfix. It mostly replaces
the open-coded bit operations scattered throughout the MGLRU code with
new helpers, with proper kdocs, sanity debug checks, and hardens a few
MGLRU functions.
A subtle generation counter leak is also found during the refactoring
and the fix is included.
Also collected review feedbacks on the cleanup part from the posted
series.
Link: https://lore.kernel.org/linux-mm/20260804-mglru-fg-v1-0-4d8dad39dad6@xxxxxxxxxxx/ [1]
Signed-off-by: Kairui Song <kasong@xxxxxxxxxxx>
---
Changes in v5:
- Change atomic long to plain long in patch 1, and adjust commit message.
- Rename is_file to type, and avoid touching folio_is_file_lru in patch 7/7.
- Link to v4: https://patch.msgid.link/20260831-mglru-flags-cleanup-v4-0-2d15dde0d7ee@xxxxxxxxxxx
Changes in v4:
- Rename lru_gen_set_flags/lru_gen_from_flags to
lru_set_gen_flags/lru_get_gen_flags, similiar with lru refs helpers,
as suggested by [ Barry Song ].
- Added some comments to cover concerned raised by [ Barry Song ] and [
Baolin Wang ].
- A little code shuffling and better sanity check for concerns raised by
[ Ridong Chen ].
- Collect tags.
- Link to v3: https://patch.msgid.link/20260826-mglru-flags-cleanup-v3-0-d9f1c75549c8@xxxxxxxxxxx
Changes in v3:
- Set PG_workingset before setting LRU refs bit to avoid any potential
under report of folio tier.
- Migrate folio's referenced status before PG_update so lockless readers
won't set stale referenced status.
- Link to v2: https://patch.msgid.link/20260824-mglru-flags-cleanup-v2-0-0104132114ae@xxxxxxxxxxx
Changes in v2:
- Dropped the redundant LRU_GEN_MAX macro.
- Renamed folio_migrate_refs() to folio_migrate_lru_refs() and reworded
its commit message to describe the referenced state being transferred.
- Rewrote patch 6's commit message with a detailed analysis of the
anon/file accounting race.
- Dropped patch 7, it doesn't really fix anything, and might be in
conflict with other updating series.
- Collected Reviewed-by tags.
- Link to v1: https://patch.msgid.link/20260818-mglru-flags-cleanup-v1-0-8dbbdac0d28c@xxxxxxxxxxx
---
Kairui Song (6):
mm/memcontrol: move the lru_zone_size sanity check to the reader side
mm/mglru: introduce helpers for manipulating gen and refs flags
mm/migrate: copy all referenced state via folio_migrate_lru_refs
mm/mglru: move max_seq read into walk_update_folio
mm/mglru: use explicit tier range in read_ctrl_pos()
mm/mglru: fix potential generation folio number leak
include/linux/memcontrol.h | 9 +++-
include/linux/mm_inline.h | 115 +++++++++++++++++++++++++++++++++++++++-----
include/linux/mmzone.h | 3 ++
mm/folio.c | 19 ++++----
mm/memcontrol.c | 18 +------
mm/migrate.c | 6 +--
mm/vmscan.c | 117 +++++++++++++++++++++++----------------------
7 files changed, 188 insertions(+), 99 deletions(-)
---
base-commit: 893f78beb0477671991ccaae87df75acc484162f
change-id: 20260818-mglru-flags-cleanup-cc49cfad654f
Best regards,
--
Kairui Song <kasong@xxxxxxxxxxx>