lkml.org 
[lkml]   [2022]   [Aug]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject[PATCH v2 0/2] mm: Remember a/d bits for migration entries
Date
v2:
- Fix build for !CONFIG_SWAP [syzbot]
- Carry over dirty bit too [Nadav]

rfc: https://lore.kernel.org/all/20220729014041.21292-1-peterx@redhat.com
v1: https://lore.kernel.org/all/20220803012159.36551-1-peterx@redhat.com

Problem
=======

When migrate a page, right now we always mark the migrated page as old &
clean.

However that could lead to at least two problems:

(1) We lost the real hot/cold information while we could have persisted.
That information shouldn't change even if the backing page is changed
after the migration,

(2) There can be always extra overhead on the immediate next access to
any migrated page, because hardware MMU needs cycles to set the young
bit again for reads, and dirty bits for write, as long as the
hardware MMU supports these bits.

Many of the recent upstream works showed that (2) is not something trivial
and actually very measurable. In my test case, reading 1G chunk of memory
- jumping in page size intervals - could take 99ms just because of the
extra setting on the young bit on a generic x86_64 system, comparing to 4ms
if young set.

This issue is originally reported by Andrea Arcangeli.

Solution
========

To solve this problem, this patchset tries to remember the young/dirty bits
in the migration entries and carry them over when recovering the ptes.

We have the chance to do so because in many systems the swap offset is not
really fully used. Migration entries use swp offset to store PFN only,
while the PFN is normally not as large as swp offset and normally smaller.
It means we do have some free bits in swp offset that we can use to store
things like A/D bits, and that's how this series tried to approach this
problem.

max_swapfile_size() is used here to detect per-arch offset length in swp
entries. We'll automatically remember the A/D bits when we find that we
have enough swp offset field to keep both the PFN and the extra bits.

This series is majorly solving the bit lost issue only for the migration
procedure. Besides that, a few topics to mention related to this series:

(1) Page Idle Tracking

Before this series, idle tracking can cause false negative if an accessed
page got migrated, since after migration the young bit will get lost.
After this series, it'll be better in that after migration young bit will
be persisted, so it'll be able to be detected correctly by page idle
logic when walking the pgtable.

However there's still nothing done when page idle reset was carried out
during migration procedure in progress, but that should be a separate
topic to be addressed (e.g. to teach rmap pgtable walk code to be able to
walk with both present ptes and migration ptes).

(2) MADV_COLD/MADV_FREE

This series doesn't teach madvise() code to recognize the new entries
with reasons.

Currently MADV_COLD is not handled for migration entries containing A
bit. Logically A bit should be dropped when colding a page, but here the
more important thing is probably LRU operations which is still missing.
It may justify that it is not a major scenario for COLD on migration
entries.

Similar thing applies to MADV_FREE: logically we should consider dropping
migration entries as a whole if it is found. In all cases, this series
didn't cover any of the cases above assuming they'll be either kept as-is
or be addressed in separate patchset.

Tests
=====

After the patchset applied, the immediate read access test [1] of above 1G
chunk after migration can shrink from 99ms to 4ms. The test is done by
moving 1G pages from node 0->1->0 then read it in page size jumps. The
test is with Intel(R) Xeon(R) CPU E5-2630 v4 @ 2.20GHz.

Similar effect can also be measured when writting the memory the 1st time
after migration.

After applying the patchset, both initial immediate read/write after page
migrated will perform similarly like before migration happened.

Patch Layout
============

Patch 1: Add swp_offset_pfn() and apply to all pfn swap entries, we should
also stop treating swp_offset() as PFN anymore because it can
contain more information starting from next patch.
Patch 2: The core patch to remember young/dirty bit in swap offsets.

Please review, thanks.

[1] https://github.com/xzpeter/clibs/blob/master/misc/swap-young.c

Peter Xu (2):
mm/swap: Add swp_offset_pfn() to fetch PFN from swap entry
mm: Remember young/dirty bit for page migrations

arch/arm64/mm/hugetlbpage.c | 2 +-
include/linux/swapops.h | 126 ++++++++++++++++++++++++++++++++++--
mm/hmm.c | 2 +-
mm/huge_memory.c | 26 +++++++-
mm/memory-failure.c | 2 +-
mm/migrate.c | 6 +-
mm/migrate_device.c | 4 ++
mm/page_vma_mapped.c | 6 +-
mm/rmap.c | 5 +-
9 files changed, 163 insertions(+), 16 deletions(-)

--
2.32.0

\
 
 \ /
  Last update: 2022-08-04 22:40    [W:1.327 / U:0.552 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site