[PATCH] kexec: Fix add_usable_memory to use buf of type uint64_t.
Mahesh J Salgaonkar
mahesh at linux.vnet.ibm.com
Thu Dec 5 14:25:30 EST 2013
From: Mahesh Salgaonkar <mahesh at linux.vnet.ibm.com>
A switchover from /kexec/arch/ppc64/fs2dt.c to common code under
/kexec/fs2dt.c broke the kdump on ppc64. The function add_usable_memory()
assumes that 'memory@*' node exports two 32bit values and fails
to populate mem ranges correctly. On ppc64, 'memory@*' exports two 64bit
values as below:
# hexdump /proc/device-tree/memory\@0/reg
0000000 0000 0000 0000 0000 0000 0000 0800 0000
0000010
#
This patch fixes add_usable_memory() to use buf[2] of type uint64_t.
Signed-off-by: Mahesh Salgaonkar <mahesh at linux.vnet.ibm.com>
---
kexec/fs2dt.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/kexec/fs2dt.c b/kexec/fs2dt.c
index 5d774ae..7202dc1 100644
--- a/kexec/fs2dt.c
+++ b/kexec/fs2dt.c
@@ -285,8 +285,8 @@ static void add_dyn_reconf_usable_mem_property(struct dirent *dp, int fd) {}
static void add_usable_mem_property(int fd, size_t len)
{
char fname[MAXPATH], *bname;
- uint32_t buf[2];
- uint32_t *ranges;
+ uint64_t buf[2];
+ uint64_t *ranges;
int ranges_size = MEM_RANGE_CHUNK_SZ;
uint64_t base, end, loc_base, loc_end;
size_t range;
@@ -311,8 +311,8 @@ static void add_usable_mem_property(int fd, size_t len)
if (~0ULL - buf[0] < buf[1])
die("unrecoverable error: mem property overflow\n");
- base = be32_to_cpu(buf[0]);
- end = base + be32_to_cpu(buf[1]);
+ base = be64_to_cpu(buf[0]);
+ end = base + be64_to_cpu(buf[1]);
ranges = malloc(ranges_size * sizeof(*ranges));
if (!ranges)
More information about the kexec
mailing list