[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v3 4/9] qemu-log: Avoid function call for disabled q
From: |
Alex Bennée |
Subject: |
[Qemu-devel] [PATCH v3 4/9] qemu-log: Avoid function call for disabled qemu_log_mask logging |
Date: |
Thu, 31 Jul 2014 14:06:27 +0100 |
From: Peter Maydell <address@hidden>
Make qemu_log_mask() a macro which only calls the function to
do the actual work if the logging is enabled. This avoids making
a function call in possible fast paths where logging is disabled.
Signed-off-by: Peter Maydell <address@hidden>
diff --git a/include/qemu/log.h b/include/qemu/log.h
index d515424..da22e13 100644
--- a/include/qemu/log.h
+++ b/include/qemu/log.h
@@ -64,10 +64,17 @@ qemu_log_vprintf(const char *fmt, va_list va)
}
}
-/* log only if a bit is set on the current loglevel mask
+/* log only if a bit is set on the current loglevel mask:
+ * @mask: bit to check in the mask
+ * @fmt: printf-style format string
+ * @args: optional arguments for format string
*/
-void GCC_FMT_ATTR(2, 3) qemu_log_mask(int mask, const char *fmt, ...);
-
+#define qemu_log_mask(MASK, FMT, ...) \
+ do { \
+ if (unlikely(qemu_loglevel_mask(MASK))) { \
+ qemu_log(FMT, ## __VA_ARGS__); \
+ } \
+ } while (0)
/* Special cases: */
diff --git a/qemu-log.c b/qemu-log.c
index 35bbb56..83e0cb4 100644
--- a/qemu-log.c
+++ b/qemu-log.c
@@ -36,17 +36,6 @@ void qemu_log(const char *fmt, ...)
va_end(ap);
}
-void qemu_log_mask(int mask, const char *fmt, ...)
-{
- va_list ap;
-
- va_start(ap, fmt);
- if ((qemu_loglevel & mask) && qemu_logfile) {
- vfprintf(qemu_logfile, fmt, ap);
- }
- va_end(ap);
-}
-
/* enable or disable low levels log */
void do_qemu_set_log(int log_flags, bool use_own_buffers)
{
--
2.0.3
- [Qemu-devel] [PATCH v3 0/9] qemu-log dfilter and perf.map changes, Alex Bennée, 2014/07/31
- [Qemu-devel] [PATCH v3 6/9] qemu-log: support simple pid substitution in logfile, Alex Bennée, 2014/07/31
- [Qemu-devel] [PATCH v3 1/9] tcg: add ability to dump /tmp/perf-<pid>.map files, Alex Bennée, 2014/07/31
- [Qemu-devel] [PATCH v3 9/9] target-arm: dfilter support for in_asm, op, opt_op, Alex Bennée, 2014/07/31
- [Qemu-devel] [PATCH v3 7/9] qemu-log: new option -dfilter to limit output, Alex Bennée, 2014/07/31
- [Qemu-devel] [PATCH v3 8/9] qemu-log: dfilter-ise exec, out_asm, and op_opt, Alex Bennée, 2014/07/31
- [Qemu-devel] [PATCH v3 5/9] qemu-log: Improve the "exec" TB execution logging, Alex Bennée, 2014/07/31
- [Qemu-devel] [PATCH v3 4/9] qemu-log: Avoid function call for disabled qemu_log_mask logging,
Alex Bennée <=
- [Qemu-devel] [PATCH v3 2/9] tcg: light re-factor and pass down TranslationBlock, Alex Bennée, 2014/07/31
- [Qemu-devel] [PATCH v3 3/9] qemu-log: correct help text for -d cpu, Alex Bennée, 2014/07/31