KVM: 0.972
vnc: 0.960
other: 0.955
PID: 0.950
device: 0.944
performance: 0.939
graphic: 0.938
files: 0.937
semantic: 0.929
socket: 0.893
network: 0.882
permissions: 0.879
debug: 0.869
boot: 0.864

Compilation Error: hw/virtio/dataplane/vring.c:400:5: error: ‘ret’ may be used uninitialised in this function

Qemu git-cloned from mo. 13.01.14 (ca. 13:00 GMT), Version 1.7.50


#git clone git://git.qemu-project.org/qemu.git
# cd qemu; git submodule update --init dtc
#./configure --disable-xen --enable-kvm
...No Errors...

#CC="ccache gcc" make -j8
....
  GEN   qemu.1
  Signing optionrom/kvmvapic.bin
  GEN   qemu-img.1
  CC    qapi-types.o
hw/virtio/dataplane/vring.c: In function ‘vring_pop’:
hw/virtio/dataplane/vring.c:400:5: error: ‘ret’ may be used uninitialised in this function [-Werror=uninitialized]
cc1: all warnings being treated as errors
make: *** [hw/virtio/dataplane/vring.o] Error 1
make: *** Waiting for unfinished jobs....


Thx.



What compiler is this? The variable is quite obviously initialized before that line.

On 13 January 2014 14:40, Paolo Bonzini <email address hidden> wrote:
> What compiler is this? The variable is quite obviously initialized
> before that line.

The issue is that one of the code paths has a shadowing declaration
of 'ret' which is what gets assigned to, and so in that code path
the compiler is correct that the outer 'ret' is not assigned to.

Stefan said he was going to send out a fix for this.

thanks
-- PMM


A fix has been posted to the mailing list and will soon be merged into qemu.git:

http://thread.gmane.org/gmane.comp.emulators.qemu/250657

Thanks a lot.

Fix had been included here:
http://git.qemu.org/?p=qemu.git;a=commitdiff;h=385c04d0b66917457b6
... thus marking this ticked as fixed.