On Mon, Jun 30, 2025 at 02:59:50PM +0200, David Hildenbrand wrote: > Let's factor it out, simplifying the calling code. > > The assumption is that flush_dcache_page() is not required for > movable_ops pages: as documented for flush_dcache_folio(), it really > only applies when the kernel wrote to pagecache pages / pages in > highmem. movable_ops callbacks should be handling flushing > caches if ever required. > > Note that we can now change folio_mapping_flags() to folio_test_anon() > to make it clearer, because movable_ops pages will never take that path. > > Reviewed-by: Zi Yan <ziy@xxxxxxxxxx> > Signed-off-by: David Hildenbrand <david@xxxxxxxxxx> > --- Looks correct to me. Reviewed-by: Harry Yoo <harry.yoo@xxxxxxxxxx> -- Cheers, Harry / Hyeonggon
