Re: [Crash-utility] [PATCHv3] crash-utility/arm64: store phy_offset and memstart_addr separately

Crash encounters a bug like the following: ... License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html> This is free software: you are free to change and redistribute it. There is NO WARRANTY, to the extent permitted by law. Type "show copying" and "show warranty" for details. This GDB was configured as "aarch64-unknown-linux-gnu"... crash: read error: kernel virtual address: ffff000f789c0050 type: "IRQ stack pointer" crash: read error: kernel virtual address: ffff000f78a60050 type: "IRQ stack pointer" crash: read error: kernel virtual address: ffff000f78b00050 type: "IRQ stack pointer" ... This bug connects with kernel commit 7bc1a0f9e176 ("arm64: mm: use single quantity to represent the PA to VA translation"), memstart_addr can be negative, which makes it different from real phys_offset. In crash utility, PTOV() needs memstart_addr to calculate VA from PA, while getting PFN offset in a dumpfile, phys_offset is required. To serve the different purpose, using phys_offset_nominal and phys_offset to store them. Signed-off-by: Pingfan Liu <piliu(a)redhat.com&gt; Cc: HAGIO KAZUHITO <k-hagio-ab(a)nec.com&gt; Cc: Lianbo Jiang <lijiang(a)redhat.com&gt; Cc: Bhupesh Sharma <bhupesh.sharma(a)linaro.org&gt; Cc: Mark Salter <msalter(a)redhat.com&gt; Cc: Mark Langsdorf <mlangsdo(a)redhat.com&gt; Cc: Jeremy Linton <jlinton(a)redhat.com&gt; To: crash-utility(a)redhat.com --- v2 -> v3: rename ms->memstart_addr as ms->phys_offset_nominal ( I keep the name as phys_offset* since it is in accordance with other platform conventions) --- arm64.c | 25 ++++++++++++++++++++++--- defs.h | 3 +++ 2 files changed, 25 insertions(+), 3 deletions(-) diff --git a/arm64.c b/arm64.c index 37aed07..5e567ca 100644 --- a/arm64.c +++ b/arm64.c @@ -24,6 +24,9 @@ #define NOT_IMPLEMENTED(X) error((X), "%s: function not implemented\n", __func__) +#define MEMSTART_ADDR_OFFSET \ + (0xffffffffffffffff << 48 - 0xffffffffffffffff << 56) +