diff options
| author | Wolfgang Bumiller <w.bumiller@proxmox.com> | 2018-09-25 10:15:06 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2018-11-06 17:02:49 +0100 |
| commit | 9a3e52e3465f85d203d0fd2910ca3625b3c45420 (patch) | |
| tree | 94455be456ba3c5a1e3b7ffdd144e8d5b0461754 | |
| parent | 0ca70f19c010ccf0b10cf7cc1d2b9a87193fe787 (diff) | |
| download | focaccia-qemu-9a3e52e3465f85d203d0fd2910ca3625b3c45420.tar.gz focaccia-qemu-9a3e52e3465f85d203d0fd2910ca3625b3c45420.zip | |
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: Wolfgang Bumiller <w.bumiller@proxmox.com> Reviewed-by: Eric Blake <eblake@redhat.com> Reviewed-by: Peter Xu <peterx@redhat.com> Message-Id: <20180925081507.11873-2-w.bumiller@proxmox.com> Signed-off-by: Markus Armbruster <armbru@redhat.com>
| -rw-r--r-- | monitor.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/monitor.c b/monitor.c index 823b5a1099..66f149c1dc 100644 --- a/monitor.c +++ b/monitor.c @@ -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. |