Message ID | 20180628113817.30814-3-david@redhat.com |
---|---|
State | New |
Headers | show |
Series | s390x/kvm: legacy_s390_alloc() fixes | expand |
diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c index bbac3454e3..22e13080a5 100644 --- a/target/s390x/kvm.c +++ b/target/s390x/kvm.c @@ -765,6 +765,9 @@ static void *legacy_s390_alloc(size_t size, uint64_t *align, bool shared) if (mem == MAP_FAILED) { mem = NULL; } + if (mem && align) { + *align = QEMU_VMALLOC_ALIGN; + } return mem; }
Let's do this for completeness reason, although we don't support e.g. PCDIMM/NVDIMM, which would use the alignment for placing the memory region in guest physical memory. But maybe someday we would want to support something like this - then we don't forget about this if allowing multiple allocations in legacy_s390_alloc(). Use the same alignment as we would set in qemu_anon_ram_alloc(). Our fixed address satisfies this alignment (1MB). This implicitly sets the alignment of the underlying memory region. Signed-off-by: David Hildenbrand <david@redhat.com> --- target/s390x/kvm.c | 3 +++ 1 file changed, 3 insertions(+)