Re: [Qemu-devel] [PATCH v20 09/30] block/dirty-bitmap: fix comment for BlockDirtyBitmap.disabled field

[Max Reitz <mreitz@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1530 bytes --]

On 2017-06-02 23:02, John Snow wrote:
> 
> 
> On 06/02/2017 07:21 AM, Vladimir Sementsov-Ogievskiy wrote:
>> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
>> ---
>>  block/dirty-bitmap.c | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/block/dirty-bitmap.c b/block/dirty-bitmap.c
>> index 90af37287f..f25428868c 100644
>> --- a/block/dirty-bitmap.c
>> +++ b/block/dirty-bitmap.c
>> @@ -42,7 +42,8 @@ struct BdrvDirtyBitmap {
>>      BdrvDirtyBitmap *successor; /* Anonymous child; implies frozen status */
>>      char *name;                 /* Optional non-empty unique ID */
>>      int64_t size;               /* Size of the bitmap (Number of sectors) */
>> -    bool disabled;              /* Bitmap is read-only */
>> +    bool disabled;              /* Bitmap is disabled. It skips all writes to
>> +                                   the device */
> 
> 
> Or, "Bitmap is disabled. Writes to the device are ignored." or similar.

I'd like "It ignores all writes to the device" the most. :-)

("Writes are ignored" sounds a bit like they are ignored in general, not
just by this bitmap; and "it skips" just sounds a bit weird.)

> It's not very important.

Discussing the non-important stuff is the most fun!

Reviewed-by: Max Reitz <mreitz@redhat.com>

> Reviewed-by: John Snow <jsnow@redhat.com>
> 
>>      int active_iterators;       /* How many iterators are active */
>>      QLIST_ENTRY(BdrvDirtyBitmap) list;
>>  };
>>



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 498 bytes --]