From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51978) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UzRFS-0004VJ-7s for qemu-devel@nongnu.org; Wed, 17 Jul 2013 08:54:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UzRFP-0000hS-WB for qemu-devel@nongnu.org; Wed, 17 Jul 2013 08:54:26 -0400 Received: from mx1.redhat.com ([209.132.183.28]:50763) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UzRFP-0000hJ-NO for qemu-devel@nongnu.org; Wed, 17 Jul 2013 08:54:23 -0400 Message-ID: <51E693E9.3010000@redhat.com> Date: Wed, 17 Jul 2013 06:54:01 -0600 From: Eric Blake MIME-Version: 1.0 References: <1374053388-17431-1-git-send-email-wdongxu@linux.vnet.ibm.com> In-Reply-To: <1374053388-17431-1-git-send-email-wdongxu@linux.vnet.ibm.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="----enig2UHKQWPDREFLPNCDKDFMW" Subject: Re: [Qemu-devel] [PATCH V17 0/9] replace QEMUOptionParameter with QemuOpts parser List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Dong Xu Wang Cc: kwolf@redhat.com, wdongxu@cn.ibm.com, qemu-devel@nongnu.org, stefanha@redhat.com This is an OpenPGP/MIME signed message (RFC 4880 and 3156) ------enig2UHKQWPDREFLPNCDKDFMW Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On 07/17/2013 03:29 AM, Dong Xu Wang wrote: > These patches will replace QEMUOptionParameter with QemuOpts. Change lo= gs > please go to each patch's commit message. >=20 > Dong Xu Wang (9): > qemu-option: add def_value_str in QemuOptDesc struct and rewrite > qemu_opts_print > qemu-option: avoid duplication of default value in QemuOpts > qemu-option: create four QemuOptsList related functions > qemu-option: create some QemuOpts functons > block: use QemuOpts support in block layer > qapi: query-command-line-options outputs def_value_str > qemu-option: remove QEMUOptionParameter related functions and struct > qemu-option: make qemu_opts_del accept opts being NULL > qemu-option: use qemu_opts_del without judging NULL >=20 >=20 > --=20 Ouch - by putting your additional comments after an 'end-of-message' marker, you made it harder for me to reply. (Sane mailers intentionally drop text after a "-- " line when replying). > V17 add qemu_opts_del's support when opts is NULL, and made changes > based on Eric's comments. I tried to split PATCH 5 into small pieces, > but I found it is very hard, bdrv_* functions are all using > QEMUOptionParameter or QemuOpts, if I split them, I have to support two= > parsers in block.c and qemu-img.c, it will make code very hard to read,= Yes, if you split the patches, then there will be a window of time in the series where you have to support BOTH parsers at the same time. But since the existing parser is already present, what's so hard about that? By having two different callbacks, one with the old signature and one with the new, with exactly one of the callbacks being non-NULL, it should be relatively easy to decide which of the two callbacks to use. > so I leave them in one big patch, I am sorry it is realy hard to review= =2E I still think that the ease of review that would be added by splitting the patch is worth making the effort. Yes, it may be harder for you, but if the end result is that more people are willing to review the patch, then the community as a whole spends less collective time, and the series will be accepted sooner. --=20 Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org ------enig2UHKQWPDREFLPNCDKDFMW 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/ iQEcBAEBCAAGBQJR5pPpAAoJEKeha0olJ0NqcGYH/ieBl6i0GnDH6ms1MyuucS8P k+oKnxPlcj9wwb6EQlaGV4kf0qGnbCNa0YcfBQWlNua2uXnEKqe/AUdSiupUmh9W 15w/80xtSpv/kEminam6uoq7Vzea+/B2DxSlHvLOCdyV4IzUHrtz7/vzyq+rgv/z ztzean/ufVA+72saZts2NBG57LuMY9hdj4Mme0KUtwLRglP0r5nN+gRrLH77Dbpu O0H6FoUzLiHcSWEUywgU6cJBgXi6K9LbTPfNyMYqGcvyMMvznIxvs2sw02Myc1lL Vp96WPFMBp3pVHEJFU+ydaHJQ4V2REP+WzUAFc5JwyifAaG7rodLbkWk1qWvTSo= =3qpT -----END PGP SIGNATURE----- ------enig2UHKQWPDREFLPNCDKDFMW--