From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:45201) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ub0aL-0005x6-Vj for qemu-devel@nongnu.org; Fri, 10 May 2013 23:35:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Ub0aK-0008Cq-Qb for qemu-devel@nongnu.org; Fri, 10 May 2013 23:35:01 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38719) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ub0aK-0008Cm-AV for qemu-devel@nongnu.org; Fri, 10 May 2013 23:35:00 -0400 Message-ID: <518DBC5E.1020700@redhat.com> Date: Fri, 10 May 2013 21:34:54 -0600 From: Eric Blake MIME-Version: 1.0 References: <1367221335-22777-1-git-send-email-stefanha@redhat.com> <1367221335-22777-3-git-send-email-stefanha@redhat.com> <20130508124859.GH3093@dhcp-200-207.str.redhat.com> In-Reply-To: <20130508124859.GH3093@dhcp-200-207.str.redhat.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="----enig2QLKGAOQOEGMDTXKNVEIE" 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: Kevin Wolf Cc: Fam Zheng , qemu-devel@nongnu.org, Wenchao Xia , imain@redhat.com, Stefan Hajnoczi , pbonzini@redhat.com, dietmar@proxmox.com This is an OpenPGP/MIME signed message (RFC 4880 and 3156) ------enig2QLKGAOQOEGMDTXKNVEIE Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On 05/08/2013 06:49 AM, Kevin Wolf wrote: > Am 29.04.2013 um 09:42 hat Stefan Hajnoczi geschrieben: >> @block-backup >> > 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= =2E > created by blockdev-add). At least it would match why we named a command 'drive-mirror' instead of 'block-mirror'. Hmm, looking at qapi-schema.json, I wonder if we can rename 'BlockdevAction' to 'TransactionAction' as used in the @transaction command. It wouldn't change what is sent over the wire in JSON, and until we have full introspection, there is no visibility into the type name used. Changing the name now would let it be more generic to adding future transaction items that are not blockdev related. >=20 > 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. Agreed - existence of a command at the same time the command is made transactionable serves as a nice substitute for not having full introspection into the 'BlockdevAction' union type, whereas if we introduce the command now but not transaction support until 1.7, life becomes tougher to know when it can be used where (although I HOPE we have introspection in 1.6). --=20 Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org ------enig2QLKGAOQOEGMDTXKNVEIE Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.13 (GNU/Linux) Comment: Public key at http://people.redhat.com/eblake/eblake.gpg Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQEcBAEBCAAGBQJRjbxeAAoJEKeha0olJ0NqgsEH+gKQPfOO19yxr/SbqjKBhio/ tJYfJUzsTdB/x2/pn/wQPoSsB83bOIG++WMsorWF1u0Ltg4vS6oTMzb1diW1ZDl0 cWClxpldKr0d8VacplbuE7vBgv/TbYsKsuQKjOGPMFlw1CQQY1DuY+2Y7Tf5Mmcv 5fANxxNO4r2i/FQVckBf649XmpKWGWn51I/dHaQJsPukQPkgIDwAyNUWfv9FW0kn +6qC4KmunvvNIs59+iGnCO4NC/EYXC+h2K9M8zvrR0y8H+xHuNJn5j02uizswAw3 fTtTyuTGlcsNKspFm8LxjNbHVwJKtusPwKMbvK9GlvmwP/yw0hLkq5mW81mwDmY= =PIa/ -----END PGP SIGNATURE----- ------enig2QLKGAOQOEGMDTXKNVEIE--