From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9DA1BEB64DB for ; Tue, 20 Jun 2023 08:46:15 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 0FAB58D0002; Tue, 20 Jun 2023 04:46:15 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 0AB978D0001; Tue, 20 Jun 2023 04:46:15 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id EB5178D0002; Tue, 20 Jun 2023 04:46:14 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id D97478D0001 for ; Tue, 20 Jun 2023 04:46:14 -0400 (EDT) Received: from smtpin21.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id A3A96AFF82 for ; Tue, 20 Jun 2023 08:46:14 +0000 (UTC) X-FDA: 80922494268.21.7EEC99E Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf02.hostedemail.com (Postfix) with ESMTP id 079678001C for ; Tue, 20 Jun 2023 08:46:11 +0000 (UTC) Authentication-Results: imf02.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=ERSorwLz; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf02.hostedemail.com: domain of rppt@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=rppt@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1687250772; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=CG1RRjQCCqM8TREixbZVPdAP/Y7ulF1vCns4KvG06Wg=; b=0BT2lQ5lGX7bS+Bl/otKo6jacuKk7cttq0jpiOA6dgQzsqVmA8NGGjubFUkQECTVkm3BiB ZjMLHR8VL4kpPwb7/g8PxDSfdYMgW781OT8cMjRNoL+eyrviScG2eWTO76rqQpVpncFsb/ U520Kigaf1pJNMfUy3H1Urphnw/1L2o= ARC-Authentication-Results: i=1; imf02.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=ERSorwLz; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf02.hostedemail.com: domain of rppt@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=rppt@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1687250772; a=rsa-sha256; cv=none; b=uGvMjXWmI8qHsvtlQ+YolvuayTXipRlfd91CORlpPs5APd/D+ExaZN1o43veGziDEa+W9M yv44DJTct+CR+kFU7AD+LiqyJ3YYPC1JS76UnTK2TR0aBLhrym2dThS9BiafJsl78EH8i6 KkoYXlyvGF6Hzg75wUKMRYsvVSn8zx8= Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 024AE61003; Tue, 20 Jun 2023 08:46:11 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 54080C433C0; Tue, 20 Jun 2023 08:46:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1687250770; bh=TQjPtAYxTNuXGasBPn7DviEY9IbqSVODbZ5OTQJzlDA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ERSorwLzotKCYA7nLdIx8+nEcpGIIEqMMwz1RxTBzfi0JEjB1LTH6CntGV/xnrElf HY26odflX72zv/1dHt3ZucTnLJ1SnsxfO8BNySz3YZkaTLHoErUT9sRWPxasPc9Fjb yTb3tLckFhTim9Tsy/asVZOFqQl9+m36mjUu6VTrNsfBcMmXl7RVfdjPHEFe6T77nX Pk+YpVfZEK5mKVWO/DPaPeeWfT7LEMYO9DPiZwKI/9SC/S0GTL1UB18zzmh61cNtCq SHAQeK2pQhD/YirKg6RGh5nYTfW2oMaWT9Y2qXx4Yb5iwCp9u/Fptk0F5NqaCj6OWQ U6Cb+/LoVjG5Q== Date: Tue, 20 Jun 2023 11:45:30 +0300 From: Mike Rapoport To: Yajun Deng Cc: akpm@linux-foundation.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, kernel test robot Subject: Re: [PATCH v4] mm: pass nid to reserve_bootmem_region() Message-ID: <20230620084530.GD52412@kernel.org> References: <20230619023406.424298-1-yajun.deng@linux.dev> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230619023406.424298-1-yajun.deng@linux.dev> X-Rspamd-Queue-Id: 079678001C X-Rspam-User: X-Rspamd-Server: rspam02 X-Stat-Signature: y8qhcaob4ohpm71w1irm85rh46ynp811 X-HE-Tag: 1687250771-841992 X-HE-Meta: U2FsdGVkX1+vpVQn/E6XJR1XASt3ZHnkGX4KkCXbwJBZYQGmTColCZvubY3By5UtAI4M6T0CVblz8gh3YF2BqsJr6io+JgYsfktmHRPmGh00ruzlecNh8wOpIMo11uDNGe94A7hCzmLzFh2asaBJD2ItXUGxY/P+dav3bspY2Q0TzLUr9KEtyXM9jyglbo0H13w9eT6AaGpwChNp7FaFguTxisBwcDSJyvmbkEqy58p8f4Ki1VGvY0LN+vHiG5uj74S0srdfQmUiHqR2CMFZvnMPRcoJdINJwjPZrjEFBNwRI64+q/Z2Adu2R/+y/m9K/d3XIpTj2ccZOOvAD/bmsRB9HSAIS2SYZPirZCXGOWPXBy2X5HSSHoGV1Ujx91T8aVD8LDm3DsbCYqV3C6TWc1jwlG6IwR4IeRV4j3FiGVm2wC3BNr9N1otZC+eEOBa3GM0K0JtMdi8Gh+l4xJpB+WHZ1ca0lw0/himRjaizBI/H9lEOa5MQSrEXvgeZdT+OHCEfgaPDoMNeXAAgU7C/tTefcbvV9UPeOI5zB9jKtetMAteIoRwj9EeoYD3udx+9SnBn+rSNX8ms8DUFEhSisB+fx4FiqpH+Kx4qiQWVgeBwbmBaP7HNRrqV5XPt0B6eyDdMv7LLF6AEKg2YGcLLHtGn6xAxnH0L5/oK0vqEyaNOM+YzXbjIxrQIlUW/adFfpiKZmMLlzNJZJi+jDX9JIbaC4A5NOoxcNx+e0t8rMHAhIOgc9TvZQt3ZDPeEa0P5JOnB2R2bf1nMPbNpnaqK7zW9aXVNlhcE5KP4qw9JSJOBDObn3jOgv+1g4za3UIORRNIGC/ApTtMMIGnbzXLnvZszk1l1WqFWmKeUzOXw9gqmnwo3xloV7dsqWJergxj56U5+BqxX2I26/cUlAQn3mcUu9p3Pc4Snuiu9SFyMAF3WASML+jIOixsU6tyGxtIY1qsJ1YT5D94HCBxaMG0 4a5w+mUE 0qsQFykrKq1c9LbjfWmSa/MXdAPzov/qfsdsXlrrg41s2S6C3hZtMlkVBPu84H/N6Ce+6EcIEZ5bupnpdLOMr+HhYbVJP2B2hxKe02l8Z3hlEq6c2FDBC7Amipa87Bcg66Lj4w3U5cfPxQdFLbbmCGIoagGrz6sJ2vGvv52DJPd2FMjjalFjAyi7ZUVVfzJZYURO996LU1bjjM/gvONaXxxzqeF57pL2VbmNl2cz1d7xBOF2HNaSb1yaSgDRSZxncDUwLKVR1gcIKvPjqHikzIVY0rxjefgFa5HsCpgo7gOuuepQ28gzCSNE/ZBPTlbyCKsvT4jW6DDZR9nrOQn73eIVbo9t79JX56aKdyODRc2yCt78vmZqfEfDRZPXSRMo5zbuExXw62J4IOJghiyMt6p7lI85d1ibMtXsy X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Mon, Jun 19, 2023 at 10:34:06AM +0800, Yajun Deng wrote: > early_pfn_to_nid() is called frequently in init_reserved_page(), it > returns the node id of the PFN. These PFN are probably from the same > memory region, they have the same node id. It's not necessary to call > early_pfn_to_nid() for each PFN. > > Pass nid to reserve_bootmem_region() and drop the call to > early_pfn_to_nid() in init_reserved_page(). Also, set nid on all > reserved pages before doing this, as some reserved memory regions may > not be set nid. > > The most beneficial function is memmap_init_reserved_pages() if > CONFIG_DEFERRED_STRUCT_PAGE_INIT is enabled. > > The following data was tested on an x86 machine with 190GB of RAM. > > before: > memmap_init_reserved_pages() 67ms > > after: > memmap_init_reserved_pages() 20ms > > Signed-off-by: Yajun Deng > Reported-by: kernel test robot > Closes: https://lore.kernel.org/oe-kbuild-all/202306160145.juJMr3Bi-lkp@intel.com Reviewed-by: Mike Rapoport (IBM) > --- > V3 -> V4: make the test for early_page_initialised() inside if > (IS_ENABLED(CONFIG_DEFERRED_STRUCT_PAGE_INIT)) > V2 -> V3: set nid on all reserved pages before pass nid. > V1 -> V2: fix build error when CONFIG_NUMA is not enabled. > --- > include/linux/mm.h | 3 ++- > mm/memblock.c | 31 +++++++++++++++++++++---------- > mm/mm_init.c | 30 +++++++++++++++++------------- > 3 files changed, 40 insertions(+), 24 deletions(-) > > diff --git a/include/linux/mm.h b/include/linux/mm.h > index fdd966b11f79..a7a0e692d44d 100644 > --- a/include/linux/mm.h > +++ b/include/linux/mm.h > @@ -2960,7 +2960,8 @@ extern unsigned long free_reserved_area(void *start, void *end, > > extern void adjust_managed_page_count(struct page *page, long count); > > -extern void reserve_bootmem_region(phys_addr_t start, phys_addr_t end); > +extern void reserve_bootmem_region(phys_addr_t start, > + phys_addr_t end, int nid); > > /* Free the reserved page into the buddy system, so it gets managed. */ > static inline void free_reserved_page(struct page *page) > diff --git a/mm/memblock.c b/mm/memblock.c > index ff0da1858778..f9e61e565a53 100644 > --- a/mm/memblock.c > +++ b/mm/memblock.c > @@ -2091,19 +2091,30 @@ static void __init memmap_init_reserved_pages(void) > { > struct memblock_region *region; > phys_addr_t start, end; > - u64 i; > + int nid; > + > + /* > + * set nid on all reserved pages and also treat struct > + * pages for the NOMAP regions as PageReserved > + */ > + for_each_mem_region(region) { > + nid = memblock_get_region_node(region); > + start = region->base; > + end = start + region->size; > + > + if (memblock_is_nomap(region)) > + reserve_bootmem_region(start, end, nid); > + > + memblock_set_node(start, end, &memblock.reserved, nid); > + } > > /* initialize struct pages for the reserved regions */ > - for_each_reserved_mem_range(i, &start, &end) > - reserve_bootmem_region(start, end); > + for_each_reserved_mem_region(region) { > + nid = memblock_get_region_node(region); > + start = region->base; > + end = start + region->size; > > - /* and also treat struct pages for the NOMAP regions as PageReserved */ > - for_each_mem_region(region) { > - if (memblock_is_nomap(region)) { > - start = region->base; > - end = start + region->size; > - reserve_bootmem_region(start, end); > - } > + reserve_bootmem_region(start, end, nid); > } > } > > diff --git a/mm/mm_init.c b/mm/mm_init.c > index d393631599a7..a1963c3322af 100644 > --- a/mm/mm_init.c > +++ b/mm/mm_init.c > @@ -646,10 +646,8 @@ static inline void pgdat_set_deferred_range(pg_data_t *pgdat) > } > > /* Returns true if the struct page for the pfn is initialised */ > -static inline bool __meminit early_page_initialised(unsigned long pfn) > +static inline bool __meminit early_page_initialised(unsigned long pfn, int nid) > { > - int nid = early_pfn_to_nid(pfn); > - > if (node_online(nid) && pfn >= NODE_DATA(nid)->first_deferred_pfn) > return false; > > @@ -695,15 +693,14 @@ defer_init(int nid, unsigned long pfn, unsigned long end_pfn) > return false; > } > > -static void __meminit init_reserved_page(unsigned long pfn) > +static void __meminit init_reserved_page(unsigned long pfn, int nid) > { > pg_data_t *pgdat; > - int nid, zid; > + int zid; > > - if (early_page_initialised(pfn)) > + if (early_page_initialised(pfn, nid)) > return; > > - nid = early_pfn_to_nid(pfn); > pgdat = NODE_DATA(nid); > > for (zid = 0; zid < MAX_NR_ZONES; zid++) { > @@ -717,7 +714,7 @@ static void __meminit init_reserved_page(unsigned long pfn) > #else > static inline void pgdat_set_deferred_range(pg_data_t *pgdat) {} > > -static inline bool early_page_initialised(unsigned long pfn) > +static inline bool early_page_initialised(unsigned long pfn, int nid) > { > return true; > } > @@ -727,7 +724,7 @@ static inline bool defer_init(int nid, unsigned long pfn, unsigned long end_pfn) > return false; > } > > -static inline void init_reserved_page(unsigned long pfn) > +static inline void init_reserved_page(unsigned long pfn, int nid) > { > } > #endif /* CONFIG_DEFERRED_STRUCT_PAGE_INIT */ > @@ -738,7 +735,8 @@ static inline void init_reserved_page(unsigned long pfn) > * marks the pages PageReserved. The remaining valid pages are later > * sent to the buddy page allocator. > */ > -void __meminit reserve_bootmem_region(phys_addr_t start, phys_addr_t end) > +void __meminit reserve_bootmem_region(phys_addr_t start, > + phys_addr_t end, int nid) > { > unsigned long start_pfn = PFN_DOWN(start); > unsigned long end_pfn = PFN_UP(end); > @@ -747,7 +745,7 @@ void __meminit reserve_bootmem_region(phys_addr_t start, phys_addr_t end) > if (pfn_valid(start_pfn)) { > struct page *page = pfn_to_page(start_pfn); > > - init_reserved_page(start_pfn); > + init_reserved_page(start_pfn, nid); > > /* Avoid false-positive PageTail() */ > INIT_LIST_HEAD(&page->lru); > @@ -2579,8 +2577,14 @@ void __init set_dma_reserve(unsigned long new_dma_reserve) > void __init memblock_free_pages(struct page *page, unsigned long pfn, > unsigned int order) > { > - if (!early_page_initialised(pfn)) > - return; > + > + if (IS_ENABLED(CONFIG_DEFERRED_STRUCT_PAGE_INIT)) { > + int nid = early_pfn_to_nid(pfn); > + > + if (!early_page_initialised(pfn, nid)) > + return; > + } > + > if (!kmsan_memblock_free_pages(page, order)) { > /* KMSAN will take care of these pages. */ > return; > -- > 2.25.1 > -- Sincerely yours, Mike.