From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38103) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WXVUM-0001kY-BZ for qemu-devel@nongnu.org; Tue, 08 Apr 2014 08:51:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WXVUG-0004ZX-6K for qemu-devel@nongnu.org; Tue, 08 Apr 2014 08:50:54 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49013) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WXVUF-0004Z4-UG for qemu-devel@nongnu.org; Tue, 08 Apr 2014 08:50:48 -0400 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s38Col52000845 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Tue, 8 Apr 2014 08:50:47 -0400 From: Max Reitz Date: Tue, 8 Apr 2014 14:50:36 +0200 Message-Id: <1396961442-24046-1-git-send-email-mreitz@redhat.com> Subject: [Qemu-devel] [PATCH v2 0/6] qemu-img: Implement commit like QMP List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org 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