On 9/11/19 11:00 AM, Vladimir Sementsov-Ogievskiy wrote: > block/dirty-bitmap.c seems to be more appropriate for it and > bdrv_remove_persistent_dirty_bitmap already in it. > > Signed-off-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> No disagreements here. I think we put it in block.c initially because it's a bdrv callback/dispatch function, but dirty-block.c is just a logical subdivision of block.c, so it's fine. Reviewed-by: John Snow <js...@redhat.com>
- [Qemu-devel] [PATCH v2 0/3] proper locking on... Vladimir Sementsov-Ogievskiy
- [Qemu-devel] [PATCH v2 1/3] block: move ... Vladimir Sementsov-Ogievskiy
- Re: [Qemu-devel] [PATCH v2 1/3] bloc... John Snow
- [Qemu-devel] [PATCH v2 2/3] block/dirty-... Vladimir Sementsov-Ogievskiy
- Re: [Qemu-devel] [PATCH v2 2/3] bloc... John Snow
- Re: [Qemu-devel] [PATCH v2 2/3] ... Vladimir Sementsov-Ogievskiy
- [Qemu-devel] [PATCH v2 3/3] block/qcow2:... Vladimir Sementsov-Ogievskiy
- Re: [Qemu-devel] [PATCH v2 3/3] bloc... John Snow