[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] block/dirty-bitmap: Documentation and Comment f
Re: [Qemu-devel] [PATCH] block/dirty-bitmap: Documentation and Comment fixups
Mon, 11 Feb 2019 18:20:18 +0000
02.02.2019 4:01, John Snow wrote:
> On 1/31/19 3:29 AM, Vladimir Sementsov-Ogievskiy wrote:
>> About @frozen and @locked, we can also note that they can't be exported
>> through NBD.
>> We can summarize, that @frozen and @locked means that user can't use them in
>> command, and the only option is to list them. However even info->count
>> should not be considered as something meaningful, as bitmaps are under some
>> And we can also use same paragraph for @frozen and @locked, as a first step
>> to @frozen
> There's a subtle difference in the two, namely that:
> (1) Frozen may or may not record new writes, but they don't "show up"
> until after the operation is over, because it's using a second bitmap as
> a temporary buffer.
> (2) Locked may or may not record new writes *immediately*.
> Regardless, I'm realizing that for both Frozen and Locked bitmaps we
> want to allow users to know if the bitmap is recording or not. (And
> possibly if there is a temporary write cache or not, but maybe it's not
> Maybe we want two new fields on query:
> Recording/Enabled: [True | False]
> Busy/Locked: [True | False] (or "Locked)
> which will then deprecate the status field. This doesn't manage to
> communicate the write cache nuance, but maybe we don't need to.
> (It also doesn't help elucidate whether or not the successor is
> active/disabled, but even for migration once the guest has started, the
> successors are always enabled, right?)
Hmmm, it's all really questionable.
What we want to exporot about bitmaps?
A. For not-locked, not-frozen:
It's obvious enough: we want to say that bitmap is not locked and available for
any operation, and all information we have, is it disabled or not, dirty count,
So we have two possible states here:
B. For locked of frozen:
Firsty, dirty-count seems not useful and not reliable.
Disabled/enabled.. what it means for locked bitmaps?
1. for frozen (used only in backups), enabled means, that writes are tracked in
of this bitmaps, and after operation finish information about dirtiness will be
somehow. But details about reclaim/abdicate are related to the operation, not
to the bitmap
2. for locked bitmap, again, enabled means that it tracks dirtiness.
So, yes, [recording, not-recording] x [busy, not-busy] works. And we may
recording for busy case like "guest writes are tracked in context of the
availability of tracked information as well as bitmap status after holding
depends on the operation itself"