On Tue, Jul 31, 2012 at 5:51 PM, Dong Xu Wang
<wdon...@linux.vnet.ibm.com> wrote:
> add-cow will let raw file support snapshot_blkdev indirectly.
>
> Signed-off-by: Dong Xu Wang <wdon...@linux.vnet.ibm.com>
> ---
>  blockdev.c              |   45 +++++++++++++++++++++++++++++++++++++--------
>  docs/live-block-ops.txt |   11 ++++++++++-
>  2 files changed, 47 insertions(+), 9 deletions(-)

This patch would need to update qapi-schema.json and qmp-commands.hx
to make use of the new arguments.

I don't think QEMU compiles cleanly after this patch.  It's important
to keep the build clean after every patch so that git-bisect(1) can be
used (if you hit a broken build during a bisect it makes things
harder).

However, I think the QMP/HMP changes should not be included in this
series unless you are sure the interface is stable.  When you convert
QEMUOptionsParameter will transaction or snapshot-blkdev-sync change?
If so, please only send the add-cow image format and leave the QMP/HMP
changes until you have the QemuOpts solution.

Stefan

Reply via email to