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.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 296D2C4338F for ; Fri, 23 Jul 2021 15:17:23 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 7166160F02 for ; Fri, 23 Jul 2021 15:17:22 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 7166160F02 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=linux-m68k.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=kvack.org Received: by kanga.kvack.org (Postfix) id BFA286B0033; Fri, 23 Jul 2021 11:17:21 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id BABDD6B005D; Fri, 23 Jul 2021 11:17:21 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id A98A66B006C; Fri, 23 Jul 2021 11:17:21 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0025.hostedemail.com [216.40.44.25]) by kanga.kvack.org (Postfix) with ESMTP id 8E92E6B0033 for ; Fri, 23 Jul 2021 11:17:21 -0400 (EDT) Received: from smtpin13.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay02.hostedemail.com (Postfix) with ESMTP id 3D28D202EE for ; Fri, 23 Jul 2021 15:17:21 +0000 (UTC) X-FDA: 78394206282.13.5EF4239 Received: from mail-ua1-f48.google.com (mail-ua1-f48.google.com [209.85.222.48]) by imf25.hostedemail.com (Postfix) with ESMTP id E063CB007D56 for ; Fri, 23 Jul 2021 15:17:20 +0000 (UTC) Received: by mail-ua1-f48.google.com with SMTP id n15so637773uao.6 for ; Fri, 23 Jul 2021 08:17:20 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Sie/IxwBmILJtt5CfCHMRTqh7diZyseV1fFMGEm6yOk=; b=Z54B/50A8ER79in826x9vah80ebWnbcLKl4Hq/MKQEpmDr5nV7/rulPPGJryRfqr41 c15w+/kkcf6TKCUn3YSTaMp0a4sMxJHwltHXfoC5sjtOdFrpnyxPRIJq9Kf+kD+tyzrV jrI0YnfSenYwCdX2E6e3Ptzbdh38tANzNvvYjNj0uuhWqhG7VmyO6BY9xMLFChCYbjnN lXc57s4HpLnfEIRiDMANNWvXOVjkMo9G/fuUB3uncUPajg2pAE/71atGjg0M+0ZnQZDo Qhf3OL2bv+SYMHQ0KBKRZnlGkdCCAJb3Pb0AvbeqZNP+VZpyb+fLv4p4jaRaxU/9Ew2s 4i7Q== X-Gm-Message-State: AOAM533JnE9tuI38az1d4uX2Y5jA2IQGVsDuxWUnExBmAiXSbIODBx/O n8LqS4LwUBvRuSc+SCA63kd+eAECGxX7gYFPimE= X-Google-Smtp-Source: ABdhPJwjie6QeabkOjY7IEyTELcNhWlJ1IBBcsCliDIJ6B3GHGV6iAk2cW/JLMoH9H/oEu8zcHMlVqite6WQsRrTwGc= X-Received: by 2002:ab0:2554:: with SMTP id l20mr5059882uan.58.1627053440203; Fri, 23 Jul 2021 08:17:20 -0700 (PDT) MIME-Version: 1.0 References: <170e7daa55aced148b8e57827f7c8fc65a0a8c7a.1626266516.git.geert+renesas@glider.be> In-Reply-To: From: Geert Uytterhoeven Date: Fri, 23 Jul 2021 17:17:08 +0200 Message-ID: Subject: Re: [PATCH v4 06/10] riscv: Remove non-standard linux,elfcorehdr handling To: Palmer Dabbelt Cc: Rob Herring , Russell King , Nicolas Pitre , Ard Biesheuvel , Linus Walleij , Catalin Marinas , Will Deacon , Nick Kossifidis , Paul Walmsley , Albert Ou , Frank Rowand , Dave Young , Mike Rapoport , Baoquan He , Vivek Goyal , Andrew Morton , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Linux ARM , linux-riscv , kexec@lists.infradead.org, Linux MM , Linux-Renesas , Linux Kernel Mailing List Content-Type: text/plain; charset="UTF-8" X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: E063CB007D56 Authentication-Results: imf25.hostedemail.com; dkim=none; spf=pass (imf25.hostedemail.com: domain of geertuytterhoeven@gmail.com designates 209.85.222.48 as permitted sender) smtp.mailfrom=geertuytterhoeven@gmail.com; dmarc=none X-Stat-Signature: tgymis8scisybb3if1hb14zcdnwt3bqk X-HE-Tag: 1627053440-15348 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 Palmer, On Tue, Jul 20, 2021 at 5:43 PM Palmer Dabbelt wrote: > On Wed, 14 Jul 2021 05:50:16 PDT (-0700), geert+renesas@glider.be wrote: > > RISC-V uses platform-specific code to locate the elf core header in > > memory. However, this does not conform to the standard > > "linux,elfcorehdr" DT bindings, as it relies on a reserved memory node > > with the "linux,elfcorehdr" compatible value, instead of on a > > "linux,elfcorehdr" property under the "/chosen" node. > > > > The non-compliant code can just be removed, as the standard behavior is > > already implemented by platform-agnostic handling in the FDT core code. > > > > Fixes: 5640975003d0234d ("RISC-V: Add crash kernel support") > > Signed-off-by: Geert Uytterhoeven > > --- > > v4: > > - No changes. > > --- > > arch/riscv/mm/init.c | 20 -------------------- > > 1 file changed, 20 deletions(-) > > > > diff --git a/arch/riscv/mm/init.c b/arch/riscv/mm/init.c > > index 269fc648ef3d84b2..4aa4b6a034086df6 100644 > > --- a/arch/riscv/mm/init.c > > +++ b/arch/riscv/mm/init.c > > @@ -829,26 +829,6 @@ static void __init reserve_crashkernel(void) > > } > > #endif /* CONFIG_KEXEC_CORE */ > > > > -#ifdef CONFIG_CRASH_DUMP > > -/* > > - * We keep track of the ELF core header of the crashed > > - * kernel with a reserved-memory region with compatible > > - * string "linux,elfcorehdr". Here we register a callback > > - * to populate elfcorehdr_addr/size when this region is > > - * present. Note that this region will be marked as > > - * reserved once we call early_init_fdt_scan_reserved_mem() > > - * later on. > > - */ > > -static int __init elfcore_hdr_setup(struct reserved_mem *rmem) > > -{ > > - elfcorehdr_addr = rmem->base; > > - elfcorehdr_size = rmem->size; > > - return 0; > > -} > > - > > -RESERVEDMEM_OF_DECLARE(elfcorehdr, "linux,elfcorehdr", elfcore_hdr_setup); > > -#endif > > - > > void __init paging_init(void) > > { > > setup_bootmem(); > > Reviewed-by: Palmer Dabbelt > Acked-by: Palmer Dabbelt > > LMK if you wanted me to take this series, otherwise I'm going to assume > it's going in via some other tree. There will be a v5 of the series in response to the review comments. But I think this patch can already be applied. Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds