[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 11/18] block/backup: upgrade copy_bitmap to B
Re: [Qemu-block] [PATCH v2 11/18] block/backup: upgrade copy_bitmap to BdrvDirtyBitmap
Mon, 8 Jul 2019 14:02:48 +0200
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0
On 05.07.19 18:52, John Snow wrote:
> On 7/4/19 1:29 PM, Max Reitz wrote:
>> Which brings me to a question: Why is the copy bitmap assigned to the
>> target in the first place? Wouldn’t it make more sense on the source?
> the idea was that the target is *most likely* to be the temporary node
> created for the purpose of this backup, even though backup does not
> explicitly create the node.
> So ... by creating it on the target, we avoid cluttering up the results
> in block query; and otherwise it doesn't actually matter what node we
> created it on, really.
> I can move it over to the source, but the testing code has to get a
> little smarter in order to find the "right" anonymous bitmap, which is
> not impossible; but I thought this would actually be a convenience for
You didn’t really say whether you think it makes more sense on the
source or on the target.
This is an internal bitmap, so from my understanding, the user better
not sees it at all. It should be easy for them to ignore it, regardless
of which node it is on. (I don’t consider “clutter” a strong point,
because, well, most of our current query interfaces are nothing but
clutter.) Sure, that makes it a bit difficult for testing, but testing
shouldn’t really be the focus.
So for me, it comes down to where it makes more sense. And I think it
makes more sense on the source, because it flags source clusters to copy.
Description: OpenPGP digital signature
[Qemu-block] [PATCH v2 10/18] block/dirty-bitmap: add bdrv_dirty_bitmap_get, John Snow, 2019/07/03
[Qemu-block] [PATCH v2 11/18] block/backup: upgrade copy_bitmap to BdrvDirtyBitmap, John Snow, 2019/07/03
[Qemu-block] [PATCH v2 12/18] block/backup: add 'always' bitmap sync policy, John Snow, 2019/07/03
[Qemu-block] [PATCH v2 13/18] iotests: add testing shim for script-style python tests, John Snow, 2019/07/03
[Qemu-block] [PATCH v2 14/18] iotests: teach run_job to cancel pending jobs, John Snow, 2019/07/03
- Re: [Qemu-block] [PATCH v2 09/18] block/dirty-bitmap: add bdrv_dirty_bitmap_merge_internal, (continued)