]> git.proxmox.com Git - qemu.git/commit
qemu: Use valgrind annotations to mark kvm guest memory as defined
authorChristian Borntraeger <borntraeger@de.ibm.com>
Fri, 10 Aug 2012 13:11:45 +0000 (15:11 +0200)
committerAvi Kivity <avi@redhat.com>
Sun, 9 Sep 2012 13:48:34 +0000 (16:48 +0300)
commit62fe83318d2fc5b31f473d66326910d94c1c4907
tree89d23660c8b2cefc9b309a0de03dee5c7ac3ec04
parent0c267217ca9985e6d118ec8368bebd382db7a099
qemu: Use valgrind annotations to mark kvm guest memory as defined

valgrind with kvm produces a big amount of false positives regarding
"Conditional jump or move depends on uninitialised value(s)". This
happens because the guest memory is allocated with qemu_vmalloc which
boils down posix_memalign etc. This function is (correctly) considered
by valgrind as returning undefined memory.

Since valgrind is based on jitting code, it will not be able to see
changes made by the guest to guest memory if this is done by KVM_RUN,
thus keeping most of the guest memory undefined.

Now lots of places in qemu will then use guest memory to change behaviour.
To avoid the flood of these messages, lets declare the whole guest
memory as defined. This will reduce the noise and allows us to see real
problems.

In the future we might want to make this conditional, since there
is actually something that we can use those false positives for:
These messages will point to code that depends on guest memory, so
we can use these backtraces to actually make an audit that is focussed
only at those code places. For normal development we dont want to
see those messages, though.

Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
configure
kvm-all.c