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=-15.5 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 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 C413FC433E0 for ; Wed, 6 Jan 2021 06:11:11 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 3CDA322CE3 for ; Wed, 6 Jan 2021 06:11:11 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 3CDA322CE3 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id A14778D00EB; Wed, 6 Jan 2021 01:11:10 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 9C5448D0090; Wed, 6 Jan 2021 01:11:10 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8DB038D00EB; Wed, 6 Jan 2021 01:11:10 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0150.hostedemail.com [216.40.44.150]) by kanga.kvack.org (Postfix) with ESMTP id 7775A8D0090 for ; Wed, 6 Jan 2021 01:11:10 -0500 (EST) Received: from smtpin21.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay02.hostedemail.com (Postfix) with ESMTP id 45B6C362D for ; Wed, 6 Jan 2021 06:11:10 +0000 (UTC) X-FDA: 77674327500.21.smile24_210a178274df Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin21.hostedemail.com (Postfix) with ESMTP id 1BA71180442C0 for ; Wed, 6 Jan 2021 06:11:10 +0000 (UTC) X-HE-Tag: smile24_210a178274df X-Filterd-Recvd-Size: 4501 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by imf26.hostedemail.com (Postfix) with ESMTP for ; Wed, 6 Jan 2021 06:11:09 +0000 (UTC) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id A8C4E31B; Tue, 5 Jan 2021 22:11:08 -0800 (PST) Received: from [10.163.87.111] (unknown [10.163.87.111]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 614FE3F70D; Tue, 5 Jan 2021 22:11:07 -0800 (PST) Subject: Re: [PATCH 1/1] arm64: make section size configurable for memory hotplug To: Sudarshan Rajagopalan , akpm@linux-foundation.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org References: <66f79b0c06602c22df4da8ff4a5c2b97c9275250.1609895500.git.sudaraja@codeaurora.org> From: Anshuman Khandual Message-ID: <055b0aca-af60-12ad-cd68-e15440ade64b@arm.com> Date: Wed, 6 Jan 2021 11:41:26 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <66f79b0c06602c22df4da8ff4a5c2b97c9275250.1609895500.git.sudaraja@codeaurora.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit 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: Hi Sudershan, This patch (and the cover letter) does not copy LAKML even though the entire change here is arm64 specific. Please do copy all applicable mailing lists for a given patch. On 1/6/21 6:58 AM, Sudarshan Rajagopalan wrote: > Currently on arm64, memory section size is hard-coded to 1GB. > Make this configurable if memory-hotplug is enabled, to support > more finer granularity for hotplug-able memory. Section size has always been decided by the platform. It cannot be a configurable option because the user would not know the constraints for memory representation on the platform and besides it also cannot be trusted. > > Signed-off-by: Sudarshan Rajagopalan > --- > arch/arm64/Kconfig | 11 +++++++++++ > arch/arm64/include/asm/sparsemem.h | 4 ++++ > 2 files changed, 15 insertions(+) > > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig > index 6d232837cbee..34124eee65da 100644 > --- a/arch/arm64/Kconfig > +++ b/arch/arm64/Kconfig > @@ -294,6 +294,17 @@ config ARCH_ENABLE_MEMORY_HOTREMOVE > config SMP > def_bool y > > +config HOTPLUG_SIZE_BITS > + int "Memory hotplug block size(29 => 512MB 30 => 1GB)" > + depends on SPARSEMEM > + depends on MEMORY_HOTPLUG > + range 28 30 28 would not work for 64K pages. > + default 30 > + help > + Selects granularity of hotplug memory. Block size for > + memory hotplug is represent as a power of 2. > + If unsure, stick with default value. > + > config KERNEL_MODE_NEON > def_bool y > > diff --git a/arch/arm64/include/asm/sparsemem.h b/arch/arm64/include/asm/sparsemem.h > index 1f43fcc79738..3d5310f3aad5 100644 > --- a/arch/arm64/include/asm/sparsemem.h > +++ b/arch/arm64/include/asm/sparsemem.h > @@ -7,7 +7,11 @@ > > #ifdef CONFIG_SPARSEMEM > #define MAX_PHYSMEM_BITS CONFIG_ARM64_PA_BITS > +#ifndef CONFIG_MEMORY_HOTPLUG > #define SECTION_SIZE_BITS 30 > +#else > +#define SECTION_SIZE_BITS CONFIG_HOTPLUG_SIZE_BITS > +#endif > #endif > > #endif > There was an inconclusive discussion regarding this last month. https://lore.kernel.org/linux-arm-kernel/20201204014443.43329-1-liwei213@huawei.com/ I have been wondering if this would solve the problem for 4K page size config which requires PMD mapping for the vmemmap mapping while making section size bits dependent on max order. But this has not been tested properly. diff --git a/arch/arm64/include/asm/sparsemem.h b/arch/arm64/include/asm/sparsemem.h index 1f43fcc79738..fe4353cb1dce 100644 --- a/arch/arm64/include/asm/sparsemem.h +++ b/arch/arm64/include/asm/sparsemem.h @@ -7,7 +7,18 @@ #ifdef CONFIG_SPARSEMEM #define MAX_PHYSMEM_BITS CONFIG_ARM64_PA_BITS -#define SECTION_SIZE_BITS 30 -#endif + +#ifdef CONFIG_ARM64_4K_PAGES +#define SECTION_SIZE_BITS 27 +#else +#ifdef CONFIG_FORCE_MAX_ZONEORDER +#define SECTION_SIZE_BITS (CONFIG_FORCE_MAX_ZONEORDER - 1 + PAGE_SHIFT) +#else +#define SECTION_SIZE_BITS 30 +#endif /* CONFIG_FORCE_MAX_ZONEORDER */ + +#endif /* CONFIG_ARM64_4K_PAGES */ + +#endif /* CONFIG_SPARSEMEM*/ #endif