qemuProcessQMPConnectMonitor: Connect to probing monitor with 'retry' set to false

In 'qemuProcessQMPLaunch' qemu is very specifically launched using it's
internal '-daemonize' flag (see comment in the function) to ensure that
the monitor socket is ready and opened prior to attempting the monitor
connection.

This means we don't have to retry the connection to the monitor in
qemuMonitorOpen as the socket will be already there.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Jonathon Jongsma <jjongsma@redhat.com>

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index 1cd55fe989a92746cc5e0305f08f94102bb88434..c51d704af802ea86b633494c574fd0f2b39ed664 100644 (file)
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -9493,7 +9493,7 @@ qemuProcessQMPConnectMonitor(qemuProcessQMP *proc)
 
     proc->vm->pid = proc->pid;
 
-    if (!(proc->mon = qemuMonitorOpen(proc->vm, &monConfig, true,
+    if (!(proc->mon = qemuMonitorOpen(proc->vm, &monConfig, false,
                                       virEventThreadGetContext(proc->eventThread),
                                       &callbacks)))
         return -1;