[Crash-utility] Re: [PATCH v3 00/10] add LoongArch64 platform support

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

 



Hi, Ming

Thank you for the update.

On 12/28/23 19:47, devel-request@xxxxxxxxxxxxxxxxxxxxxxxxxxx wrote:
Date: Thu, 28 Dec 2023 19:46:24 +0800
From: Ming Wang <wangming01@xxxxxxxxxxx>
Subject:  [Crash-utility][PATCH v3 00/10] add
	LoongArch64 platform support
To: devel@xxxxxxxxxxxxxxxxxxxxxxxxxxx,  lijiang@xxxxxxxxxx,
	k-hagio-ab@xxxxxxx
Cc: gaojuxin@xxxxxxxxxxx, liweihao@xxxxxxxxxxx
Message-ID: <20231228114634.1085279-1-wangming01@xxxxxxxxxxx>

This patch set are for Crash-utility tool, it make crash tool support on
loongarch64 architecture and the common commands(bt, p, rd, mod, log, set,
dis, and so on).

The patch sets were tested on a loongArch64 Loongson-3C5000 processor. Can
successfully enter the crash command line and support for common command.

...
       KERNEL: vmlinux
     DUMPFILE: /proc/kcore
         CPUS: 16
         DATE: Thu Jul 27 19:51:21 CST 2023
       UPTIME: 06:35:11
LOAD AVERAGE: 0.15, 0.03, 0.01
        TASKS: 257
     NODENAME: localhost.localdomain
      RELEASE: 5.10.0-60.102.0.128.oe2203.loongarch64
      VERSION: #1 SMP Fri Jul 14 04:17:09 UTC 2023
      MACHINE: loongarch64  (2200 Mhz)
       MEMORY: 64 GB
          PID: 2964
      COMMAND: "crash"
         TASK: 9000000098805500  [THREAD_INFO: 9000000094d48000]
          CPU: 6
        STATE: TASK_RUNNING (ACTIVE)
crash>
crash> dis -l start_kernel
/linux-loongarch64/init/main.c: 883
0x9000000001030818 <start_kernel>:      0x0141ee40
/linux-loongarch64/init/main.c: 879
0x900000000103081c <start_kernel+4>:    0x90000000
/linux-loongarch64/init/main.c: 883
0x9000000001030820 <start_kernel+8>:    addu16i.d       $zero, $t8, 8179(0x1ff3)
/linux-loongarch64/init/main.c: 879
...

About the LoongArch64 Architecture:
https://www.kernel.org/doc/html/latest/arch/loongarch/introduction.html

Changes between v2 and v3:
- Fix some compilation warnings.
- Fix build errors when without target.
- Some minor code adjustments.

Thanks and regards,
Ming

Ming Wang (10):
   Add LoongArch64 framework code support
   LoongArch64: Make the crash tool successfully enter the crash command
     line
   LoongArch64: Add 'pte' command support
   LoongArch64: Add 'mach' command support
   LoongArch64: Add 'bt' command support
   LoongArch64: Add 'help -m/M' command support
   LoongArch64: Add 'help -r' command support
   LoongArch64: Add 'irq' command support
   LoongArch64: Add "--kaslr" command line option support
   LoongArch64: Add LoongArch64 architecture support information

  Makefile            |     7 +-
  README              |     4 +-
  configure.c         |    43 +-
  crash.8             |     2 +-
  defs.h              |   164 +-
  diskdump.c          |    24 +-
  gdb-10.2.patch      | 12822 +++++++++++++++++++++++++++++++++++++++++-

This is really a big change for the gdb, there are 12822 lines of code. :-) Is it possible to support the LoongArch64 feature if a very smaller change is made in the gdb patch? For example: trying to remove redundant(unnecessary) changes, as I mentioned in the V1.

Or is it hard to implement this feature based on the gdb-10.2? Looks like all code changes are backported from the latest gdb?


Thanks

Lianbo

  help.c              |    13 +-
  lkcd_vmdump_v1.h    |     2 +-
  lkcd_vmdump_v2_v3.h |     5 +-
  loongarch64.c       |  1368 +++++
  main.c              |     3 +-
  netdump.c           |    27 +-
  ramdump.c           |     2 +
  symbols.c           |    33 +-
  15 files changed, 14493 insertions(+), 26 deletions(-)
  create mode 100644 loongarch64.c


base-commit: 53d2577cef98b76b122aade94349637a11e06138
--
Crash-utility mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxxxxxx
https://${domain_name}/admin/lists/devel.lists.crash-utility.osci.io/
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