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 004C7C433EF for ; Wed, 22 Jun 2022 06:21:40 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 8920C6B0116; Wed, 22 Jun 2022 02:21:40 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 819BF8E0084; Wed, 22 Jun 2022 02:21:40 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 6BAC68E0083; Wed, 22 Jun 2022 02:21:40 -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 578CD6B0116 for ; Wed, 22 Jun 2022 02:21:40 -0400 (EDT) Received: from smtpin13.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 2A9F66139A for ; Wed, 22 Jun 2022 06:21:40 +0000 (UTC) X-FDA: 79604875560.13.8CEAA1F Received: from mail-pj1-f48.google.com (mail-pj1-f48.google.com [209.85.216.48]) by imf15.hostedemail.com (Postfix) with ESMTP id CB0D3A00A0 for ; Wed, 22 Jun 2022 06:21:33 +0000 (UTC) Received: by mail-pj1-f48.google.com with SMTP id g10-20020a17090a708a00b001ea8aadd42bso15819867pjk.0 for ; Tue, 21 Jun 2022 23:21:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bytedance-com.20210112.gappssmtp.com; s=20210112; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=2l9YHnTCHEZbmX1vqIKgTsBl4LuPnmzfnEVIEO95Yuk=; b=Vl93CrA/UtQLYvuumeHVdrEweooBfop0R+dDLntKS4n2ikh/TXiUNXQbmHovjn/s6w G0c1JCo8UyQc3UNEzuYovHxq0f2lsG4hhC8lrA/vVLRD1C4atRIBGy7b+fpsfpE+ymVg e9wEzdOabfUlvuIV4H2GfIUL21uG76AZEqSjXIH/8jwYkL5ZF/H5014CciBuUtQy0e8T Ab+U6g6fxmIGgtKmsTCC+gtDBqrS08XRYJv6udAhG8Y6SZGBftVk+Rh4KKJ5f1T+Ke/5 fb7vDdswWfQ7Bcpks0fig5C67kQBVCteU9kBD4ikKxjYnK9IF1/DTFiEdCvIA8UIyeeS X/3w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=2l9YHnTCHEZbmX1vqIKgTsBl4LuPnmzfnEVIEO95Yuk=; b=VHgsKcFPplyZuq8AW1NkncWPEHiGcEj7qHnhb9dWbNncZPhZ/LSmfcb065omPbHoNt qUVPMEPfTR8gNLPjZg6gSjBSFLNH5g2bwuSYH7Xeh6gAJYKRMR1UMCDYLA1dkn6o2VSr PVOpWAqOfN7La5IbNcQWauGcXfu1MB8guMLkF6pSYCXHeFYM4q9SsjULi73sq+cneXUN ViQ2z6lQp7SkCc+NN1JraI4Mo4bE0NewijYH6Ne3BHz1kMb9VEzakyS1Dws7v1Qc8CCh jq/6L2ADLBT5gotnFBE2keWRUEXmZwK6hbWu5PqGIRxX/T1uG5cP7aM7ewgZdhvb4YhB /23g== X-Gm-Message-State: AJIora8wnw6rWA8D5RE288PbCiPZOum4JSF1o8CD8NzvxdeA5qyos+XR CREayblSO+52midOY55Dpt4vcA== X-Google-Smtp-Source: AGRyM1uYRu1M8vsdu4fWKSIrkfoZvszKA5zB9WnFh2GsKIMSN2uogA/zjFuVFCiBFUMQ21o8FhZhqQ== X-Received: by 2002:a17:90a:6809:b0:1ec:c213:56c8 with SMTP id p9-20020a17090a680900b001ecc21356c8mr9884251pjj.82.1655878889648; Tue, 21 Jun 2022 23:21:29 -0700 (PDT) Received: from localhost ([139.177.225.231]) by smtp.gmail.com with ESMTPSA id z8-20020a1709027e8800b0015e8d4eb1easm8942900pla.52.2022.06.21.23.21.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 21 Jun 2022 23:21:29 -0700 (PDT) Date: Wed, 22 Jun 2022 14:21:25 +0800 From: Muchun Song To: Gang Li Cc: Mike Kravetz , Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v1] mm, hugetlb: skip irrelevant nodes in hugetlb_show_meminfo() Message-ID: References: <20220622040043.25462-1-ligang.bdlg@bytedance.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220622040043.25462-1-ligang.bdlg@bytedance.com> ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1655878899; 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=2l9YHnTCHEZbmX1vqIKgTsBl4LuPnmzfnEVIEO95Yuk=; b=jcKgWoXrgoGwAs2Z1jzAWCJp95QMKkKK2tXV/zEWkmJguwtlyY8FcUje8qa14gzu1LmWiC 8IlucjhErVbFvxO4k1ead17tOyYXuYctx95G/7kGqyzsXTXqUx9MxlPC5A6/qTlUw446fQ O2YDfFVa/KoANF+dErivU510de64KcU= ARC-Authentication-Results: i=1; imf15.hostedemail.com; dkim=pass header.d=bytedance-com.20210112.gappssmtp.com header.s=20210112 header.b="Vl93CrA/"; spf=pass (imf15.hostedemail.com: domain of songmuchun@bytedance.com designates 209.85.216.48 as permitted sender) smtp.mailfrom=songmuchun@bytedance.com; dmarc=pass (policy=none) header.from=bytedance.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1655878899; a=rsa-sha256; cv=none; b=4Wr5mswOzl5imau61Yok4Xnup/H4dMxC+a3r8YKaX6An3qIseTQtK5IYS/aNplFzm5OTsN n97xcr+pglEaTcAw1WhceIODEfowYcTvtLAj9BevDaPXPUgXzPtwTpxfXE5ww5/pRnmLfH 8GxcnIlumCi3Qj7dVNyciJFFw91TKf4= X-Rspamd-Queue-Id: CB0D3A00A0 Authentication-Results: imf15.hostedemail.com; dkim=pass header.d=bytedance-com.20210112.gappssmtp.com header.s=20210112 header.b="Vl93CrA/"; spf=pass (imf15.hostedemail.com: domain of songmuchun@bytedance.com designates 209.85.216.48 as permitted sender) smtp.mailfrom=songmuchun@bytedance.com; dmarc=pass (policy=none) header.from=bytedance.com X-Rspam-User: X-Stat-Signature: mewbfm818ktmb15xojpa9k8myi4gtjm6 X-Rspamd-Server: rspam04 X-HE-Tag: 1655878893-706746 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, Jun 22, 2022 at 12:00:42PM +0800, Gang Li wrote: > show_free_areas() allows to filter out node specific data which is > irrelevant to the allocation request. But hugetlb_show_meminfo() still > show hugetlb on all nodes, which is redundant and unnecessary. > > Use show_mem_node_skip() to skip irrelevant nodes in > hugetlb_show_meminfo(). > > Signed-off-by: Gang Li > --- > include/linux/hugetlb.h | 4 ++-- > include/linux/mm.h | 1 + > mm/hugetlb.c | 7 +++++-- > mm/page_alloc.c | 4 ++-- > 4 files changed, 10 insertions(+), 6 deletions(-) > > diff --git a/include/linux/hugetlb.h b/include/linux/hugetlb.h > index 642a39016f9a..1913ac6bf10a 100644 > --- a/include/linux/hugetlb.h > +++ b/include/linux/hugetlb.h > @@ -155,7 +155,7 @@ void __unmap_hugepage_range_final(struct mmu_gather *tlb, > struct page *ref_page, zap_flags_t zap_flags); > void hugetlb_report_meminfo(struct seq_file *); > int hugetlb_report_node_meminfo(char *buf, int len, int nid); > -void hugetlb_show_meminfo(void); > +void hugetlb_show_meminfo(unsigned int filter, nodemask_t *nodemask); > unsigned long hugetlb_total_pages(void); > vm_fault_t hugetlb_fault(struct mm_struct *mm, struct vm_area_struct *vma, > unsigned long address, unsigned int flags); > @@ -300,7 +300,7 @@ static inline int hugetlb_report_node_meminfo(char *buf, int len, int nid) > return 0; > } > > -static inline void hugetlb_show_meminfo(void) > +static inline void hugetlb_show_meminfo(unsigned int filter, nodemask_t *nodemask) > { > } > > diff --git a/include/linux/mm.h b/include/linux/mm.h > index 764dc0fdae5e..f72c1b21cde3 100644 > --- a/include/linux/mm.h > +++ b/include/linux/mm.h > @@ -1873,6 +1873,7 @@ extern void pagefault_out_of_memory(void); > */ > #define SHOW_MEM_FILTER_NODES (0x0001u) /* disallowed nodes */ > > +extern bool show_mem_node_skip(unsigned int flags, int nid, nodemask_t *nodemask); Exporting this to the only user of HugeTLB is not a good idea. > extern void show_free_areas(unsigned int flags, nodemask_t *nodemask); > > #ifdef CONFIG_MMU > diff --git a/mm/hugetlb.c b/mm/hugetlb.c > index 98492733cc64..632826e6fea5 100644 > --- a/mm/hugetlb.c > +++ b/mm/hugetlb.c > @@ -4486,7 +4486,7 @@ int hugetlb_report_node_meminfo(char *buf, int len, int nid) > nid, h->surplus_huge_pages_node[nid]); > } > > -void hugetlb_show_meminfo(void) > +void hugetlb_show_meminfo(unsigned int filter, nodemask_t *nodemask) > { I suggest making this function show meminfo for a specific node. like hugetlb_show_meminfo_node(int nid) which only show meminfo for node @nid. > struct hstate *h; > int nid; > @@ -4494,7 +4494,9 @@ void hugetlb_show_meminfo(void) > if (!hugepages_supported()) > return; > > - for_each_node_state(nid, N_MEMORY) > + for_each_node_state(nid, N_MEMORY) { > + if (show_mem_node_skip(filter, nid, nodemask)) > + continue; > for_each_hstate(h) > pr_info("Node %d hugepages_total=%u hugepages_free=%u hugepages_surp=%u hugepages_size=%lukB\n", > nid, > @@ -4502,6 +4504,7 @@ void hugetlb_show_meminfo(void) > h->free_huge_pages_node[nid], > h->surplus_huge_pages_node[nid], > huge_page_size(h) / SZ_1K); > + } > } > > void hugetlb_report_usage(struct seq_file *m, struct mm_struct *mm) > diff --git a/mm/page_alloc.c b/mm/page_alloc.c > index 84781094b478..5896b5a9101f 100644 > --- a/mm/page_alloc.c > +++ b/mm/page_alloc.c > @@ -5951,7 +5951,7 @@ void si_meminfo_node(struct sysinfo *val, int nid) > * Determine whether the node should be displayed or not, depending on whether > * SHOW_MEM_FILTER_NODES was passed to show_free_areas(). > */ > -static bool show_mem_node_skip(unsigned int flags, int nid, nodemask_t *nodemask) > +bool show_mem_node_skip(unsigned int flags, int nid, nodemask_t *nodemask) > { > if (!(flags & SHOW_MEM_FILTER_NODES)) > return false; > @@ -6196,7 +6196,7 @@ void show_free_areas(unsigned int filter, nodemask_t *nodemask) > printk(KERN_CONT "= %lukB\n", K(total)); > } > Just like the above case. We can check nid here instead of in hugetlb_show_meminfo(). Then you do not need to export show_mem_node_skip(). for_each_online_node(nid) { if (show_mem_node_skip(filter, nid, nodemask)) continue; hugetlb_show_meminfo_node(nid); } Thanks. > - hugetlb_show_meminfo(); > + hugetlb_show_meminfo(filter, nodemask); > > printk("%ld total pagecache pages\n", global_node_page_state(NR_FILE_PAGES)); > > -- > 2.20.1 > >