Re: [Qemu-devel] [PATCH v4 00/15] block: generic image streaming

[Luiz Capitulino <lcapitulino@redhat.com>]

On Fri,  6 Jan 2012 14:01:26 +0000
Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> wrote:

> This series adds the 'block_stream' command which copies the contents of a
> backing file into the image file while the VM is running.  The series builds on
> the zero detection features which I sent out before Christmas. I suggest
> grabbing my git tree to try it out without merging this dependency:
> 
> https://github.com/stefanha/qemu/tree/image-streaming-api
> 
> The image streaming HMP/QMP commands are documented in the patch and also
> described here:
> 
> http://wiki.qemu.org/Features/LiveBlockMigration/ImageStreamingAPI

From a QMP perspective (and given that we'll make this obsolete when
we have a proper async API):

 Acked-by: Luiz Capitulino <lcapitulino@redhat.com>

> 
> The basic idea is to execute 'block_stream virtio0' while the guest is running.
> Progress can be monitored using 'info block-jobs'.  When the streaming
> operation completes it raises a QMP event.
> 
> Note: The last patch includes includes a Python test script called
> test-stream.py, I do not propose to merge it.  When run in a QEMU source tree
> it performs basic image streaming QMP tests.
> 
> v4:
>  * Drop SQMP/EQMP docs from qmp-commands.hx [Luiz]
>  * Follow QAPI doc conventions [Luiz]
>  * Document QMP events in QMP/qmp-events.txt [Luiz]
>  * Protect against hotplug, resize, eject, etc [Kevin]
>  * Move block job functions from header to block.c [Kevin]
>  * Return error from bdrg_change_backing_file() [Kevin]
>  * Merge Marcelo's block_stream base partial streaming series [Marcelo]
> 
> Marcelo Tosatti (4):
>   block: add bdrv_find_backing_image
>   add QERR_BASE_ID_NOT_FOUND
>   block stream: add support for partial streaming
>   add doc to describe live block operations
> 
> Stefan Hajnoczi (11):
>   coroutine: add co_sleep_ns() coroutine sleep function
>   block: check bdrv_in_use() before blockdev operations
>   block: add BlockJob interface for long-running operations
>   block: add image streaming block job
>   block: rate-limit streaming operations
>   qmp: add block_stream command
>   qmp: add block_job_set_speed command
>   qmp: add block_job_cancel command
>   qmp: add query-block-jobs
>   blockdev: make image streaming safe across hotplug
>   test: add image streaming test cases
> 
>  Makefile.objs           |    2 +
>  QMP/qmp-events.txt      |   53 ++++++++++++
>  block.c                 |  133 ++++++++++++++++++++++++++++++
>  block.h                 |    5 +
>  block/stream.c          |  192 +++++++++++++++++++++++++++++++++++++++++++
>  block_int.h             |   44 ++++++++++
>  blockdev.c              |  199 ++++++++++++++++++++++++++++++++++++++++++++-
>  docs/live-block-ops.txt |   58 +++++++++++++
>  hmp-commands.hx         |   41 +++++++++
>  hmp.c                   |   68 +++++++++++++++
>  hmp.h                   |    4 +
>  monitor.c               |   13 +++
>  monitor.h               |    2 +
>  qapi-schema.json        |  115 ++++++++++++++++++++++++++
>  qemu-coroutine-sleep.c  |   38 +++++++++
>  qemu-coroutine.h        |    6 ++
>  qerror.c                |    8 ++
>  qerror.h                |    6 ++
>  qmp-commands.hx         |   24 ++++++
>  test-stream.py          |  208 +++++++++++++++++++++++++++++++++++++++++++++++
>  trace-events            |    9 ++
>  21 files changed, 1227 insertions(+), 1 deletions(-)
>  create mode 100644 block/stream.c
>  create mode 100644 docs/live-block-ops.txt
>  create mode 100644 qemu-coroutine-sleep.c
>  create mode 100644 test-stream.py
>