Re: [PATCH V3 0/9] Support RISCV64 arch and common commands

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi lijiang

On Mon, Aug 22, 2022 at 3:56 AM lijiang <lijiang@xxxxxxxxxx> wrote:
>
> Hi, Xianting
> Thank you for the update.
> On Sat, Aug 13, 2022 at 11:18 AM Xianting Tian <xianting.tian@xxxxxxxxxxxxxxxxx> 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, 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/20220811074150.3020189-1-xianting.tian@xxxxxxxxxxxxxxxxx/
>> [ Patch 1 ~ 4 already merged to Linux for-next branch, targeted for 5.20;
>>   Patch 5, 6 already merged to Palmer's riscv-crash branch:
>>   https://git.kernel.org/pub/scm/linux/kernel/git/palmer/linux.git/log/?h=riscv-crash ]
>>
>
> Can you also help to share the patch link for kexec-tools? Or share your loading steps and generate a vmcore in riscv64 system?
>
> I tried it with kexec-tools and got the following error:
>
> [root@fedora-riscv kexec-tools]# ./configure
> checking for gcc... gcc
> checking whether the C compiler works... yes
> checking for C compiler default output file name... a.out
> checking for suffix of executables...
> checking whether we are cross compiling... no
> checking for suffix of object files... o
> checking whether we are using the GNU C compiler... yes
> checking whether gcc accepts -g... yes
> checking for gcc option to accept ISO C89... none needed
> checking build system type... riscv64-unknown-linux-gnu
> checking host system type... riscv64-unknown-linux-gnu
> checking target system type... riscv64-unknown-linux-gnu
> configure: error: unsupported architecture riscv64
>
> Thanks.
> Lianbo
>
>> 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.
>> Changes V2 -> V3:
>>  1, Fix coding style, avoid including the header twice, move free() to right place,
>>     introduce VM_FLAGS and so on based on Li Jiang's comments.
>>  2, Adjust the implementation of riscv64_verify_symbol(refer to the logic of x86_64_verify_symbol)
>>     as KSYMS_START isn't set when verify symbol in some case.
>>
>>
>> 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              |  252 +++++++-
>>  diskdump.c          |   21 +-
>>  help.c              |    2 +-
>>  lkcd_vmdump_v1.h    |    8 +-
>>  lkcd_vmdump_v2_v3.h |    8 +-
>>  netdump.c           |   22 +-
>>  ramdump.c           |    2 +
>>  riscv64.c           | 1491 +++++++++++++++++++++++++++++++++++++++++++
>>  symbols.c           |   10 +
>>  12 files changed, 1848 insertions(+), 22 deletions(-)
>>  create mode 100644 riscv64.c
>>
>> --
>> 2.17.1
>>

FYI, take a look at
lists.infradead.org/pipermail/linux-riscv/2021-October/008773.html

also need this extra fix
lists.infradead.org/pipermail/linux-riscv/2022-August/018398.html

and just follow normal scheme to use kexec + crash, there is nothing different

Yixun

--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/crash-utility
Contribution Guidelines: https://github.com/crash-utility/crash/wiki




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux