From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:60961) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ua4ZF-0006Ij-FC for qemu-devel@nongnu.org; Wed, 08 May 2013 09:38:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Ua4ZE-0003Gj-AQ for qemu-devel@nongnu.org; Wed, 08 May 2013 09:38:01 -0400 Received: from mx1.redhat.com ([209.132.183.28]:13734) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ua4ZD-0003GQ-Ue for qemu-devel@nongnu.org; Wed, 08 May 2013 09:38:00 -0400 Date: Wed, 8 May 2013 14:49:00 +0200 From: Kevin Wolf Message-ID: <20130508124859.GH3093@dhcp-200-207.str.redhat.com> References: <1367221335-22777-1-git-send-email-stefanha@redhat.com> <1367221335-22777-3-git-send-email-stefanha@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1367221335-22777-3-git-send-email-stefanha@redhat.com> Subject: Re: [Qemu-devel] [PATCH v2 2/3] block: add block-backup QMP command List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Fam Zheng , qemu-devel@nongnu.org, Wenchao Xia , imain@redhat.com, pbonzini@redhat.com, dietmar@proxmox.com Am 29.04.2013 um 09:42 hat Stefan Hajnoczi geschrieben: > @block-backup > > Start a point-in-time copy of a block device to a new destination. The > status of ongoing block backup operations can be checked with > query-block-jobs. The operation can be stopped before it has completed using > the block-job-cancel command. > > @device: the name of the device whose writes should be mirrored. > > @target: the target of the new image. If the file exists, or if it > is a device, the existing file/device will be used as the new > destination. If it does not exist, a new file will be created. > > @format: #optional the format of the new destination, default is to > probe if @mode is 'existing', else the format of the source > > @mode: #optional whether and how QEMU should create a new image, default is > 'absolute-paths'. > > @speed: #optional the maximum speed, in bytes per second > > Returns: nothing on success > If @device is not a valid block device, DeviceNotFound > > Since 1.6 > > Signed-off-by: Stefan Hajnoczi drive-backup would probably be a more consistent naming. We would then still have block-backup for a future low-level command that doesn't create everything by itself but takes an existing BlockDriverState (e.g. created by blockdev-add). We should also make it transactionable from the beginning, as we don't have schema introspection yet. This way we allow to assume that if the standalone command exists, the transaction subcommand exists as well. Kevin