2024-02-22 09:28:52

by Hao Ge

Subject: [PATCH] mm/page_alloc: Make check_new_page return bool

Make check_new_page return bool like check_new_pages

Signed-off-by: Hao Ge <[email protected]>
---
mm/page_alloc.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index 150d4f23b010..c7a0bc546d2c 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -1422,14 +1422,14 @@ static void check_new_page_bad(struct page *page)
/*
* This page is about to be returned from the page allocator
*/
-static int check_new_page(struct page *page)
+static bool check_new_page(struct page *page)
{
if (likely(page_expected_state(page,
PAGE_FLAGS_CHECK_AT_PREP|__PG_HWPOISON)))
- return 0;
+ return false;

check_new_page_bad(page);
- return 1;
+ return true;
}

static inline bool check_new_pages(struct page *page, unsigned int order)
--
2.25.1

2024-02-22 12:47:15

by Matthew Wilcox

Subject: Re: [PATCH] mm/page_alloc: Make check_new_page return bool

On Thu, Feb 22, 2024 at 05:19:32PM +0800, Hao Ge wrote:
> Make check_new_page return bool like check_new_pages
>
> Signed-off-by: Hao Ge <[email protected]>

Reviewed-by: Matthew Wilcox (Oracle) <[email protected]>