[v2,0/3] memblock: some fix for memmap_init_reserved_pages()

Message

Wei Yang March 18, 2025, 7:19 a.m. UTC

During code review, I found we may leave some reserved region with invalid nid
when memblock_set_node() double the array. The patch set propose a fix and
test case.

Also I found memblock_set_node() accept size instead of end as second
parameter. Currently it is misused.

Patch 1: fix the second argument passed to memblock_set_node()
Patch 2: fix the invalid node id we may have in memmap_init_reserved_pages()
Patch 3: add a test case to verify it

v2: move check out side of the loop

Wei Yang (3):
  mm/memblock: pass size instead of end to memblock_set_node()
  mm/memblock: repeat setting reserved region nid if array is doubled
  memblock tests: add test for memblock_set_node

 mm/memblock.c                            |  12 ++-
 tools/testing/memblock/tests/basic_api.c | 102 +++++++++++++++++++++++
 2 files changed, 113 insertions(+), 1 deletion(-)