Re: [Qemu-devel] [PATCH 3/3] COW: Skip setting already set bits

[Charlie Shepherd]

On 06/11/2013 12:29, Paolo Bonzini wrote:
> Il 06/11/2013 13:23, Charlie Shepherd ha scritto:
> > +        set = cow_find_streak(bitmap, 1, bitnum, sector_bits);
> > +        if (set == sector_bits) {
> > +            continue;
> I think this shouldn't be a continue; these lines should be executed:
>
>          bitnum += sector_bits;
>          nb_sectors -= sector_bits;
>          offset += BDRV_SECTOR_SIZE;

Good point, this is basically a poor man's for-loop. I'll turn it into a 
for loop then continue will make sense here.

> > +        }
> > +        bitnum += set;
> Here you're adjusting bitnum but not nb_sectors and sector_bits.

Good catch.

Charlie