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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5D3D6C433F5 for ; Thu, 21 Oct 2021 01:14:35 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 4675A611CC for ; Thu, 21 Oct 2021 01:14:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231472AbhJUBQs (ORCPT ); Wed, 20 Oct 2021 21:16:48 -0400 Received: from szxga08-in.huawei.com ([45.249.212.255]:26105 "EHLO szxga08-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230268AbhJUBQq (ORCPT ); Wed, 20 Oct 2021 21:16:46 -0400 Received: from dggemv704-chm.china.huawei.com (unknown [172.30.72.54]) by szxga08-in.huawei.com (SkyGuard) with ESMTP id 4HZTx516ZTz1DHb1; Thu, 21 Oct 2021 09:12:41 +0800 (CST) Received: from dggpemm500006.china.huawei.com (7.185.36.236) by dggemv704-chm.china.huawei.com (10.3.19.47) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.15; Thu, 21 Oct 2021 09:14:28 +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.15; Thu, 21 Oct 2021 09:14:27 +0800 Subject: Re: [PATCH v15 09/10] of: fdt: Add memory for devices by DT property "linux,usable-memory-range" To: Rob Herring CC: Thomas Gleixner , Ingo Molnar , Borislav Petkov , , "H . Peter Anvin" , , Dave Young , Baoquan He , Vivek Goyal , Eric Biederman , , Catalin Marinas , "Will Deacon" , , "Frank Rowand" , , "Jonathan Corbet" , , Randy Dunlap , Nicolas Saenz Julienne , Feng Zhou , Kefeng Wang References: <20211020020317.1220-1-thunder.leizhen@huawei.com> <20211020020317.1220-10-thunder.leizhen@huawei.com> From: "Leizhen (ThunderTown)" Message-ID: Date: Thu, 21 Oct 2021 09:14:26 +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-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.174.178.55] X-ClientProxiedBy: dggems704-chm.china.huawei.com (10.3.19.181) To dggpemm500006.china.huawei.com (7.185.36.236) X-CFilter-Loop: Reflected Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2021/10/20 22:19, Rob Herring wrote: > On Wed, Oct 20, 2021 at 10:03:16AM +0800, Zhen Lei wrote: >> From: Chen Zhou >> >> When reserving crashkernel in high memory, some low memory is reserved >> for crash dump kernel devices and never mapped by the first kernel. >> This memory range is advertised to crash dump kernel via DT property >> under /chosen, >> linux,usable-memory-range = >> >> We reused the DT property linux,usable-memory-range and made the low >> memory region as the second range "BASE2 SIZE2", which keeps compatibility >> with existing user-space and older kdump kernels. >> >> Crash dump kernel reads this property at boot time and call memblock_add() >> to add the low memory region after memblock_cap_memory_range() has been >> called. >> >> Signed-off-by: Chen Zhou >> Signed-off-by: Zhen Lei >> --- >> drivers/of/fdt.c | 47 ++++++++++++++++++++++++++++++++++++----------- >> 1 file changed, 36 insertions(+), 11 deletions(-) >> >> diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c >> index 4546572af24bbf1..cf59c847b2c28a5 100644 >> --- a/drivers/of/fdt.c >> +++ b/drivers/of/fdt.c >> @@ -969,8 +969,16 @@ static void __init early_init_dt_check_for_elfcorehdr(unsigned long node) >> elfcorehdr_addr, elfcorehdr_size); >> } >> >> -static phys_addr_t cap_mem_addr; >> -static phys_addr_t cap_mem_size; >> +/* >> + * The main usage of linux,usable-memory-range is for crash dump kernel. >> + * Originally, the number of usable-memory regions is one. Now there may >> + * be two regions, low region and high region. >> + * To make compatibility with existing user-space and older kdump, the low >> + * region is always the last range of linux,usable-memory-range if exist. >> + */ >> +#define MAX_USABLE_RANGES 2 >> + >> +static struct memblock_region cap_mem_regions[MAX_USABLE_RANGES]; >> >> /** >> * early_init_dt_check_for_usable_mem_range - Decode usable memory range >> @@ -979,20 +987,30 @@ static phys_addr_t cap_mem_size; >> */ >> static void __init early_init_dt_check_for_usable_mem_range(unsigned long node) >> { >> - const __be32 *prop; >> - int len; >> + const __be32 *prop, *endp; >> + int len, nr = 0; >> + struct memblock_region *rgn = &cap_mem_regions[0]; >> >> pr_debug("Looking for usable-memory-range property... "); >> >> prop = of_get_flat_dt_prop(node, "linux,usable-memory-range", &len); >> - if (!prop || (len < (dt_root_addr_cells + dt_root_size_cells))) >> + if (!prop) >> return; >> >> - cap_mem_addr = dt_mem_next_cell(dt_root_addr_cells, &prop); >> - cap_mem_size = dt_mem_next_cell(dt_root_size_cells, &prop); >> + endp = prop + (len / sizeof(__be32)); >> + while ((endp - prop) >= (dt_root_addr_cells + dt_root_size_cells)) { >> + rgn->base = dt_mem_next_cell(dt_root_addr_cells, &prop); >> + rgn->size = dt_mem_next_cell(dt_root_size_cells, &prop); >> + >> + pr_debug("cap_mem_regions[%d]: base=%pa, size=%pa\n", >> + nr, &rgn->base, &rgn->size); >> + >> + if (++nr >= MAX_USABLE_RANGES) >> + break; >> + >> + rgn++; >> + } >> >> - pr_debug("cap_mem_start=%pa cap_mem_size=%pa\n", &cap_mem_addr, >> - &cap_mem_size); >> } >> >> #ifdef CONFIG_SERIAL_EARLYCON >> @@ -1265,7 +1283,8 @@ bool __init early_init_dt_verify(void *params) >> >> void __init early_init_dt_scan_nodes(void) >> { >> - int rc = 0; >> + int i, rc = 0; >> + struct memblock_region *rgn = &cap_mem_regions[0]; >> >> /* Initialize {size,address}-cells info */ >> of_scan_flat_dt(early_init_dt_scan_root, NULL); >> @@ -1279,7 +1298,13 @@ void __init early_init_dt_scan_nodes(void) >> of_scan_flat_dt(early_init_dt_scan_memory, NULL); >> >> /* Handle linux,usable-memory-range property */ >> - memblock_cap_memory_range(cap_mem_addr, cap_mem_size); >> + memblock_cap_memory_range(rgn->base, rgn->size); >> + for (i = 1; i < MAX_USABLE_RANGES; i++) { >> + rgn++; > > Just use rgn[i]. OK. > >> + >> + if (rgn->size) > > This check can be in the 'for' conditions check. Yes, this node is added by kexec tool, it is impossible that the first range is zero and the second range is not zero. > >> + memblock_add(rgn->base, rgn->size); >> + } > > > There's not really any point in doing all this in 2 steps. I'm > assuming this needs to be handled after scanning the memory nodes, so > can you refactor this moving early_init_dt_check_for_usable_mem_range > out of early_init_dt_scan_chosen() and call it here. You'll have to get > the offset for /chosen twice or save the offset. This's a good suggestion. I'll refactor it. Thanks. Zhen Lei > > Rob > . >