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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id A5433CD98CC for ; Thu, 11 Jun 2026 12:04:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To: Content-Transfer-Encoding:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=f/j69pXYfj34ZmqA6bJCKc0nePFg+h49mJ8/N7zh1iA=; b=r4FzlaTyeR9P7L+KETf+xNXbw2 59LCAtLTBxZZQzsXxLDhwU8n1x4tz4qE8ew7+j8oJ91C7OZoYJlGWr4BROhTEIdUSYYdIjbppoG0J wevd3g2RpHNct6l5z9SPoFo30WMPy9UBC5gUZ5GX/cmJg3ux+HSQrBRMCtJgXTyqE1skOnyqFh7w/ B+NR5dUgccQRgX8mgrlLP5JtcYpAcQXq0n3O87gfebEqiZTOJSOr/AgnbhxcQF4H7GsbdtyZMU5y1 qGTwS1SjgSmgi1haMZ3V7R3dmHr4mhuHEXCNPkXX3hotjdq67Vt0Gzw7/4WTxZTINAzbv441D2ELU l07qo0Vg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wXe8y-00000009Mao-0Cpl; Thu, 11 Jun 2026 12:04:20 +0000 Received: from out-185.mta0.migadu.com ([91.218.175.185]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wXe8u-00000009MZf-3sHD for linux-arm-kernel@lists.infradead.org; Thu, 11 Jun 2026 12:04:18 +0000 Date: Thu, 11 Jun 2026 20:03:53 +0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1781179452; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=f/j69pXYfj34ZmqA6bJCKc0nePFg+h49mJ8/N7zh1iA=; b=C7VD/ntPHBl4YIG9Watr5T2GZLCvEmAzup5tfdjNbozip65PyigVCCrPseQbQKMgC2+M7G fYJkNSWFsX/dRktAsP7CNtY62sQ3fk1oEHQwJa+H4KpymFRmYbGmZ4jTCJo6jBN0nw2upe Z54mPkCYOx99N+izHFLdL+L2Fx6cijE= X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Baoquan He To: Wandun Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, loongarch@lists.linux.dev, linux-riscv@lists.infradead.org, devicetree@vger.kernel.org, kexec@lists.infradead.org, iommu@lists.linux.dev, zhaomeijing@lixiang.com, Rob Herring , saravanak@kernel.org, bhe@redhat.com, rppt@kernel.org, pjw@kernel.org, palmer@dabbelt.com, aou@eecs.berkeley.edu, chenhuacai@kernel.org, kernel@xen0n.name, catalin.marinas@arm.com, will@kernel.org, alex@ghiti.fr, akpm@linux-foundation.org, pasha.tatashin@soleen.com, pratyush@kernel.org, ruirui.yang@linux.dev, m.szyprowski@samsung.com, robin.murphy@arm.com Subject: Re: [PATCH v3 00/11] kdump: reduce vmcore size and capture time Message-ID: References: <20260527032917.3385849-1-chenwandun1@gmail.com> <7ed207fa-4c86-426a-8570-495902ce04c3@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Migadu-Flow: FLOW_OUT X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260611_050417_115819_09BBCFE5 X-CRM114-Status: GOOD ( 32.75 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 06/11/26 at 11:09am, Wandun wrote: > > > On 6/11/26 10:09, Wandun wrote: > > > > > > On 5/27/26 11:29, Wandun Chen wrote: > >> From: Wandun Chen > >> > >> On SoCs that carve out large firmware-owned reserved memory (GPU > >> firmware, DSP, modem, camera ISP, NPU, ...), kdump currently dumps > >> those carveouts as part of system RAM even though their contents are > >> firmware state that is not useful for kernel crash analysis. > >> > >> This series introduces an opt-in 'dumpable' flag [1] on struct > >> reserved_mem and uses it to filter the elfcorehdr PT_LOAD ranges on > >> DT-based architectures (arm64, riscv, loongarch). By default reserved > >> regions are treated as non-dumpable; CMA regions are explicitly opted > >> in because their pages are returned to the buddy allocator and may > >> carry key crash-analysis data. > >> > >> The series is organized as follows: > >> Patches 1-3: Pre-existing fixes and a small prep change. > >> Patches 4-5: Restructure to allow appending /memreserve/ entries. > >> Patches 6-7: Add a dumpable flag and append /memreserve/ entries. > >> Patch 8: Add generic kdump helpers. > >> Patches 9-11: Wire the helpers into arm64, riscv and loongarch kdump > >>                elfcorehdr preparation. > > Hi, > > > > Gentle ping on this series. > > > > Status summary: > > -patch 03: respun separately per Rob's suggestion, picked up for 7.2 > > -patch 06: Acked-by: Marek Szyprowski -patch 09: Acked-by: Will Deacon > > The remaining patches (01, 02, 04, 05, 07, 08, 10, 11) are still > > awaiting review. your feedback would be greately appreciated. I know we > > are at the end of 7.1 -rc cycle, I don't want to rush this series, just > > collecting more feedback, and will send next version based on 7.2-rc1. > > If spliting the series into smaller logical group would make review > > easier, please let me know. Best regards, Wandun > > Apologies for the formatting issue in my previous email. > Here is the properly formatted version. > > Gentle ping on this series. Thanks for the effort, the overral looks good to me at 1st glance. I will check if there's concern on generic part. And meanwhile, I am wondering if there's any chance x86 or other ARCH-es w/o OF/FDT can also choose to not dump some areas, e.g GPU stolen memory. Surely, that's another story. > > Status summary: > - patch 03: respun separately per Rob's suggestion, picked up for 7.2 > - patch 06: Acked-by: Marek Szyprowski > - patch 09: Acked-by: Will Deacon > > The remaining patches (01, 02, 04, 05, 07, 08, 10, 11) are still > awaiting review. Your feedback would be greatly appreciated. > > I know we are at the end of 7.1-rc cycle, I don't want to rush this > series, just collecting more feedback, and will send next version based > on 7.2-rc1. > > If splitting the series into smaller logical groups would make review > easier, please let me know. > > Best regards, > Wandun > > > >> > >> v2 --> v3: > >> 1. Fix out-of-bounds issue if device tree lacks /reserved-memory node.[2] > >> 2. Fix UAF issue when alloc_reserved_mem_array() fails. > >> 3. Add some prepare patches. > >> > >> v1 --> v2: > >> 1. v1 added an opt-out DT property ('linux,no-dump'). Per Rob's > >>     feedback [1], v2 drop that property and exclude reserve memory > >>     by default. > >> 2. Split some prepared patches from the original patches. > >> 3. Address coding-style comments on patch 5 from Rob. > >> > >> [1] https://lore.kernel.org/lkml/20260506144542.GA2072596- > >> robh@kernel.org/ > >> [2] https://sashiko.dev/#/patchset/20260520091844.592753-1- > >> chenwandun%40lixiang.com?part=4 > >> > >> Wandun Chen (11): > >>    of: reserved_mem: handle NULL name in of_reserved_mem_lookup() > >>    kexec/crash: provide crash_exclude_mem_range() stub when > >>      CONFIG_CRASH_DUMP=n > >>    of: reserved_mem: avoid post-init UAF when alloc_reserved_mem_array() > >>      fails > >>    of: reserved_mem: zero total_reserved_mem_cnt if no valid > >>      /reserved-memory entry > >>    of: reserved_mem: split alloc_reserved_mem_array() from > >>      fdt_scan_reserved_mem_late() > >>    of: reserved_mem: add dumpable flag to opt-in vmcore > >>    of: reserved_mem: save /memreserve/ entries into the reserved_mem > >>      array > >>    of: reserved_mem: add kdump helpers to exclude non-dumpable regions > >>    arm64: kdump: exclude non-dumpable reserved memory regions from vmcore > >>    riscv: kdump: exclude non-dumpable reserved memory regions from vmcore > >>    loongarch: kdump: exclude non-dumpable reserved memory regions from > >>      vmcore > >> > >>   arch/arm64/kernel/machine_kexec_file.c     |   6 ++ > >>   arch/loongarch/kernel/machine_kexec_file.c |   6 ++ > >>   arch/riscv/kernel/machine_kexec_file.c     |   4 + > >>   drivers/of/fdt.c                           |  11 +- > >>   drivers/of/of_private.h                    |   3 + > >>   drivers/of/of_reserved_mem.c               | 117 +++++++++++++++++++-- > >>   include/linux/crash_core.h                 |   6 ++ > >>   include/linux/of_reserved_mem.h            |  15 +++ > >>   kernel/dma/contiguous.c                    |   1 + > >>   9 files changed, 157 insertions(+), 12 deletions(-) > >> > > >