On 2022/08/01 13:30, Xianting Tian wrote: > This series of patches are for Crash-utility tool, it make crash tool support > RISCV64 arch and the common commands(*, bt, p, rd, mod, log, set, struct, task, > dis, help -r, help -m, and so on). > > To make the crash tool work normally for RISCV64 arch, we need a Linux kernel > patch(wait for apply), which exports the kernel virtual memory layout, va_bits, > phys_ram_base to vmcoreinfo, it can simplify the development of crash tool. > > The Linux kernel patch set: > https://lore.kernel.org/linux-riscv/20220726093729.1231867-1-xianting.tian@xxxxxxxxxxxxxxxxx/ > > This series of patches are tested on QEMU RISCV64 env and SoC platform of > T-head Xuantie 910 RISCV64 CPU. > > ==================================== > Some test examples list as below > ==================================== > ... ... > KERNEL: vmlinux > DUMPFILE: vmcore > CPUS: 1 > DATE: Fri Jul 15 10:24:25 CST 2022 > UPTIME: 00:00:33 > LOAD AVERAGE: 0.05, 0.01, 0.00 > TASKS: 41 > NODENAME: buildroot > RELEASE: 5.18.9 > VERSION: #30 SMP Fri Jul 15 09:47:03 CST 2022 > MACHINE: riscv64 (unknown Mhz) > MEMORY: 1 GB > PANIC: "Kernel panic - not syncing: sysrq triggered crash" > PID: 113 > COMMAND: "sh" > TASK: ff60000002269600 [THREAD_INFO: ff60000002269600] > CPU: 0 > STATE: TASK_RUNNING (PANIC) > > carsh> > > crash> p mem_map > mem_map = $1 = (struct page *) 0xff6000003effbf00 > > crash> p /x *(struct page *) 0xff6000003effbf00 > $5 = { > flags = 0x1000, > { > { > { > lru = { > next = 0xff6000003effbf08, > prev = 0xff6000003effbf08 > }, > { > __filler = 0xff6000003effbf08, > mlock_count = 0x3effbf08 > } > }, > mapping = 0x0, > index = 0x0, > private = 0x0 > }, > ... ... > > crash> mod > MODULE NAME BASE SIZE OBJECT FILE > ffffffff0113e740 nvme_core ffffffff01133000 98304 (not loaded) [CONFIG_KALLSYMS] > ffffffff011542c0 nvme ffffffff0114c000 61440 (not loaded) [CONFIG_KALLSYMS] > > crash> rd ffffffff0113e740 8 > ffffffff0113e740: 0000000000000000 ffffffff810874f8 .........t...... > ffffffff0113e750: ffffffff011542c8 726f635f656d766e .B......nvme_cor > ffffffff0113e760: 0000000000000065 0000000000000000 e............... > ffffffff0113e770: 0000000000000000 0000000000000000 ................ > > crash> vtop ffffffff0113e740 > VIRTUAL PHYSICAL > ffffffff0113e740 8254d740 > > PGD: ffffffff810e9ff8 => 2ffff001 > P4D: 0000000000000000 => 000000002fffec01 > PUD: 00005605c2957470 => 0000000020949801 > PMD: 00007fff7f1750c0 => 0000000020947401 > PTE: 0 => 209534e7 > PAGE: 000000008254d000 > > PTE PHYSICAL FLAGS > 209534e7 8254d000 (PRESENT|READ|WRITE|GLOBAL|ACCESSED|DIRTY) > > PAGE PHYSICAL MAPPING INDEX CNT FLAGS > ff6000003f0777d8 8254d000 0 0 1 0 > > crash> bt > PID: 113 TASK: ff6000000226c200 CPU: 0 COMMAND: "sh" > #0 [ff20000010333b90] riscv_crash_save_regs at ffffffff800078f8 > #1 [ff20000010333cf0] panic at ffffffff806578c6 > #2 [ff20000010333d50] sysrq_reset_seq_param_set at ffffffff8038c03c > #3 [ff20000010333da0] __handle_sysrq at ffffffff8038c604 > #4 [ff20000010333e00] write_sysrq_trigger at ffffffff8038cae4 > #5 [ff20000010333e20] proc_reg_write at ffffffff801b7ee8 > #6 [ff20000010333e40] vfs_write at ffffffff80152bb2 > #7 [ff20000010333e80] ksys_write at ffffffff80152eda > #8 [ff20000010333ed0] sys_write at ffffffff80152f52 > > ------- > Changes V1 -> V2: > 1, Do the below fixes based on HAGIO KAZUHITO's comments: > Fix build warnings, > Use MACRO for Linux version, > Add description of x86_64 binary for riscv64 in README, > Fix build error for the "sticky" target for build on x86_64, > Fix the mixed indent. > 2, Add 'help -m/M' support patch to this patch set. > 3, Support native compiling approach, which means the host OS distro > is also a riscv64 (lp64d) Linux, based on Yixun Lan's comments. > 4, Use __riscv and __riscv_xlen instead of __riscv64__ based on Yixun Lan's comments. Thank you for the v2, looks good to me. So with the kernel patch set merged, Acked-by: Kazuhito Hagio <k-hagio-ab@xxxxxxx> (no need to add this tag to the patches) Thanks, Kazu > > Xianting Tian (9): > Add RISCV64 framework code support > RISCV64: Make crash tool enter command line and support some commands > RISCV64: Add 'dis' command support > RISCV64: Add 'irq' command support > RISCV64: Add 'bt' command support > RISCV64: Add 'help -r' command support > RISCV64: Add 'help -m/M' command support > RISCV64: Add 'mach' command support > RISCV64: Add the implementation of symbol verify > > Makefile | 7 +- > README | 4 +- > configure.c | 43 +- > defs.h | 248 +++++++- > diskdump.c | 21 +- > help.c | 2 +- > lkcd_vmdump_v1.h | 2 + > lkcd_vmdump_v2_v3.h | 4 +- > netdump.c | 22 +- > ramdump.c | 2 + > riscv64.c | 1492 +++++++++++++++++++++++++++++++++++++++++++ > symbols.c | 10 + > 12 files changed, 1847 insertions(+), 10 deletions(-) > create mode 100644 riscv64.c > -- Crash-utility mailing list Crash-utility@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/crash-utility Contribution Guidelines: https://github.com/crash-utility/crash/wiki
