Hi, HATAYAMA
Thanks for your trying this.
On 2/15/24 18:04, devel-request(a)lists.crash-utility.osci.io wrote:
 Date: Thu, 15 Feb 2024 19:02:36 +0900
 From: HATAYAMA Daisuke<d.hatayama(a)fujitsu.com>
 Subject: [Crash-utility] [RFC PATCH 0/9] Add feature to validate page
 	descriptor table in kdump-compressed format
 To:devel@lists.crash-utility.osci.io
 Message-ID:<20240215100246.437-1-d.hatayama@fujitsu.com>
 I've made a RFC patch set to make sanity check of page descriptor
 table in kdump compressed format strict.
 This work has arose from the past issue that a produced crash dump
 file was broken not only in data segment but also in headers including
 page descriptor table. I've ever explained a bit this on crash-devel
 in the following thread:
      
https://listman.redhat.com/archives/crash-utility/2023-September/010957.html 
For the patchset, the patch [2] and [6] should be able to fix the 
current issue?
After the crash tool knows which pages are incomplete or invalid, how to 
handle these pages in the next steps?  Or crash-utility can not do 
anything except for printing those bad pages. Could you please explain 
it a little more? That can help me understand the other 
patches[1/3/4/5/7/8].
Thanks
Lianbo
 In this past issue, I couldn't find out its root cause after all
 because when I began investigating the issue, problematic system where
 the issue was reproduced was already unavailable.
 This patch set is aimed at allowing us to figure out this kind of
 issue more quickly and in more detail using crash utility. The code is
 based on the tool I made to analyze the broken crash dump in the past
 issue.
 This is still incomplete, for example, the sanity check has not yet
 supported split dump files in the kdump-compressed format and old
 header version up to 5. It would be appreciated if I can get comments
 for this RFC version.
 HATAYAMA Daisuke (9):
    diskdump: Add stat object in diskdump_data
    diskdump: Add function sanity_check_page_desc() that sanity checks an
      entry of page descriptor table
    diskdump: Add function check_kdump_headers() that validates page
      descriptor table
    defs.h: Introduce flag VALIDATE_KDUMP_HEADERS
    diskdump, main: Add --validate_kdump_headers command-line option
    diskdump: Make sanity check in cache_page() strict
    help: Add description of --validate_kdump_headers command-line option
    man: Add description of --validate_kdump_headers command-line option
    diskdump, debug: Print elapsed time consumed in validation of page
      descriptor table
   crash.8    |  4 +++
   defs.h     |  1 +
   diskdump.c | 74 +++++++++++++++++++++++++++++++++++++++++++++++++++++-
   help.c     |  4 +++
   main.c     |  5 ++++
   5 files changed, 87 insertions(+), 1 deletion(-)
 -- 2.43.1