From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53123) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a3mQj-0007qN-Nw for qemu-devel@nongnu.org; Tue, 01 Dec 2015 10:01:26 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a3mQd-0005qF-U9 for qemu-devel@nongnu.org; Tue, 01 Dec 2015 10:01:21 -0500 Received: from mx1.redhat.com ([209.132.183.28]:44528) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a3mQd-0005q5-Oi for qemu-devel@nongnu.org; Tue, 01 Dec 2015 10:01:15 -0500 References: <1447687950-29350-1-git-send-email-den@openvz.org> <1447687950-29350-3-git-send-email-den@openvz.org> <87ziyd0wdm.fsf@blackfin.pond.sub.org> <87mvubmtef.fsf@emacs.mitica> <565DAEAB.6060104@openvz.org> From: Eric Blake Message-ID: <565DB635.5070306@redhat.com> Date: Tue, 1 Dec 2015 08:01:09 -0700 MIME-Version: 1.0 In-Reply-To: <565DAEAB.6060104@openvz.org> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="lrTIgM5MFIaroW84Tc12t23sNJPh54oFa" Subject: Re: [Qemu-devel] [PATCH 2/5] qmp: create qmp_savevm command List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Denis V. Lunev" , quintela@redhat.com, Markus Armbruster Cc: Amit Shah , qemu-devel@nongnu.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --lrTIgM5MFIaroW84Tc12t23sNJPh54oFa Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 12/01/2015 07:28 AM, Denis V. Lunev wrote: >>>> +void qmp_savevm(bool has_name, const char *name, Error **errp) >>>> +{ >>>> + do_savevm(has_name ? name : NULL, errp); >>>> +} >>>> + >>> Please name do_savevm() qmp_savevm() and drop this wrapper. >>> >>> We're working on omitting has_FOO for pointer-valued FOO. >> Agreed. >> >=20 > is there a tree with this stuff in or something I could be based on? > This series is simple enough to be written fast and be merged > early when the hard freeze will be done. I have started the work in my local tree, but have not posted it yet. I'll try and remember to cc you when it's ready (right now, there are more than 20 qapi patches from me and another 35 or so from Marc-Andre that need reviewing before we'd even get to the point of my work on dropping has_FOO for pointers). >=20 > Above stuff could have a way harder way due to the amount of > changes to be performed. Dropping has_FOO will be rather invasive; my plan is to add a per-struct marker to the .json files on which structs are ready to use the leaner layout, then do a series of patches that adjusts a couple structs at a time and catches all fallout for those structs. Then, if everything converts nicely to the leaner form, I can remove the per-struct marker in the end. So it's not a problem to just keep the three-argument form with has_name for now. --=20 Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org --lrTIgM5MFIaroW84Tc12t23sNJPh54oFa Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Public key at http://people.redhat.com/eblake/eblake.gpg Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQEcBAEBCAAGBQJWXbY1AAoJEKeha0olJ0Nqq7wH/0QccOn7iWwYcFDF2/EttiGB 1YNBtbG7YgRsLJygJKSNrKY/LvXGRfghoQwtbRJ9AEFnl1FlbpGKPqp4/DkZSBzO gDDsR+O3MKgKjcaBZJ86sjQkyV5J85gJIlf4c5PA5X3sbshCJ6zMEz0gIVJK1QXb DEjiriYKwPeNoLvxPq9u+ag6kGn3StA+ocm7BORNTW2u7c+tHpXf2K16ydG5fk+3 WS50PX/rW/hTHwWTsRTyMAB+V0TOZ0Z9aZNsLRHY+ssA/Ym/l2BemgtY7pAJaLlU JU8pszu3+SAKjaTbJZ6l6a4e6yJUOA4xPLuPnl/ETyTHwrZ/TYJwHBycjniUVh8= =EsTY -----END PGP SIGNATURE----- --lrTIgM5MFIaroW84Tc12t23sNJPh54oFa--