[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 08/16] block: Use blk_remove_bs() in blk_dele
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v8 08/16] block: Use blk_remove_bs() in blk_delete() |
Date: |
Thu, 28 Jan 2016 11:28:56 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, 01/27 18:59, Max Reitz wrote:
> Signed-off-by: Max Reitz <address@hidden>
> Reviewed-by: Kevin Wolf <address@hidden>
> ---
> block/block-backend.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/block/block-backend.c b/block/block-backend.c
> index 621787c..7f5ad59 100644
> --- a/block/block-backend.c
> +++ b/block/block-backend.c
> @@ -166,10 +166,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);
> }
> assert(QLIST_EMPTY(&blk->remove_bs_notifiers.notifiers));
> assert(QLIST_EMPTY(&blk->insert_bs_notifiers.notifiers));
> @@ -351,6 +348,8 @@ void blk_hide_on_behalf_of_hmp_drive_del(BlockBackend
> *blk)
> */
> void blk_remove_bs(BlockBackend *blk)
> {
> + assert(blk->bs->blk == blk);
> +
> notifier_list_notify(&blk->remove_bs_notifiers, blk);
>
> blk_update_root_state(blk);
> --
> 2.7.0
>
Reviewed-by: Fam Zheng <address@hidden>
- [Qemu-devel] [PATCH v8 05/16] virtio-scsi: Catch BDS-BB removal/insertion, (continued)
- [Qemu-devel] [PATCH v8 05/16] virtio-scsi: Catch BDS-BB removal/insertion, Max Reitz, 2016/01/27
- [Qemu-devel] [PATCH v8 09/16] blockdev: Use blk_remove_bs() in do_drive_del(), Max Reitz, 2016/01/27
- [Qemu-devel] [PATCH v8 07/16] block: Remove BDS close notifier, Max Reitz, 2016/01/27
- [Qemu-devel] [PATCH v8 11/16] block: Add list of all BlockDriverStates, Max Reitz, 2016/01/27
- [Qemu-devel] [PATCH v8 08/16] block: Use blk_remove_bs() in blk_delete(), Max Reitz, 2016/01/27
- Re: [Qemu-devel] [PATCH v8 08/16] block: Use blk_remove_bs() in blk_delete(),
Fam Zheng <=
- [Qemu-devel] [PATCH v8 12/16] blockdev: Keep track of monitor-owned BDS, Max Reitz, 2016/01/27
- [Qemu-devel] [PATCH v8 10/16] block: Make bdrv_close() static, Max Reitz, 2016/01/27
- [Qemu-devel] [PATCH v8 13/16] block: Add blk_remove_all_bs(), Max Reitz, 2016/01/27
- [Qemu-devel] [PATCH v8 14/16] block: Rewrite bdrv_close_all(), Max Reitz, 2016/01/27
- [Qemu-devel] [PATCH v8 16/16] iotests: Add test for block jobs and BDS ejection, Max Reitz, 2016/01/27