Re: [libvirt] [PATCH v5 22/36] qemu_process: Stop locking QMP process monitor immediately

2019-01-03 Thread Jiri Denemark
On Sun, Dec 02, 2018 at 23:10:16 -0600, Chris Venteicher wrote: > Locking the monitor object immediately after call to qemuMonitorOpen > doesn't make sense now that we have expanded the QEMU process code to > cover more than the original capabilities usecase. The expanded use case has nothing to d

[libvirt] [PATCH v5 22/36] qemu_process: Stop locking QMP process monitor immediately

2018-12-02 Thread Chris Venteicher
Locking the monitor object immediately after call to qemuMonitorOpen doesn't make sense now that we have expanded the QEMU process code to cover more than the original capabilities usecase. Removing the monitor lock makes the qemuProcessQmpConnectMonitor code consistent with the qemuConnectMonitor