qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] block migration and dirty bitmap reset


From: Fam Zheng
Subject: Re: [Qemu-block] block migration and dirty bitmap reset
Date: Thu, 8 Mar 2018 09:28:42 +0800
User-agent: Mutt/1.9.2 (2017-12-15)

On Wed, 03/07 09:06, Peter Lieven wrote:
> Hi,
> 
> while looking at the code I wonder if the blk_aio_preadv and the 
> bdrv_reset_dirty_bitmap order must
> be swapped in mig_save_device_bulk:
> 
>     qemu_mutex_lock_iothread();
>     aio_context_acquire(blk_get_aio_context(bmds->blk));
>     blk->aiocb = blk_aio_preadv(bb, cur_sector * BDRV_SECTOR_SIZE, &blk->qiov,
>                                 0, blk_mig_read_cb, blk);
> 
>     bdrv_reset_dirty_bitmap(bmds->dirty_bitmap, cur_sector * BDRV_SECTOR_SIZE,
>                             nr_sectors * BDRV_SECTOR_SIZE);
>     aio_context_release(blk_get_aio_context(bmds->blk));
>     qemu_mutex_unlock_iothread();
> 
> In mig_save_device_dirty we first reset the dirty bitmap and read then which 
> shoulds like
> a better idea.

Yes, that sounds right to me.

Fam



reply via email to

[Prev in Thread] Current Thread [Next in Thread]