memblock: Return NUMA_NO_NODE instead of -1 to improve code readability
authorYuntao Wang <ytcoode@gmail.com>
Thu, 7 Dec 2023 13:10:01 +0000 (21:10 +0800)
committerMike Rapoport (IBM) <rppt@kernel.org>
Fri, 8 Dec 2023 08:31:00 +0000 (10:31 +0200)
When no corresponding memory region is found for the given pfn, return
NUMA_NO_NODE instead of -1. This improves code readability and aligns with
the existing logic of the memblock_search_pfn_nid() function's user.

Signed-off-by: Yuntao Wang <ytcoode@gmail.com>
Link: https://lore.kernel.org/r/20231207131001.224914-1-ytcoode@gmail.com
Signed-off-by: Mike Rapoport (IBM) <rppt@kernel.org>
mm/memblock.c

index 5a88d6d24d793807803139254b5804827251adf1..c1f579c23396a85c4003c3db0fded7e690b5b0d3 100644 (file)
@@ -1851,7 +1851,7 @@ int __init_memblock memblock_search_pfn_nid(unsigned long pfn,
        int mid = memblock_search(type, PFN_PHYS(pfn));
 
        if (mid == -1)
-               return -1;
+               return NUMA_NO_NODE;
 
        *start_pfn = PFN_DOWN(type->regions[mid].base);
        *end_pfn = PFN_DOWN(type->regions[mid].base + type->regions[mid].size);