The 'base' argument of mtree_print_mr() actually represents an offset, not a base address. Rename it as such.
Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> --- softmmu/memory.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/softmmu/memory.c b/softmmu/memory.c index 874a8fccdee..e4d93b2fd6f 100644 --- a/softmmu/memory.c +++ b/softmmu/memory.c @@ -2925,7 +2925,7 @@ static void mtree_print_mr_owner(const MemoryRegion *mr) } static void mtree_print_mr(const MemoryRegion *mr, unsigned int level, - hwaddr base, + hwaddr offset, MemoryRegionListHead *alias_print_queue, bool owner, bool display_disabled) { @@ -2939,7 +2939,7 @@ static void mtree_print_mr(const MemoryRegion *mr, unsigned int level, return; } - cur_start = base + mr->addr; + cur_start = offset + mr->addr; cur_end = cur_start + MR_SIZE(mr->size); /* @@ -2947,7 +2947,7 @@ static void mtree_print_mr(const MemoryRegion *mr, unsigned int level, * happen normally. When it happens, we dump something to warn the * user who is observing this. */ - if (cur_start < base || cur_end < cur_start) { + if (cur_start < offset || cur_end < cur_start) { qemu_printf("[DETECTED OVERFLOW!] "); } -- 2.26.2