From 13320bf77effb24c316221a0c5a36553a58806fd Mon Sep 17 00:00:00 2001 From: Peter Krempa Date: Tue, 2 Aug 2022 14:26:41 +0200 Subject: [PATCH] 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 Reviewed-by: Jonathon Jongsma --- src/qemu/qemu_process.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c index 1cd55fe989..c51d704af8 100644 --- 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;