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 <da...@redhat.com> > --- > 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 <z...@nvidia.com> -- Best Regards, Yan, Zi