Commit fc574488 authored by Mike Rapoport (IBM)'s avatar Mike Rapoport (IBM) Committed by Andrew Morton

mm/page_alloc: invert logic for early page initialisation checks

Rename early_page_uninitialised() to early_page_initialised() and invert
its logic to make the code more readable.

Link: https://lkml.kernel.org/r/20230104191805.2535864-1-rppt@kernel.orgSigned-off-by: default avatarMike Rapoport (IBM) <rppt@kernel.org>
Reviewed-by: default avatarDavid Hildenbrand <david@redhat.com>
Reviewed-by: default avatarAnshuman Khandual <anshuman.khandual@arm.com>
Acked-by: default avatarMel Gorman <mgorman@suse.de>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent f78dfc7b
...@@ -443,15 +443,15 @@ static inline bool deferred_pages_enabled(void) ...@@ -443,15 +443,15 @@ static inline bool deferred_pages_enabled(void)
return static_branch_unlikely(&deferred_pages); return static_branch_unlikely(&deferred_pages);
} }
/* Returns true if the struct page for the pfn is uninitialised */ /* Returns true if the struct page for the pfn is initialised */
static inline bool __meminit early_page_uninitialised(unsigned long pfn) static inline bool __meminit early_page_initialised(unsigned long pfn)
{ {
int nid = early_pfn_to_nid(pfn); int nid = early_pfn_to_nid(pfn);
if (node_online(nid) && pfn >= NODE_DATA(nid)->first_deferred_pfn) if (node_online(nid) && pfn >= NODE_DATA(nid)->first_deferred_pfn)
return true; return false;
return false; return true;
} }
/* /*
...@@ -498,9 +498,9 @@ static inline bool deferred_pages_enabled(void) ...@@ -498,9 +498,9 @@ static inline bool deferred_pages_enabled(void)
return false; return false;
} }
static inline bool early_page_uninitialised(unsigned long pfn) static inline bool early_page_initialised(unsigned long pfn)
{ {
return false; return true;
} }
static inline bool defer_init(int nid, unsigned long pfn, unsigned long end_pfn) static inline bool defer_init(int nid, unsigned long pfn, unsigned long end_pfn)
...@@ -1643,7 +1643,7 @@ static void __meminit init_reserved_page(unsigned long pfn) ...@@ -1643,7 +1643,7 @@ static void __meminit init_reserved_page(unsigned long pfn)
pg_data_t *pgdat; pg_data_t *pgdat;
int nid, zid; int nid, zid;
if (!early_page_uninitialised(pfn)) if (early_page_initialised(pfn))
return; return;
nid = early_pfn_to_nid(pfn); nid = early_pfn_to_nid(pfn);
...@@ -1806,7 +1806,7 @@ int __meminit early_pfn_to_nid(unsigned long pfn) ...@@ -1806,7 +1806,7 @@ int __meminit early_pfn_to_nid(unsigned long pfn)
void __init memblock_free_pages(struct page *page, unsigned long pfn, void __init memblock_free_pages(struct page *page, unsigned long pfn,
unsigned int order) unsigned int order)
{ {
if (early_page_uninitialised(pfn)) if (!early_page_initialised(pfn))
return; return;
if (!kmsan_memblock_free_pages(page, order)) { if (!kmsan_memblock_free_pages(page, order)) {
/* KMSAN will take care of these pages. */ /* KMSAN will take care of these pages. */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment