diff options
Diffstat (limited to 'results/classifier/118/all/1268596')
| -rw-r--r-- | results/classifier/118/all/1268596 | 83 |
1 files changed, 83 insertions, 0 deletions
diff --git a/results/classifier/118/all/1268596 b/results/classifier/118/all/1268596 new file mode 100644 index 00000000..61ba6d25 --- /dev/null +++ b/results/classifier/118/all/1268596 @@ -0,0 +1,83 @@ +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. + |