From patchwork Wed Jul 14 12:50:10 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Geert Uytterhoeven X-Patchwork-Id: 12376871 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=-11.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT 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 524FDC07E9C for ; Wed, 14 Jul 2021 12:51:06 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 05901613B6 for ; Wed, 14 Jul 2021 12:51:06 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 05901613B6 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=glider.be Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 6A32B8D0007; Wed, 14 Jul 2021 08:50:52 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 654798D0006; Wed, 14 Jul 2021 08:50:52 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 20E3B8D0007; Wed, 14 Jul 2021 08:50:52 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0079.hostedemail.com [216.40.44.79]) by kanga.kvack.org (Postfix) with ESMTP id E8EFA8D0005 for ; Wed, 14 Jul 2021 08:50:51 -0400 (EDT) Received: from smtpin22.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay01.hostedemail.com (Postfix) with ESMTP id D6C35183C8AA3 for ; Wed, 14 Jul 2021 12:50:50 +0000 (UTC) X-FDA: 78361177860.22.1117F4D Received: from baptiste.telenet-ops.be (baptiste.telenet-ops.be [195.130.132.51]) by imf11.hostedemail.com (Postfix) with ESMTP id 66462F000204 for ; Wed, 14 Jul 2021 12:50:50 +0000 (UTC) Received: from ramsan.of.borg ([IPv6:2a02:1810:ac12:ed10:39cc:190a:2775:cfe7]) by baptiste.telenet-ops.be with bizsmtp id V0qW250061ccfby010qWMr; Wed, 14 Jul 2021 14:50:49 +0200 Received: from rox.of.borg ([192.168.97.57]) by ramsan.of.borg with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.93) (envelope-from ) id 1m3eLO-0018dL-En; Wed, 14 Jul 2021 14:50:30 +0200 Received: from geert by rox.of.borg with local (Exim 4.93) (envelope-from ) id 1m3eLN-00AaDl-Lw; Wed, 14 Jul 2021 14:50:29 +0200 From: Geert Uytterhoeven To: Rob Herring , Russell King , Nicolas Pitre , Ard Biesheuvel , Linus Walleij , Catalin Marinas , Will Deacon , Nick Kossifidis , Paul Walmsley , Palmer Dabbelt , Albert Ou , Frank Rowand , Dave Young , Mike Rapoport Cc: Baoquan He , Vivek Goyal , Andrew Morton , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-riscv@lists.infradead.org, kexec@lists.infradead.org, linux-mm@kvack.org, linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Geert Uytterhoeven Subject: [PATCH v4 00/10] Add generic support for kdump DT properties Date: Wed, 14 Jul 2021 14:50:10 +0200 Message-Id: X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Authentication-Results: imf11.hostedemail.com; dkim=none; spf=none (imf11.hostedemail.com: domain of geert@linux-m68k.org has no SPF policy when checking 195.130.132.51) smtp.mailfrom=geert@linux-m68k.org; dmarc=none X-Stat-Signature: hecinny78gprk3knjxugyw373na9yomj X-Rspamd-Queue-Id: 66462F000204 X-Rspamd-Server: rspam01 X-HE-Tag: 1626267050-73313 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 all, This patch series adds generic support for parsing DT properties related to crash dump kernels ("linux,elfcorehdr" and "linux,elfcorehdr" under the "/chosen" node), makes use of it on arm64, arm32, and (partially) riscv, and performs a few cleanups. It is an evolution of the combination of [1] and [2]. The series consists of 5 parts: 1. Patches 1-2 prepare (the visibility of) variables used to hold information retrieved from the DT properties. 2. Patches 3-5 add support to the FDT core for parsing the properties. This can co-exist safely with architecture-specific parsing, until the latter has been removed. 3. Patch 6 removes the non-standard handling of "linux,elfcorehdr" on riscv. I think this can be applied independently, as the non-standard handling is in v5.13, but upstream riscv kdump support is still incomplete. 4. Patches 7-9 convert arm64 to use the generic handling instead of its own implementation. 5. Patch 10 adds support for kdump properties to arm32. The corresponding patch for kexec-tools is "[PATCH] arm: kdump: Add DT properties to crash dump kernel's DTB"[3], which is still valid. Changes compared to the previous versions: - Make elfcorehdr_{addr,size} always visible, - Add variables for usable memory limitation, - Use IS_ENABLED() instead of #ifdef (incl. initrd and arm64), - Clarify what architecture-specific code is still responsible for, - Add generic support for parsing linux,usable-memory-range, - Remove custom linux,usable-memory-range parsing on arm64, - Use generic handling on ARM. This has been tested on arm32 and arm64, and compile-tested on riscv64. Thanks for your comments! [1] "[PATCH v3] ARM: Parse kdump DT properties" https://lore.kernel.org/r/20210317113130.2554368-1-geert+renesas@glider.be/ [2] "[PATCH 0/3] Add generic-support for linux,elfcorehdr and fix riscv" https://lore.kernel.org/r/cover.1623780059.git.geert+renesas@glider.be/ [3] "[PATCH] arm: kdump: Add DT properties to crash dump kernel's DTB" https://lore.kernel.org/linux-arm-kernel/20200902154129.6358-1-geert+renesas@glider.be/ Geert Uytterhoeven (10): crash_dump: Make elfcorehdr_{addr,size} always visible memblock: Add variables for usable memory limitation of: fdt: Add generic support for parsing elf core headers property of: fdt: Add generic support for parsing usable memory range property of: fdt: Use IS_ENABLED(CONFIG_BLK_DEV_INITRD) instead of #ifdef riscv: Remove non-standard linux,elfcorehdr handling arm64: kdump: Remove custom linux,elfcorehdr parsing arm64: kdump: Remove custom linux,usable-memory-range parsing arm64: kdump: Use IS_ENABLED(CONFIG_CRASH_DUMP) instead of #ifdef ARM: Parse kdump DT properties Documentation/devicetree/bindings/chosen.txt | 12 ++-- .../arm/boot/compressed/fdt_check_mem_start.c | 48 ++++++++++++-- arch/arm/mm/init.c | 30 +++++++++ arch/arm64/mm/init.c | 63 +------------------ arch/riscv/mm/init.c | 20 ------ drivers/of/fdt.c | 63 ++++++++++++++++--- include/linux/crash_dump.h | 7 ++- include/linux/memblock.h | 6 ++ mm/memblock.c | 2 + 9 files changed, 148 insertions(+), 103 deletions(-)