On Wed, 14 Mar 2012 10:03:15 +0800
Wen Congyang<address@hidden> wrote:
Changes from v8 to v9:
1. remove async support(it will be reimplemented after QAPI async commands
support
is finished)
I gave my review on this one (concentrating on the QMP part only), and one
important aspect of this command is that it's a long synchronous operation.
As it runs with vCPUs stopped, the only drawback is that libvirt won't be able
to run other commands in parallel and won't be able to cancel it either. Also
note that this command is most likely to be executed when the guest crashes.
Given all this, it's fine with me to have this as a synchronous command, but
I'd like to get an ACK from libvirt and another one from Jan and/or Anthony.