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 3F793C7EE29 for ; Wed, 7 Jun 2023 10:33:40 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id B02098E0001; Wed, 7 Jun 2023 06:33:39 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id A89A46B0072; Wed, 7 Jun 2023 06:33:39 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 92B3B8E0001; Wed, 7 Jun 2023 06:33:39 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 7E05E6B0071 for ; Wed, 7 Jun 2023 06:33:39 -0400 (EDT) Received: from smtpin19.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 2841DA0215 for ; Wed, 7 Jun 2023 10:33:38 +0000 (UTC) X-FDA: 80875590558.19.EA143E6 Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by imf04.hostedemail.com (Postfix) with ESMTP id BB35B4000A for ; Wed, 7 Jun 2023 10:33:36 +0000 (UTC) Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=I1TTb94c; dmarc=pass (policy=quarantine) header.from=suse.com; spf=pass (imf04.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.28 as permitted sender) smtp.mailfrom=mhocko@suse.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1686134017; 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=2GQDwm9XkXFV1nznF3vSR6+NBAxeeNTuAQMVtcoe+2I=; b=lubAWFlCXQfxiFMkZi5b/kmOx6y6lSVftCUBtFynoNvlJnVQgUR+/tqJoaTDtAWPSDGHQi A/CJI0psTRgGa+lE9bbt3v8+PZvCzGLRqnSn3k0A0yRGJSdSr5yTr6e3ICvxjf3m2LyvM3 4J23Z3cXmySVyK4cRveqGys8EjSwwDs= ARC-Authentication-Results: i=1; imf04.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=I1TTb94c; dmarc=pass (policy=quarantine) header.from=suse.com; spf=pass (imf04.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.28 as permitted sender) smtp.mailfrom=mhocko@suse.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1686134017; a=rsa-sha256; cv=none; b=i49eA0j4HPKCQAQJsC7T104/07lMPx7XYjC9/PZC2iwj7Bi0Twc3/VRkT0Fbr+0sw5qiPR yWgrfBrQ79ecPCUR+DPBZeKTQfWXfixEG+eRwSbuv5Ji5fQ06mPLEwy69foBI+PgX6G/NC oWVJu/hVRb6yK0B4XDa5PrYmUh30Ox4= Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id 486C0219F7; Wed, 7 Jun 2023 10:33:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1686134015; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=2GQDwm9XkXFV1nznF3vSR6+NBAxeeNTuAQMVtcoe+2I=; b=I1TTb94ckPTKnPVhzz6RBMCKJSPvHt55awTKzeRhsiiJDlx93lQ76y3oJSKU86F9H99TeM aqYhiPKTXEfFpzgE8oZxI8WTbWx52u86xBIlDGnO4CH0F3rgh17wJj2xDZTJVOhrX7imTw 4q7a7mP8wNqHfZb38UIR6VTvXLzLByU= Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 349971346D; Wed, 7 Jun 2023 10:33:35 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id H1IxDP9cgGR/BAAAMHmgww (envelope-from ); Wed, 07 Jun 2023 10:33:35 +0000 Date: Wed, 7 Jun 2023 12:33:34 +0200 From: Michal Hocko To: Haifeng Xu Cc: david@redhat.com, rppt@kernel.org, osalvador@suse.de, akpm@linux-foundation.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 2/2] mm/mm_init.c: remove reset_node_present_pages() Message-ID: References: <20230607025056.1348-1-haifeng.xu@shopee.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230607025056.1348-1-haifeng.xu@shopee.com> X-Rspam-User: X-Stat-Signature: i7cc945s7jma5wujuax49h649himnqg8 X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: BB35B4000A X-HE-Tag: 1686134016-140145 X-HE-Meta: U2FsdGVkX1+IRXvjTLvTbk/t1dKfHSRXWCr8C9d3cl3zOr/03TC2qDTZk9IBOE7Jy4zNvsCXeh3PrSHvzmeEvMUQGeaE/nTr7GMBh3APQtzUtrW3W8h1KemRgRzUuX3RGOtsQT0XFkn9QaMP9sDnLlJ5wX0drr1ggPU17g578/lkcPwrDInctOdhrMg/qd5JZ/VHouO2ta51dMyzxxVoOqS5/11wy8rFFvYI65dS9QtMWyBMbJ5WSJQrNzyDkCg5TIHPND1IZNE+PQbteIOe+IMk2F0zGFMRrJxLLpaGfUq1PMhklmwltzDV+wOVZ9NWGfPxMNp35N6ZdvaKfQ2kfa2Cj7kd2kA7a44vcHKi9vEL/ugCBgZ18W8IDeR+KA+LjxyLF+dmg1KZd69XJW6Kc0XpwvlTmiIiDgzLBvBK7S3siLnFbBlcr4Wi597gXfUmCFpilww49l1e5UwmE6DOqZJPkDkx/GoZC0N3L7wXR1BNoTLQiWYl+lMjE04B8WR7wWz7jYtqckHmCVfFCgl07gz4EKBhAy26GacNUshCXpL00DTA8lvWuydrFmHPDziOxU8i4IZYK0thvgP/XFXL6dJpYvGjphwRH3gLy/an3yZOLGloXR2AwArVfY25R/xAPRHsmKQ+B0THLhcFC7mqiQoiokx4qFT0hOMbdUGcKywdY+m5BmJ0Vbmv9xIRLkp0DGHo5pfQDVzA20CFn3hkS17wdLyqALA7JGCySC8JHQI6r0kmwYnit6krWH9xyO6g+kbYaRJLEI5f6IXGopQzloapT8QkuNn1DVmz6V6S/Kxe5QZxYwqoKPxT01KFb0+hTRqprF0S/CwrSgu2Qlzky4lcryshS11wOsjsh2iZccpBJt89lYHY1M+7FxNRRpXQZWyO4CPDccvSp4940HBrMemSI0x/L27Jsy/zWr0+WOaB8QJ7sSG5yEuWzSfvtq5wpzD0+PJVDo46nLnKsay qn1gqOzZ d74JWygGBavun9jagw9qW1BEKwSBAUv3yyBlMWhCQBiUmPPeqlavhX/+NWRbiZnNchHf0F+KYLt+k+sgYy2vOC0++ff8TNGodJjY6kctaTu2IlYaKP7R9aEe9d9Sb3futATWNVfdxcDJL+FKnWhmhGU8NIUFRX0YhBdiwyjLhmCFtIPuOe6LQzZzRfdaO2of91mMcgRKpWdfa7/EAmvOB6rX4Qn3MRHv4mIuELxrTVl03aEm8GP+QrLk1FpDtRcKpfNtylfmwAITe/N+tAEO/jKRLgKufkCJ6cqpsva52Mm9SQnVcFAUW7QJnTQ27wfPKOgenjB6aBbA6uHmIeMSSGhgWLQ== 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 Wed 07-06-23 02:50:56, Haifeng Xu wrote: > reset_node_present_pages() only get called in hotadd_init_pgdat(), move > the action that clear present pages to free_area_init_core_hotplug(), so > the helper can be removed. There is no explanation why moving the action is desirable and/or needed. I do agree that removing this trivial helper which only has a single caller. I guess the incentive is an overal simplification but please spell that out. > Signed-off-by: Haifeng Xu > Suggested-by: David Hildenbrand The patch itself makes sense Acked-by: Michal Hocko > --- > v2: > - declare 'struct zone' within the for loop > - fix comment layout > --- > mm/memory_hotplug.c | 18 ------------------ > mm/mm_init.c | 15 +++++++++++++-- > 2 files changed, 13 insertions(+), 20 deletions(-) > > diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c > index 65e385f34679..ee1c0d9a4d5a 100644 > --- a/mm/memory_hotplug.c > +++ b/mm/memory_hotplug.c > @@ -1172,16 +1172,6 @@ int __ref online_pages(unsigned long pfn, unsigned long nr_pages, > return ret; > } > > -static void reset_node_present_pages(pg_data_t *pgdat) > -{ > - struct zone *z; > - > - for (z = pgdat->node_zones; z < pgdat->node_zones + MAX_NR_ZONES; z++) > - z->present_pages = 0; > - > - pgdat->node_present_pages = 0; > -} > - > /* we are OK calling __meminit stuff here - we have CONFIG_MEMORY_HOTPLUG */ > static pg_data_t __ref *hotadd_init_pgdat(int nid) > { > @@ -1204,14 +1194,6 @@ static pg_data_t __ref *hotadd_init_pgdat(int nid) > */ > build_all_zonelists(pgdat); > > - /* > - * When memory is hot-added, all the memory is in offline state. So > - * clear all zones' present_pages because they will be updated in > - * online_pages() and offline_pages(). > - * TODO: should be in free_area_init_core_hotplug? > - */ > - reset_node_present_pages(pgdat); > - > return pgdat; > } > > diff --git a/mm/mm_init.c b/mm/mm_init.c > index 78e67041ae9f..a02ba9a79b3d 100644 > --- a/mm/mm_init.c > +++ b/mm/mm_init.c > @@ -1507,6 +1507,8 @@ void __ref free_area_init_core_hotplug(struct pglist_data *pgdat) > pgdat->kswapd_order = 0; > pgdat->kswapd_highest_zoneidx = 0; > pgdat->node_start_pfn = 0; > + pgdat->node_present_pages = 0; > + > for_each_online_cpu(cpu) { > struct per_cpu_nodestat *p; > > @@ -1514,8 +1516,17 @@ void __ref free_area_init_core_hotplug(struct pglist_data *pgdat) > memset(p, 0, sizeof(*p)); > } > > - for (z = 0; z < MAX_NR_ZONES; z++) > - zone_init_internals(&pgdat->node_zones[z], z, nid, 0); > + /* > + * When memory is hot-added, all the memory is in offline state. So > + * clear all zones' present_pages and managed_pages because they will > + * be updated in online_pages() and offline_pages(). > + */ > + for (z = 0; z < MAX_NR_ZONES; z++) { > + struct zone *zone = pgdat->node_zones + z; > + > + zone->present_pages = 0; > + zone_init_internals(zone, z, nid, 0); > + } > } > #endif > > -- > 2.25.1 -- Michal Hocko SUSE Labs