diff options
author | Luiz Capitulino <lcapitulino@redhat.com> | 2010-05-11 18:08:20 -0300 |
---|---|---|
committer | Luiz Capitulino <lcapitulino@redhat.com> | 2010-05-19 12:45:54 -0300 |
commit | 39b59d2606b3360458361ab0f5419b3262dbf46d (patch) | |
tree | 0a38cbad3c63f4b383b659e586f13e0a13a3da08 /qerror.c | |
parent | a691d41a118289e8eb8210e1a824a848497d14a9 (diff) | |
download | qemu-39b59d2606b3360458361ab0f5419b3262dbf46d.zip qemu-39b59d2606b3360458361ab0f5419b3262dbf46d.tar.gz qemu-39b59d2606b3360458361ab0f5419b3262dbf46d.tar.bz2 |
Monitor: Return before exiting with 'quit'
This is a new version of the (now reverted) following commit:
0e8d2b5575938b8876a3c4bb66ee13c5d306fb6d
The 'quit' Monitor command (implemented by do_quit()) calls
exit() directly, this is problematic under QMP because QEMU
exits before having a chance to send the ok response.
Clients don't know if QEMU exited because of a problem or
because the 'quit' command has been executed.
This commit fixes that by making do_quit() use
qemu_system_shutdown_request(), so that we exit gracefully.
Thanks to Paolo Bonzini <pbonzini@redhat.com> for suggesting
this solution.
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
Diffstat (limited to 'qerror.c')
0 files changed, 0 insertions, 0 deletions