From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:51317) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SPK9P-0004fP-KF for qemu-devel@nongnu.org; Tue, 01 May 2012 16:58:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SPK9N-0003Ri-TZ for qemu-devel@nongnu.org; Tue, 01 May 2012 16:58:23 -0400 Received: from cantor2.suse.de ([195.135.220.15]:39739 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SPJvW-0007DI-R1 for qemu-devel@nongnu.org; Tue, 01 May 2012 16:44:02 -0400 Message-ID: <4FA04B0E.5040500@suse.de> Date: Tue, 01 May 2012 22:43:58 +0200 From: =?ISO-8859-15?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <1335896294-9530-1-git-send-email-aliguori@us.ibm.com> <1335896294-9530-4-git-send-email-aliguori@us.ibm.com> <4FA033EC.1050800@suse.de> <4FA0497D.3060103@us.ibm.com> In-Reply-To: <4FA0497D.3060103@us.ibm.com> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 03/14] qdev: add qdev_add_properties List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Wanpeng Li , Peter Maydell , qemu-devel@nongnu.org, Paolo Bonzini Am 01.05.2012 22:37, schrieb Anthony Liguori: > On 05/01/2012 02:05 PM, Andreas F=E4rber wrote: >> Am 01.05.2012 20:18, schrieb Anthony Liguori: >>> This allows a base class to easily add properties. >>> >>> Signed-off-by: Anthony Liguori >> >> Implementation looks okay but /me not so happy with it: This conflicts >> with the move of the qdev static property infrastructure from >> DeviceState to Object. >> >> Consider rebasing this onto part of Paolo's series and call it >> object_add_properties? >=20 > Eh? There's nothing object_ about these properties and there's no way > I'm willing to put legacy properties in object... >=20 > So I'm not quite sure what you're suggesting. You just suggested to Peter using qdev_add_properties() in new QOM ARM classes of his. I'd rather not propagate using qdev_* functions in new QOM code because either it remains forever or renaming becomes another touch-all series. In Paolo's series the Property* concept is moved from qdev to QOM; thus if it's in Object we usually use an object_ prefix. Not too long ago you were willing to merge the large part of Paolo's series which included this code movement, so if you don't want that after all you should communicate that openly as a reply there. :) 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