TCG: 0.982
KVM: 0.972
ppc: 0.967
hypervisor: 0.961
vnc: 0.960
user-level: 0.951
PID: 0.950
virtual: 0.947
device: 0.944
performance: 0.939
graphic: 0.938
mistranslation: 0.937
files: 0.937
peripherals: 0.936
x86: 0.934
semantic: 0.929
kernel: 0.928
VMM: 0.917
register: 0.909
architecture: 0.902
socket: 0.893
network: 0.882
risc-v: 0.880
permissions: 0.879
debug: 0.869
i386: 0.867
boot: 0.864
arm: 0.859
assembly: 0.838

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.