[Qemu-devel] [PATCH v4 0/8] Support streaming to an intermediate layer

[Alberto Garcia <berto@igalia.com>]

New series.

As I wrote in a separate e-mail, we don't need to have any additional
checks on the top image before reopening it in R/W mode since
bdrv_reopen_prepare() already takes care of checking the
BDRV_O_ALLOW_RDWR flag, which serves our purpose.

v4:
- Refactor find_block_job to use the error from bdrv_lookup_bs()
- Don't use QERR_DEVICE_IN_USE in block_job_create() since we can be
  dealing with nodes now.
- Fix @device comment in the BlockJobInfo documentation
- stream_start(): simplify the bdrv_reopen() call and use
  bdrv_get_device_or_node_name() for error messages.
- Use a different variable name for BlockDriverState *i
- Documentation fixes in docs/live-block-ops.txt
- Update iotest 30 since now test_device_not_found() returns
  GenericError
- Fix test case test_stream_partial()
- Add new test case test_stream_intermediate()
- Fix typos

v3: https://lists.gnu.org/archive/html/qemu-devel/2015-04/msg00806.html
- Keep a list of block jobs and make qmp_query_block_jobs() iterate
  over it.

v2: https://lists.gnu.org/archive/html/qemu-devel/2015-03/msg04798.html
- The 'block-stream' command does not have a 'node-name' parameter
  anymore and reuses 'device' for that purpose.
- Block jobs can now be owned by any intermediate node, and not just
  by the ones at the root. query-block-jobs is updated to reflect that
  change.
- The 'device' parameter of all 'block-job-*' commands can now take a
  node name.
- The BlockJobInfo type and all BLOCK_JOB_* events report the node
  name in the 'device' field if the node does not have a device name.
- All intermediate nodes are blocked (and checked for blockers) during
  the streaming operation.

Regards,

Berto

Alberto Garcia (8):
  block: keep a list of block jobs
  block: allow block jobs in any arbitrary node
  block: never cancel a streaming job without running stream_complete()
  block: Support streaming to an intermediate layer
  block: Add QMP support for streaming to an intermediate layer
  docs: Document how to stream to an intermediate layer
  qemu-iotests: fix test_stream_partial()
  qemu-iotests: test streaming to an intermediate layer

 block.c                    |  4 +++-
 block/mirror.c             |  5 +++--
 block/stream.c             | 44 ++++++++++++++++++++++++++++++++-----
 blockdev.c                 | 55 ++++++++++++++++++++++------------------------
 blockjob.c                 | 31 +++++++++++++++++++-------
 docs/live-block-ops.txt    | 30 +++++++++++++++----------
 docs/qmp/qmp-events.txt    |  8 +++----
 include/block/blockjob.h   | 14 ++++++++++++
 include/qapi/qmp/qerror.h  |  3 ---
 qapi/block-core.json       | 30 ++++++++++++++-----------
 tests/qemu-iotests/030     | 23 ++++++++++++++++---
 tests/qemu-iotests/030.out |  4 ++--
 12 files changed, 170 insertions(+), 81 deletions(-)

-- 
2.1.4