From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:60863)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1WC8pM-0005d0-DS
	for qemu-devel@nongnu.org; Sat, 08 Feb 2014 09:24:22 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1WC8pG-0000rO-J9
	for qemu-devel@nongnu.org; Sat, 08 Feb 2014 09:24:16 -0500
Received: from cantor2.suse.de ([195.135.220.15]:44207 helo=mx2.suse.de)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1WC8pG-0000r3-CX
	for qemu-devel@nongnu.org; Sat, 08 Feb 2014 09:24:10 -0500
Message-ID: <52F63E05.409@suse.de>
Date: Sat, 08 Feb 2014 15:24:05 +0100
From: =?ISO-8859-15?Q?Andreas_F=E4rber?= <afaerber@suse.de>
MIME-Version: 1.0
References: <1391853717-3837-1-git-send-email-pbonzini@redhat.com>
In-Reply-To: <1391853717-3837-1-git-send-email-pbonzini@redhat.com>
Content-Type: text/plain; charset=ISO-8859-15
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PULL 00/14] Cleanup qdev legacy properties
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Paolo Bonzini <pbonzini@redhat.com>, qemu-devel@nongnu.org
Cc: Peter Maydell <peter.maydell@linaro.org>, Anthony Liguori <aliguori@amazon.com>

Paolo,

Am 08.02.2014 11:01, schrieb Paolo Bonzini:
> Anthony, Peter,
>=20
> The following changes since commit 0169c511554cb0014a00290b0d3d26c31a49=
818f:
>=20
>   Merge remote-tracking branch 'qemu-kvm/uq/master' into staging (2014-=
01-24 15:52:44 -0800)
>=20
> are available in the git repository at:
>=20
>   git://github.com/bonzini/qemu.git qdev-props
>=20
> for you to fetch changes up to 94fb9add077db8a8f0be3796f44785694c4686bb=
:
>=20
>   qapi: refine human printing of sizes (2014-02-08 10:44:41 +0100)
>=20
> ----------------------------------------------------------------
> 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.

Apart from that issue all patches look okay on brief skimming over.

Patch 10 should've been slightly more verbose because changing the type
of properties is normally NOT allowed. The reason it is possible here is
because they are format-compatible, which the commit message does not
explain.

Andreas

--=20
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrnbe=
rg