]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: monitor: Decrease logging verbosity
authorPeter Krempa <pkrempa@redhat.com>
Wed, 20 Dec 2017 12:09:07 +0000 (13:09 +0100)
committerPeter Krempa <pkrempa@redhat.com>
Wed, 3 Jan 2018 14:21:30 +0000 (15:21 +0100)
The PROBE macro used in qemuMonitorIOProcess and the VIR_DEBUG message
in qemuMonitorJSONIOProcess create a lot of logging churn when debug
logging is enabled during monitor communication.

The messages logged from the PROBE macro are rather useless since they
are reporting the partial state of receiving the reply from qemu. The
actual full reply is still logged in qemuMonitorJSONIOProcessLine once
the full message is received.

src/qemu/qemu_monitor.c
src/qemu/qemu_monitor_json.c

index 4db12c512a24ae0acebe3ef5d7943c780cf521d4..046caf001c9cfedbc10a33d142299ba495fbde97 100644 (file)
@@ -435,8 +435,8 @@ qemuMonitorIOProcess(qemuMonitorPtr mon)
 # endif
 #endif
 
-    PROBE(QEMU_MONITOR_IO_PROCESS,
-          "mon=%p buf=%s len=%zu", mon, mon->buffer, mon->bufferOffset);
+    PROBE_QUIET(QEMU_MONITOR_IO_PROCESS, "mon=%p buf=%s len=%zu",
+                mon, mon->buffer, mon->bufferOffset);
 
     if (mon->json)
         len = qemuMonitorJSONIOProcess(mon,
index e45868b01fc87cb80b8a8bc19901aff9e046d115..ea1b8e1f5f2a55a9bff6439ae44e1fc9a3ffae18 100644 (file)
@@ -259,7 +259,10 @@ int qemuMonitorJSONIOProcess(qemuMonitorPtr mon,
         }
     }
 
+#if DEBUG_IO
     VIR_DEBUG("Total used %d bytes out of %zd available in buffer", used, len);
+#endif
+
     return used;
 }