From: Mike Rapoport <rppt@kernel.org>
To: Wandun <chenwandun1@gmail.com>
Cc: robh@kernel.org, baoquan.he@linux.dev, m.szyprowski@samsung.com,
catalin.marinas@arm.com, will@kernel.org, mark.rutland@arm.com,
chenhuacai@kernel.org, kernel@xen0n.name, pjw@kernel.org,
palmer@dabbelt.com, aou@eecs.berkeley.edu, alex@ghiti.fr,
tglx@kernel.org, mingo@redhat.com, bp@alien8.de,
dave.hansen@linux.intel.com, x86@kernel.org, hpa@zytor.com,
saravanak@kernel.org, akpm@linux-foundation.org,
pasha.tatashin@soleen.com, pratyush@kernel.org,
ruirui.yang@linux.dev, robin.murphy@arm.com,
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, linux-mm@kvack.org,
iommu@lists.linux.dev
Subject: Re: [PATCH v7 0/9] kdump: reduce vmcore size and capture time
Date: Fri, 9 Oct 2026 18:26:36 +0200 [thread overview]
Message-ID: <askVvEPHyrhpF8aw@kernel.org> (raw)
In-Reply-To: <80af60db-8c39-461c-a488-3dfee990bce8@gmail.com>
On Fri, Oct 09, 2026 at 11:04:47AM +0800, Wandun wrote:
> On 10/2/26 16:17, Mike Rapoport wrote:
> > Hi Wandun,
> >
> > On Tue, Sep 22, 2026 at 04:42:47PM +0800, Wandun Chen wrote:
> >> From: Wandun Chen <chenwandun@lixiang.com>
> >>
> >> On SoCs that carve out large firmware-owned reserved memory (GPU,
> >> camera ISP, ...), 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 a MEMBLOCK_NODUMP flag in memblock to filter
> >> reserved memory on DT-based architectures (arm64, riscv, loongarch).
> >> Reserved regions are marked MEMBLOCK_NODUMP by default so kdump omits
> >> them; reusable CMA regions are different because their pages are handed
> >> back to the buddy allocator and may carry crash-relevant data.
> >>
> >> ACPI systems already filter reserved memory out of the vmcore through
> >> their existing path; only DT-based systems currently fail to filter these
> >> regions, which is what this series addresses. The flag lives in memblock
> >> itself rather than in a DT-only structure, so both ACPI and DT systems
> >> can benefit from it (suggested by Rob, thanks).
> >
> Hi Mike,
>
> Thanks for the review.
>
> > Can you elaborate how ACPI systems filter their reserved memory?
>
> In the EFI memory map, firmware carveouts are reported with the type
> EFI_RESERVED_TYPE. This type of memory gets the nomap flag set in
> memblock.memory (is_usable_memory() returns false in reserve_regions()
> and mark these regions with nomap flag), during prepare_elf_headers(),
> for_each_mem_range() filters out these nomap regions. Therefore, the
> memory reserved on ACPI based systems will not be dumped into the vmcore.
>
> I will add these description in next version.
>
> > With your patches, the memblock flag becomes a part of DT-only
> > infrastructure, so that claim that both ACPI and DT benefit from it does
> > not hold, doesn't it?
>
> You are right, the wording was misleading. Let me clarify what I meant.
>
> In this series only DT marks regions as MEMBLOCK_NODUMP:
> /reserved-memory and /memreserve/ entries are tagged in the OF
> paths (of_reserved_mem.c, fdt.c). ACPI does not set the flag and
> keeps filtering its reserved memory through the existing path, as
> described above.
>
>
> What I intended to convey is that the flag itself lives in memblock
> rather than on the DT-only struct reserved_mem, so it is available
> for reuse. If an ACPI system ever needed to exclude a firmware-owned
> region from the vmcore, it could mark that region with MEMBLOCK_NODUMP
> the same way DT does, instead of having to invent a parallel mechanism.
> That is a future option, not something this series enables for ACPI
> today.
So, EFI uses NOMAP for the same purpose, right?
Why when we need a new memblock flag then?
> Also will update these description in next verison.
>
> Best regards,
> Wandun
--
Sincerely yours,
Mike.
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
WARNING: multiple messages have this Message-ID (diff)
From: Mike Rapoport <rppt@kernel.org>
To: Wandun <chenwandun1@gmail.com>
Cc: robh@kernel.org, baoquan.he@linux.dev, m.szyprowski@samsung.com,
catalin.marinas@arm.com, will@kernel.org, mark.rutland@arm.com,
chenhuacai@kernel.org, kernel@xen0n.name, pjw@kernel.org,
palmer@dabbelt.com, aou@eecs.berkeley.edu, alex@ghiti.fr,
tglx@kernel.org, mingo@redhat.com, bp@alien8.de,
dave.hansen@linux.intel.com, x86@kernel.org, hpa@zytor.com,
saravanak@kernel.org, akpm@linux-foundation.org,
pasha.tatashin@soleen.com, pratyush@kernel.org,
ruirui.yang@linux.dev, robin.murphy@arm.com,
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, linux-mm@kvack.org,
iommu@lists.linux.dev
Subject: Re: [PATCH v7 0/9] kdump: reduce vmcore size and capture time
Date: Fri, 9 Oct 2026 18:26:36 +0200 [thread overview]
Message-ID: <askVvEPHyrhpF8aw@kernel.org> (raw)
In-Reply-To: <80af60db-8c39-461c-a488-3dfee990bce8@gmail.com>
On Fri, Oct 09, 2026 at 11:04:47AM +0800, Wandun wrote:
> On 10/2/26 16:17, Mike Rapoport wrote:
> > Hi Wandun,
> >
> > On Tue, Sep 22, 2026 at 04:42:47PM +0800, Wandun Chen wrote:
> >> From: Wandun Chen <chenwandun@lixiang.com>
> >>
> >> On SoCs that carve out large firmware-owned reserved memory (GPU,
> >> camera ISP, ...), 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 a MEMBLOCK_NODUMP flag in memblock to filter
> >> reserved memory on DT-based architectures (arm64, riscv, loongarch).
> >> Reserved regions are marked MEMBLOCK_NODUMP by default so kdump omits
> >> them; reusable CMA regions are different because their pages are handed
> >> back to the buddy allocator and may carry crash-relevant data.
> >>
> >> ACPI systems already filter reserved memory out of the vmcore through
> >> their existing path; only DT-based systems currently fail to filter these
> >> regions, which is what this series addresses. The flag lives in memblock
> >> itself rather than in a DT-only structure, so both ACPI and DT systems
> >> can benefit from it (suggested by Rob, thanks).
> >
> Hi Mike,
>
> Thanks for the review.
>
> > Can you elaborate how ACPI systems filter their reserved memory?
>
> In the EFI memory map, firmware carveouts are reported with the type
> EFI_RESERVED_TYPE. This type of memory gets the nomap flag set in
> memblock.memory (is_usable_memory() returns false in reserve_regions()
> and mark these regions with nomap flag), during prepare_elf_headers(),
> for_each_mem_range() filters out these nomap regions. Therefore, the
> memory reserved on ACPI based systems will not be dumped into the vmcore.
>
> I will add these description in next version.
>
> > With your patches, the memblock flag becomes a part of DT-only
> > infrastructure, so that claim that both ACPI and DT benefit from it does
> > not hold, doesn't it?
>
> You are right, the wording was misleading. Let me clarify what I meant.
>
> In this series only DT marks regions as MEMBLOCK_NODUMP:
> /reserved-memory and /memreserve/ entries are tagged in the OF
> paths (of_reserved_mem.c, fdt.c). ACPI does not set the flag and
> keeps filtering its reserved memory through the existing path, as
> described above.
>
>
> What I intended to convey is that the flag itself lives in memblock
> rather than on the DT-only struct reserved_mem, so it is available
> for reuse. If an ACPI system ever needed to exclude a firmware-owned
> region from the vmcore, it could mark that region with MEMBLOCK_NODUMP
> the same way DT does, instead of having to invent a parallel mechanism.
> That is a future option, not something this series enables for ACPI
> today.
So, EFI uses NOMAP for the same purpose, right?
Why when we need a new memblock flag then?
> Also will update these description in next verison.
>
> Best regards,
> Wandun
--
Sincerely yours,
Mike.
next prev parent reply other threads:[~2026-10-09 16:27 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-22 8:42 [PATCH v7 0/9] kdump: reduce vmcore size and capture time Wandun Chen
2026-09-22 8:42 ` Wandun Chen
2026-09-22 8:42 ` [PATCH v7 1/9] riscv: build crash_mem ranges from memblock instead of resource tree Wandun Chen
2026-09-22 8:42 ` Wandun Chen
2026-09-22 8:59 ` sashiko-bot
2026-09-22 8:59 ` sashiko-bot
2026-09-22 8:42 ` [PATCH v7 2/9] crash_core: fold duplicated memblock arch hooks into the weak default Wandun Chen
2026-09-22 8:42 ` Wandun Chen
2026-09-22 8:56 ` sashiko-bot
2026-09-22 8:56 ` sashiko-bot
2026-09-22 8:42 ` [PATCH v7 3/9] crash_core: serialize crash header preparation against hotplug Wandun Chen
2026-09-22 8:42 ` Wandun Chen
2026-09-22 9:03 ` sashiko-bot
2026-09-22 9:03 ` sashiko-bot
2026-09-22 8:42 ` [PATCH v7 4/9] crash_core: replace for_each_mem_range() with for_each_mem_region() Wandun Chen
2026-09-22 8:42 ` Wandun Chen
2026-09-22 8:56 ` sashiko-bot
2026-09-22 8:56 ` sashiko-bot
2026-09-22 8:42 ` [PATCH v7 5/9] memblock: introduce MEMBLOCK_NODUMP flag Wandun Chen
2026-09-22 8:42 ` Wandun Chen
2026-09-22 9:02 ` sashiko-bot
2026-09-22 9:02 ` sashiko-bot
2026-09-22 8:42 ` [PATCH v7 6/9] of: reserved_mem: add dumpable flag to opt-in vmcore Wandun Chen
2026-09-22 8:42 ` Wandun Chen
2026-09-22 8:54 ` sashiko-bot
2026-09-22 8:54 ` sashiko-bot
2026-09-22 8:42 ` [PATCH v7 7/9] of: reserved_mem: mark /reserved-memory entries with MEMBLOCK_NODUMP Wandun Chen
2026-09-22 8:42 ` Wandun Chen
2026-09-22 9:01 ` sashiko-bot
2026-09-22 9:01 ` sashiko-bot
2026-09-22 8:42 ` [PATCH v7 8/9] of: reserved_mem: mark /memreserve/ entries as MEMBLOCK_NODUMP Wandun Chen
2026-09-22 8:42 ` Wandun Chen
2026-09-22 9:04 ` sashiko-bot
2026-09-22 9:04 ` sashiko-bot
2026-09-22 8:42 ` [PATCH v7 9/9] crash_core: skip MEMBLOCK_NODUMP regions when building vmcore ELF header Wandun Chen
2026-09-22 8:42 ` Wandun Chen
2026-09-22 9:02 ` sashiko-bot
2026-09-22 9:02 ` sashiko-bot
2026-10-02 8:17 ` [PATCH v7 0/9] kdump: reduce vmcore size and capture time Mike Rapoport
2026-10-02 8:17 ` Mike Rapoport
2026-10-09 3:04 ` Wandun
2026-10-09 3:04 ` Wandun
2026-10-09 16:26 ` Mike Rapoport [this message]
2026-10-09 16:26 ` Mike Rapoport
2026-10-09 17:10 ` Rob Herring
2026-10-09 17:10 ` Rob Herring
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=askVvEPHyrhpF8aw@kernel.org \
--to=rppt@kernel.org \
--cc=akpm@linux-foundation.org \
--cc=alex@ghiti.fr \
--cc=aou@eecs.berkeley.edu \
--cc=baoquan.he@linux.dev \
--cc=bp@alien8.de \
--cc=catalin.marinas@arm.com \
--cc=chenhuacai@kernel.org \
--cc=chenwandun1@gmail.com \
--cc=dave.hansen@linux.intel.com \
--cc=devicetree@vger.kernel.org \
--cc=hpa@zytor.com \
--cc=iommu@lists.linux.dev \
--cc=kernel@xen0n.name \
--cc=kexec@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linux-riscv@lists.infradead.org \
--cc=loongarch@lists.linux.dev \
--cc=m.szyprowski@samsung.com \
--cc=mark.rutland@arm.com \
--cc=mingo@redhat.com \
--cc=palmer@dabbelt.com \
--cc=pasha.tatashin@soleen.com \
--cc=pjw@kernel.org \
--cc=pratyush@kernel.org \
--cc=robh@kernel.org \
--cc=robin.murphy@arm.com \
--cc=ruirui.yang@linux.dev \
--cc=saravanak@kernel.org \
--cc=tglx@kernel.org \
--cc=will@kernel.org \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.