From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38241) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YRO1d-0000L9-Uw for qemu-devel@nongnu.org; Fri, 27 Feb 2015 11:44:30 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YRO1X-0006M1-QR for qemu-devel@nongnu.org; Fri, 27 Feb 2015 11:44:29 -0500 From: Max Reitz Date: Fri, 27 Feb 2015 11:43:55 -0500 Message-Id: <1425055440-18038-7-git-send-email-mreitz@redhat.com> In-Reply-To: <1425055440-18038-1-git-send-email-mreitz@redhat.com> References: <1425055440-18038-1-git-send-email-mreitz@redhat.com> Subject: [Qemu-devel] [PATCH v4 06/11] block: Use blk_remove_bs() in blk_delete() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-block@nongnu.org Cc: Kevin Wolf , Fam Zheng , qemu-devel@nongnu.org, Markus Armbruster , Stefan Hajnoczi , Paolo Bonzini , Max Reitz Signed-off-by: Max Reitz Reviewed-by: Eric Blake --- block/block-backend.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/block/block-backend.c b/block/block-backend.c index 254fde4..7e9d53a 100644 --- a/block/block-backend.c +++ b/block/block-backend.c @@ -163,12 +163,7 @@ static void blk_delete(BlockBackend *blk) { assert(!blk->refcnt); assert(!blk->dev); - if (blk->bs) { - assert(blk->bs->blk == blk); - blk->bs->blk = NULL; - bdrv_unref(blk->bs); - blk->bs = NULL; - } + blk_remove_bs(blk); /* Avoid double-remove after blk_hide_on_behalf_of_do_drive_del() */ if (blk->name[0]) { QTAILQ_REMOVE(&blk_backends, blk, link); @@ -324,6 +319,8 @@ void blk_remove_bs(BlockBackend *blk) return; } + assert(blk->bs->blk == blk); + notifier_list_notify(&blk->remove_bs_notifiers, blk); blk_update_root_state(blk); -- 2.1.0