[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH v5 00/11] Support streaming to an intermediate layer
From: |
Alberto Garcia |
Subject: |
[Qemu-block] [PATCH v5 00/11] Support streaming to an intermediate layer |
Date: |
Fri, 24 Apr 2015 14:40:44 +0300 |
New version of the patchset. Apart from a couple of minor
documentation fixes, this one has new test cases.
Since we now allow block jobs in arbitrary nodes, I wanted to test
that given a sufficiently long snapshot chain, it's possible to launch
several block-stream operations in parallel as long as they involve
different sets of nodes.
Regards,
Berto
v5:
- Fix a few typos
- Minor documentation updates
- Update test_stream_partial() to test no-ops
- New test case: test_stream_parallel()
- New test case: test_stream_overlapping()
v4: https://lists.gnu.org/archive/html/qemu-devel/2015-04/msg01878.html
- 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.
Alberto Garcia (11):
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: add no-op streaming test
qemu-iotests: test streaming to an intermediate layer
qemu-iotests: test block-stream operations in parallel
qemu-iotests: test overlapping block-stream operations
block.c | 4 +-
block/mirror.c | 5 +-
block/stream.c | 44 ++++++++++++--
blockdev.c | 55 +++++++++---------
blockjob.c | 31 +++++++---
docs/live-block-ops.txt | 31 ++++++----
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 | 141 ++++++++++++++++++++++++++++++++++++++++++++-
tests/qemu-iotests/030.out | 4 +-
12 files changed, 290 insertions(+), 80 deletions(-)
--
2.1.4