monitor/qmp: Stop processing requests when shutdown is requested
commitb248e61652e20c3353af4b0ccb90f17d76f4db21
authorKevin Wolf <kwolf@redhat.com>
Fri, 12 Feb 2021 17:20:28 +0000 (12 18:20 +0100)
committerKevin Wolf <kwolf@redhat.com>
Mon, 15 Feb 2021 14:10:14 +0000 (15 15:10 +0100)
treee38633fc57e709e8b27c9821deba353629099357
parentc81219a7dd36a815bd85beed9932fc973d4f5d51
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>
monitor/qmp.c