From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39418) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XPYwj-0007D9-2e for qemu-devel@nongnu.org; Thu, 04 Sep 2014 11:27:41 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XPYwb-0006lD-Ut for qemu-devel@nongnu.org; Thu, 04 Sep 2014 11:27:37 -0400 Date: Thu, 4 Sep 2014 18:30:37 +0300 From: "Michael S. Tsirkin" Message-ID: <20140904153037.GA21203@redhat.com> References: <1409797106-9076-1-git-send-email-arei.gonglei@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1409797106-9076-1-git-send-email-arei.gonglei@huawei.com> Subject: Re: [Qemu-devel] [PATCH v6 0/3] Refactor device_set_realized to avoid resource leak List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: arei.gonglei@huawei.com Cc: peter.crosthwaite@xilinx.com, weidong.huang@huawei.com, qemu-stable@nongnu.org, luonengjun@huawei.com, qemu-devel@nongnu.org, peter.huangpeng@huawei.com, pbonzini@redhat.com, imammedo@redhat.com, afaerber@suse.de On Thu, Sep 04, 2014 at 10:18:23AM +0800, arei.gonglei@huawei.com wrote: > From: Gonglei > > after committing > [PATCH v6 0/9] Refactor PCI/SHPC/PCIE hotplug to use a more generic hotplug API > > If devcie hotplugging failed, will casuse resource leak. > This patch series include address resouce leak and two other issuses. > > v6 -> v5: > - rework patch 2/3 by Peter's suggestion. > - add 'Reviewed-by' tag. > - remove a patch about PCIe hotplugging by Michael's idea. > > v5 -> v4: > - add patch 1/4 'Reviewed-by' tag. > - change patch 2/4, propagate firsh child unrealizing failure, and > change this patch's commit message.(Peter) > > v4 -> v3: > - add patch 2/4.(Thanks for Peter's suggestion) > - rework patch 3/4 based on patch 2/4. > > v3 -> v2: > - add cleanup logic for set bus/child_bus realized/unrealized failed. > - change patch 1/3 commit message, add 'Reviewed-by' tag. > > v2 -> v1: > - rewrite patch 1/3, using error_abort instead of local_err. > - rewrite patch 2/3, add cleanup logic for different error embranchment. > - rewrite title of patch 3/3, and a syntax fix. > > Gonglei (3): > qdev: using error_abort instead of using local_err > qdev: using NULL instead of local_err for qbus_child unrealize > qdev: add cleanup logic in device_set_realized() to avoid resource > leak Applied and Cc qemu-stable, thanks! > hw/core/qdev.c | 70 +++++++++++++++++++++++++++++++++++++++------------------- > 1 file changed, 47 insertions(+), 23 deletions(-) > > -- > 1.7.12.4 > >