On Thu, 21 May 2026 12:01:18 +0800, Li Zhe <[email protected]> wrote: > diff --git a/mm/mm_init.c b/mm/mm_init.c > index f9f8e1af921c..4ba506df93bc 100644 > --- a/mm/mm_init.c > +++ b/mm/mm_init.c > @@ -987,11 +987,36 @@ static void __init memmap_init(void) > } > > #ifdef CONFIG_ZONE_DEVICE > -static void __ref __init_zone_device_page(struct page *page, unsigned long > pfn, > +static inline int zone_device_page_init_refcount( > + const struct dev_pagemap *pgmap)
The name zone_device_page_init_refcount() implies that this function actually intializes the refcount, but it's actually used as a predicate based on pagemap properties, so let's make it bool and name accordingly. How about static inline bool pagemap_resets_refcount(const struct dev_pagemap *pagemap) Other than that: Reviewed-by: Mike Rapoport (Microsoft) <[email protected]> -- Sincerely yours, Mike.
