From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48818) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a7hXb-0008AA-5l for qemu-devel@nongnu.org; Sat, 12 Dec 2015 05:36:40 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a7hXX-00085q-Un for qemu-devel@nongnu.org; Sat, 12 Dec 2015 05:36:39 -0500 Received: from mail-wm0-x234.google.com ([2a00:1450:400c:c09::234]:32872) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a7hXX-00085i-P4 for qemu-devel@nongnu.org; Sat, 12 Dec 2015 05:36:35 -0500 Received: by mail-wm0-x234.google.com with SMTP id l68so3404438wml.0 for ; Sat, 12 Dec 2015 02:36:35 -0800 (PST) Sender: Paolo Bonzini From: Paolo Bonzini Date: Sat, 12 Dec 2015 11:36:16 +0100 Message-Id: <1449916587-18312-1-git-send-email-pbonzini@redhat.com> Subject: [Qemu-devel] [PATCH 00/11] Avoid always-active qemu_log calls List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: peter.maydell@linaro.org qemu_log calls should generally be wrapped by DEBUG_* preprocessor symbols or (preferred) qemu_log_mask should be used instead. This avoids that the upcoming integration of logging and tracing prints some log messages to stderr unconditionally. Similarly, most occurrences of qemu_log_enabled() should be changed to qemu_log_mask(). One exception is when these always-active qemu_log calls are coupled with printfs to stdout or stderr. These should be skipped if stderr is _already_ the logging destination, and the series thus introduces a new predicate qemu_log_separate() to replace qemu_log_enabled() in some cases. This series removes all the instances that I could find of these problems. Paolo Paolo Bonzini (11): qemu-log: introduce qemu_log_separate alpha: convert "naked" qemu_log to tracepoint cris: avoid "naked" qemu_log microblaze: avoid "naked" qemu_log s390x: avoid "naked" qemu_log ppc: cleanup logging tricore: avoid "naked" qemu_log xtensa: avoid "naked" qemu_log user: introduce "-d page" linux-user: avoid "naked" qemu_log linux-user: convert DEBUG_SIGNAL logging to tracepoints bsd-user/main.c | 4 +- bsd-user/signal.c | 2 - exec.c | 2 +- hw/alpha/pci.c | 3 +- hw/char/etraxfs_ser.c | 2 +- include/qemu/log.h | 8 +++ linux-user/elfload.c | 8 +-- linux-user/main.c | 75 ++++++++++++--------------- linux-user/signal.c | 118 +++++++++++++++++++++--------------------- qemu-log.c | 2 + target-cris/helper.h | 1 - target-cris/op_helper.c | 5 -- target-cris/translate.c | 8 +-- target-cris/translate_v10.c | 2 +- target-microblaze/helper.c | 2 +- target-microblaze/mmu.c | 20 +++---- target-microblaze/op_helper.c | 8 +-- target-microblaze/translate.c | 2 +- target-ppc/excp_helper.c | 8 +-- target-ppc/mmu-hash32.c | 12 +---- target-ppc/mmu-hash64.c | 9 +--- target-ppc/mmu_helper.c | 15 +++--- target-ppc/timebase_helper.c | 10 ++-- target-ppc/translate.c | 62 +++++++++++----------- target-s390x/cc_helper.c | 2 +- target-s390x/helper.c | 2 +- target-s390x/misc_helper.c | 2 +- target-s390x/mmu_helper.c | 2 +- target-tricore/helper.c | 4 +- target-xtensa/gdbstub.c | 8 +-- target-xtensa/helper.c | 4 +- target-xtensa/op_helper.c | 20 +++---- target-xtensa/translate.c | 28 +++++----- target-xtensa/xtensa-semi.c | 2 +- trace-events | 14 +++++ 35 files changed, 237 insertions(+), 239 deletions(-) -- 2.5.0