linux-toradex.git/mm/memory_hotplug.c, branch v6.17-rc1

mm: rename __PageMovable() to page_has_movable_ops()

2025-07-13T23:38:29+00:00

Let's make it clearer that we are talking about movable_ops pages.

While at it, convert a VM_BUG_ON to a VM_WARN_ON_ONCE_PAGE.

Link: https://lkml.kernel.org/r/20250704102524.326966-17-david@redhat.com
Signed-off-by: David Hildenbrand 
Reviewed-by: Zi Yan 
Reviewed-by: Lorenzo Stoakes 
Reviewed-by: Harry Yoo 
Cc: Alistair Popple 
Cc: Al Viro 
Cc: Arnd Bergmann 
Cc: Brendan Jackman 
Cc: Byungchul Park 
Cc: Chengming Zhou 
Cc: Christian Brauner 
Cc: Christophe Leroy 
Cc: Eugenio Pé rez 
Cc: Greg Kroah-Hartman 
Cc: Gregory Price 
Cc: "Huang, Ying" 
Cc: Jan Kara 
Cc: Jason Gunthorpe 
Cc: Jason Wang 
Cc: Jerrin Shaji George 
Cc: Johannes Weiner 
Cc: John Hubbard 
Cc: Jonathan Corbet 
Cc: Joshua Hahn 
Cc: Liam Howlett 
Cc: Madhavan Srinivasan 
Cc: Mathew Brost 
Cc: Matthew Wilcox (Oracle) 
Cc: Miaohe Lin 
Cc: Michael Ellerman 
Cc: "Michael S. Tsirkin" 
Cc: Michal Hocko 
Cc: Mike Rapoport 
Cc: Minchan Kim 
Cc: Naoya Horiguchi 
Cc: Nicholas Piggin 
Cc: Oscar Salvador 
Cc: Peter Xu 
Cc: Qi Zheng 
Cc: Rakie Kim 
Cc: Rik van Riel 
Cc: Sergey Senozhatsky 
Cc: Shakeel Butt 
Cc: Suren Baghdasaryan 
Cc: Vlastimil Babka 
Cc: Xuan Zhuo 
Cc: xu xin 
Signed-off-by: Andrew Morton

mm/page_isolation: remove migratetype parameter from more functions

2025-07-13T23:38:18+00:00

migratetype is no longer overwritten during pageblock isolation,
start_isolate_page_range(), has_unmovable_pages(), and
set_migratetype_isolate() no longer need which migratetype to restore
during isolation failure.

For has_unmoable_pages(), it needs to know if the isolation is for CMA
allocation, so adding PB_ISOLATE_MODE_CMA_ALLOC provide the information. 
At the same time change isolation flags to enum pb_isolate_mode
(PB_ISOLATE_MODE_MEM_OFFLINE, PB_ISOLATE_MODE_CMA_ALLOC,
PB_ISOLATE_MODE_OTHER).  Remove REPORT_FAILURE and check
PB_ISOLATE_MODE_MEM_OFFLINE, since only PB_ISOLATE_MODE_MEM_OFFLINE
reports isolation failures.

alloc_contig_range() no longer needs migratetype.  Replace it with a newly
defined acr_flags_t to tell if an allocation is for CMA.  So does
__alloc_contig_migrate_range().  Add ACR_FLAGS_NONE (set to 0) to indicate
ordinary allocations.

Link: https://lkml.kernel.org/r/20250617021115.2331563-7-ziy@nvidia.com
Signed-off-by: Zi Yan 
Reviewed-by: Vlastimil Babka 
Acked-by: David Hildenbrand 
Cc: Baolin Wang 
Cc: Brendan Jackman 
Cc: Johannes Weiner 
Cc: Kirill A. Shuemov 
Cc: Mel Gorman 
Cc: Michal Hocko 
Cc: Oscar Salvador 
Cc: Richard Chang 
Cc: Suren Baghdasaryan 
Signed-off-by: Andrew Morton

mm/page_isolation: remove migratetype from undo_isolate_page_range()

2025-07-13T23:38:18+00:00

Since migratetype is no longer overwritten during pageblock isolation,
undoing pageblock isolation no longer needs which migratetype to restore.

Link: https://lkml.kernel.org/r/20250617021115.2331563-6-ziy@nvidia.com
Signed-off-by: Zi Yan 
Acked-by: David Hildenbrand 
Reviewed-by: Vlastimil Babka 
Cc: Baolin Wang 
Cc: Brendan Jackman 
Cc: Johannes Weiner 
Cc: Kirill A. Shuemov 
Cc: Mel Gorman 
Cc: Michal Hocko 
Cc: Oscar Salvador 
Cc: Richard Chang 
Cc: Suren Baghdasaryan 
Signed-off-by: Andrew Morton

mm/page_alloc: add support for initializing pageblock as isolated

2025-07-13T23:38:17+00:00

MIGRATE_ISOLATE is a standalone bit, so a pageblock cannot be initialized
to just MIGRATE_ISOLATE.  Add init_pageblock_migratetype() to enable
initialize a pageblock with a migratetype and isolated.

Link: https://lkml.kernel.org/r/20250617021115.2331563-4-ziy@nvidia.com
Signed-off-by: Zi Yan 
Reviewed-by: Vlastimil Babka 
Acked-by: David Hildenbrand 
Cc: Baolin Wang 
Cc: Brendan Jackman 
Cc: Johannes Weiner 
Cc: Kirill A. Shuemov 
Cc: Mel Gorman 
Cc: Michal Hocko 
Cc: Oscar Salvador 
Cc: Richard Chang 
Cc: Suren Baghdasaryan 
Signed-off-by: Andrew Morton

mm/page_alloc: pageblock flags functions clean up

2025-07-13T23:38:17+00:00

Patch series "Make MIGRATE_ISOLATE a standalone bit", v10.

This patchset moves MIGRATE_ISOLATE to a standalone bit to avoid being
overwritten during pageblock isolation process.  Currently,
MIGRATE_ISOLATE is part of enum migratetype (in include/linux/mmzone.h),
thus, setting a pageblock to MIGRATE_ISOLATE overwrites its original
migratetype.  This causes pageblock migratetype loss during
alloc_contig_range() and memory offline, especially when the process fails
due to a failed pageblock isolation and the code tries to undo the
finished pageblock isolations.

In terms of performance for changing pageblock types, no performance
change is observed:

1. I used perf to collect stats of offlining and onlining all memory
   of a 40GB VM 10 times and see that get_pfnblock_flags_mask() and
   set_pfnblock_flags_mask() take about 0.12% and 0.02% of the whole
   process respectively with and without this patchset across 3 runs.

2. I used perf to collect stats of dd from /dev/random to a 40GB tmpfs
   file and find get_pfnblock_flags_mask() takes about 0.05% of the
   process with and without this patchset across 3 runs.


This patch (of 6):

No functional change is intended.

1. Add __NR_PAGEBLOCK_BITS for the number of pageblock flag bits and use
   roundup_pow_of_two(__NR_PAGEBLOCK_BITS) as NR_PAGEBLOCK_BITS to take
   right amount of bits for pageblock flags.
2. Rename PB_migrate_skip to PB_compact_skip.
3. Add {get,set,clear}_pfnblock_bit() to operate one a standalone bit,
   like PB_compact_skip.
3. Make {get,set}_pfnblock_flags_mask() internal functions and use
   {get,set}_pfnblock_migratetype() for pageblock migratetype operations.
4. Move pageblock flags common code to get_pfnblock_bitmap_bitidx().
3. Use MIGRATETYPE_MASK to get the migratetype of a pageblock from its
   flags.
4. Use PB_migrate_end in the definition of MIGRATETYPE_MASK instead of
   PB_migrate_bits.
5. Add a comment on is_migrate_cma_folio() to prevent one from changing it
   to use get_pageblock_migratetype() and causing issues.

Link: https://lkml.kernel.org/r/20250617021115.2331563-1-ziy@nvidia.com
Link: https://lkml.kernel.org/r/20250617021115.2331563-2-ziy@nvidia.com
Signed-off-by: Zi Yan 
Reviewed-by: Vlastimil Babka 
Acked-by: David Hildenbrand 
Cc: Baolin Wang 
Cc: Brendan Jackman 
Cc: Johannes Weiner 
Cc: Kirill A. Shuemov 
Cc: Mel Gorman 
Cc: Michal Hocko 
Cc: Oscar Salvador 
Cc: Richard Chang 
Cc: Suren Baghdasaryan 
Signed-off-by: Andrew Morton

mm,memory_hotplug: drop status_change_nid parameter from memory_notify

2025-07-13T23:38:17+00:00

There no users left of status_change_nid, so drop it from memory_notify
struct.

Link: https://lkml.kernel.org/r/20250616135158.450136-12-osalvador@suse.de
Signed-off-by: Oscar Salvador 
Suggested-by: David Hildenbrand 
Acked-by: David Hildenbrand 
Reviewed-by: Vlastimil Babka 
Cc: Harry Yoo 
Cc: Hyeonggon Yoo <42.hyeyoo@gmail.com>
Cc: Joanthan Cameron 
Cc: Rakie Kim 
Signed-off-by: Andrew Morton

mm,memory_hotplug: implement numa node notifier

2025-07-13T23:38:15+00:00

There are at least six consumers of hotplug_memory_notifier that what they
really are interested in is whether any numa node changed its state, e.g:
going from having memory to not having memory and vice versa.

Implement a specific notifier for numa nodes when their state gets
changed, which will later be used by those consumers that are only
interested in numa node state changes.

Add documentation as well.

[dan.carpenter@linaro.org: set failure reason in offline_pages()]
  Link: https://lkml.kernel.org/r/be4fd31b-7d09-46b0-8329-6d0464ffa7a5@sabinyo.mountain
Link: https://lkml.kernel.org/r/20250616135158.450136-4-osalvador@suse.de
Signed-off-by: Oscar Salvador 
Signed-off-by: Dan Carpenter 
Reviewed-by: Jonathan Cameron 
Reviewed-by: Harry Yoo 
Reviewed-by: Vlastimil Babka 
Acked-by: David Hildenbrand 
Cc: Hyeonggon Yoo <42.hyeyoo@gmail.com>
Cc: Rakie Kim 
Signed-off-by: Andrew Morton

mm,memory_hotplug: remove status_change_nid_normal and update documentation

2025-07-13T23:38:14+00:00

Now that the last user of status_change_nid_normal is gone, we can remove
it.  Update documentation accordingly.

Link: https://lkml.kernel.org/r/20250616135158.450136-3-osalvador@suse.de
Signed-off-by: Oscar Salvador 
Reviewed-by: Vlastimil Babka 
Acked-by: David Hildenbrand 
Cc: Harry Yoo 
Cc: Hyeonggon Yoo <42.hyeyoo@gmail.com>
Cc: Joanthan Cameron 
Cc: Rakie Kim 
Signed-off-by: Andrew Morton

drivers/base/node: rename __register_one_node() to register_one_node()

2025-07-10T05:42:00+00:00

The register_one_node() function was a simple wrapper around
__register_one_node().  To simplify the code, register_one_node() has been
removed, and __register_one_node() has been renamed to
register_one_node().

Link: https://lkml.kernel.org/r/8262cd0f44eeb048a1fcd3ac8382760d7f7dea60.1748452242.git.donettom@linux.ibm.com
Signed-off-by: Donet Tom 
Acked-by: David Hildenbrand 
Cc: Mike Rapoport (Microsoft) 
Cc: Oscar Salvador 
Cc: Zi Yan 
Signed-off-by: Andrew Morton

drivers/base/node: rename register_memory_blocks_under_node() and remove context argument

2025-07-10T05:42:00+00:00

The function register_memory_blocks_under_node() is now only called from
the memory hotplug path, as register_memory_blocks_under_node_early()
handles registration during early boot.  Therefore, the context argument
used to differentiate between early boot and hotplug is no longer needed
and was removed.

Since the function is only called from the hotplug path, we renamed
register_memory_blocks_under_node() to
register_memory_blocks_under_node_hotplug()

Link: https://lkml.kernel.org/r/907c22292b0ee4975107876efc875c75c11badd9.1748452242.git.donettom@linux.ibm.com
Signed-off-by: Donet Tom 
Acked-by: Oscar Salvador 
Acked-by: Mike Rapoport (Microsoft) 
Acked-by: David Hildenbrand 
Acked-by: Zi Yan 
Signed-off-by: Andrew Morton