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=-13.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,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 0E34BC433DF for ; Fri, 22 May 2020 03:24:26 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id EB13F20748 for ; Fri, 22 May 2020 03:24:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727995AbgEVDYZ (ORCPT ); Thu, 21 May 2020 23:24:25 -0400 Received: from szxga05-in.huawei.com ([45.249.212.191]:5272 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727024AbgEVDYY (ORCPT ); Thu, 21 May 2020 23:24:24 -0400 Received: from DGGEMS404-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id 51717A9057906BA0EFA3; Fri, 22 May 2020 11:24:22 +0800 (CST) Received: from [127.0.0.1] (10.166.213.90) by DGGEMS404-HUB.china.huawei.com (10.3.19.204) with Microsoft SMTP Server id 14.3.487.0; Fri, 22 May 2020 11:24:12 +0800 Subject: Re: [PATCH v8 5/5] dt-bindings: chosen: Document linux,low-memory-range for arm64 kdump To: Rob Herring References: <20200521093805.64398-1-chenzhou10@huawei.com> <20200521093805.64398-6-chenzhou10@huawei.com> CC: Thomas Gleixner , Ingo Molnar , Catalin Marinas , Will Deacon , , Baoquan He , Arnd Bergmann , , , "Simon Horman" , Hanjun Guo , "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" , , "Linux Doc Mailing List" , "linux-kernel@vger.kernel.org" , From: chenzhou Message-ID: Date: Fri, 22 May 2020 11:24:11 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.166.213.90] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, On 2020/5/21 21:29, Rob Herring wrote: > On Thu, May 21, 2020 at 3:35 AM Chen Zhou wrote: >> Add documentation for DT property used by arm64 kdump: >> linux,low-memory-range. >> "linux,low-memory-range" is an another memory region used for crash >> dump kernel devices. >> >> Signed-off-by: Chen Zhou >> --- >> Documentation/devicetree/bindings/chosen.txt | 25 ++++++++++++++++++++ >> 1 file changed, 25 insertions(+) > chosen is now a schema documented here[1]. Ok, that is, i don't need to modify the doc in kernel, just create a pull request in github [1]? > >> diff --git a/Documentation/devicetree/bindings/chosen.txt b/Documentation/devicetree/bindings/chosen.txt >> index 45e79172a646..bfe6fb6976e6 100644 >> --- a/Documentation/devicetree/bindings/chosen.txt >> +++ b/Documentation/devicetree/bindings/chosen.txt >> @@ -103,6 +103,31 @@ While this property does not represent a real hardware, the address >> and the size are expressed in #address-cells and #size-cells, >> respectively, of the root node. >> >> +linux,low-memory-range >> +---------------------- >> +This property (arm64 only) holds a base address and size, describing a >> +limited region below 4G. Similar to "linux,usable-memory-range", it is >> +an another memory range which may be considered available for use by the >> +kernel. > Why can't you just add a range to "linux,usable-memory-range"? It > shouldn't be hard to figure out which part is below 4G. I did like this in my previous version, such as v5. After discussed with James, i modified it to the current way. We think the existing behavior should be unchanged, which helps with keeping compatibility with existing user-space and older kdump kernels. The comments from James: > linux,usable-memory-range = . Won't this break if your kdump kernel doesn't know what the extra parameters are? Or if it expects two ranges, but only gets one? These DT properties should be treated as ABI between kernel versions, we can't really change it like this. I think the 'low' region is an optional-extra, that is never mapped by the first kernel. I think the simplest thing to do is to add an 'linux,low-memory-range' that we memblock_add() after memblock_cap_memory_range() has been called. If its missing, or the new kernel doesn't know what its for, everything keeps working. previous discusses: https://lkml.org/lkml/2019/6/5/674 https://lkml.org/lkml/2019/6/13/229 Thanks, Chen Zhou > > Rob > > [1] https://github.com/devicetree-org/dt-schema/blob/master/schemas/chosen.yaml > > . > 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=-13.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=unavailable 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 33744C433DF for ; Fri, 22 May 2020 03:24:45 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id C6A9320748 for ; Fri, 22 May 2020 03:24:44 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="ko2uAyIa" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C6A9320748 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=huawei.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date: Message-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description :Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=FxvhtIWTZqtWVND1kDmC0oCWKov/wQ96mMzu9N/O7Zw=; b=ko2uAyIafB45wP 11DIEGXcbqYYKj+9f46WfQeZ621iJa0wTc0yMJSCnRfMYVfms2MqU0l7y0J3YYDBMRIcHwAlG6Cix /CD7AhQQNChO6Bk+Cu04dv5LP/YZolcPzPez4gC3cAYfX1Yca0aOmiFoHFce0cWuX3KyEF0Fcne5X Y/KfGOl21uEyhX5xwp4zKwRww+ICkejoRijvA09PwFui2d4JgEG/PzL++RTkoKX4ubACuPRydjeQ1 wpq3EMqIXqCBJsanSjI5vBsC5vtovU5+TUGdqGjFjTVXm00CtnDptMo8gvL9uOyGMr4/CSjDnrdqy I4fZ9vEdQPENCmetrfIA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jbyIV-0007Oy-3O; Fri, 22 May 2020 03:24:35 +0000 Received: from szxga05-in.huawei.com ([45.249.212.191] helo=huawei.com) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jbyIR-0007Nc-6p; Fri, 22 May 2020 03:24:33 +0000 Received: from DGGEMS404-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id 51717A9057906BA0EFA3; Fri, 22 May 2020 11:24:22 +0800 (CST) Received: from [127.0.0.1] (10.166.213.90) by DGGEMS404-HUB.china.huawei.com (10.3.19.204) with Microsoft SMTP Server id 14.3.487.0; Fri, 22 May 2020 11:24:12 +0800 Subject: Re: [PATCH v8 5/5] dt-bindings: chosen: Document linux,low-memory-range for arm64 kdump To: Rob Herring References: <20200521093805.64398-1-chenzhou10@huawei.com> <20200521093805.64398-6-chenzhou10@huawei.com> From: chenzhou Message-ID: Date: Fri, 22 May 2020 11:24:11 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: X-Originating-IP: [10.166.213.90] X-CFilter-Loop: Reflected X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200521_202431_414675_FEE819FB X-CRM114-Status: GOOD ( 16.00 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Simon Horman , John.p.donnelly@oracle.com, Baoquan He , Will Deacon , devicetree@vger.kernel.org, Catalin Marinas , Linux Doc Mailing List , kexec@lists.infradead.org, "linux-kernel@vger.kernel.org" , Ingo Molnar , Arnd Bergmann , Hanjun Guo , Thomas Gleixner , pkushwaha@marvell.com, dyoung@redhat.com, "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Rob, On 2020/5/21 21:29, Rob Herring wrote: > On Thu, May 21, 2020 at 3:35 AM Chen Zhou wrote: >> Add documentation for DT property used by arm64 kdump: >> linux,low-memory-range. >> "linux,low-memory-range" is an another memory region used for crash >> dump kernel devices. >> >> Signed-off-by: Chen Zhou >> --- >> Documentation/devicetree/bindings/chosen.txt | 25 ++++++++++++++++++++ >> 1 file changed, 25 insertions(+) > chosen is now a schema documented here[1]. Ok, that is, i don't need to modify the doc in kernel, just create a pull request in github [1]? > >> diff --git a/Documentation/devicetree/bindings/chosen.txt b/Documentation/devicetree/bindings/chosen.txt >> index 45e79172a646..bfe6fb6976e6 100644 >> --- a/Documentation/devicetree/bindings/chosen.txt >> +++ b/Documentation/devicetree/bindings/chosen.txt >> @@ -103,6 +103,31 @@ While this property does not represent a real hardware, the address >> and the size are expressed in #address-cells and #size-cells, >> respectively, of the root node. >> >> +linux,low-memory-range >> +---------------------- >> +This property (arm64 only) holds a base address and size, describing a >> +limited region below 4G. Similar to "linux,usable-memory-range", it is >> +an another memory range which may be considered available for use by the >> +kernel. > Why can't you just add a range to "linux,usable-memory-range"? It > shouldn't be hard to figure out which part is below 4G. I did like this in my previous version, such as v5. After discussed with James, i modified it to the current way. We think the existing behavior should be unchanged, which helps with keeping compatibility with existing user-space and older kdump kernels. The comments from James: > linux,usable-memory-range = . Won't this break if your kdump kernel doesn't know what the extra parameters are? Or if it expects two ranges, but only gets one? These DT properties should be treated as ABI between kernel versions, we can't really change it like this. I think the 'low' region is an optional-extra, that is never mapped by the first kernel. I think the simplest thing to do is to add an 'linux,low-memory-range' that we memblock_add() after memblock_cap_memory_range() has been called. If its missing, or the new kernel doesn't know what its for, everything keeps working. previous discusses: https://lkml.org/lkml/2019/6/5/674 https://lkml.org/lkml/2019/6/13/229 Thanks, Chen Zhou > > Rob > > [1] https://github.com/devicetree-org/dt-schema/blob/master/schemas/chosen.yaml > > . > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Subject: Re: [PATCH v8 5/5] dt-bindings: chosen: Document linux,low-memory-range for arm64 kdump References: <20200521093805.64398-1-chenzhou10@huawei.com> <20200521093805.64398-6-chenzhou10@huawei.com> From: chenzhou Message-ID: Date: Fri, 22 May 2020 11:24:11 +0800 MIME-Version: 1.0 In-Reply-To: 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+dwmw2=infradead.org@lists.infradead.org To: Rob Herring Cc: Simon Horman , John.p.donnelly@oracle.com, Baoquan He , Will Deacon , devicetree@vger.kernel.org, Catalin Marinas , Linux Doc Mailing List , kexec@lists.infradead.org, "linux-kernel@vger.kernel.org" , Ingo Molnar , Arnd Bergmann , Hanjun Guo , Thomas Gleixner , pkushwaha@marvell.com, dyoung@redhat.com, "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" Hi Rob, On 2020/5/21 21:29, Rob Herring wrote: > On Thu, May 21, 2020 at 3:35 AM Chen Zhou wrote: >> Add documentation for DT property used by arm64 kdump: >> linux,low-memory-range. >> "linux,low-memory-range" is an another memory region used for crash >> dump kernel devices. >> >> Signed-off-by: Chen Zhou >> --- >> Documentation/devicetree/bindings/chosen.txt | 25 ++++++++++++++++++++ >> 1 file changed, 25 insertions(+) > chosen is now a schema documented here[1]. Ok, that is, i don't need to modify the doc in kernel, just create a pull request in github [1]? > >> diff --git a/Documentation/devicetree/bindings/chosen.txt b/Documentation/devicetree/bindings/chosen.txt >> index 45e79172a646..bfe6fb6976e6 100644 >> --- a/Documentation/devicetree/bindings/chosen.txt >> +++ b/Documentation/devicetree/bindings/chosen.txt >> @@ -103,6 +103,31 @@ While this property does not represent a real hardware, the address >> and the size are expressed in #address-cells and #size-cells, >> respectively, of the root node. >> >> +linux,low-memory-range >> +---------------------- >> +This property (arm64 only) holds a base address and size, describing a >> +limited region below 4G. Similar to "linux,usable-memory-range", it is >> +an another memory range which may be considered available for use by the >> +kernel. > Why can't you just add a range to "linux,usable-memory-range"? It > shouldn't be hard to figure out which part is below 4G. I did like this in my previous version, such as v5. After discussed with James, i modified it to the current way. We think the existing behavior should be unchanged, which helps with keeping compatibility with existing user-space and older kdump kernels. The comments from James: > linux,usable-memory-range = . Won't this break if your kdump kernel doesn't know what the extra parameters are? Or if it expects two ranges, but only gets one? These DT properties should be treated as ABI between kernel versions, we can't really change it like this. I think the 'low' region is an optional-extra, that is never mapped by the first kernel. I think the simplest thing to do is to add an 'linux,low-memory-range' that we memblock_add() after memblock_cap_memory_range() has been called. If its missing, or the new kernel doesn't know what its for, everything keeps working. previous discusses: https://lkml.org/lkml/2019/6/5/674 https://lkml.org/lkml/2019/6/13/229 Thanks, Chen Zhou > > Rob > > [1] https://github.com/devicetree-org/dt-schema/blob/master/schemas/chosen.yaml > > . > _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec