[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 04/15] migration: Trivial cleanup save_page_header() on same b
From: |
Peter Xu |
Subject: |
[PATCH v2 04/15] migration: Trivial cleanup save_page_header() on same block check |
Date: |
Tue, 11 Oct 2022 17:55:48 -0400 |
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>
---
migration/ram.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 562646609e..c90feedb13 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);
--
2.37.3
[PATCH v2 10/15] migration: Add pss_init(), Peter Xu, 2022/10/11
[PATCH v2 05/15] migration: Remove RAMState.f references in compression code, Peter Xu, 2022/10/11
[PATCH v2 11/15] migration: Make PageSearchStatus part of RAMState, Peter Xu, 2022/10/11
[PATCH v2 08/15] migration: Teach PSS about host page, Peter Xu, 2022/10/11
[PATCH v2 15/15] migration: Drop rs->f, Peter Xu, 2022/10/11