Thread (87 messages) 87 messages, 5 authors, 2025-06-30

Re: [PATCH RFC 18/29] mm: remove __folio_test_movable()

From: Zi Yan <ziy@nvidia.com>
Date: 2025-06-20 20:41:46
Also in: linux-doc, linux-fsdevel, linux-mm, linuxppc-dev, lkml

On 18 Jun 2025, at 13:40, David Hildenbrand wrote:
Convert to page_has_movable_ops(). While at it, cleanup relevant code
a bit.

The data_race() in migrate_folio_unmap() is questionable: we already
hold a page reference, and concurrent modifications can no longer
happen (iow: __ClearPageMovable() no longer exists). Drop it for now,
we'll rework page_has_movable_ops() soon either way to no longer
rely on page->mapping.

Wherever we cast from folio to page now is a clear sign that this
code has to be decoupled.

Signed-off-by: David Hildenbrand <redacted>
---
 include/linux/page-flags.h |  6 ------
 mm/migrate.c               | 43 ++++++++++++--------------------------
 mm/vmscan.c                |  6 ++++--
 3 files changed, 17 insertions(+), 38 deletions(-)
Reviewed-by: Zi Yan <ziy@nvidia.com>

--
Best Regards,
Yan, Zi
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help