Commit 9a3e52e3 authored by Wolfgang Bumiller's avatar Wolfgang Bumiller Committed by Markus Armbruster
Browse files

monitor: guard iothread access by mon->use_io_thread



monitor_resume() and monitor_suspend() both want to
"kick" the I/O thread if it is there, but in
monitor_suspend() lacked the use_io_thread flag condition.
This is required when we later only spawn the thread on
first use.

Signed-off-by: default avatarWolfgang Bumiller <w.bumiller@proxmox.com>
Reviewed-by: default avatarEric Blake <eblake@redhat.com>
Reviewed-by: default avatarPeter Xu <peterx@redhat.com>
Message-Id: <20180925081507.11873-2-w.bumiller@proxmox.com>
Signed-off-by: default avatarMarkus Armbruster <armbru@redhat.com>
parent 0ca70f19
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -4292,7 +4292,7 @@ int monitor_suspend(Monitor *mon)

    atomic_inc(&mon->suspend_cnt);

    if (monitor_is_qmp(mon)) {
    if (monitor_is_qmp(mon) && mon->use_io_thread) {
        /*
         * Kick I/O thread to make sure this takes effect.  It'll be
         * evaluated again in prepare() of the watch object.