diff options
author | Kevin Wolf <kwolf@redhat.com> | 2021-02-12 18:20:28 +0100 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2021-02-15 15:10:14 +0100 |
commit | b248e61652e20c3353af4b0ccb90f17d76f4db21 (patch) | |
tree | e38633fc57e709e8b27c9821deba353629099357 /monitor/qmp.c | |
parent | c81219a7dd36a815bd85beed9932fc973d4f5d51 (diff) | |
download | qemu-b248e61652e20c3353af4b0ccb90f17d76f4db21.zip qemu-b248e61652e20c3353af4b0ccb90f17d76f4db21.tar.gz qemu-b248e61652e20c3353af4b0ccb90f17d76f4db21.tar.bz2 |
monitor/qmp: Stop processing requests when shutdown is requested
Before this patch, monitor_qmp_dispatcher_co() used to check whether
shutdown is requested only when it would have to wait for new requests.
If there were still some queued requests, it would try to execute all of
them before shutting down.
This can be surprising when the queued QMP commands take long or hang
because Ctrl-C may not actually exit QEMU as soon as possible.
Change monitor_qmp_dispatcher_co() so that it additionally checks
whether shutdown is request before it gets a new request from the queue.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Message-Id: <20210212172028.288825-3-kwolf@redhat.com>
Tested-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'monitor/qmp.c')
-rw-r--r-- | monitor/qmp.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/monitor/qmp.c b/monitor/qmp.c index 43880fa..2326bd7 100644 --- a/monitor/qmp.c +++ b/monitor/qmp.c @@ -227,6 +227,11 @@ void coroutine_fn monitor_qmp_dispatcher_co(void *data) */ qatomic_mb_set(&qmp_dispatcher_co_busy, false); + /* On shutdown, don't take any more requests from the queue */ + if (qmp_dispatcher_co_shutdown) { + return; + } + while (!(req_obj = monitor_qmp_requests_pop_any_with_lock())) { /* * No more requests to process. Wait to be reentered from |