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 CB42EC433F5 for ; Mon, 21 Feb 2022 06:40:56 +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:Date: Message-ID:From:References:CC:To:Subject:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=GF5XwdGxafzTn1JCi2KGUlBBAG/XVQeKMZr38cccfAc=; b=SkHeTvuaGYJuPTtdJdNrlnodRI h7Wdxn3IRZdq3SX7iiI7gNLH255YvQ8LGpvYN5utqYEMWk7ueP+Y9qFK9jwKptOzyeua65QtzbTiO wBltFoGHCDgYlGT6NjWmNuhIC4Dx5Za/Ct1rYv8BIs+Q34MFZ2lUEyqBNPIVDnDS6Ov0M7UxZPGIK YDnbsCn1VLWAA12orKrnx6ZMrZEiERnK1wZ2Ni//Gu6xcQKDaXKI0JqKtiCLfvT4YiQJGsxecEUFk Oq5i5HvaZE70Rzrt+Adtw1iFrYlXNnywB84PRj+l2BS0Pwv4r8jKbVzXSiuHzGeIlf0GhMJNKZ/gG 6YWez15A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nM2La-003kG5-VN; Mon, 21 Feb 2022 06:38:59 +0000 Received: from szxga02-in.huawei.com ([45.249.212.188]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nM2LH-003k8J-5V; Mon, 21 Feb 2022 06:38:46 +0000 Received: from dggpemm500023.china.huawei.com (unknown [172.30.72.57]) by szxga02-in.huawei.com (SkyGuard) with ESMTP id 4K2CK22CRMzcfdn; Mon, 21 Feb 2022 14:37:26 +0800 (CST) Received: from dggpemm500006.china.huawei.com (7.185.36.236) by dggpemm500023.china.huawei.com (7.185.36.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.21; Mon, 21 Feb 2022 14:38:36 +0800 Received: from [10.174.178.55] (10.174.178.55) by dggpemm500006.china.huawei.com (7.185.36.236) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.21; Mon, 21 Feb 2022 14:38:35 +0800 Subject: Re: [PATCH v20 5/5] kdump: update Documentation about crashkernel To: Baoquan He CC: Thomas Gleixner , Ingo Molnar , Borislav Petkov , , "H . Peter Anvin" , , Dave Young , Vivek Goyal , Eric Biederman , , Catalin Marinas , Will Deacon , , Rob Herring , Frank Rowand , , "Jonathan Corbet" , , Randy Dunlap , Feng Zhou , Kefeng Wang , Chen Zhou , "John Donnelly" , Dave Kleikamp References: <20220124084708.683-1-thunder.leizhen@huawei.com> <20220124084708.683-6-thunder.leizhen@huawei.com> From: "Leizhen (ThunderTown)" Message-ID: <8ada665b-1917-2f01-477f-baaa562f5502@huawei.com> Date: Mon, 21 Feb 2022 14:38:35 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-Originating-IP: [10.174.178.55] X-ClientProxiedBy: dggems703-chm.china.huawei.com (10.3.19.180) To dggpemm500006.china.huawei.com (7.185.36.236) X-CFilter-Loop: Reflected X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220220_223839_659113_0330296B X-CRM114-Status: GOOD ( 29.66 ) X-BeenThere: linux-arm-kernel@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: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 2022/2/21 11:48, Baoquan He wrote: > On 01/24/22 at 04:47pm, Zhen Lei wrote: >> From: Chen Zhou >> >> For arm64, the behavior of crashkernel=X has been changed, which >> tries low allocation in DMA zone and fall back to high allocation >> if it fails. >> >> We can also use "crashkernel=X,high" to select a high region above >> DMA zone, which also tries to allocate at least 256M low memory in >> DMA zone automatically and "crashkernel=Y,low" can be used to allocate >> specified size low memory. >> >> So update the Documentation. >> >> Signed-off-by: Chen Zhou >> Signed-off-by: Zhen Lei >> --- >> Documentation/admin-guide/kdump/kdump.rst | 11 +++++++++-- >> Documentation/admin-guide/kernel-parameters.txt | 11 +++++++++-- >> 2 files changed, 18 insertions(+), 4 deletions(-) >> >> diff --git a/Documentation/admin-guide/kdump/kdump.rst b/Documentation/admin-guide/kdump/kdump.rst >> index cb30ca3df27c9b2..d4c287044be0c70 100644 >> --- a/Documentation/admin-guide/kdump/kdump.rst >> +++ b/Documentation/admin-guide/kdump/kdump.rst >> @@ -361,8 +361,15 @@ Boot into System Kernel >> kernel will automatically locate the crash kernel image within the >> first 512MB of RAM if X is not given. >> >> - On arm64, use "crashkernel=Y[@X]". Note that the start address of >> - the kernel, X if explicitly specified, must be aligned to 2MiB (0x200000). >> + On arm64, use "crashkernel=X" to try low allocation in DMA zone and >> + fall back to high allocation if it fails. >> + We can also use "crashkernel=X,high" to select a high region above >> + DMA zone, which also tries to allocate at least 256M low memory in >> + DMA zone automatically. >> + "crashkernel=Y,low" can be used to allocate specified size low memory. >> + Use "crashkernel=Y@X" if you really have to reserve memory from >> + specified start address X. Note that the start address of the kernel, >> + X if explicitly specified, must be aligned to 2MiB (0x200000). > > Hmm, we may not need the details related to crashkernel,high|low in this > section. This just gives examples of basic configation for each ARCH. > The detailed configuration of all crashkernel setting can be found in > "crashkernel syntax" section, I don't think arm64 is so special to need > a specific one. > >> >> Load the Dump-capture Kernel >> ============================ >> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt >> index f5a27f067db9ed9..65780c2ca830be0 100644 >> --- a/Documentation/admin-guide/kernel-parameters.txt >> +++ b/Documentation/admin-guide/kernel-parameters.txt >> @@ -792,6 +792,9 @@ >> [KNL, X86-64] Select a region under 4G first, and >> fall back to reserve region above 4G when '@offset' >> hasn't been specified. >> + [KNL, ARM64] Try low allocation in DMA zone and fall back >> + to high allocation if it fails when '@offset' hasn't been >> + specified. >> See Documentation/admin-guide/kdump/kdump.rst for further details. > > How about add ARM64 like below to avoid redundant words? > [KNL, X86-64, ARM64] Select a region under 4G first, and > fall back to reserve region above 4G when '@offset' > hasn't been specified. I agree very much, and I think that's the right thing to do. Thanks. I wanted to do the same before, just wasn't sure if the format was correct. I just looked at the other descriptions in kernel-parameters.txt and found that there are many precedents. "movablecore= [KNL,X86,IA-64,PPC]" > >> >> crashkernel=range1:size1[,range2:size2,...][@offset] >> @@ -808,6 +811,8 @@ >> Otherwise memory region will be allocated below 4G, if >> available. >> It will be ignored if crashkernel=X is specified. >> + [KNL, ARM64] range in high memory. >> + Allow kernel to allocate physical memory region from top. > > Ditto, please don't add redundent words if it's similar to x86_64 > handling. > >> crashkernel=size[KMG],low >> [KNL, X86-64] range under 4G. When crashkernel=X,high >> is passed, kernel could allocate physical memory region >> @@ -816,13 +821,15 @@ >> requires at least 64M+32K low memory, also enough extra >> low memory is needed to make sure DMA buffers for 32-bit >> devices won't run out. Kernel would try to allocate at >> - at least 256M below 4G automatically. >> + least 256M below 4G automatically. >> This one let user to specify own low range under 4G >> for second kernel instead. >> 0: to disable low allocation. >> 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 let user to specify a low range in DMA zone for >> + crash dump kernel. > > Ditto. > >> cryptomgr.notests >> [KNL] Disable crypto self-tests >> >> -- >> 2.25.1 >> > > . > -- Regards, Zhen Lei _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel