qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v2 0/6] qemu-img: Implement commit like QMP
@ 2014-04-08 12:50 Max Reitz
  2014-04-08 12:50 ` [Qemu-devel] [PATCH v2 1/6] block-commit: Expose granularity Max Reitz
                   ` (5 more replies)
  0 siblings, 6 replies; 26+ messages in thread
From: Max Reitz @ 2014-04-08 12:50 UTC (permalink / raw)
  To: qemu-devel; +Cc: Kevin Wolf, Stefan Hajnoczi, Max Reitz

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).


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 (6):
  block-commit: Expose granularity
  block-commit: speed is an optional parameter
  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                 |   26 +-
 include/block/block_int.h  |    6 +-
 qapi-schema.json           |    6 +-
 qemu-img-cmds.hx           |    4 +-
 qemu-img.c                 |  113 ++-
 qemu-img.texi              |    8 +-
 tests/qemu-iotests/020     |   57 +-
 tests/qemu-iotests/020.out | 2168 ++++++++++++++++++++++++++++++++++++++++++++
 11 files changed, 2368 insertions(+), 42 deletions(-)

-- 
1.9.1

^ permalink raw reply	[flat|nested] 26+ messages in thread

end of thread, other threads:[~2014-04-11 12:54 UTC | newest]

Thread overview: 26+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-08 12:50 [Qemu-devel] [PATCH v2 0/6] qemu-img: Implement commit like QMP Max Reitz
2014-04-08 12:50 ` [Qemu-devel] [PATCH v2 1/6] block-commit: Expose granularity Max Reitz
2014-04-08 15:09   ` Kevin Wolf
2014-04-08 16:20   ` Eric Blake
2014-04-10 14:40     ` Max Reitz
2014-04-08 12:50 ` [Qemu-devel] [PATCH v2 2/6] block-commit: speed is an optional parameter Max Reitz
2014-04-08 15:07   ` Kevin Wolf
2014-04-10 14:41     ` Max Reitz
2014-04-08 16:24   ` Eric Blake
2014-04-08 12:50 ` [Qemu-devel] [PATCH v2 3/6] qemu-img: Implement commit like QMP Max Reitz
2014-04-08 15:14   ` Kevin Wolf
2014-04-08 16:39     ` Eric Blake
2014-04-10 14:32     ` Max Reitz
2014-04-11 12:54       ` Kevin Wolf
2014-04-08 12:50 ` [Qemu-devel] [PATCH v2 4/6] qemu-img: Enable progress output for commit Max Reitz
2014-04-08 15:34   ` Kevin Wolf
2014-04-08 16:53     ` Eric Blake
2014-04-09  8:27       ` Kevin Wolf
2014-04-10 14:37     ` Max Reitz
2014-04-08 12:50 ` [Qemu-devel] [PATCH v2 5/6] qemu-img: Specify backing file " Max Reitz
2014-04-08 17:01   ` Eric Blake
2014-04-10 14:42     ` Max Reitz
2014-04-10  9:05   ` Fam Zheng
2014-04-10 14:45     ` Max Reitz
2014-04-08 12:50 ` [Qemu-devel] [PATCH v2 6/6] iotests: Commit tests for two-layer backing chains Max Reitz
2014-04-08 17:10   ` Eric Blake

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).