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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B7291C43334 for ; Fri, 17 Jun 2022 08:27:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=6GvaxLVoXn/dCL4IRGaZcuwZgNyOjXy7Jo+F7SMGc3c=; b=MIBcLB0H8fg7Y0 KIM1jB/UOPwwBhweC3E7SaFsO9tn7VRii69EAppyMlxy8dHWYae3GW+fBsjrq6hiGndebUvJ1gu5E xUUFNUJPIDaGvHYgXCmrKvEXPf+X+6FpV8Livetcx09DcwrY98WYs6hPN2fTmzVL4D3r0BbO3sDga t4c0VawZa0T4MHZBNdwzJ8xVyHriNyIUEIbh6ejFDJBPKd5pL/3F4qgdOSPPOEi4XxM2bkC76MAjM G4eQi4RBrdycWRdXQbPcx4fEyFd9cUoWmZTw5od61AZmt8+V5PI81pqYHuOUAHqSONMH5GCBJcYK5 whkYA7S6l37+LZCPiuUw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o27J4-006IE2-Ig; Fri, 17 Jun 2022 08:26:18 +0000 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1o27J0-006IC0-Gh for kexec@lists.infradead.org; Fri, 17 Jun 2022 08:26:16 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1655454373; h=from:from:reply-to:subject:subject: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=hTHK8486lIIQN7NgMgmBqHT4WzEpoAXoz+nMVgMQJgE=; b=bkg17Hgm/Mb48+Yzv0qDrha0bW2n9fJ2VrhTYPYTlYWak6xlIItAUyb0cLbK6iDygaejDq KL5M2kpIBvEPTDwuIseaV9UQ9SIm41WcBVD1M9kZp+y8pFuxWVXgQjkWT/CdtIUlow8SMZ Oi01ruXzv/TPMHCS9HIDvKRCxgfbJTQ= Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-463-Oa1fsMubMqqWgbWDxIYN4Q-1; Fri, 17 Jun 2022 04:26:09 -0400 X-MC-Unique: Oa1fsMubMqqWgbWDxIYN4Q-1 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.rdu2.redhat.com [10.11.54.7]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id C34AC3C10223; Fri, 17 Jun 2022 08:26:08 +0000 (UTC) Received: from localhost (ovpn-12-144.pek2.redhat.com [10.72.12.144]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 43C211415106; Fri, 17 Jun 2022 08:26:07 +0000 (UTC) Date: Fri, 17 Jun 2022 16:26:02 +0800 From: Baoquan He To: Zhen Lei , msalter@redhat.com, ctatman@redhat.com Cc: Thomas Gleixner , Ingo Molnar , Borislav Petkov , x86@kernel.org, "H . Peter Anvin" , Eric Biederman , Rob Herring , Frank Rowand , devicetree@vger.kernel.org, Dave Young , Vivek Goyal , kexec@lists.infradead.org, linux-kernel@vger.kernel.org, Catalin Marinas , Will Deacon , linux-arm-kernel@lists.infradead.org, Jonathan Corbet , linux-doc@vger.kernel.org, Randy Dunlap , Feng Zhou , Kefeng Wang , Chen Zhou , John Donnelly , Dave Kleikamp Subject: Re: [PATCH 1/5] arm64: kdump: Provide default size when crashkernel=Y,low is not specified Message-ID: <20220617082602.GE234358@MiWiFi-R3L-srv> References: <20220613080932.663-1-thunder.leizhen@huawei.com> <20220613080932.663-2-thunder.leizhen@huawei.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220613080932.663-2-thunder.leizhen@huawei.com> X-Scanned-By: MIMEDefang 2.85 on 10.11.54.7 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220617_012615_102240_44F963A4 X-CRM114-Status: GOOD ( 32.12 ) X-BeenThere: kexec@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "kexec" Errors-To: kexec-bounces+kexec=archiver.kernel.org@lists.infradead.org On 06/13/22 at 04:09pm, Zhen Lei wrote: > To be consistent with the implementation of x86 and improve cross-platform > user experience. Try to allocate at least 256 MiB low memory automatically > when crashkernel=Y,low is not specified. > > Signed-off-by: Zhen Lei > --- > Documentation/admin-guide/kernel-parameters.txt | 8 +------- > arch/arm64/mm/init.c | 12 +++++++++++- > 2 files changed, 12 insertions(+), 8 deletions(-) > > diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt > index 8090130b544b070..61b179232b68001 100644 > --- a/Documentation/admin-guide/kernel-parameters.txt > +++ b/Documentation/admin-guide/kernel-parameters.txt > @@ -843,7 +843,7 @@ > available. > It will be ignored if crashkernel=X is specified. > crashkernel=size[KMG],low > - [KNL, X86-64] range under 4G. When crashkernel=X,high > + [KNL, X86-64, ARM64] range under 4G. When crashkernel=X,high > is passed, kernel could allocate physical memory region > above 4G, that cause second kernel crash on system > that require some amount of low memory, e.g. swiotlb > @@ -857,12 +857,6 @@ > It will be ignored when crashkernel=X,high is not used > or memory reserved is below 4G. > > - [KNL, ARM64] range in low memory. > - This one lets the user specify a low range in the > - DMA zone for the crash dump kernel. > - It will be ignored when crashkernel=X,high is not used > - or memory reserved is located in the DMA zones. > - > cryptomgr.notests > [KNL] Disable crypto self-tests > > diff --git a/arch/arm64/mm/init.c b/arch/arm64/mm/init.c > index 339ee84e5a61a0b..5390f361208ccf7 100644 > --- a/arch/arm64/mm/init.c > +++ b/arch/arm64/mm/init.c > @@ -96,6 +96,14 @@ phys_addr_t __ro_after_init arm64_dma_phys_limit = PHYS_MASK + 1; > #define CRASH_ADDR_LOW_MAX arm64_dma_phys_limit > #define CRASH_ADDR_HIGH_MAX (PHYS_MASK + 1) > > +/* > + * This is an empirical value in x86_64 and taken here directly. Please > + * refer to the code comment in reserve_crashkernel_low() of x86_64 for more > + * details. > + */ > +#define DEFAULT_CRASH_KERNEL_LOW_SIZE \ > + max(swiotlb_size_or_default() + (8UL << 20), 256UL << 20) About this default low value, 256M, I am not sure if it can be lowered down. We have Ampere Mt-Jade systems in Redhat and their biggest contiguous memory is less than 256M under low 4G when the firmware 32bit option is disabled. Obviously this will fail the default crashkernel,low value. I am not sure how common the 32bit option is disabled. If it's an important feature and widely set, it need be taken into consideration when deciding this default crashkernel,low value. Otherwise, the crashkernel=xM won't work in Ampere Mt-Jade system with 32bit option disabled, and people need specify crashkernel=xM,high, crashkernel=yM,low explicitly. The omission of crashkernel,low is also not allowed in the case. Hi, Mark and Christopher, Add you in CC. If you happen to know contact person from Ampere, please also feel free to add them in this thread. Thanks Baoquan > static int __init reserve_crashkernel_low(unsigned long long low_size) > { > unsigned long long low_base; > @@ -147,7 +155,9 @@ static void __init reserve_crashkernel(void) > * is not allowed. > */ > ret = parse_crashkernel_low(cmdline, 0, &crash_low_size, &crash_base); > - if (ret && (ret != -ENOENT)) > + if (ret == -ENOENT) > + crash_low_size = DEFAULT_CRASH_KERNEL_LOW_SIZE; > + else if (ret) > return; > > crash_max = CRASH_ADDR_HIGH_MAX; > -- > 2.25.1 > _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec