qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

  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).