From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33000) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V1bAD-0002Eo-3M for qemu-devel@nongnu.org; Tue, 23 Jul 2013 07:53:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V1bAA-0007F2-4k for qemu-devel@nongnu.org; Tue, 23 Jul 2013 07:53:57 -0400 Received: from mx1.redhat.com ([209.132.183.28]:20152) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V1bA9-0007Et-Rb for qemu-devel@nongnu.org; Tue, 23 Jul 2013 07:53:54 -0400 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r6NBrr0F005452 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Tue, 23 Jul 2013 07:53:53 -0400 Date: Tue, 23 Jul 2013 13:53:51 +0200 From: Stefan Hajnoczi Message-ID: <20130723115351.GH20256@stefanha-thinkpad.redhat.com> References: <1374530960-22031-1-git-send-email-imain@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1374530960-22031-1-git-send-email-imain@redhat.com> Subject: Re: [Qemu-devel] [PATCH V6 0/3] Implement sync modes for drive-backup. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Ian Main Cc: kwolf@redhat.com, famz@redhat.com, qemu-devel@nongnu.org, rjones@redhat.com, pbonzini@redhat.com On Mon, Jul 22, 2013 at 03:09:17PM -0700, Ian Main wrote: > This patch adds sync modes on top of the work that Stefan Hajnoczi has done. > > These patches apply on kevin/block. > > Hopefully all is in order as this is my first QEMU patch. Many thanks to > Stephan and Fam Zheng for their help. > > V2: > > - No longer poll, instead use qemu_coroutine_yield(). > - Use bdrv_co_is_allocated(). > - Much better SYNC_MODE_NONE test. > > V3: > > - A few style fixes. > - Better commit message explaining how TOP and NONE operate. > - Verified using checkpatch.pl. > > V4: > > - Add patch to use the source as a backing hd during backup. > - Add patch to default sync mode none to qcow2. > > V5: > > - Fix qcow2 patch. Forgot to git add final version. > > V6: > > - Default to requiring 'format' when mode is absolute-paths. > - Removed one bad hunk that was misapplying. > - Fixed docs, examples and tests to match changes. > - Added tests for format bad/missing. > - Added bdrv_set_in_use() to target. > - Default to qcow2 patch not required. > > Ian Main (3): > Implement sync modes for drive-backup. > Add tests for sync modes 'TOP' and 'NONE' > Add backing drive while performing backup. > > block/backup.c | 107 +++++++++++++++++++++++++++++------------ > blockdev.c | 36 +++++++++----- > include/block/block_int.h | 4 +- > qapi-schema.json | 4 +- > qmp-commands.hx | 2 + > tests/qemu-iotests/055 | 108 +++++++++++++++++++++++++++++++++++++----- > tests/qemu-iotests/055.out | 4 +- > tests/qemu-iotests/group | 2 +- > tests/qemu-iotests/iotests.py | 5 ++ > 9 files changed, 211 insertions(+), 61 deletions(-) This patch mostly takes care of image fleecing except it does not give the target a device name which can be used by nbd-server-add. Fam's series tackles the target device name and some of the overlapping problems with your series. The core feature in your series is sync=top|none and that needs to be merged. Now we need to figure out which patches to take and what must be changed. Please see the sub-threads on Fam's series. Perhaps we can reach a consensus there. Stefan