diff options
author | Michael S. Tsirkin <mst@redhat.com> | 2013-11-04 08:06:08 +0200 |
---|---|---|
committer | Michael S. Tsirkin <mst@redhat.com> | 2013-11-04 15:38:49 +0200 |
commit | 818f86b88394b7b2b59d313e51043fe15a8004db (patch) | |
tree | eb232e56b97fe1374415e7fc3c1f92a593c4f82f /exec.c | |
parent | 98af2ac93fa6bd83e19f7cff1cc9513fa7c4d58d (diff) | |
download | qemu-818f86b88394b7b2b59d313e51043fe15a8004db.zip qemu-818f86b88394b7b2b59d313e51043fe15a8004db.tar.gz qemu-818f86b88394b7b2b59d313e51043fe15a8004db.tar.bz2 |
exec: limit system memory size
The page table logic in exec.c assumes
that memory addresses are at most TARGET_PHYS_ADDR_SPACE_BITS.
But pci addresses are full 64 bit so if we try to render them ignoring
the extra bits, we get strange effects with sections overlapping each
other.
To fix, simply limit the system memory size to
1 << TARGET_PHYS_ADDR_SPACE_BITS,
pci addresses will be rendered within that.
Cc: qemu-stable@nongnu.org
Reported-by: Andreas Färber <afaerber@suse.de>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'exec.c')
-rw-r--r-- | exec.c | 7 |
1 files changed, 6 insertions, 1 deletions
@@ -1741,7 +1741,12 @@ void address_space_destroy_dispatch(AddressSpace *as) static void memory_map_init(void) { system_memory = g_malloc(sizeof(*system_memory)); - memory_region_init(system_memory, NULL, "system", INT64_MAX); + + assert(TARGET_PHYS_ADDR_SPACE_BITS <= 64); + + memory_region_init(system_memory, NULL, "system", + TARGET_PHYS_ADDR_SPACE_BITS == 64 ? + UINT64_MAX : (0x1ULL << TARGET_PHYS_ADDR_SPACE_BITS)); address_space_init(&address_space_memory, system_memory, "memory"); system_io = g_malloc(sizeof(*system_io)); |