aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWolfgang Bumiller <w.bumiller@proxmox.com>2018-09-25 10:15:06 +0200
committerMarkus Armbruster <armbru@redhat.com>2018-11-06 17:02:49 +0100
commit9a3e52e3465f85d203d0fd2910ca3625b3c45420 (patch)
tree94455be456ba3c5a1e3b7ffdd144e8d5b0461754
parent0ca70f19c010ccf0b10cf7cc1d2b9a87193fe787 (diff)
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.c2
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.