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 X-Spam-Level: X-Spam-Status: No, score=-9.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 20E14C43461 for ; Mon, 14 Sep 2020 08:59:15 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 7727820756 for ; Mon, 14 Sep 2020 08:59:14 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7727820756 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=suse.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id E27726B0003; Mon, 14 Sep 2020 04:59:13 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id DD7E96B0037; Mon, 14 Sep 2020 04:59:13 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id CED776B0055; Mon, 14 Sep 2020 04:59:13 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0175.hostedemail.com [216.40.44.175]) by kanga.kvack.org (Postfix) with ESMTP id B93E16B0003 for ; Mon, 14 Sep 2020 04:59:13 -0400 (EDT) Received: from smtpin26.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id 7439C181B10F2 for ; Mon, 14 Sep 2020 08:59:13 +0000 (UTC) X-FDA: 77261067786.26.nose60_2100d3f27107 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin26.hostedemail.com (Postfix) with ESMTP id 32CFF18150A4F for ; Mon, 14 Sep 2020 08:59:13 +0000 (UTC) X-HE-Tag: nose60_2100d3f27107 X-Filterd-Recvd-Size: 6190 Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) by imf17.hostedemail.com (Postfix) with ESMTP for ; Mon, 14 Sep 2020 08:59:12 +0000 (UTC) X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id 1B2F9AC46; Mon, 14 Sep 2020 08:59:27 +0000 (UTC) Date: Mon, 14 Sep 2020 10:59:11 +0200 From: Michal Hocko To: Laurent Dufour Cc: akpm@linux-foundation.org, David Hildenbrand , Oscar Salvador , Greg Kroah-Hartman , linux-mm@kvack.org, "Rafael J . Wysocki" , nathanl@linux.ibm.com, cheloha@linux.ibm.com, Tony Luck , Fenghua Yu , linux-ia64@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/3] mm: don't panic when links can't be created in sysfs Message-ID: <20200914085911.GC16999@dhcp22.suse.cz> References: <20200911134831.53258-1-ldufour@linux.ibm.com> <20200911134831.53258-4-ldufour@linux.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200911134831.53258-4-ldufour@linux.ibm.com> X-Rspamd-Queue-Id: 32CFF18150A4F X-Spamd-Result: default: False [0.00 / 100.00] X-Rspamd-Server: rspam03 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 Fri 11-09-20 15:48:31, Laurent Dufour wrote: > At boot time, or when doing memory hot-add operations, if the links in > sysfs can't be created, the system is still able to run, so just report the > error in the kernel log. .. rather than BUG_ON and potentially make system unusable because the callpath can be called with locks held etc... > Since the number of memory blocks managed could be high, the messages are > rate limited. > > As a consequence, link_mem_sections() has no status to report anymore. > > Signed-off-by: Laurent Dufour > Cc: David Hildenbrand Acked-by: Michal Hocko Thanks! > --- > drivers/base/node.c | 25 +++++++++++++++++-------- > include/linux/node.h | 17 ++++++++--------- > mm/memory_hotplug.c | 5 ++--- > 3 files changed, 27 insertions(+), 20 deletions(-) > > diff --git a/drivers/base/node.c b/drivers/base/node.c > index 862516c5a5ae..749a1c8ea992 100644 > --- a/drivers/base/node.c > +++ b/drivers/base/node.c > @@ -811,12 +811,21 @@ static int register_mem_sect_under_node(struct memory_block *mem_blk, > ret = sysfs_create_link_nowarn(&node_devices[nid]->dev.kobj, > &mem_blk->dev.kobj, > kobject_name(&mem_blk->dev.kobj)); > - if (ret) > - return ret; > + if (ret && ret != -EEXIST) > + pr_err_ratelimited( > + "can't create %s to %s link in sysfs (%d)\n", > + kobject_name(&node_devices[nid]->dev.kobj), > + kobject_name(&mem_blk->dev.kobj), ret); > > - return sysfs_create_link_nowarn(&mem_blk->dev.kobj, > + ret = sysfs_create_link_nowarn(&mem_blk->dev.kobj, > &node_devices[nid]->dev.kobj, > kobject_name(&node_devices[nid]->dev.kobj)); > + if (ret && ret != -EEXIST) > + pr_err_ratelimited( > + "can't create %s to %s link in sysfs (%d)\n", > + kobject_name(&mem_blk->dev.kobj), > + kobject_name(&node_devices[nid]->dev.kobj), > + ret); > } > /* mem section does not span the specified node */ > return 0; > @@ -837,17 +846,17 @@ void unregister_memory_block_under_nodes(struct memory_block *mem_blk) > kobject_name(&node_devices[mem_blk->nid]->dev.kobj)); > } > > -int link_mem_sections(int nid, unsigned long start_pfn, unsigned long end_pfn, > - enum memplug_context context) > +void link_mem_sections(int nid, unsigned long start_pfn, unsigned long end_pfn, > + enum memplug_context context) > { > struct rmsun_args args = { > .nid = nid, > .context = context, > }; > > - return walk_memory_blocks(PFN_PHYS(start_pfn), > - PFN_PHYS(end_pfn - start_pfn), (void *)&args, > - register_mem_sect_under_node); > + walk_memory_blocks(PFN_PHYS(start_pfn), > + PFN_PHYS(end_pfn - start_pfn), (void *)&args, > + register_mem_sect_under_node); > } > > #ifdef CONFIG_HUGETLBFS > diff --git a/include/linux/node.h b/include/linux/node.h > index 8ff08520488c..6bdd6f3ed3aa 100644 > --- a/include/linux/node.h > +++ b/include/linux/node.h > @@ -99,15 +99,14 @@ extern struct node *node_devices[]; > typedef void (*node_registration_func_t)(struct node *); > > #if defined(CONFIG_MEMORY_HOTPLUG_SPARSE) && defined(CONFIG_NUMA) > -extern int link_mem_sections(int nid, unsigned long start_pfn, > - unsigned long end_pfn, > - enum memplug_context context); > +void link_mem_sections(int nid, unsigned long start_pfn, > + unsigned long end_pfn, > + enum memplug_context context); > #else > -static inline int link_mem_sections(int nid, unsigned long start_pfn, > - unsigned long end_pfn, > - enum memplug_context context) > +static inline void link_mem_sections(int nid, unsigned long start_pfn, > + unsigned long end_pfn, > + enum memplug_context context) > { > - return 0; > } > #endif > > @@ -130,8 +129,8 @@ static inline int register_one_node(int nid) > if (error) > return error; > /* link memory sections under this node */ > - error = link_mem_sections(nid, start_pfn, end_pfn, > - MEMPLUG_EARLY); > + link_mem_sections(nid, start_pfn, end_pfn, > + MEMPLUG_EARLY); > } > > return error; > diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c > index 912d355ca446..668418071a49 100644 > --- a/mm/memory_hotplug.c > +++ b/mm/memory_hotplug.c > @@ -1080,9 +1080,8 @@ int __ref add_memory_resource(int nid, struct resource *res) > } > > /* link memory sections under this node.*/ > - ret = link_mem_sections(nid, PFN_DOWN(start), PFN_UP(start + size - 1), > - MEMPLUG_HOTPLUG); > - BUG_ON(ret); > + link_mem_sections(nid, PFN_DOWN(start), PFN_UP(start + size - 1), > + MEMPLUG_HOTPLUG); > > /* create new memmap entry */ > if (!strcmp(res->name, "System RAM")) > -- > 2.28.0 -- Michal Hocko SUSE Labs