[PATCH 3/4] Log dtrace/systemd probes at INFO level instead of DEBUG

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Every dtrace/systemd probe also include a libvirt log message.
These are logged at level DEBUG currently, which means if you
want to see all probes they are drowned by the rest of the
DEBUG messages. Since we don't really use the INFO log level
for much, it seems reasonable to suggest we log all probes at
level INFO.
---
 src/util/virprobe.h | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/util/virprobe.h b/src/util/virprobe.h
index fe3c422..7565954 100644
--- a/src/util/virprobe.h
+++ b/src/util/virprobe.h
@@ -83,8 +83,8 @@
 
 #  define PROBE_EXPAND(NAME, ARGS) NAME(ARGS)
 #  define PROBE(NAME, FMT, ...)                              \
-    VIR_DEBUG_INT(&virLogSelf,                               \
-                  NULL, __LINE__, __func__,                  \
+    VIR_INFO_INT(&virLogSelf,                                \
+                  __FILE__, __LINE__, __func__,              \
                   #NAME ": " FMT, __VA_ARGS__);              \
     if (LIBVIRT_ ## NAME ## _ENABLED()) {                    \
         PROBE_EXPAND(LIBVIRT_ ## NAME,                       \
@@ -92,9 +92,9 @@
     }
 # else
 #  define PROBE(NAME, FMT, ...)                              \
-    VIR_DEBUG_INT(&virLogSelf,                               \
-                  NULL, __LINE__, __func__,                  \
-                  #NAME ": " FMT, __VA_ARGS__);
+    VIR_INFO_INT(&virLogSelf,                                \
+                 __FILE__, __LINE__, __func__,               \
+                 #NAME ": " FMT, __VA_ARGS__);
 # endif
 
 #endif /* __VIR_PROBE_H__ */
-- 
2.1.0

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]