Re: [Qemu-devel] ping2 Re: [PATCH v4 0/8] dirty-bitmap: rewrite bdrv_dirty_iter_next_area

2018-12-13 Thread Denis Lunev
On 12/13/18 9:18 PM, John Snow wrote: > > On 12/13/18 6:07 AM, Vladimir Sementsov-Ogievskiy wrote: >> 12.12.2018 23:41, John Snow wrote: >>> >>> On 12/12/18 4:27 AM, Vladimir Sementsov-Ogievskiy wrote: ping. No dependencies, apply to master. >>> Sure thing. >>> >>> Staged to

Re: [Qemu-devel] ping2 Re: [PATCH v4 0/8] dirty-bitmap: rewrite bdrv_dirty_iter_next_area

2018-12-13 Thread John Snow
On 12/13/18 6:07 AM, Vladimir Sementsov-Ogievskiy wrote: > 12.12.2018 23:41, John Snow wrote: >> >> >> On 12/12/18 4:27 AM, Vladimir Sementsov-Ogievskiy wrote: >>> ping. No dependencies, apply to master. >>> >> >> Sure thing. >> >> Staged to jsnow/bitmaps. > > > Thank you! > > Oops, I've

Re: [Qemu-devel] ping2 Re: [PATCH v4 0/8] dirty-bitmap: rewrite bdrv_dirty_iter_next_area

2018-12-13 Thread Vladimir Sementsov-Ogievskiy
12.12.2018 23:41, John Snow wrote: > > > On 12/12/18 4:27 AM, Vladimir Sementsov-Ogievskiy wrote: >> ping. No dependencies, apply to master. >> > > Sure thing. > > Staged to jsnow/bitmaps. Thank you! Oops, I've missed your email about s/firt_dirty_off/first_dirty_off I can resend, or you

Re: [Qemu-devel] ping2 Re: [PATCH v4 0/8] dirty-bitmap: rewrite bdrv_dirty_iter_next_area

2018-12-12 Thread John Snow
On 12/12/18 4:27 AM, Vladimir Sementsov-Ogievskiy wrote: > ping. No dependencies, apply to master. > Sure thing. Staged to jsnow/bitmaps. --js > 17.09.2018 17:57, Vladimir Sementsov-Ogievskiy wrote: >> Hi all. >> >> 1. bdrv_dirty_iter_next_area don't use hbitmap_next_zero and uses >>

[Qemu-devel] ping2 Re: [PATCH v4 0/8] dirty-bitmap: rewrite bdrv_dirty_iter_next_area

2018-12-12 Thread Vladimir Sementsov-Ogievskiy
ping. No dependencies, apply to master. 17.09.2018 17:57, Vladimir Sementsov-Ogievskiy wrote: > Hi all. > > 1. bdrv_dirty_iter_next_area don't use hbitmap_next_zero and uses > inefficient loop instead. Let's improve it. > > 2. bdrv_dirty_iter_next_area don't handle unaligned offset and >