[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_c
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close() |
Date: |
Thu, 12 Nov 2015 14:23:49 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, 11/09 23:39, Max Reitz wrote:
> bdrv_delete() is not very happy about deleting BlockDriverStates with
> dirty bitmaps still attached to them. In the past, we got around that
> very easily by relying on bdrv_close_all() bypassing bdrv_delete(), and
> bdrv_close() simply ignoring that condition. We should fix that by
> releasing all dirty bitmaps in bdrv_close() and drop the assertion in
> bdrv_delete().
What bitmaps are attached when bdrv_close() is called? The ones created from
the monitor should probably be removed by the monitor, and the internal ones
like in migration and block jobs should probably be removed by stopping the
respective job.
Fam
- [Qemu-devel] [PATCH v7 01/24] blockdev: Add missing bdrv_unref() in drive-backup, (continued)
- [Qemu-devel] [PATCH v7 01/24] blockdev: Add missing bdrv_unref() in drive-backup, Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 02/24] blockjob: Call bdrv_unref() on creation error, Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 04/24] iotests: Rename filter_nbd to _filter_nbd in 083, Max Reitz, 2015/11/09
- [Qemu-devel] [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close(), Max Reitz, 2015/11/09
- Re: [Qemu-devel] [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close(), John Snow, 2015/11/11
- Re: [Qemu-devel] [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close(),
Fam Zheng <=
- Re: [Qemu-devel] [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close(), John Snow, 2015/11/13
- Re: [Qemu-devel] [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close(), Fam Zheng, 2015/11/16
- Re: [Qemu-devel] [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close(), John Snow, 2015/11/16
- Re: [Qemu-devel] [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close(), Fam Zheng, 2015/11/16
- [Qemu-devel] Closing Bitmaps (Was: Re: [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close()), John Snow, 2015/11/17
- Re: [Qemu-devel] Closing Bitmaps (Was: Re: [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close()), Fam Zheng, 2015/11/17
- Re: [Qemu-devel] Closing Bitmaps (Was: Re: [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close()), John Snow, 2015/11/18
- Re: [Qemu-devel] Closing Bitmaps (Was: Re: [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close()), Kevin Wolf, 2015/11/18
- Re: [Qemu-devel] Closing Bitmaps (Was: Re: [PATCH v7 03/24] block: Release dirty bitmaps in bdrv_close()), John Snow, 2015/11/18
[Qemu-devel] [PATCH v7 05/24] iotests: Change coding style of _filter_nbd in 083, Max Reitz, 2015/11/09