From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:43654) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UV2NF-00082Y-8o for qemu-devel@nongnu.org; Wed, 24 Apr 2013 12:16:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UV2NA-0003s5-DD for qemu-devel@nongnu.org; Wed, 24 Apr 2013 12:16:49 -0400 Received: from mx1.redhat.com ([209.132.183.28]:12304) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UV2NA-0003qX-4G for qemu-devel@nongnu.org; Wed, 24 Apr 2013 12:16:44 -0400 Message-ID: <5178053A.1000102@redhat.com> Date: Wed, 24 Apr 2013 10:15:54 -0600 From: Eric Blake MIME-Version: 1.0 References: In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="----enig2KAQPEJHSXBTEPKKSFCRS" Subject: Re: [Qemu-devel] [PATCH v2 00/12] covert savevm, loadvm and delvm into qapi List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Pavel Hrdina Cc: kwolf@redhat.com, xiawenc@linux.vnet.ibm.com, lcapitulino@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com This is an OpenPGP/MIME signed message (RFC 4880 and 3156) ------enig2KAQPEJHSXBTEPKKSFCRS Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On 04/24/2013 09:31 AM, Pavel Hrdina wrote: > I'm sending patches for all commands in one patch series because the > savevm command depends on delvm command. I debated whether to call this out, but here goes: I pointed out s/covert/convert/ on the subject line of v1; that can't bode too well if the v2 still has the same problem, even if the problem does not affect anything that gets committed into git. Here's hoping the rest of the series is better... >=20 > This patch series introduces new design of these commands: At this point, we've missed soft freeze. Is this something we are still trying to get into 1.5, or is it okay to slip it to 1.6 where we can feel more comfortable about having a solid review in place? Ultimately, it's the maintainer's call, but I'm personally leaning towards deferring - libvirt has survived with HMP savevm long enough that another qemu release without QMP savevm won't be the end of the world. I'm also hoping that the final product is clean enough that a distro could feel comfortable with the idea of rebasing to 1.5 while still backporting the QMP commands that go into 1.6. >=20 > changes from v1: > - patch for updating bdrv_snapshot_goto and bdrv_snapshot_list is s= plit > into two patches > - fixes typos and grammar > - vm-snapshot-delete and vm-snapshot-load now returns an error also= if > snapshot for delete or load not exists > - all error messages starts with uppercase and are without trailing= dot > - updated error messages recording to comments =2E..at least the summary makes it sound like you made a good effort. It= might also help to provide a bit of context on how we are planning on tackling any merge conflicts regarding the algorithm for snapshot lookup, and whether there are any rebase dependencies between your series and Wenchao's. --=20 Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org ------enig2KAQPEJHSXBTEPKKSFCRS 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/ iQEcBAEBCAAGBQJReAU6AAoJEKeha0olJ0NqXjUIAKUsJCYNYwcKJWLlaHHVhG8/ KXM0A+8AFInf0al7nZ1H5sSZGUFZmlVyGOitDLWQYO+2XIDt7ZlE2Wy3TSahzJCO p0q0DJRrT2zUAKfNA819xlAk3l0w0ik8PYtR3x2PdaeEfg/GjpMsLj4xFrQOpNOw QmEbgVkDzE4CFsQtA8tEK1dRxb2oEwqbnkNFxQe19Dl0YLmsUBtWjkDOOykAKzQU 4gIOjpjXe35pd8fMtGs1ZAz6wv5PmGjjMAzkjoB2wU1U9uj42OecgHBP5VM257MX RhfiZTFM7lbQlFoYAKIsBJ3oB+0ucx4t0CFqWy7XqN3eX/2dMromOJXSQRjVYiE= =n4t3 -----END PGP SIGNATURE----- ------enig2KAQPEJHSXBTEPKKSFCRS--