On Mon, Jan 10, 2011 at 10:05:17AM +0800, Hao, Xudong wrote: > Qemu-kvm commit 92d675d1c1f23f3617e24b63c825074a1d1da44b changed sx, dx and > others variables into a judgement, there was a build failure on RHEL5 system. > > ... > CC x86_64-softmmu/debugcon.o > cc1: warnings being treated as errors > /workspace/ia32-pae/nightly/qemu-kvm/hw/cirrus_vga.c: In function > 'cirrus_bitblt_start': > /workspace/ia32-pae/nightly/qemu-kvm/hw/cirrus_vga.c:678: warning: 'dx' may > be used uninitialized in this function > /workspace/ia32-pae/nightly/qemu-kvm/hw/cirrus_vga.c:677: warning: 'sx' may > be used uninitialized in this function > /workspace/ia32-pae/nightly/qemu-kvm/hw/cirrus_vga.c:679: warning: 'depth' > may be used uninitialized in this function > /workspace/ia32-pae/nightly/qemu-kvm/hw/cirrus_vga.c:678: warning: 'dy' may > be used uninitialized in this function > /workspace/ia32-pae/nightly/qemu-kvm/hw/cirrus_vga.c:677: warning: 'sy' may > be used uninitialized in this function > CC x86_64-softmmu/multiboot.o > make[1]: *** [cirrus_vga.o] Error 1 > make[1]: *** Waiting for unfinished jobs.... > make: *** [subdir-x86_64-softmmu] Error 2 >
Yes, the code is correct, but some compilers choke on that, some not. Anyway it's fixed in 78935c4a4bfec8ef2f4f924cfc35ac09a963e81e. Aurelien -- Aurelien Jarno GPG: 1024D/F1BCDB73 [email protected] http://www.aurel32.net -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
