From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44752) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WCs4D-0008Cl-1f for qemu-devel@nongnu.org; Mon, 10 Feb 2014 09:42:42 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WCs45-00039J-3v for qemu-devel@nongnu.org; Mon, 10 Feb 2014 09:42:36 -0500 Received: from cantor2.suse.de ([195.135.220.15]:40230 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WCs44-000394-TM for qemu-devel@nongnu.org; Mon, 10 Feb 2014 09:42:29 -0500 Message-ID: <52F8E551.8010605@suse.de> Date: Mon, 10 Feb 2014 15:42:25 +0100 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1391853717-3837-1-git-send-email-pbonzini@redhat.com> <52F63E05.409@suse.de> <87iosnb9ya.fsf@blackfin.pond.sub.org> In-Reply-To: <87iosnb9ya.fsf@blackfin.pond.sub.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster Cc: Paolo Bonzini , qemu-devel@nongnu.org, Anthony Liguori , Peter Maydell Am 10.02.2014 10:20, schrieb Markus Armbruster: > Andreas F=C3=A4rber writes: >=20 >> Paolo, >> >> Am 08.02.2014 11:01, schrieb Paolo Bonzini: >>> Anthony, Peter, >>> >>> The following changes since commit 0169c511554cb0014a00290b0d3d26c31a= 49818f: >>> >>> Merge remote-tracking branch 'qemu-kvm/uq/master' into staging (201= 4-01-24 15:52:44 -0800) >>> >>> are available in the git repository at: >>> >>> git://github.com/bonzini/qemu.git qdev-props >>> >>> for you to fetch changes up to 94fb9add077db8a8f0be3796f44785694c4686= bb: >>> >>> qapi: refine human printing of sizes (2014-02-08 10:44:41 +0100) >>> >>> ---------------------------------------------------------------- >>> Paolo Bonzini (14): >>> qapi: add size parser to StringInputVisitor >>> qdev: sizes are now parsed by StringInputVisitor >>> qdev: remove legacy parsers for hex8/32/64 >>> qdev: legacy properties are now read-only >>> qdev: legacy properties are just strings >>> qdev: inline qdev_prop_parse >>> qapi: add human mode to StringOutputVisitor >>> qdev: use human mode in "info qtree" >>> qdev: remove most legacy printers >>> qdev: remove hex8/32/64 property types >>> block: handle "rechs" and "large" translation options >>> qdev: add enum property types to QAPI schema >>> qdev: use QAPI type names for properties >>> qapi: refine human printing of sizes >> >> I had specifically requested to review and take these through qom-next= , >> like most qdev changes have gone lately. Why are you sending a pull >> nontheless? In particular Luiz has not yet replied to the QERR issue I >> pointed out. >=20 > I guess Luiz didn't reply for the same reason I didn't chime in then: > Paolo and Eric explained the use of QERR_INVALID_PARAMETER_TYPE > adquately. Guess that's one of the social aspects again: My lack of response does not mean "I'm okay with it" but rather "I haven't read it yet or am investigating alternatives", and I assume the same for other people. ;) > You're right to challenge new uses of QERR_*, but the use you spotted i= s > appropriate, since we want consistency with the existing visitors. OK, I still don't fully understand the logic why sometimes we shouldn't use QERR_ at all, in some cases inline the error message for compatibility without reusing the QERR_ and sometimes can use QERR_ directly, but I don't mind it getting applied that way - QERR_ is not my fight. :) Since Peter has not pulled yet, I'll pull or apply into my tree to check if anything collides. Cheers, Andreas E.g., http://repo.or.cz/w/qemu.git/commit/d44bb8604e87ecd3823f12f0c92d5e56d613d= e0d (which collided with the QOM realize conversion so that I noticed) --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC= rnberg