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 60A83C87FDC for ; Wed, 7 Jun 2023 08:48:34 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id C3A296B0071; Wed, 7 Jun 2023 04:48:33 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id BC2AC6B0072; Wed, 7 Jun 2023 04:48:33 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id A144C8E0001; Wed, 7 Jun 2023 04:48:33 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 8E3A86B0071 for ; Wed, 7 Jun 2023 04:48:33 -0400 (EDT) Received: from smtpin27.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 58D831C7318 for ; Wed, 7 Jun 2023 08:48:33 +0000 (UTC) X-FDA: 80875325706.27.B610156 Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) by imf19.hostedemail.com (Postfix) with ESMTP id 7A5531A0007 for ; Wed, 7 Jun 2023 08:48:31 +0000 (UTC) Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=T0FTkw8Y; dmarc=pass (policy=quarantine) header.from=suse.com; spf=pass (imf19.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.29 as permitted sender) smtp.mailfrom=mhocko@suse.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1686127711; a=rsa-sha256; cv=none; b=JT5RONkdE4ijW5Ktb7YNN3J+xyNrVDNHSMpyYS/1zH5Td/zQl6y8rL+iWhhW+93+KGw7PW Oc8U1w6pCT9qg+F9fPqjj43O+85JEXVxRbWyxfdYyGgYRxNI0U3uL+uokWt0+BMhbf2sG1 ammx4PueoovnYO2HFnXKkbjnnSZ4Ooc= ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=T0FTkw8Y; dmarc=pass (policy=quarantine) header.from=suse.com; spf=pass (imf19.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.29 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=1686127711; 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=4nuQ9zLS68BCE4lTKPyHUGHAuv4TPfQjF/DOF3031s0=; b=TQDKlsAsw+uu898Jabzqb5LVJAr0VGtKLBowLsMp+YQWVfnYptGzJ799HHdkaBqsA/Kg2B ddVgUSgJ851ZlyllqfkGDqc3thYTyNC1ZfxMnYx0npJYFF/Zz/nTE7Qzjonf2M5hzZOu/R 7HffvT2PslawRnM24zJ5kjI+LoFSSfw= 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-out2.suse.de (Postfix) with ESMTPS id E33B41FDAA; Wed, 7 Jun 2023 08:48:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1686127709; 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=4nuQ9zLS68BCE4lTKPyHUGHAuv4TPfQjF/DOF3031s0=; b=T0FTkw8Y/YEC/IL1O4BJBomDGULhPsbWtQZAwyLk+kag29HOqfG+K3r4mmcFlVMt7CziyY 3qhQO+ZWFqp89VxMDO5hoQzJlW6DJGEA0q69uIy7NPG2d76PHMc4q+Q69aEmkGrGmetIJZ MI21lemIIQo8muMfUz41Y7kBONpGTjI= 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 D5E2E1346D; Wed, 7 Jun 2023 08:48:29 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id f3h9M11EgGSqSQAAMHmgww (envelope-from ); Wed, 07 Jun 2023 08:48:29 +0000 Date: Wed, 7 Jun 2023 10:48:29 +0200 From: Michal Hocko To: Haifeng Xu Cc: rppt@kernel.org, akpm@linux-foundation.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] mm/mm_init.c: drop 'nid' parameter from check_for_memory() Message-ID: References: <20230607032402.4679-1-haifeng.xu@shopee.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230607032402.4679-1-haifeng.xu@shopee.com> X-Rspam-User: X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: 7A5531A0007 X-Stat-Signature: cdpp9f6597b6dcut7zhycu8s9q781p7q X-HE-Tag: 1686127711-839369 X-HE-Meta: U2FsdGVkX1/wfDMec8IxmgkmUVQDTWjI2nDuC/VdFcOAUY2vk5Ei2UahneVt2rso63wtWpL5PUNFLg8DWD8lP3S4b2FMq1wtgfUJoo26p7Mw0DWpgMqLdfuc8y7mdDdGCzE4rzZX4ULFKZaPKXxowTcOwL//FZjLn1JYtxY6hjGQ4xRunsOF4yJWTwczQMOo7VqV+cAKpYyh58KTFmC7blBYoOEzCjjzvkNriYvoq5zv3DH7y25V6loiORnnnmhdeBS86bjSs1E26/3/PMg294GSybdiN2CczskxNdd+JdUhuEzZ7Og0r16RzgRzWoalL9nfde3No6yc2hQUF1icMZQD2aT+lPvft3Qo3P1fjrYIMvaXmDHP+ReiUSpDb+A6LbYBO9DrE1g63rOnQeysDeE/9A7MHxMoVgkS6Ajm95dDHGuaCLQ3bBRPoHvhAGxryO2bsYPn0o8p8J6o2IuxjtrBkFNIiaWAaFt51yCoWhGKgxOUmePJqszVrwQH6panzofNIVb7l+9B5Sp1R8s1z9obFzmuEokkxclPB6043buN3d4sjHgAPzWk52L3fQn0MF1heRb2HdcOe/LD+Egq/FeOXCPJwoYGTZmUPg8ClJu9QpuGC09ZHWEbGynu3YUChZq6/auF2QqqNSpD9Kb4k23P6FthFNsdXJsMoGanDagBV14e7DUweTgjmnE7AO/y/B7CjR9RmcrlNDn8aOAoByOTdjgv2vOvTWMfW9fTA2XoKEg+Hd+OyJawOA6RaTKX7Yp0oVyV0AyEJrSCVk86PyynSsRX1KlO/l5XVJhmZUSuisuYjUZBHhMHvKgEjT97+xLygLwpZvriba6jF6RMpENLLR3UhYpVbgcJCcdlQH+Q0R/GvgzSOy+9RSvSs6KD9yT+XPVPEj+ZRJfI8dmRatgaXyu5hPQoeqXlfTJqJSDjp7WWQqLoANPkzgGz7mfcIu/LFYCyA7b7CcocPxz sLGuNjHA SCjPuBztxirhP6ScHbR0Ec7MPmAP40OA6nXAaUoLU/CUMyYOonl11oC9jlhs+VQRZCo0RX5ki2DgRF6gNRfRKPrXcGcnqmCds8gbbJU9nfchNAQn6o7/Iy4jeLeQ+mdtB+Sll6yFNB+QX2+8hQiCUuM8OEE4Fi7YEyXDP1cZkHNsp+H22eb93/CQzt2utPu+wFgHR4bKe5dz3Zw5dmhSYjSH9uNk0zVSLVop42qqtUeEN4pYKV3k0d11o0yeDQ44Caryv/Ylgdvxq6Fyjmw5DFgZp3hsb3Siyr7rS2vQdqm3KJJKdn5oLVJA55g== 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 03:24:02, Haifeng Xu wrote: > The node_id in pgdat has already been set in free_area_init_node(), > so use it internally instead of passing a redundant parameter. Yes and this even improves the code because having both pgdat and nid arguments is confusing and can be error prone because those might mismatch and we would end up allocating from a different node in the end. > Signed-off-by: Haifeng Xu Acked-by: Michal Hocko > --- > mm/mm_init.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/mm/mm_init.c b/mm/mm_init.c > index b82718d596c3..78e67041ae9f 100644 > --- a/mm/mm_init.c > +++ b/mm/mm_init.c > @@ -1725,7 +1725,7 @@ static void __init free_area_init_node(int nid) > } > > /* Any regular or high memory on that node ? */ > -static void check_for_memory(pg_data_t *pgdat, int nid) > +static void check_for_memory(pg_data_t *pgdat) > { > enum zone_type zone_type; > > @@ -1733,9 +1733,9 @@ static void check_for_memory(pg_data_t *pgdat, int nid) > struct zone *zone = &pgdat->node_zones[zone_type]; > if (populated_zone(zone)) { > if (IS_ENABLED(CONFIG_HIGHMEM)) > - node_set_state(nid, N_HIGH_MEMORY); > + node_set_state(pgdat->node_id, N_HIGH_MEMORY); > if (zone_type <= ZONE_NORMAL) > - node_set_state(nid, N_NORMAL_MEMORY); > + node_set_state(pgdat->node_id, N_NORMAL_MEMORY); > break; > } > } > @@ -1887,7 +1887,7 @@ void __init free_area_init(unsigned long *max_zone_pfn) > /* Any memory on that node */ > if (pgdat->node_present_pages) > node_set_state(nid, N_MEMORY); > - check_for_memory(pgdat, nid); > + check_for_memory(pgdat); > } > > memmap_init(); > -- > 2.25.1 -- Michal Hocko SUSE Labs