From patchwork Mon Nov 15 21:03:37 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stefan Weil X-Patchwork-Id: 71301 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [199.232.76.165]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 314A1B7129 for ; Tue, 16 Nov 2010 08:24:26 +1100 (EST) Received: from localhost ([127.0.0.1]:51885 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PI6XH-0007He-9B for incoming@patchwork.ozlabs.org; Mon, 15 Nov 2010 16:24:23 -0500 Received: from [140.186.70.92] (port=47527 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PI6Dh-0001XY-9Q for qemu-devel@nongnu.org; Mon, 15 Nov 2010 16:04:25 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PI6DH-00031z-8x for qemu-devel@nongnu.org; Mon, 15 Nov 2010 16:04:06 -0500 Received: from moutng.kundenserver.de ([212.227.17.8]:53899) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PI6DG-00031I-8w for qemu-devel@nongnu.org; Mon, 15 Nov 2010 16:03:43 -0500 Received: from flocke.fritz.box (p5086F8F5.dip.t-dialin.net [80.134.248.245]) by mrelayeu.kundenserver.de (node=mreu1) with ESMTP (Nemesis) id 0LlJ5u-1OkQ9c0iv8-00aUaH; Mon, 15 Nov 2010 22:03:38 +0100 Received: from localhost ([127.0.0.1] ident=stefan) by flocke.fritz.box with esmtp (Exim 4.72) (envelope-from ) id 1PI6DB-0006k5-6f; Mon, 15 Nov 2010 22:03:37 +0100 Message-ID: <4CE1A029.4090800@mail.berlios.de> Date: Mon, 15 Nov 2010 22:03:37 +0100 From: Stefan Weil 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 MIME-Version: 1.0 To: Blue Swirl References: <1289852574-21577-1-git-send-email-weil@mail.berlios.de> In-Reply-To: <1289852574-21577-1-git-send-email-weil@mail.berlios.de> X-Provags-ID: V02:K0:o/kmgLroxblihLvLjnXlwZ3GxaUFIAmhLaSBMZ9Cmkt X07Qi95YKsQXpDZwRJMIE9HFSXzImmKUHbBL03PmUDWX+y6XwK YbrZsRKWziZuFa6pm2ErITv/lcRSh9m/rp29/4xk6+csLxZdMw R28XgmcPMy9U4Y0fDkkgURZ+Zf6qE8bpFiy50Zmcv56d2O7X6I eU8b0tExXoJTb8owYeQCUM2Kof8o17IWpLqSSuq+5CKfkEcXHv FW3KNWAJlQXCp X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. Cc: QEMU Developers Subject: [Qemu-devel] Re: [PATCH] configure: Add compiler option -Wmissing-format-attribute X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org 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 > Signed-off-by: Stefan Weil > --- [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- 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(-)