From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45860) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VTEEr-0003Ud-OE for qemu-devel@nongnu.org; Mon, 07 Oct 2013 13:05:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VTEEm-0000dm-GI for qemu-devel@nongnu.org; Mon, 07 Oct 2013 13:04:57 -0400 Received: from mail-qa0-x231.google.com ([2607:f8b0:400d:c00::231]:55410) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VTEEm-0000dH-C0 for qemu-devel@nongnu.org; Mon, 07 Oct 2013 13:04:52 -0400 Received: by mail-qa0-f49.google.com with SMTP id k15so3194369qaq.1 for ; Mon, 07 Oct 2013 10:04:51 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <5252E9AF.6010606@redhat.com> Date: Mon, 07 Oct 2013 19:04:47 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <1381164241-6566-1-git-send-email-afaerber@suse.de> In-Reply-To: <1381164241-6566-1-git-send-email-afaerber@suse.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH qom-next 0/2] qdev-monitor: Reference counting follow-ups List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= Cc: Igor Mammedov , Stefan Hajnoczi , qemu-devel@nongnu.org, Anthony Liguori Il 07/10/2013 18:43, Andreas Färber ha scritto: > Hello, > > I have queued bug fixes by Igor and Stefan for device_add on qom-next and > am rearranging the following changes of mine on top. > > 1) Further naming cleanups, now rebased on the bugfixes for easier backporting. > 2) Inlining of qdev_init(), so that we always have unparent+unref pairs. > > If there's no objections, planning to include this in a pull tonight or tomorrow. > > Regards, > Andreas > > Cc: Igor Mammedov > Cc: Stefan Hajnoczi > Cc: Paolo Bonzini > Cc: Anthony Liguori > > Andreas Färber (2): > qdev-monitor: Avoid qdev as variable name > qdev-monitor: Inline qdev_init() for device_add > > qdev-monitor.c | 37 +++++++++++++++++++++---------------- > 1 file changed, 21 insertions(+), 16 deletions(-) > Yes, looks good. Paolo