Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp2229961pxj; Sun, 9 May 2021 20:05:44 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwrsCpwf0csN/2RMvZTGkRm2I/8p2XacjeuENcP8ds9eEF0oEevVJB6zgh0Ysdma8/d+xcn X-Received: by 2002:a5e:dd47:: with SMTP id u7mr11746570iop.62.1620615943939; Sun, 09 May 2021 20:05:43 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1620615943; cv=none; d=google.com; s=arc-20160816; b=sbLkK4aaKWtOeC4VCYvq5+yv+tJST9MDgKyl2AHcYFmgHHEzzD2j23G2t74EYLSOG9 HfUXqz70Z73FDB1MkPHTpnLFNNBwkYEXioU62ytqoSVXtv2xBlwf/7+85Hysm0THym6W I3R2yVM8QrbetNqKY4tOAqkR+qkNSylapvT1IBhb6iuKUln65TSaiVQK/77vxZvz1leB 3pbX775rBYDG19MPjaeEus11Yyc6DbCw1NRXo298sz+/uExoznOXV+IU71/5zFya0e1y 7Y+RE4A3nACVd28SpOdLGiCpSLzGibr/wuHlNYUuaOE9TEl4Rw/uvREl6gfFNqC+lH0I LK+Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=sYTUTjFUbmon1SGhbccDrj5pYUlY6vb4o2URlhlRuqM=; b=SiawagZIJJeYRCICTNXQMhyYBE32N0dASTUx8BNJwfRAUbIr24Ww8SFSisEymBhmIN l/qmJzrWbGa66m7dcAanUuLpjj4eB83V5i502R5L5n084zyX3SscSyjosT+AoBFgRoI5 GPCJ8RQ9na+/IX8m9i3Ho2MJDqfC9OD2Teb7+FH/3aDwTM+a0mvefXb1V5iGYIOceUeW u7C3nNXDZwibVLqZZYsY2Nk5BDFs/bpqolV79ZL+0HOWpZ9ilRnpxLjinvgpezy6QuKM HyKdk30nmtNwc1e7YjBnLtUnhItWpc0aMQubex4U1EsVscLs6aswUU3hxMdcRAPj0HWc 9ghA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@bytedance-com.20150623.gappssmtp.com header.s=20150623 header.b=UlYurxDU; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=bytedance.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id j23si14981811ioo.31.2021.05.09.20.05.32; Sun, 09 May 2021 20:05:43 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@bytedance-com.20150623.gappssmtp.com header.s=20150623 header.b=UlYurxDU; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=bytedance.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230202AbhEJDFo (ORCPT + 99 others); Sun, 9 May 2021 23:05:44 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52532 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230281AbhEJDFo (ORCPT ); Sun, 9 May 2021 23:05:44 -0400 Received: from mail-pl1-x631.google.com (mail-pl1-x631.google.com [IPv6:2607:f8b0:4864:20::631]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 87256C061574 for ; Sun, 9 May 2021 20:04:40 -0700 (PDT) Received: by mail-pl1-x631.google.com with SMTP id t21so8428234plo.2 for ; Sun, 09 May 2021 20:04:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bytedance-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=sYTUTjFUbmon1SGhbccDrj5pYUlY6vb4o2URlhlRuqM=; b=UlYurxDUOYaTylyy3bFLR3po7g0XViDSRns7W24iohK5GSrihpYUpFPQ/2ZHXExdYy lEJ58mpiWlV/UvjAmS9BW0oVSbllli8c12A8bjSCuRLxoGUMMhSX5jflOzfKCD0iJPWL MmbuK7dMaQV7e+I60pgjih1Qj0HHHU1Pd0nphZuAuR0OW+mex4w2Ij+Ud7BEHWhICzP9 NIgMtOin8pmw6LIm9uuTeLV1aPhT+dSsBApcH7It89OqwUn/4A9T7lrUpeoGZS6mnY2E TrzOt2MX5C7FljGZpUBye/JBe6M3PeKZMV309gzweIEX4/mvpqD8o/ONjxWNTFcoKVoI NOSA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=sYTUTjFUbmon1SGhbccDrj5pYUlY6vb4o2URlhlRuqM=; b=e7rpfC0T4Vpa6dDD5RB+9j7A4YBZLr8j0xRt3+DfJeiu7fLAla7zWYSp6Wg3HFc4GZ PukfqmviT6JC6JjRPW8qPUrCsGquxzJoOKfz9Mx95VG7lv1DM/7cL5TZ9bdEpCexltlZ BrMja9XcZJa5xE2Aip81KAiIxuDboocWTNTiQfOFPrY5T+8tExoFoWFt0mDL+tbD0uMw g7kv/gTSFYwbFAc5e+i/DQ55Lpm3B7BG1ZcJ28I9gRhVPOGxS+kCdDmeJx5Q5wvHpmGM ij5IBMl4UcLWT1pL4Su+af7lGxU+xR2xk8BGDG3kJ+nQayzy9tKABTUBLgjYJrrDzeTL nqFg== X-Gm-Message-State: AOAM5315IxGXHJuXlhvl4MxOcCwxSmaeM/Lruey01SXTulgWcnre52fW 18Fxbe/pzczCi248Wm6Bxcg11A== X-Received: by 2002:a17:90a:454c:: with SMTP id r12mr25539597pjm.52.1620615879874; Sun, 09 May 2021 20:04:39 -0700 (PDT) Received: from localhost.localdomain ([139.177.225.236]) by smtp.gmail.com with ESMTPSA id a128sm9777003pfd.115.2021.05.09.20.04.28 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 09 May 2021 20:04:39 -0700 (PDT) From: Muchun Song To: corbet@lwn.net, mike.kravetz@oracle.com, tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, x86@kernel.org, hpa@zytor.com, dave.hansen@linux.intel.com, luto@kernel.org, peterz@infradead.org, viro@zeniv.linux.org.uk, akpm@linux-foundation.org, paulmck@kernel.org, pawan.kumar.gupta@linux.intel.com, rdunlap@infradead.org, oneukum@suse.com, anshuman.khandual@arm.com, jroedel@suse.de, almasrymina@google.com, rientjes@google.com, willy@infradead.org, osalvador@suse.de, mhocko@suse.com, song.bao.hua@hisilicon.com, david@redhat.com, naoya.horiguchi@nec.com, joao.m.martins@oracle.com Cc: duanxiongchun@bytedance.com, fam.zheng@bytedance.com, zhengqi.arch@bytedance.com, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, Muchun Song , Miaohe Lin , Chen Huang , Bodeddula Balasubramaniam Subject: [PATCH v23 7/9] mm: hugetlb: add a kernel parameter hugetlb_free_vmemmap Date: Mon, 10 May 2021 11:00:25 +0800 Message-Id: <20210510030027.56044-8-songmuchun@bytedance.com> X-Mailer: git-send-email 2.21.0 (Apple Git-122) In-Reply-To: <20210510030027.56044-1-songmuchun@bytedance.com> References: <20210510030027.56044-1-songmuchun@bytedance.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Add a kernel parameter hugetlb_free_vmemmap to enable the feature of freeing unused vmemmap pages associated with each hugetlb page on boot. We disables PMD mapping of vmemmap pages for x86-64 arch when this feature is enabled. Because vmemmap_remap_free() depends on vmemmap being base page mapped. Signed-off-by: Muchun Song Reviewed-by: Oscar Salvador Reviewed-by: Barry Song Reviewed-by: Miaohe Lin Tested-by: Chen Huang Tested-by: Bodeddula Balasubramaniam Reviewed-by: Mike Kravetz --- Documentation/admin-guide/kernel-parameters.txt | 17 +++++++++++++++++ Documentation/admin-guide/mm/hugetlbpage.rst | 3 +++ arch/x86/mm/init_64.c | 8 ++++++-- include/linux/hugetlb.h | 19 +++++++++++++++++++ mm/hugetlb_vmemmap.c | 24 ++++++++++++++++++++++++ 5 files changed, 69 insertions(+), 2 deletions(-) diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt index 1d56ad77189b..3cc19cb78b85 100644 --- a/Documentation/admin-guide/kernel-parameters.txt +++ b/Documentation/admin-guide/kernel-parameters.txt @@ -1621,6 +1621,23 @@ Documentation/admin-guide/mm/hugetlbpage.rst. Format: size[KMG] + hugetlb_free_vmemmap= + [KNL] Reguires CONFIG_HUGETLB_PAGE_FREE_VMEMMAP + enabled. + Allows heavy hugetlb users to free up some more + memory (6 * PAGE_SIZE for each 2MB hugetlb page). + This feauture is not free though. Large page + tables are not used to back vmemmap pages which + can lead to a performance degradation for some + workloads. Also there will be memory allocation + required when hugetlb pages are freed from the + pool which can lead to corner cases under heavy + memory pressure. + Format: { on | off (default) } + + on: enable the feature + off: disable the feature + hung_task_panic= [KNL] Should the hung task detector generate panics. Format: 0 | 1 diff --git a/Documentation/admin-guide/mm/hugetlbpage.rst b/Documentation/admin-guide/mm/hugetlbpage.rst index 6988895d09a8..8abaeb144e44 100644 --- a/Documentation/admin-guide/mm/hugetlbpage.rst +++ b/Documentation/admin-guide/mm/hugetlbpage.rst @@ -153,6 +153,9 @@ default_hugepagesz will all result in 256 2M huge pages being allocated. Valid default huge page size is architecture dependent. +hugetlb_free_vmemmap + When CONFIG_HUGETLB_PAGE_FREE_VMEMMAP is set, this enables freeing + unused vmemmap pages associated with each HugeTLB page. When multiple huge page sizes are supported, ``/proc/sys/vm/nr_hugepages`` indicates the current number of pre-allocated huge pages of the default size. diff --git a/arch/x86/mm/init_64.c b/arch/x86/mm/init_64.c index 65ea58527176..9d9d18d0c2a1 100644 --- a/arch/x86/mm/init_64.c +++ b/arch/x86/mm/init_64.c @@ -34,6 +34,7 @@ #include #include #include +#include #include #include @@ -1609,7 +1610,8 @@ int __meminit vmemmap_populate(unsigned long start, unsigned long end, int node, VM_BUG_ON(!IS_ALIGNED(start, PAGE_SIZE)); VM_BUG_ON(!IS_ALIGNED(end, PAGE_SIZE)); - if (end - start < PAGES_PER_SECTION * sizeof(struct page)) + if ((is_hugetlb_free_vmemmap_enabled() && !altmap) || + end - start < PAGES_PER_SECTION * sizeof(struct page)) err = vmemmap_populate_basepages(start, end, node, NULL); else if (boot_cpu_has(X86_FEATURE_PSE)) err = vmemmap_populate_hugepages(start, end, node, altmap); @@ -1637,6 +1639,8 @@ void register_page_bootmem_memmap(unsigned long section_nr, pmd_t *pmd; unsigned int nr_pmd_pages; struct page *page; + bool base_mapping = !boot_cpu_has(X86_FEATURE_PSE) || + is_hugetlb_free_vmemmap_enabled(); for (; addr < end; addr = next) { pte_t *pte = NULL; @@ -1662,7 +1666,7 @@ void register_page_bootmem_memmap(unsigned long section_nr, } get_page_bootmem(section_nr, pud_page(*pud), MIX_SECTION_INFO); - if (!boot_cpu_has(X86_FEATURE_PSE)) { + if (base_mapping) { next = (addr + PAGE_SIZE) & PAGE_MASK; pmd = pmd_offset(pud, addr); if (pmd_none(*pmd)) diff --git a/include/linux/hugetlb.h b/include/linux/hugetlb.h index c5cc16af897c..3258177f6494 100644 --- a/include/linux/hugetlb.h +++ b/include/linux/hugetlb.h @@ -895,6 +895,20 @@ static inline void huge_ptep_modify_prot_commit(struct vm_area_struct *vma, } #endif +#ifdef CONFIG_HUGETLB_PAGE_FREE_VMEMMAP +extern bool hugetlb_free_vmemmap_enabled; + +static inline bool is_hugetlb_free_vmemmap_enabled(void) +{ + return hugetlb_free_vmemmap_enabled; +} +#else +static inline bool is_hugetlb_free_vmemmap_enabled(void) +{ + return false; +} +#endif + #else /* CONFIG_HUGETLB_PAGE */ struct hstate {}; @@ -1054,6 +1068,11 @@ static inline void set_huge_swap_pte_at(struct mm_struct *mm, unsigned long addr pte_t *ptep, pte_t pte, unsigned long sz) { } + +static inline bool is_hugetlb_free_vmemmap_enabled(void) +{ + return false; +} #endif /* CONFIG_HUGETLB_PAGE */ static inline spinlock_t *huge_pte_lock(struct hstate *h, diff --git a/mm/hugetlb_vmemmap.c b/mm/hugetlb_vmemmap.c index a897c7778246..3070e1465b1b 100644 --- a/mm/hugetlb_vmemmap.c +++ b/mm/hugetlb_vmemmap.c @@ -168,6 +168,8 @@ * (last) level. So this type of HugeTLB page can be optimized only when its * size of the struct page structs is greater than 2 pages. */ +#define pr_fmt(fmt) "HugeTLB: " fmt + #include "hugetlb_vmemmap.h" /* @@ -180,6 +182,28 @@ #define RESERVE_VMEMMAP_NR 2U #define RESERVE_VMEMMAP_SIZE (RESERVE_VMEMMAP_NR << PAGE_SHIFT) +bool hugetlb_free_vmemmap_enabled; + +static int __init early_hugetlb_free_vmemmap_param(char *buf) +{ + /* We cannot optimize if a "struct page" crosses page boundaries. */ + if ((!is_power_of_2(sizeof(struct page)))) { + pr_warn("cannot free vmemmap pages because \"struct page\" crosses page boundaries\n"); + return 0; + } + + if (!buf) + return -EINVAL; + + if (!strcmp(buf, "on")) + hugetlb_free_vmemmap_enabled = true; + else if (strcmp(buf, "off")) + return -EINVAL; + + return 0; +} +early_param("hugetlb_free_vmemmap", early_hugetlb_free_vmemmap_param); + static inline unsigned long free_vmemmap_pages_size_per_hpage(struct hstate *h) { return (unsigned long)free_vmemmap_pages_per_hpage(h) << PAGE_SHIFT; -- 2.11.0