Re: [Qemu-devel] [PATCH 0/4] backup: fix skipping unallocated clusters

[John Snow]

On 8/20/19 4:28 AM, Vladimir Sementsov-Ogievskiy wrote:
> 16.08.2019 22:11, John Snow wrote:
>>
>>
>> On 8/14/19 12:54 PM, Vladimir Sementsov-Ogievskiy wrote:
>>>
>>>
>>> 14 авг. 2019 г. 17:43 пользователь Vladimir Sementsov-Ogievskiy
>>> <address@hidden> написал:
>>>
>>>      Hi all!
>>>
>>>      There is a bug in not yet merged patch
>>>      "block/backup: teach TOP to never copy unallocated regions"
>>>      in https://github.com/jnsnow/qemu bitmaps. 04 fixes it. So, I propose
>>>      to put 01-03 somewhere before
>>>      "block/backup: teach TOP to never copy unallocated regions"
>>>      and squash 04 into "block/backup: teach TOP to never copy
>>>      unallocated regions"
>>>
>>>
>>> Hmm, don't bother with it. Simpler is fix the bug in your commit by just
>>> use skip_bytes variable when initializing dirty_end.
>>>
>>
>> OK, just use Max's fix instead of this entire 4 patch series?
>>
>> --js
>>
> 
> Yes, I think it's OK
> 

ACK, thank you!