diff options
| author | Peter Xu <peterx@redhat.com> | 2022-10-11 17:55:48 -0400 |
|---|---|---|
| committer | Juan Quintela <quintela@redhat.com> | 2022-12-15 10:30:37 +0100 |
| commit | 10661f118048265df33cc30597ecb346c0f092c1 (patch) | |
| tree | 644380b77dd21be76cf164f0e97d1141ee4f3272 /migration | |
| parent | ef5c3d13916f04176ed2d6d8405fb678e50b043f (diff) | |
| download | qemu-10661f118048265df33cc30597ecb346c0f092c1.zip qemu-10661f118048265df33cc30597ecb346c0f092c1.tar.gz qemu-10661f118048265df33cc30597ecb346c0f092c1.tar.bz2 | |
migration: Trivial cleanup save_page_header() on same block check
The 2nd check on RAM_SAVE_FLAG_CONTINUE is a bit redundant. Use a boolean
to be clearer.
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
Diffstat (limited to 'migration')
| -rw-r--r-- | migration/ram.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/migration/ram.c b/migration/ram.c index 7124ff5..4147543 100644 --- a/migration/ram.c +++ b/migration/ram.c @@ -661,14 +661,15 @@ static size_t save_page_header(RAMState *rs, QEMUFile *f, RAMBlock *block, ram_addr_t offset) { size_t size, len; + bool same_block = (block == rs->last_sent_block); - if (block == rs->last_sent_block) { + if (same_block) { offset |= RAM_SAVE_FLAG_CONTINUE; } qemu_put_be64(f, offset); size = 8; - if (!(offset & RAM_SAVE_FLAG_CONTINUE)) { + if (!same_block) { len = strlen(block->idstr); qemu_put_byte(f, len); qemu_put_buffer(f, (uint8_t *)block->idstr, len); |