From: Eduardo Habkost <ehabkost@redhat.com>
To: qemu-devel@nongnu.org
Cc: Eduardo Habkost <ehabkost@redhat.com>
Subject: [Qemu-devel] [RFC PATCH URL 2/2] Big conversion to the new logging macros
Date: Fri, 12 Dec 2008 19:24:20 -0200 [thread overview]
Message-ID: <1229117060-8467-3-git-send-email-ehabkost@redhat.com> (raw)
In-Reply-To: <1229117060-8467-1-git-send-email-ehabkost@redhat.com>
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
next prev parent reply other threads:[~2008-12-12 21:25 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-12-12 21:24 [Qemu-devel] [RFC PATCH 0/2] Eliminate references to logfile/loglevel global vars Eduardo Habkost
2008-12-12 21:24 ` [Qemu-devel] [RFC PATCH 1/2] Create macros that will help define the logging API Eduardo Habkost
2008-12-12 21:24 ` Eduardo Habkost [this message]
2008-12-12 21:30 ` [Qemu-devel] [RFC PATCH URL 2/2] Big conversion to the new logging macros Anthony Liguori
2008-12-12 21:52 ` Blue Swirl
2008-12-12 22:27 ` [Qemu-devel] [RFC PATCH 0/2] Eliminate references to logfile/loglevel global vars Dor Laor
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1229117060-8467-3-git-send-email-ehabkost@redhat.com \
--to=ehabkost@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).