aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlex Bennée <alex.bennee@linaro.org>2020-04-03 20:11:42 +0100
committerAlex Bennée <alex.bennee@linaro.org>2020-04-07 16:19:49 +0100
commitb859040dc44b271e9ad29f729cac71d2389b05fd (patch)
treef302354abb5d827934db9ae0b091c815df2aca86
parentaf7fc47f2c28fe0183ce98aa070e1b899cd08199 (diff)
downloadqemu-b859040dc44b271e9ad29f729cac71d2389b05fd.zip
qemu-b859040dc44b271e9ad29f729cac71d2389b05fd.tar.gz
qemu-b859040dc44b271e9ad29f729cac71d2389b05fd.tar.bz2
linux-user: more debug for init_guest_space
Searching for memory space can cause problems so lets extend the CPU_LOG_PAGE output so you can watch init_guest_space fail to allocate memory. A more involved fix is actually required to make this function play nicely with the large guard pages the sanitiser likes to use. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Laurent Vivier <laurent@vivier.eu> Message-Id: <20200403191150.863-5-alex.bennee@linaro.org>
-rw-r--r--linux-user/elfload.c8
1 files changed, 7 insertions, 1 deletions
diff --git a/linux-user/elfload.c b/linux-user/elfload.c
index 8198be0..619c054 100644
--- a/linux-user/elfload.c
+++ b/linux-user/elfload.c
@@ -2172,6 +2172,8 @@ unsigned long init_guest_space(unsigned long host_start,
/* Check to see if the address is valid. */
if (host_start && real_start != current_start) {
+ qemu_log_mask(CPU_LOG_PAGE, "invalid %lx && %lx != %lx\n",
+ host_start, real_start, current_start);
goto try_again;
}
@@ -2240,7 +2242,11 @@ unsigned long init_guest_space(unsigned long host_start,
* probably a bad strategy if not, which means we got here
* because of trouble with ARM commpage setup.
*/
- munmap((void *)real_start, real_size);
+ if (munmap((void *)real_start, real_size) != 0) {
+ error_report("%s: failed to unmap %lx:%lx (%s)", __func__,
+ real_start, real_size, strerror(errno));
+ abort();
+ }
current_start += align;
if (host_start == current_start) {
/* Theoretically possible if host doesn't have any suitably