From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1LBFVm-0007dv-V7 for qemu-devel@nongnu.org; Fri, 12 Dec 2008 16:25:27 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1LBFVm-0007dd-EB for qemu-devel@nongnu.org; Fri, 12 Dec 2008 16:25:26 -0500 Received: from [199.232.76.173] (port=45302 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1LBFVl-0007dM-Vt for qemu-devel@nongnu.org; Fri, 12 Dec 2008 16:25:26 -0500 Received: from mx2.redhat.com ([66.187.237.31]:33586) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1LBFVl-00009G-BV for qemu-devel@nongnu.org; Fri, 12 Dec 2008 16:25:25 -0500 Received: from int-mx2.corp.redhat.com (int-mx2.corp.redhat.com [172.16.27.26]) by mx2.redhat.com (8.13.8/8.13.8) with ESMTP id mBCLPM5m005858 for ; Fri, 12 Dec 2008 16:25:24 -0500 From: Eduardo Habkost Date: Fri, 12 Dec 2008 19:24:20 -0200 Message-Id: <1229117060-8467-3-git-send-email-ehabkost@redhat.com> In-Reply-To: <1229117060-8467-1-git-send-email-ehabkost@redhat.com> References: <1229117060-8467-1-git-send-email-ehabkost@redhat.com> Subject: [Qemu-devel] [RFC PATCH URL 2/2] Big conversion to the new logging macros Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Eduardo Habkost This is a really big series, so I've made it available via HTTP instead, at: http://raisama.net/tmp/qemu-log-macros/ You can get the 62 individual patches there, or look at all changes on a big single patch at: http://raisama.net/tmp/qemu-log-macros/0000-BIG-SINGLE-PATCH.patch With this series, almost all references to the logfile/loglevel global variables outside of qemu-log.h were eliminated. except for: * cpu_set_log*() functions at exec.c, that I plan to move to a qemu-log.c file on further patches; and * these two cases at tcg/tcg.c: tcg/tcg.c:1920: tcg_dump_ops(s, logfile); tcg/tcg.c:1936: tcg_dump_ops(s, logfile); The whole series is also available on a git reposity at: git://github.com/ehabkost/qemu-hacks.git hacking/log-macros --- The following changes since commit 303dccf5fba94342c5001648d60ed2caf88898a1: Eduardo Habkost (1): Define macros that will become the new logging API are available in the git repository at: git://github.com/ehabkost/qemu-hacks.git hacking/log-macros Eduardo Habkost (62): target-i386/op_helper.c: qemu_log_mask() replacements target-i386/op_helper.c: simultaneous qemu_log_mask()/log_cpu_state_mask() replacements target-i386/op_helper.c: qemu_log()/log_cpu_state() replacements target-i386/op_helper.c: debug macros: use qemu_log*() target-i386/translate.c: use log_cpu_state_mask() target-i386/translate.c: use qemu_log()/log_target_disas() target-mips/helper.c: use qemu_log()/log_cpu_state() target-mips/helper.c: remaining qemu_log() replacements target-mips/op_helper.c: move loglevel check into debug_pre_eret() target-mips/op_helper.c: move loglevel check into debug_post_eret() target-mips/op_helper.c: qemu_log() replacements target-mips/op_helper.c: remaining qemu_log() replacement target-mips/translate.c: debug macros: use qemu_log_mask() target-mips/translate.c: remaining qemu_log*() conversions hw/mips_timer.c: qemu_log() conversions target-ppc/helper.c: qemu_log() conversions target-ppc/helper.c: qemu_log_mask() conversions target-ppc/helper.c: qemu_log_enabled()/qemu_log() conversions target-ppc/helper.c: move loglevel check into dump_syscall() target-ppc/helper.c: dump_syscall(): use qemu_log_mask() target-ppc/helper.c: debug macros: use qemu_log() target-ppc/helper.c: cpu_dump_rfi(): use qemu_log() target-ppc/op_helper.c: qemu_log() conversions target-ppc/op_helper.c: LOG_SWTLB: use qemu_log() target-ppc/translate.c: qemu_log() conversions target-ppc/translate.c: qemu_log_enabled()/qemu_log() conversions target-ppc/translate.c: LOG_DISAS: use qemu_log_mask() target-ppc/translate.c: remaining qemu_log*() conversions hw/ppc.c: debug macros: use qemu_log*() hw/ppc4xx_devs.c: debug macros: use qemu_log*() target-sparc/op_helper.c: qemu_log*() conversions target-sparc/translate.c: qemu_log*() conversions target-cris/op_helper.c: helper_dump: use qemu_log() target-cris/translate.c: qemu_log*() conversions target-cris/translate.c: LOG_DIS: use qemu_log_mask() target-cris/translate.c: use qemu_log_try_set_file() target-cris/*: D_LOG: use qemu_log() hw/alpha_palcode.c: qemu_log() conversions target-alpha/translate.c: qemu_log*() conversions target-alpha/translate.c: LOG_DISAS: use qemu_log() target-sh4/helper.c: qemu_log*() conversions target-sh4/translate.c: qemu_log_*() conversions target-m68k/translate.c: qemu_log*() conversions target-arm/translate.c: qemu_log*() conversions kqemu.c: LOG_INT: use qemu_log_mask() kqemu.c: LOG_INT_STATE: use log_cpu_state_mask() linux-user/main.c: qemu_log*() conversions linux-user/vm86.c: LOG_VM86: use qemu_log() linux-user/elfload.c: load_elf_binary(): use qemu_log_enabled() bsd-user/main.c: use log_page_dump()/qemu_log() bsd-user/elfload.c: load_elf_binary(): use qemu_log_enabled() darwin-user/main.c: qemu_log*() conversions darwin-user: DPRINTF: use qemu_log() block-raw-posix.c: qemu_log*() conversions translate-all.c: qemu_log*() conversions cpu-exec.c: qemu_log*() conversions exec.c: qemu_log*() conversions exec.c: cpu_abort(): use qemu_log_close() tcg.c: qemu_log() conversions vl.c: LOG_IOPORT: use qemu_log_mask() global s/fflush(logfile)/qemu_log_flush()/ global s/loglevel & X/qemu_loglevel_mask(X)/ block-raw-posix.c | 4 +- bsd-user/elfload.c | 2 +- bsd-user/main.c | 28 ++++++------- cpu-exec.c | 35 +++++++--------- darwin-user/commpage.c | 4 +- darwin-user/machload.c | 4 +- darwin-user/main.c | 6 +-- darwin-user/syscall.c | 4 +- exec.c | 26 ++++++------- hw/alpha_palcode.c | 15 ++----- hw/mips_timer.c | 4 +- hw/ppc.c | 10 +---- hw/ppc4xx_devs.c | 5 +-- hw/ppc_prep.c | 11 +---- kqemu.c | 11 +---- linux-user/elfload.c | 2 +- linux-user/main.c | 34 +++++++--------- linux-user/vm86.c | 2 +- target-alpha/translate.c | 17 +++----- target-arm/translate.c | 10 ++-- target-cris/helper.c | 2 +- target-cris/mmu.c | 2 +- target-cris/op_helper.c | 4 +- target-cris/translate.c | 28 ++++++------- target-i386/op_helper.c | 79 +++++++++++++----------------------- target-i386/translate.c | 24 +++++------ target-m68k/translate.c | 12 +++--- target-mips/helper.c | 39 +++++++------------ target-mips/op_helper.c | 68 ++++++++++++++++---------------- target-mips/translate.c | 41 +++++++------------ target-ppc/helper.c | 98 +++++++++++++-------------------------------- target-ppc/op_helper.c | 33 ++++------------ target-ppc/translate.c | 57 ++++++++++---------------- target-sh4/helper.c | 8 ++-- target-sh4/translate.c | 19 ++++----- target-sparc/op_helper.c | 24 ++++++------ target-sparc/translate.c | 19 ++++----- tcg/tcg.c | 12 +++--- translate-all.c | 10 ++-- vl.c | 5 +-- 40 files changed, 321 insertions(+), 497 deletions(-) -- Eduardo