summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWei Huang2018-03-09 18:03:23 +0100
committerPeter Maydell2018-03-19 19:23:24 +0100
commit68cbecfdd7afbfdf9cb06a87a2a297e8a6add7d7 (patch)
treeda1cccbf7d88c589e93028467efa51430e63a947
parentfsl-imx6: Swap Ethernet interrupt defines (diff)
downloadqemu-68cbecfdd7afbfdf9cb06a87a2a297e8a6add7d7.tar.gz
qemu-68cbecfdd7afbfdf9cb06a87a2a297e8a6add7d7.tar.xz
qemu-68cbecfdd7afbfdf9cb06a87a2a297e8a6add7d7.zip
dump: Update correct kdump phys_base field for AArch64
For guest kernel that supports KASLR, the load address can change every time when guest VM runs. To find the physical base address correctly, current QEMU dump searches VMCOREINFO for the string "NUMBER(phys_base)=". However this string pattern is only available on x86_64. AArch64 uses a different field, called "NUMBER(PHYS_OFFSET)=". This patch makes sure QEMU dump uses the correct string on AArch64. Signed-off-by: Wei Huang <wei@redhat.com> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Message-id: 1520615003-20869-1-git-send-email-wei@redhat.com Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
-rw-r--r--dump.c14
1 files changed, 11 insertions, 3 deletions
diff --git a/dump.c b/dump.c
index 097e60b2b3..6bdb0dbe23 100644
--- a/dump.c
+++ b/dump.c
@@ -1609,10 +1609,18 @@ static void vmcoreinfo_update_phys_base(DumpState *s)
lines = g_strsplit((char *)vmci, "\n", -1);
for (i = 0; lines[i]; i++) {
- if (g_str_has_prefix(lines[i], "NUMBER(phys_base)=")) {
- if (qemu_strtou64(lines[i] + 18, NULL, 16,
+ const char *prefix = NULL;
+
+ if (s->dump_info.d_machine == EM_X86_64) {
+ prefix = "NUMBER(phys_base)=";
+ } else if (s->dump_info.d_machine == EM_AARCH64) {
+ prefix = "NUMBER(PHYS_OFFSET)=";
+ }
+
+ if (prefix && g_str_has_prefix(lines[i], prefix)) {
+ if (qemu_strtou64(lines[i] + strlen(prefix), NULL, 16,
&phys_base) < 0) {
- warn_report("Failed to read NUMBER(phys_base)=");
+ warn_report("Failed to read %s", prefix);
} else {
s->dump_info.phys_base = phys_base;
}