other: 0.142
semantic: 0.129
KVM: 0.097
PID: 0.097
files: 0.083
device: 0.076
graphic: 0.067
vnc: 0.066
performance: 0.053
permissions: 0.042
socket: 0.041
debug: 0.040
network: 0.038
boot: 0.030
debug: 0.414
files: 0.102
KVM: 0.083
PID: 0.072
other: 0.060
boot: 0.042
network: 0.040
device: 0.037
semantic: 0.033
socket: 0.030
performance: 0.027
graphic: 0.023
vnc: 0.019
permissions: 0.017

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.