[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 07/11] block: hold hard reference for backup/
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v2 07/11] block: hold hard reference for backup/mirror target |
Date: |
Thu, 25 Jul 2013 14:08:42 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, 07/23 11:52, Stefan Hajnoczi wrote:
> On Wed, Jul 17, 2013 at 05:42:12PM +0800, Fam Zheng wrote:
> > Signed-off-by: Fam Zheng <address@hidden>
> > ---
> > block/backup.c | 3 ++-
> > block/mirror.c | 4 ++--
> > 2 files changed, 4 insertions(+), 3 deletions(-)
>
> Should we update the blockjob.c in_use code instead of adding
> refcounting to specific block jobs? This ought to be handled
> generically for all block jobs.
Target is not common in block jobs (e.g. doesn't apply to block-commit),
so it seems only specific block job knows about this.
--
Fam
- [Qemu-devel] [PATCH v2 02/11] block: use refcnt for bs->backing_hd and bs->file, (continued)
- [Qemu-devel] [PATCH v2 02/11] block: use refcnt for bs->backing_hd and bs->file, Fam Zheng, 2013/07/17
- [Qemu-devel] [PATCH v2 03/11] block: use refcnt for drive_init/drive_uninit, Fam Zheng, 2013/07/17
- [Qemu-devel] [PATCH v2 04/11] block: use refcnt for device attach/detach, Fam Zheng, 2013/07/17
- [Qemu-devel] [PATCH v2 05/11] migration: omit drive ref as we have bdrv_ref now, Fam Zheng, 2013/07/17
- [Qemu-devel] [PATCH v2 06/11] xen_disk: simplify blk_disconnect with refcnt, Fam Zheng, 2013/07/17
- [Qemu-devel] [PATCH v2 07/11] block: hold hard reference for backup/mirror target, Fam Zheng, 2013/07/17
- [Qemu-devel] [PATCH v2 08/11] block: simplify bdrv_drop_intermediate, Fam Zheng, 2013/07/17
- [Qemu-devel] [PATCH v2 09/11] block: add assertion to check refcount before deleting, Fam Zheng, 2013/07/17
- [Qemu-devel] [PATCH v2 10/11] block: add option 'backing' to -drive options, Fam Zheng, 2013/07/17