qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] Re: [PATCH] configure: Add compiler option -Wmissing-format


From: Stefan Weil
Subject: [Qemu-devel] Re: [PATCH] configure: Add compiler option -Wmissing-format-attribute
Date: Mon, 15 Nov 2010 22:03:37 +0100
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.15) Gecko/20101030 Iceowl/1.0b1 Icedove/3.0.10

Am 15.11.2010 21:22, schrieb Stefan Weil:
With the previous patches, hopefully all functions with
printf like arguments use gcc's format checking.

This was tested with default build configuration on linux
and windows hosts (including some cross compilations),
so chances are good that there remain few (if any) functions
without format checking.

Cc: Blue Swirl <address@hidden>
Signed-off-by: Stefan Weil <address@hidden>
---
[snip]

Hi,

to make testing and committing of this and the previous
patches easier, I have now set up a git repository which
contains all of them (more at the bottom of this mail).

I'd appreciate if someone could test compilation especially
with these environment (which I could not test):

* BSD / Darwin host
* Special configure options

This was my configuration (+ cross compilations for windows / mips / powerpc):

$ ./configure --enable-debug-tcg --trace-backend=simple --audio-drv-list="oss,alsa,sdl,esd,pa" --enable-attr
Install prefix    /usr/local
BIOS directory    /usr/local/share/qemu
binary directory  /usr/local/bin
config directory  /usr/local/etc
Manual directory  /usr/local/share/man
ELF interp prefix /usr/gnemul/qemu-%M
Source path       /home/stefan/src/qemu
C compiler        gcc
Host C compiler   gcc
CFLAGS            -O2 -g
QEMU_CFLAGS -Werror -m64 -I. -I$(SRC_PATH) -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wmissing-format-attribute -Wall -Wundef -Wendif-labels -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fstack-protector-all -Wempty-body -Wnested-externs -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration -Wold-style-definition -Wtype-limits
LDFLAGS           -Wl,--warn-common -m64 -g
make              make
install           install
host CPU          x86_64
host big endian   no
target list i386-softmmu x86_64-softmmu arm-softmmu cris-softmmu m68k-softmmu microblaze-softmmu mips-softmmu mipsel-softmmu mips64-softmmu mips64el-softmmu ppc-softmmu ppcemb-softmmu ppc64-softmmu sh4-softmmu sh4eb-softmmu sparc-softmmu sparc64-softmmu i386-linux-user x86_64-linux-user alpha-linux-user arm-linux-user armeb-linux-user cris-linux-user m68k-linux-user microblaze-linux-user mips-linux-user mipsel-linux-user ppc-linux-user ppc64-linux-user ppc64abi32-linux-user sh4-linux-user sh4eb-linux-user sparc-linux-user sparc64-linux-user sparc32plus-linux-user
tcg debug enabled yes
Mon debug enabled no
gprof enabled     no
sparse enabled    no
strip binaries    yes
profiler          no
static build      no
-Werror enabled   yes
SDL support       yes
curses support    yes
curl support      yes
check support     no
mingw32 support   no
Audio drivers     oss alsa sdl esd pa
Extra audio cards ac97 es1370 sb16 hda
Block whitelist
Mixer emulation   no
VNC TLS support   no
VNC SASL support  no
VNC JPEG support  yes
VNC PNG support   yes
VNC thread        no
xen support       no
brlapi support    no
bluez  support    no
Documentation     yes
NPTL support      yes
GUEST_BASE        yes
PIE user targets  no
vde support       yes
IO thread         no
Linux AIO support yes
ATTR/XATTR support yes
Install blobs     yes
KVM support       yes
fdt support       no
preadv support    yes
fdatasync         yes
madvise           yes
posix_madvise     yes
uuid support      yes
vhost-net support no
Trace backend     simple
Trace output file trace-<pid>
spice support     no

Regards,
Stefan



The following changes since commit cf2c1839a955482f2e208d7400594bf076c222f2:

  add copyright to spiceaudio (2010-11-11 17:59:25 +0300)

are available in the git repository at:
  git://git.weilnetz.de/git/qemu for-blueswirl

Stefan Weil (7):
      *-dis: Replace fprintf_ftype by fprintf_function (format checking)
      target-sparc: Use fprintf_function (format checking)
      trace: Use fprintf_function (format checking)
      audio: Use GCC_FMT_ATTR (format checking)
      darwin-user: Use GCC_FMT_ATTR (format checking)
      slirp: Remove unused code for bad sprintf
      configure: Add compiler option -Wmissing-format-attribute

 HACKING                |    3 ---
 alpha-dis.c            |    3 +++
 arm-dis.c              |   14 +++++++-------
 audio/audio_pt_int.c   |    3 ++-
 configure              |    1 +
 darwin-user/machload.c |    2 +-
 darwin-user/qemu.h     |    2 +-
 dis-asm.h              |   10 ++--------
 m68k-dis.c             |    2 +-
 microblaze-dis.c       |    2 +-
 mips-dis.c             |    2 --
 sh4-dis.c              |   16 +++++-----------
 simpletrace.h          |    6 +++---
 slirp/misc.c           |   42 ------------------------------------------
 slirp/slirp.h          |   14 --------------
 slirp/slirp_config.h   |    6 ------
 target-sparc/cpu.h     |    4 ++--
 17 files changed, 29 insertions(+), 103 deletions(-)




reply via email to

[Prev in Thread] Current Thread [Next in Thread]