Hi Pingfan, On Mon, 18 Dec 2023 13:23:22 +0800 Pingfan Liu <kernelfans@xxxxxxxxx> wrote: > From: Pingfan Liu <piliu@xxxxxxxxxx> > > > First of all, this series is only for proof of concept. It only passes compilation. > > For years, CMA is proposed to be used as crashkernel reserved memory. > But DIO prevent us to follow it since DMA may be in-flight and ruin the > kdump kernel. > > This series exports the crash kernel's CMA area information through > device-tree, and kdump kernel skips any page, which refcnt!=mapcount and > has a potential DMA activity. > > The exported information include: > u64 kdump_cma_pfn; > u64 kdump_cma_pg_cnt; > u64 kdump_cma_pg_paddr; > > And they should be filled with Jiri's series "[PATCH 0/4] kdump: > crashkernel reservation from CMA" > > After the conjunction of two series, the CMA used for kdump has only the > following risk, where the following conditions: > -1.a wrong code forges _refcnt and mapcount to the same value > -2.the page is also used by DIO > > > Is it acceptable, or any rescue e.g. CRC on page? > > Please share your thoughts. I don't think your approach will work as intended. The problem is that we are dealing with two separate kernels and there is no guarantee that both kernels are identical. So you cannot rely on the definition of struct page in the crash kernel to be identical to the one in the panicked kernel. Meaning check_poison_page from the crash kernel cannot simply operate on the struct pages from the panicked kernel. To get this approach to work I see three possible "fixes" 1) enforce in kexec that only the currently running kernel can be loaded as crash kernel. 2) pass all required "debuginfo" to the crash kernel so it can parse the required data reliably from the dump. This also requires to have all the mm helper functions to be reimplemented to work in check_poison_page. 3) the required information is passed via a new data structure which is designed in a way that it can easily be passed in between different kernels. But this would require the mm subsystem to maintain the page states in the CMA in two separate data structures. Personally I don't think that any of the three "fixes" is desirable. Thanks Philipp > Thanks, > > Pingfan > > > Cc: Jiri Bohac <jbohac@xxxxxxx> > Cc: Michal Hocko <mhocko@xxxxxxxx> > Cc: Philipp Rudo <prudo@xxxxxxxxxx> > Cc: Baoquan He <bhe@xxxxxxxxxx> > Cc: Dave Young <dyoung@xxxxxxxxxx> > To: kexec@xxxxxxxxxxxxxxxxxxx > --- > Pingfan Liu (3): > crash_dump: Parse the CMA's mem_map in kdump > of: kexec: Set up properties for reusing CMA in kdump > of: fdt: Parse properties of reusing CMA in kdump > > drivers/of/fdt.c | 43 +++++++++++++++++++++++ > drivers/of/kexec.c | 14 ++++++++ > include/linux/kexec.h | 5 +++ > init/main.c | 4 +++ > kernel/crash_dump.c | 80 +++++++++++++++++++++++++++++++++++++++++++ > 5 files changed, 146 insertions(+) > _______________________________________________ kexec mailing list kexec@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/kexec
