[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v3 0/5] qemu-img: Implement commit like QMP
From: |
Max Reitz |
Subject: |
[Qemu-devel] [PATCH v3 0/5] qemu-img: Implement commit like QMP |
Date: |
Thu, 10 Apr 2014 20:11:03 +0200 |
qemu-img should use QMP commands whenever possible in order to ensure
feature completeness of both online and offline image operations. For
the "commit" command, this is relatively easy, so implement it first
(in the hope that indeed others will follow).
As qemu-img does not have access to QMP (due to QMP being intertwined
with basically everything in qemu), we cannot directly use QMP, but at
least use the functions the corresponding QMP commands are using (which
would be "block-commit", in this case).
This series depends on "block-commit: speed is an optional parameter".
v3:
- Patch 1: fixed indentation of commit_start() declaration in
block_int.h [Eric]
- Patch 2: moved out of this series which now depends on it [Kevin, in
a sense]
- Patch 3 (now 2/5): wait for actual completion of the blockjob (CB
called) and evaluate the return value [Kevin]
- Patch 4 (now 3/5): factor in the granularity of the blockjob [Kevin]
- Patch 5 (now 4/5):
- s/backing_file/base/g in documentation [Fam]
- s/unto/into/g, s/shall/will/ [Eric]
- s/filename/base/ in a certain spot [Fam]
v2 (mostly following Eric's comments):
- Patch 1:
- allow giving the granularity as 0 in order to enforce the default
value
- use is_power_of_2() instead of manually inlining it
- use "if (!has_x) { x = DEFAULT_VALUE; }" (which is the common
pattern in blockdev.c) instead of "x = has_x ? x : DEFAULT_VALUE;"
- Patch 2:
- "if (!has_x) { x = DEFAULT_VALUE; }" with a fixed condition
- Patch 3:
- don't use obsolete error_is_set()
- use bs->backing_hd instead of bdrv_find_base(), as the latter
actually finds the very base of the backing chain which does not
correspond to the current qemu-img commit behavior (which is to
commit into the first backing file)
- Added patches 5 and 6
Max Reitz (5):
block-commit: Expose granularity
qemu-img: Implement commit like QMP
qemu-img: Enable progress output for commit
qemu-img: Specify backing file for commit
iotests: Commit tests for two-layer backing chains
block/Makefile.objs | 2 +-
block/commit.c | 16 +-
block/mirror.c | 4 +-
blockdev.c | 22 +-
include/block/block_int.h | 10 +-
qapi-schema.json | 6 +-
qemu-img-cmds.hx | 4 +-
qemu-img.c | 144 ++-
qemu-img.texi | 8 +-
tests/qemu-iotests/020 | 57 +-
tests/qemu-iotests/020.out | 2168 ++++++++++++++++++++++++++++++++++++++++++++
11 files changed, 2397 insertions(+), 44 deletions(-)
--
1.9.1
- [Qemu-devel] [PATCH v3 0/5] qemu-img: Implement commit like QMP,
Max Reitz <=
- [Qemu-devel] [PATCH v3 5/5] iotests: Commit tests for two-layer backing chains, Max Reitz, 2014/04/10
- [Qemu-devel] [PATCH v3 1/5] block-commit: Expose granularity, Max Reitz, 2014/04/10
- [Qemu-devel] [PATCH v3 3/5] qemu-img: Enable progress output for commit, Max Reitz, 2014/04/10
- [Qemu-devel] [PATCH v3 2/5] qemu-img: Implement commit like QMP, Max Reitz, 2014/04/10
- [Qemu-devel] [PATCH v3 4/5] qemu-img: Specify backing file for commit, Max Reitz, 2014/04/10