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 98AB2EB64D9 for ; Thu, 15 Jun 2023 08:14:44 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 10CFB6B0072; Thu, 15 Jun 2023 04:14:44 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 096FE6B0074; Thu, 15 Jun 2023 04:14:44 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E79336B0078; Thu, 15 Jun 2023 04:14:43 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id D095B6B0072 for ; Thu, 15 Jun 2023 04:14:43 -0400 (EDT) Received: from smtpin09.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 9AD4A80B27 for ; Thu, 15 Jun 2023 08:14:43 +0000 (UTC) X-FDA: 80904270846.09.F336424 Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) by imf21.hostedemail.com (Postfix) with ESMTP id B1DEB1C0003 for ; Thu, 15 Jun 2023 08:14:41 +0000 (UTC) Authentication-Results: imf21.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=NqJM711W; dmarc=pass (policy=quarantine) header.from=suse.com; spf=pass (imf21.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=1686816881; a=rsa-sha256; cv=none; b=ynho6eRpUXXGL2dbtMhcJQiuHJr7zo3ZUi+dcTlivQ4Mf4SFklbq2w0i76eedQcCGTVsPy 49ae5XO9p3AoHjeu1ZxrVATkzITEblszj8VhM6wvjVg7d4VHs6ZJpFO1S85/ulIlfe8sQJ gmCibsWz10rOB6yqDn8/wV/xE48U6k4= ARC-Authentication-Results: i=1; imf21.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=NqJM711W; dmarc=pass (policy=quarantine) header.from=suse.com; spf=pass (imf21.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=1686816881; 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=u7K1dHo3suajr80VD6qXH9vvOWaAMLy7kkTVGXspF0U=; b=AjjHqY7BpD8w6z5XNXGes7AVTWQRZ7FIur7aixhDPMfX6PtxaO+UIRGTMgilm6YnFa2dmZ UaoQde3HMcYAjQAPtX419kGSjFP36CLmJ5QknGSgJ0RC3rngBuiTE89idRq7ttGdFRZS8D oPEYI7Xeeuhj7PqKefE2XIYO+tqvcOI= 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 5697A1FDDA; Thu, 15 Jun 2023 08:14:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1686816880; 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=u7K1dHo3suajr80VD6qXH9vvOWaAMLy7kkTVGXspF0U=; b=NqJM711WfcvRab2CCaoqcmrc5nGT/TblaFgad0EStN8zcLon2DOpDqyckZfG0d1tMo9TYO njZ+Dt7dQ6Vq0Buvu34Dj+dF0oXQ+hHimcR/YKbjQMAexlPM/zyPJ2g0Pwlw9fMTyzXqLD YLlR1w9ba2JUdWw9auY99dxXXZMSTDk= 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 46BBF13A47; Thu, 15 Jun 2023 08:14:40 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id zuOQEHDIimQnKAAAMHmgww (envelope-from ); Thu, 15 Jun 2023 08:14:40 +0000 Date: Thu, 15 Jun 2023 10:14:39 +0200 From: Michal Hocko To: Haifeng Xu Cc: roman.gushchin@linux.dev, hannes@cmpxchg.org, shakeelb@google.com, akpm@linux-foundation.org, cgroups@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] mm/memcontrol: do not tweak node in mem_cgroup_init() Message-ID: References: <20230615073226.1343-1-haifeng.xu@shopee.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230615073226.1343-1-haifeng.xu@shopee.com> X-Rspam-User: X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: B1DEB1C0003 X-Stat-Signature: dhujngfj7np53fiu5thzioes3bsze5sp X-HE-Tag: 1686816881-787658 X-HE-Meta: U2FsdGVkX1/I0jCkuPInWO1fTb/tZ4WcvLEBBSRmaRiQ1lgWz/THUHPlnYCo6sZc9lyzf0g2FOkwt6NgeQn4wCJzTBj79aywaxyWkZlFbEyFS32vZBWRzage9DzveMrumAyrtvV6SSnoTE+VGNLn9NFKP5lqA2f3pujW+FHpWP6rK+gxI0cNQGCga4K+rk5Fhx+uvWI7p/4Nosg9vnnmpVlV0bGVHOyn8WwCRYJC/w0rted4BQEVZF9poi4ZwDSWNH/iNHdY9rimgZVfNKbbR5V2vAnd04lMRfqGdnME1w1H2Kl48j35pHAr0MaZpkDj3QF9jAGiUaHqARH8PAnNew4KO+IQyVlwcmaqXRth70k0B5zvsn/yWSrgqLkblp1VoZ9U5IToDDnurkjmbhBpFz7y4vCKWP7dDrmh6+qE37eZ0ISXtNBK+xGdMWuCwYn7kBEQtPt4N93gpMZHgUCxE2enW1eTtLXTHzREZYVK2CpHqrhy6Ny9qI76NTCB/LKz4cziq2JOpVoojozo+02kggvc3SM5YFJCQpxb+or5TXgvjftkf/fFyzxV0NV1hWBj7FHgGcNrYwuYa1kp55vBQjaduJgcwtPL44d9/04FyyRSU/b39eEqMf577G9Uh1aDyVBmSzCfOClTK4mSGfvGiRmp2KYuinttnt3ZgX2S8aKn8LIuSwKJvhmoSo5TU25NfBbItNEchnYM89YnmY6Uea8UXp2/deoIGxv8jhoOBL1CgtYVHNFoosBpgCcOozyJpdU+0n2Lwm9i9kInRsSCRW229D+txCPMwfjX45324VFtsADLCWV8orO/1pgjYRz6TfABTc+ct5xyoSipTVPebGnizP47R+u1KfURgz1uGPX21I5u73KRH1dyiHlxnXm77qjofx5o7hDQj0bUY92Wffos7rY+gKH6cGoQxSTE7ZesBRXxDrwUB7MIbAUfE3iEXl7y1XF3n2Cc4w1PJEX vhPXOvym bsHdyqmqEm7Yv7ejzQ5EZPEfsAJk+CETxo9ciCr6GaB2y9P11/FxRcjltboHOR15MjpoVrGwtfvrTRsXTweYjO4bptcGQH4fkLIY5Gjb2rRHwts5J2cb4ER4XgMVpBCgZLSWAkngMsOSikCX6ipJ/boyUgKf62RWgt/Wx8NBmWACmVjaLXzyvcUAtOSOihNnAGkl2+rkNQcSSsO0uNKYQgW1ki3wSq/wMc2ZjU/ikKG7pbX0R1XSi4BDi8WlFiIrWDZ2e 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 Thu 15-06-23 07:32:25, Haifeng Xu wrote: > mem_cgroup_init() request for allocations from each possible node, and > it's used to be a problem because NODE_DATA is not allocated for offline > node. Things have already changed since commit 09f49dca570a9 ("mm: handle > uninitialized numa nodes gracefully"), so it's unnecessary to check for > !node_online nodes here. How have you tested this patch? I am not saying it is wrong and it looks like the right thing to do. But the early init code has proven to be more subtle than expected so it is definitely good to know that this has been tested on memory less setup and passed. > Signed-off-by: Haifeng Xu > --- > mm/memcontrol.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index 4b27e245a055..c73c5fb33f65 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c > @@ -7421,8 +7421,7 @@ static int __init mem_cgroup_init(void) > for_each_node(node) { > struct mem_cgroup_tree_per_node *rtpn; > > - rtpn = kzalloc_node(sizeof(*rtpn), GFP_KERNEL, > - node_online(node) ? node : NUMA_NO_NODE); > + rtpn = kzalloc_node(sizeof(*rtpn), GFP_KERNEL, node); > > rtpn->rb_root = RB_ROOT; > rtpn->rb_rightmost = NULL; > -- > 2.25.1 -- Michal Hocko SUSE Labs