3:29 p.m.
- Introduced support for upstream xensource ELF format dumpfiles,
which will replace the current xendump format in xen 3.0.5. The
new xen format uses ELF in a non-standard manner such that memory
contents are defined in section headers instead of the traditional
manner of using program headers. Testing has been on paravirtualized
x86, x86 PAE, x86_64 and ia64 dumpfiles. Fully-virtualized dumpfiles
have not been tested. (anderson(a)redhat.com)
- A number of "xencrash" (where the session is run against a xen-syms
binary) fixes have been applied:
1) "bt" did not switch from the ia64 MCA stack to the vcpu stack.
2) "bt" caused an infinite loop if ar_bspstore contained an illegal
value.
3) "bt" shows unnecessary unwind warning message. (ia64)
4) "man log" caused crash to fail with a segmentation violation.
5) "man log" did not have an example.
(oda(a)valinux.co.jp)
- Fix for "vtop" on x86 PAE kernels, which could abort upon reaching
the PTE translation section, showing the error message: "vtop:
cannot determine the swap location". (anderson(a)redhat.com)
- Fix for "vm -p" or "vtop" on 2.6 x86 PAE kernels, which could show
incorrect swap offsets, because the swap type/offset encoding was
moved to the high word of the 64-bit PTE. (anderson(a)redhat.com)
- Fix for "vm -p" on x86_64 kernels when a PTE referenced a swap
location, it would show "(not mapped)" instead of the swap location.
(anderson(a)redhat.com)
- In current 2.6 kernels, it is now possible to recognize ppc BOOKE
processors, which is the current default in crash. If the processor
is confirmed to not be BOOKE, then page table translation is done
differently. (antipov(a)ru.mvista.com)
- Fix for live system analysis of Ubuntu kernels due to a mismatch
between /proc/version and the linux_banner string. This was due
to an appendage to the linux_banner string in Ubuntu kernels.
(asid(a)hp.com)
- Fix for 2.6.21 kernels that fail during initialization with the
message: "crash: invalid (optional) structure member offsets:
zone_struct_free_pages or zone_free_pages". This was due to the
removal of the zone struct's "free_pages" member; instead the
zone's "vm_stat[NR_FREE_PAGES]" value is used. (anderson(a)redhat.com)
Download from: http://people.redhat.com/anderson
9:31 a.m.
* Dave Anderson <anderson(a)redhat.com> [2007-03-16 21:29]:
- In current 2.6 kernels, it is now possible to recognize ppc BOOKE
processors, which is the current default in crash. If the processor
is confirmed to not be BOOKE, then page table translation is done
differently. (antipov(a)ru.mvista.com)
The current code doesn't build on PPC32. Follwing patch fixes the
problem. However, I consider the patch as evil, so maybe it would be
better to change some data types that are involved here, but I don't
really know.
---
ppc.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
Index: crash-4.0-3.21/ppc.c
===================================================================
--- crash-4.0-3.21.orig/ppc.c
+++ crash-4.0-3.21/ppc.c
@@ -301,8 +301,8 @@ ppc_uvtop(struct task_context *tc, ulong
if (machdep->flags & CPU_BOOKE)
page_table = page_middle + (BTOP(vaddr) & (PTRS_PER_PTE - 1));
else
- page_table = ((page_middle & machdep->pagemask) + machdep->kvbase) +
- (BTOP(vaddr) & (PTRS_PER_PTE-1));
+ page_table = (ulong *)((ulong)( (((ulonglong)(ulong)page_middle &
machdep->pagemask)
+ + machdep->kvbase) + ((BTOP(vaddr)) & (PTRS_PER_PTE-1)) ));
if (verbose)
fprintf(fp, " PMD: %lx => %lx\n",(ulong)page_middle,
@@ -389,8 +389,8 @@ ppc_kvtop(struct task_context *tc, ulong
if (machdep->flags & CPU_BOOKE)
page_table = page_middle + (BTOP(kvaddr) & (PTRS_PER_PTE - 1));
else
- page_table = ((page_middle & machdep->pagemask) + machdep->kvbase) +
- (BTOP(kvaddr) & (PTRS_PER_PTE-1));
+ page_table = (ulong *)((ulong)( (((ulonglong)(ulong)page_middle &
machdep->pagemask)
+ + machdep->kvbase) + ((BTOP(kvaddr)) & (PTRS_PER_PTE-1)) ));
if (verbose)
fprintf(fp, " PMD: %lx => %lx\n", (ulong)page_middle,
10:12 a.m.
Bernhard Walle wrote:
* Dave Anderson <anderson(a)redhat.com> [2007-03-16 21:29]:
>
> - In current 2.6 kernels, it is now possible to recognize ppc BOOKE
> processors, which is the current default in crash. If the processor
> is confirmed to not be BOOKE, then page table translation is done
> differently. (antipov(a)ru.mvista.com)
The current code doesn't build on PPC32. Follwing patch fixes the
problem. However, I consider the patch as evil, so maybe it would be
better to change some data types that are involved here, but I don't
really know.
Figures... I took that patch on blind faith since I don't have
any 32-bit ppc hardware to compile it on, never mind test it:
https://www.redhat.com/archives/crash-utility/2007-March/msg00020.html
Anyway, I'll still defer to the MontaVista guys to come up with
a patch that they can confirm works -- and that compiles, please...
Thanks,
Dave
---
ppc.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
Index: crash-4.0-3.21/ppc.c
===================================================================
--- crash-4.0-3.21.orig/ppc.c
+++ crash-4.0-3.21/ppc.c
@@ -301,8 +301,8 @@ ppc_uvtop(struct task_context *tc, ulong
if (machdep->flags & CPU_BOOKE)
page_table = page_middle + (BTOP(vaddr) & (PTRS_PER_PTE - 1));
else
- page_table = ((page_middle & machdep->pagemask) +
machdep->kvbase) +
- (BTOP(vaddr) & (PTRS_PER_PTE-1));
+ page_table = (ulong *)((ulong)( (((ulonglong)(ulong)page_middle &
machdep->pagemask)
+ + machdep->kvbase) + ((BTOP(vaddr))
& (PTRS_PER_PTE-1)) ));
if (verbose)
fprintf(fp, " PMD: %lx => %lx\n",(ulong)page_middle,
@@ -389,8 +389,8 @@ ppc_kvtop(struct task_context *tc, ulong
if (machdep->flags & CPU_BOOKE)
page_table = page_middle + (BTOP(kvaddr) & (PTRS_PER_PTE - 1));
else
- page_table = ((page_middle & machdep->pagemask) +
machdep->kvbase) +
- (BTOP(kvaddr) & (PTRS_PER_PTE-1));
+ page_table = (ulong *)((ulong)( (((ulonglong)(ulong)page_middle &
machdep->pagemask)
+ + machdep->kvbase) + ((BTOP(kvaddr))
& (PTRS_PER_PTE-1)) ));
if (verbose)
fprintf(fp, " PMD: %lx => %lx\n", (ulong)page_middle,
------------------------------------------------------------------------------------------------------------------------
Part 1.1.2Type: application/pgp-signature
------------------------------------------------------------------------------------------------------------------------
--
Crash-utility mailing list
Crash-utility(a)redhat.com
https://www.redhat.com/mailman/listinfo/crash-utility
11:35 a.m.
Dave Anderson wrote:
Bernhard Walle wrote:
> * Dave Anderson <anderson(a)redhat.com> [2007-03-16 21:29]:
>> - In current 2.6 kernels, it is now possible to recognize ppc BOOKE
>> processors, which is the current default in crash. If the processor
>> is confirmed to not be BOOKE, then page table translation is done
>> differently. (antipov(a)ru.mvista.com)
> The current code doesn't build on PPC32. Follwing patch fixes the
> problem. However, I consider the patch as evil, so maybe it would be
> better to change some data types that are involved here, but I don't
> really know.
>
Figures... I took that patch on blind faith since I don't have
any 32-bit ppc hardware to compile it on, never mind test it:
https://www.redhat.com/archives/crash-utility/2007-March/msg00020.html
Anyway, I'll still defer to the MontaVista guys to come up with
a patch that they can confirm works -- and that compiles, please...
Argh, I've broke the build :-(. What a silly error :-((.
But, it may be better to avoid conversion to long long and back, like
the following:
--- .orig-ppc.c 2007-03-16 22:35:05.000000000 +0300
+++ ppc.c 2007-03-19 19:23:48.000000000 +0300
@@ -301,8 +301,8 @@
if (machdep->flags & CPU_BOOKE)
page_table = page_middle + (BTOP(vaddr) & (PTRS_PER_PTE - 1));
else
- page_table = ((page_middle & machdep->pagemask) +
machdep->kvbase) +
- (BTOP(vaddr) & (PTRS_PER_PTE-1));
+ page_table = (ulong *)(((pgd_pte & (ulong)machdep->pagemask) +
machdep->kvbase) +
+ ((ulong)BTOP(vaddr) & (PTRS_PER_PTE-1)));
if (verbose)
fprintf(fp, " PMD: %lx => %lx\n",(ulong)page_middle,
@@ -389,8 +389,8 @@
if (machdep->flags & CPU_BOOKE)
page_table = page_middle + (BTOP(kvaddr) & (PTRS_PER_PTE - 1));
else
- page_table = ((page_middle & machdep->pagemask) +
machdep->kvbase) +
- (BTOP(kvaddr) & (PTRS_PER_PTE-1));
+ page_table = (ulong *)(((pgd_pte & (ulong)machdep->pagemask) +
machdep->kvbase) +
+ ((ulong)BTOP(kvaddr) & (PTRS_PER_PTE-1)));
if (verbose)
fprintf(fp, " PMD: %lx => %lx\n", (ulong)page_middle,
But I should check that we didn't loss anything while doing the conversion.
Dmitry
11:47 a.m.
* Dmitry Antipov <antipov(a)ru.mvista.com> [2007-03-19 17:35]:
Argh, I've broke the build :-(. What a silly error :-((.
But, it may be better to avoid conversion to long long and back, like
the following:
The patch is quite unreadable. Please consider using attachments if
your mail client is not able to send inline patches without breaking
them.
Regards,
Bernhard
12:55 p.m.
Dmitry Antipov wrote:
Dave Anderson wrote:
> Bernhard Walle wrote:
>
>> * Dave Anderson <anderson(a)redhat.com> [2007-03-16 21:29]:
>>> - In current 2.6 kernels, it is now possible to recognize ppc BOOKE
>>> processors, which is the current default in crash. If the processor
>>> is confirmed to not be BOOKE, then page table translation is done
>>> differently. (antipov(a)ru.mvista.com)
>> The current code doesn't build on PPC32. Follwing patch fixes the
>> problem. However, I consider the patch as evil, so maybe it would be
>> better to change some data types that are involved here, but I don't
>> really know.
>>
>
> Figures... I took that patch on blind faith since I don't have
> any 32-bit ppc hardware to compile it on, never mind test it:
>
> https://www.redhat.com/archives/crash-utility/2007-March/msg00020.html
>
> Anyway, I'll still defer to the MontaVista guys to come up with
> a patch that they can confirm works -- and that compiles, please...
Argh, I've broke the build :-(. What a silly error :-((.
But, it may be better to avoid conversion to long long and back, like
the following:
--- .orig-ppc.c 2007-03-16 22:35:05.000000000 +0300
+++ ppc.c 2007-03-19 19:23:48.000000000 +0300
@@ -301,8 +301,8 @@
if (machdep->flags & CPU_BOOKE)
page_table = page_middle + (BTOP(vaddr) & (PTRS_PER_PTE - 1));
else
- page_table = ((page_middle & machdep->pagemask) +
machdep->kvbase) +
- (BTOP(vaddr) & (PTRS_PER_PTE-1));
+ page_table = (ulong *)(((pgd_pte & (ulong)machdep->pagemask) +
machdep->kvbase) +
+ ((ulong)BTOP(vaddr) & (PTRS_PER_PTE-1)));
if (verbose)
fprintf(fp, " PMD: %lx => %lx\n",(ulong)page_middle,
@@ -389,8 +389,8 @@
if (machdep->flags & CPU_BOOKE)
page_table = page_middle + (BTOP(kvaddr) & (PTRS_PER_PTE - 1));
else
- page_table = ((page_middle & machdep->pagemask) +
machdep->kvbase) +
- (BTOP(kvaddr) & (PTRS_PER_PTE-1));
+ page_table = (ulong *)(((pgd_pte & (ulong)machdep->pagemask) +
machdep->kvbase) +
+ ((ulong)BTOP(kvaddr) & (PTRS_PER_PTE-1)));
if (verbose)
fprintf(fp, " PMD: %lx => %lx\n", (ulong)page_middle,
But I should check that we didn't loss anything while doing the conversion.
Dmitry
It compiles OK -- just let me know if you're happy with it.
Thanks,
Dave
1:14 p.m.
Bernhard Walle wrote:
* Dave Anderson <anderson(a)redhat.com> [2007-03-19 18:55]:
> It compiles OK -- just let me know if you're happy with it.
He, your old Netscape 4.7 doesn't know format=flowed so you got the
patch readable .....
Regards,
Bernhard
Actually Dmitry's patch was readable, although both sections were rejected
for some reason, so I just applied it manually.
On the other hand, I thought that your patch was flowed -- here's what the
"saved" version looks like:
ppc.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
Index: crash-4.0-3.21/ppc.c
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- crash-4.0-3.21.orig/ppc.c
+++ crash-4.0-3.21/ppc.c
@@ -301,8 +301,8 @@ ppc_uvtop(struct task_context *tc, ulong
if (machdep->flags & CPU_BOOKE)
page_table =3D page_middle + (BTOP(vaddr) & (PTRS_PER_PTE - 1));
else
- page_table =3D ((page_middle & machdep->pagemask) +
machdep->kvbase) +
- (BTOP(vaddr) & (PTRS_PER_PTE-1));
+ page_table =3D (ulong *)((ulong)( (((ulonglong)(ulong)page_middle &
mach=
dep->pagemask)
+ + machdep->kvbase) + ((BTOP(vaddr))
& (PTRS_PER_PTE-1)) ));
=20
if (verbose)
fprintf(fp, " PMD: %lx =3D> %lx\n",(ulong)page_middle,=20
@@ -389,8 +389,8 @@ ppc_kvtop(struct task_context *tc, ulong
if (machdep->flags & CPU_BOOKE)
page_table =3D page_middle + (BTOP(kvaddr) & (PTRS_PER_PTE - 1));
else
- page_table =3D ((page_middle & machdep->pagemask) +
machdep->kvbase) +
- (BTOP(kvaddr) & (PTRS_PER_PTE-1));
+ page_table =3D (ulong *)((ulong)( (((ulonglong)(ulong)page_middle &
mach=
dep->pagemask)
+ + machdep->kvbase) + ((BTOP(kvaddr))
& (PTRS_PER_PTE-1)) ));
=20
if (verbose)
fprintf(fp, " PMD: %lx =3D> %lx\n", (ulong)page_middle,=20
--r5Pyd7+fXNt84Ff3
Content-Type: application/pgp-signature
Dave
8:58 p.m.
Hi, Dave
- Introduced support for upstream xensource ELF format dumpfiles,
which will replace the current xendump format in xen 3.0.5. The
new xen format uses ELF in a non-standard manner such that memory
contents are defined in section headers instead of the traditional
manner of using program headers. Testing has been on paravirtualized
x86, x86 PAE, x86_64 and ia64 dumpfiles. Fully-virtualized dumpfiles
have not been tested. (anderson(a)redhat.com)
I tested vmcore with ELF format "xm dump-core" and crash-4.0-3.21 on IA64.
The result is good. Thank you for your work.
# crash /usr/lib/debug/lib/modules/2.6.9-22.EL/vmlinux /var/xen/dump/2007-0405-
2138.38-vti.1.core
crash 4.0-3.21
Copyright (C) 2002, 2003, 2004, 2005, 2006, 2007 Red Hat, Inc.
Copyright (C) 2004, 2005, 2006 IBM Corporation
Copyright (C) 1999-2006 Hewlett-Packard Co
Copyright (C) 2005, 2006 Fujitsu Limited
Copyright (C) 2006, 2007 VA Linux Systems Japan K.K.
Copyright (C) 2005 NEC Corporation
Copyright (C) 1999, 2002 Silicon Graphics, Inc.
Copyright (C) 1999, 2000, 2001, 2002 Mission Critical Linux, Inc.
This program is free software, covered by the GNU General Public License,
and you are welcome to change it and/or distribute copies of it under
certain conditions. Enter "help copying" to see the conditions.
This program has absolutely no warranty. Enter "help warranty" for details.
GNU gdb 6.1
Copyright 2004 Free Software Foundation, Inc.
GDB is free software, covered by the GNU General Public License, and you are
welcome to change it and/or distribute copies of it under certain conditions.
Type "show copying" to see the conditions.
There is absolutely no warranty for GDB. Type "show warranty" for details.
This GDB was configured as "ia64-unknown-linux-gnu"...
KERNEL: /usr/lib/debug/lib/modules/2.6.9-22.EL/vmlinux
DUMPFILE: /var/xen/dump/2007-0405-2138.38-vti.1.core
CPUS: 4
DATE: Thu Apr 5 12:38:36 2007
UPTIME: 00:00:19
LOAD AVERAGE: 0.14, 0.03, 0.01
TASKS: 40
NODENAME: tiger2
RELEASE: 2.6.9-22.EL
VERSION: #1 SMP Mon Sep 19 17:54:55 EDT 2005
MACHINE: ia64 (1595 Mhz)
MEMORY: 480.7 MB
PANIC: ""
PID: 0
COMMAND: "swapper"
TASK: a0000001006c4000 (1 of 4) [THREAD_INFO: a0000001006c4dc0]
CPU: 0
STATE: TASK_RUNNING
WARNING: panic task not found
crash>
Best Regards,
Akio Takebe
6290
days inactive
6315
days old
devel@lists.crash-utility.osci.io
9 comments
4 participants
participants (4)
-
Akio Takebe -
Bernhard Walle -
Dave Anderson -
Dmitry Antipov