Eric Blake <ebl...@redhat.com> wrote: > On 12/17/2013 08:25 AM, Juan Quintela wrote: >> Move index and size fields from int to long. We need that for >> migration. long is 64 bits on sane architectures, and 32bits should >> be enough on all the 32bits architectures. >> >> Signed-off-by: Juan Quintela <quint...@redhat.com> >> --- >> include/qemu/bitmap.h | 77 >> ++++++++++++++++++++++++++------------------------- >> include/qemu/bitops.h | 14 +++++----- >> util/bitmap.c | 60 +++++++++++++++++++-------------------- >> 3 files changed, 76 insertions(+), 75 deletions(-) >> >> diff --git a/include/qemu/bitmap.h b/include/qemu/bitmap.h >> index 308bbb7..afdd257 100644 >> --- a/include/qemu/bitmap.h >> +++ b/include/qemu/bitmap.h >> @@ -31,7 +31,7 @@ >> * bitmap_andnot(dst, src1, src2, nbits) *dst = *src1 & ~(*src2) >> * bitmap_complement(dst, src, nbits) *dst = ~(*src) >> * bitmap_equal(src1, src2, nbits) Are *src1 and *src2 equal? >> - * bitmap_intersects(src1, src2, nbits) Do *src1 and *src2 overlap? >> + * bitmap_intersects(src1, src2, nbits) Do *src1 and *src2 overlap? > > Spurious whitespace change? > > Worth cleaning up this whitespace while in the area?
Yeap. After passing checkpatch, it complained about lots of lines. And after that only three tabs or so were left, so I fixed them. And forget to fix the comment. > > Content changes seem sane to me: > Reviewed-by: Eric Blake <ebl...@redhat.com> Thanks, Juan.
