From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:33757) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Sw92e-0006jI-2X for qemu-devel@nongnu.org; Tue, 31 Jul 2012 05:47:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Sw92Z-0008Gf-OQ for qemu-devel@nongnu.org; Tue, 31 Jul 2012 05:47:04 -0400 Received: from mx1.redhat.com ([209.132.183.28]:19296) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Sw92Z-0008FP-Eb for qemu-devel@nongnu.org; Tue, 31 Jul 2012 05:46:59 -0400 Message-ID: <5017A98E.8000109@redhat.com> Date: Tue, 31 Jul 2012 11:46:54 +0200 From: Kevin Wolf MIME-Version: 1.0 References: <1343127865-16608-1-git-send-email-pbonzini@redhat.com> <1343127865-16608-29-git-send-email-pbonzini@redhat.com> <5017A4DA.80602@redhat.com> <5017A680.8050305@redhat.com> In-Reply-To: <5017A680.8050305@redhat.com> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 28/47] qmp: add drive-mirror command List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: jcody@redhat.com, eblake@redhat.com, qemu-devel@nongnu.org, stefanha@linux.vnet.ibm.com Am 31.07.2012 11:33, schrieb Paolo Bonzini: > Il 31/07/2012 11:26, Kevin Wolf ha scritto: >> Am 24.07.2012 13:04, schrieb Paolo Bonzini: >>> This adds the monitor commands that start the mirroring job. >>> >>> Signed-off-by: Paolo Bonzini >> >> [ Moving the discussion upstream ] >> >>> Why make all of it inaccessible? Everything except target device access >>> does have a stable API. The target device access can be delayed to 1.3, >>> together with the much-needed QMP schema introspection. >> >> I'm not even sure about the QMP mirror command itself. >> >> I don't really like it, it does too many things at once: It can create >> the target image file, it opens the target and it actually starts the >> mirroring. It's rather bad at the first two steps, because it doesn't >> take any options. This means that it can't create qcow2v3 images, for >> example. Or you can't mirror into a backup with cache=unsafe while >> running your real VM on cache=writethrough. > > Yes, though this can be worked around with mode: 'existing'. True. Only the problem with image creation, though, not the one with bdrv_open() flags, right? >> Having an all-in-one mirror command is a nice feature for HMP, but for >> QMP it's more like a design problem. >> >> Now I see you have called it drive-mirror > > I thought this was your idea. :) Hm, then probably we discussed similar things before. :-) >> , so that kind of implies that >> it's not the final blockdev-mirror but just a QMP version of a command >> primarily designed for HMP. As such this restricted functionality may be >> acceptable, but it's not like everything is already perfect and there's >> no room for discussion. > > We keep going back to the same point that we do not have -blockdev, but > it's becoming a bit frustrating to always rehash this same point... The question is whether we need it at all. We do have a drive_add if=none, and for creating a mirror target that should actually be enough. Kevin