3:57 a.m.
Hi, Ming
Thank you for the update.
On 12/28/23 19:47, devel-request(a)lists.crash-utility.osci.io wrote:
Date: Thu, 28 Dec 2023 19:46:24 +0800
From: Ming Wang <wangming01(a)loongson.cn>
Subject: [Crash-utility] [Crash-utility][PATCH v3 00/10] add
LoongArch64 platform support
To: devel(a)lists.crash-utility.osci.io, lijiang(a)redhat.com,
k-hagio-ab(a)nec.com
Cc: gaojuxin(a)loongson.cn, liweihao(a)loongson.cn
Message-ID: <20231228114634.1085279-1-wangming01(a)loongson.cn>
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
9:46 p.m.
New subject: [PATCH v3 00/10] add LoongArch64 platform support
Hi, Lianbo
On 1/3/24 17:57, Lianbo Jiang wrote:
Hi, Ming
Thank you for the update.
>
> 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?
Yes, it is. The upstream loongarch gdb
does not work until 13.2.
I also tried to build a very small compact version myself,but it
didn't work.I'm still looking for the reason.
Perhaps waiting for crash to rebase embedded gdb to a later version
is the best way to solve this problem.At present, I have also seen
someone submit a preview version of 13.2[1].
[1]: https://github.com/liutgnu/crash-preview
I can do some testing work based on this preview version first.
Thanks,
Ming
1:56 a.m.
New subject: [PATCH v3 00/10] add LoongArch64 platform support
On 1/4/24 11:46, Ming Wang wrote:
Hi, Lianbo
On 1/3/24 17:57, Lianbo Jiang wrote:
> Hi, Ming
>
> Thank you for the update.
>> 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?
Yes, it is. The upstream loongarch gdb does not work until 13.2.
Got it.
I also tried to build a very small compact version myself,but it
didn't work.I'm still looking for the reason.
Thanks for trying, Ming.
This is ideal situation if the gdb patch is very small, it will be easy
to maintain and debug.
Perhaps waiting for crash to rebase embedded gdb to a later version
is the best way to solve this problem.At present, I have also seen
You are right, there are fewer code changes based on the rebase version.
I would tend to wait for the rebase(if it is not urgent for you), but
recently we have not planned to update the embedded gdb in crash-utility.
Anyway, maybe Kazu has some comments on it(BTW:He is on vacation).
Thanks.
Lianbo
someone submit a preview version of 13.2[1].
[1]: https://github.com/liutgnu/crash-preview
I can do some testing work based on this preview version first.
Thanks,
Ming
11:21 p.m.
New subject: [PATCH v3 00/10] add LoongArch64 platform support
On 2024/01/05 16:56, Lianbo Jiang wrote:
On 1/4/24 11:46, Ming Wang wrote:
> Hi, Lianbo
>
> On 1/3/24 17:57, Lianbo Jiang wrote:
>> Hi, Ming
>>
>> Thank you for the update.
>>> 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?
> Yes, it is. The upstream loongarch gdb does not work until 13.2.
Got it.
> I also tried to build a very small compact version myself,but it
> didn't work.I'm still looking for the reason.
Thanks for trying, Ming.
This is ideal situation if the gdb patch is very small, it will be easy
to maintain and debug.
>
> Perhaps waiting for crash to rebase embedded gdb to a later version
> is the best way to solve this problem.At present, I have also seen
You are right, there are fewer code changes based on the rebase version.
I would tend to wait for the rebase(if it is not urgent for you), but
recently we have not planned to update the embedded gdb in crash-utility.
Anyway, maybe Kazu has some comments on it(BTW:He is on vacation).
with the Ming's gdb patch there is no necessity of rebasing gdb for now,
let's go with this. will review it, please wait for a while.
Thanks,
Kazu
659
days inactive
667
days old
devel@lists.crash-utility.osci.io
3 comments
3 participants
participants (3)
-
HAGIO KAZUHITO(萩尾 一仁) -
Lianbo Jiang -
Ming Wang