migration/block-dirty-bitmap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Noticed while reviewing the file for newer patches.
Fixes: b35ebdf076
Signed-off-by: Eric Blake <eblake@redhat.com>
---
This is trivial enough that I'll throw it in my pull request today.
migration/block-dirty-bitmap.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/migration/block-dirty-bitmap.c b/migration/block-dirty-bitmap.c
index 1f675b792fc9..784330ebe130 100644
--- a/migration/block-dirty-bitmap.c
+++ b/migration/block-dirty-bitmap.c
@@ -97,7 +97,7 @@
#define DIRTY_BITMAP_MIG_START_FLAG_ENABLED 0x01
#define DIRTY_BITMAP_MIG_START_FLAG_PERSISTENT 0x02
-/* 0x04 was "AUTOLOAD" flags on elder versions, no it is ignored */
+/* 0x04 was "AUTOLOAD" flags on older versions, now it is ignored */
#define DIRTY_BITMAP_MIG_START_FLAG_RESERVED_MASK 0xf8
/* State of one bitmap during save process */
@@ -180,7 +180,7 @@ static uint32_t qemu_get_bitmap_flags(QEMUFile *f)
static void qemu_put_bitmap_flags(QEMUFile *f, uint32_t flags)
{
- /* The code currently do not send flags more than one byte */
+ /* The code currently does not send flags as more than one byte */
assert(!(flags & (0xffffff00 | DIRTY_BITMAP_MIG_EXTRA_FLAGS)));
qemu_put_byte(f, flags);
--
2.27.0
27.07.2020 23:32, Eric Blake wrote: > Noticed while reviewing the file for newer patches. > > Fixes: b35ebdf076 > Signed-off-by: Eric Blake <eblake@redhat.com> > --- > > This is trivial enough that I'll throw it in my pull request today. > > migration/block-dirty-bitmap.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/migration/block-dirty-bitmap.c b/migration/block-dirty-bitmap.c > index 1f675b792fc9..784330ebe130 100644 > --- a/migration/block-dirty-bitmap.c > +++ b/migration/block-dirty-bitmap.c > @@ -97,7 +97,7 @@ > > #define DIRTY_BITMAP_MIG_START_FLAG_ENABLED 0x01 > #define DIRTY_BITMAP_MIG_START_FLAG_PERSISTENT 0x02 > -/* 0x04 was "AUTOLOAD" flags on elder versions, no it is ignored */ > +/* 0x04 was "AUTOLOAD" flags on older versions, now it is ignored */ may be also s/flags/flag > #define DIRTY_BITMAP_MIG_START_FLAG_RESERVED_MASK 0xf8 > > /* State of one bitmap during save process */ > @@ -180,7 +180,7 @@ static uint32_t qemu_get_bitmap_flags(QEMUFile *f) > > static void qemu_put_bitmap_flags(QEMUFile *f, uint32_t flags) > { > - /* The code currently do not send flags more than one byte */ > + /* The code currently does not send flags as more than one byte */ Hmm, why "as more than", not just "more than"?. (this note is about the following: the protocol allows adding more than one byte of flags with use of DIRTY_BITMAP_MIG_EXTRA_FLAGS. Still, currently this possibility is not used and we assert it.) > assert(!(flags & (0xffffff00 | DIRTY_BITMAP_MIG_EXTRA_FLAGS))); > > qemu_put_byte(f, flags); > Anyway: Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> -- Best regards, Vladimir
© 2016 - 2024 Red Hat, Inc.