Commit e1b1fe45 authored Jun 16, 2025 by SeongJae Park Committed by Andrew Morton Jul 09, 2025

Revert "mm: make alloc_demote_folio externally invokable for migration"

This reverts commit a00ce85a.

Commit a00ce85a ("mm: make alloc_demote_folio externally invokable for
migration") was made to let DAMOS_MIGRATE_{HOT,COLD} call the function. 
But a previous commit made DAMOS_MIGRATE_{HOT,COLD} call
alloc_migration_target() instead.  Hence there are no more callers of the
function outside of vmscan.c.  Revert the commit to make the function
static again.

Link: https://lkml.kernel.org/r/20250616172346.67659-4-sj@kernel.org


Signed-off-by: SeongJae Park <sj@kernel.org>
Reviewed-by: Joshua Hahn <joshua.hahnjy@gmail.com>
Cc: David Hildenbrand <david@redhat.com>
Cc: Honggyu Kim <honggyu.kim@sk.com>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Cc: Michal Hocko <mhocko@kernel.org>
Cc: Qi Zheng <zhengqi.arch@bytedance.com>
Cc: Shakeel Butt <shakeel.butt@linux.dev>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

parent 29ea0409

mm/internal.h

+0 −1

Original line number	Diff line number	Diff line
		@@ -1226,7 +1226,6 @@ extern unsigned long __must_check vm_mmap_pgoff(struct file *, unsigned long,
		unsigned long, unsigned long);

		extern void set_pageblock_order(void);
		struct folio alloc_demote_folio(struct folio src, unsigned long private);
		unsigned long reclaim_pages(struct list_head *folio_list);
		unsigned int reclaim_clean_pages_from_list(struct zone *zone,
		struct list_head *folio_list);

mm/vmscan.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -1006,7 +1006,8 @@ static void folio_check_dirty_writeback(struct folio *folio,
		mapping->a_ops->is_dirty_writeback(folio, dirty, writeback);
		}

		struct folio alloc_demote_folio(struct folio src, unsigned long private)
		static struct folio alloc_demote_folio(struct folio src,
		unsigned long private)
		{
		struct folio *dst;
		nodemask_t *allowed_mask;