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=-4.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no 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 07655C4338F for ; Mon, 23 Aug 2021 14:47:12 +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 BE5DE61184 for ; Mon, 23 Aug 2021 14:47:11 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org BE5DE61184 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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:Cc:To:Subject:Message-ID:Date:From: In-Reply-To:References:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=abzJwkBuzTzPftZfN4iezbzzSyZz+/iV1R+FIrK8bfo=; b=GAooVqYLdcspBC k8/lXb4BJm5aioGvPs/h7cO7osPpRggr5MO0HIOu8YDuTUKFt1AyE8rlyYdIcda+opuKVn+t/oArX xJiO9VRPyUVtAYDod/Drrzwm3lVs+r8bRfKrgC13YYutx922smtgcofLQPRMPTjevQbDCu+4WIEBk bLd1mtTYX+Rum29ctVFzPAUvc5cb70nj4Qxj0tsFEtuUcMwIQhGX7vb3nKILM5PFPfAHk1lDjc8g0 KxERbb37qqWEV51X2V8TAC0WT4CvNl7oQzl0u4omf6/cOxudWVIffkuAkfE7u6WgWTYIQIE9VaJTt OsieDxAgbQb1nC3f+Www==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mIBCM-00HHxo-PW; Mon, 23 Aug 2021 14:45:15 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mIBCH-00HHwW-5I; Mon, 23 Aug 2021 14:45:10 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 87EAE613CE; Mon, 23 Aug 2021 14:45:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1629729908; bh=HS2M+wkDtx/G0MZSg6+SDwYSA3teAsEDvrv+10PPD1A=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=qPKYxgfB9GVNO/ci0L6xILxIQPPJGBHIQJy4YMnj0Zw7zF7wGjfzynHM07q/hdtFJ fzjZSr+FNFIn8DCHJscN7WHtmTm/qhJneZMJJj08w7k3nJ838qAJNWc+vVoaXF87+1 iNcj8INOwKrfvditV+DxlUtQ8DX4lAbEXz/sDy2ACAgKAp+HvHvaZhS0+UZJN4VgMH GPAZlRaNM4E4saszqP0w189j7NDctQFAvm5aZhrYjAQHhd9mTXKAONYbcAz4L1fskn IpHuinmaVCOcnYZwI5rcTj2KMKe0BjZ4hk50UN58HZdjfm9uvmn9CsDQYSUqstrZ9J Jky99juD8VeBQ== Received: by mail-ej1-f43.google.com with SMTP id u3so37609167ejz.1; Mon, 23 Aug 2021 07:45:08 -0700 (PDT) X-Gm-Message-State: AOAM5316Ox8HFSYpEpZMlsXV4Aw3zxOn9swcvhLF3vWzh6XiEPbls2et ofOCdjczj1I86sjSd9dE8n7BKYNZeFZ3yLWdtw== X-Google-Smtp-Source: ABdhPJxGMCnkZ6Wg5JPxtvBTf03t6XhofzKZ3GpTAEgQlSG7jl4bDW3CMJL9Gl4hxiY+ePwWhCIra0pqlfZDMAM51Ag= X-Received: by 2002:a17:907:b06:: with SMTP id h6mr3650075ejl.130.1629729907110; Mon, 23 Aug 2021 07:45:07 -0700 (PDT) MIME-Version: 1.0 References: <92b6718f5618d5469f67b48fbea189cca0c12f4b.1628670468.git.geert+renesas@glider.be> In-Reply-To: From: Rob Herring Date: Mon, 23 Aug 2021 09:44:55 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v5 1/9] MIPS: Avoid future duplicate elf core header reservation To: Mike Rapoport Cc: Geert Uytterhoeven , Russell King , Nicolas Pitre , Ard Biesheuvel , Linus Walleij , Catalin Marinas , Will Deacon , Thomas Bogendoerfer , Nick Kossifidis , Paul Walmsley , Palmer Dabbelt , Albert Ou , Frank Rowand , Dave Young , Baoquan He , Vivek Goyal , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Linux ARM , "open list:BROADCOM NVRAM DRIVER" , linux-riscv , kexec@lists.infradead.org, Linux-Renesas , Linux Kernel Mailing List X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210823_074509_283218_AD318AC5 X-CRM114-Status: GOOD ( 29.05 ) 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 Mon, Aug 23, 2021 at 8:10 AM Mike Rapoport wrote: > > On Mon, Aug 23, 2021 at 12:17:50PM +0200, Geert Uytterhoeven wrote: > > Hi Mike, > > > > On Mon, Aug 16, 2021 at 7:52 AM Mike Rapoport wrote: > > > On Wed, Aug 11, 2021 at 10:50:59AM +0200, Geert Uytterhoeven wrote: > > > > Prepare for early_init_fdt_scan_reserved_mem() reserving the memory > > > > occupied by an elf core header described in the device tree. > > > > As arch_mem_init() calls early_init_fdt_scan_reserved_mem() before > > > > mips_reserve_vmcore(), the latter needs to check if the memory has > > > > already been reserved before. > > > > > > Doing memblock_reserve() for the same region is usually fine, did you > > > encounter any issues without this patch? > > > > Does it also work if the same region is part of an earlier larger > > reservation? I am no memblock expert, so I don't know. > > I didn't run into any issues, as my MIPS platform is non-DT, but I > > assume arch/arm64/mm/init.c:reserve_elfcorehdr() had the check for > > a reason. > > The memory will be reserved regardless of the earlier reservation, the > issue may appear when the reservations are made for different purpose. E.g. > if there was crash kernel allocation before the reservation of elfcorehdr. > > The check in such case will prevent the second reservation, but, at least > in arch/arm64/mm/init.c:reserve_elfcorehdr() it does not seem to prevent > different users of the overlapping regions to step on each others toes. If the kernel has been passed in overlapping regions, is there anything you can do other than hope to get a message out? > Moreover, arm64::reserve_elfcorehdr() seems buggy to me, because of there > is only a partial overlap of the elfcorehdr with the previous reservation, > the non-overlapping part of elfcorehdr won't get reserved at all. What do you suggest as the arm64 version is not the common version? Rob _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel