[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 30/30] migration: Block migration comment or code is wrong
From: |
Juan Quintela |
Subject: |
[PATCH 30/30] migration: Block migration comment or code is wrong |
Date: |
Tue, 15 Nov 2022 13:12:26 +0100 |
And it appears that what is wrong is the code. During bulk stage we
need to make sure that some block is dirty, but no games with
max_size at all.
Signed-off-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
---
migration/block.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/migration/block.c b/migration/block.c
index 3577c815a9..4347da1526 100644
--- a/migration/block.c
+++ b/migration/block.c
@@ -880,8 +880,8 @@ static void block_save_pending(QEMUFile *f, void *opaque,
uint64_t max_size,
blk_mig_unlock();
/* Report at least one block pending during bulk phase */
- if (pending <= max_size && !block_mig_state.bulk_completed) {
- pending = max_size + BLK_MIG_BLOCK_SIZE;
+ if (!pending && !block_mig_state.bulk_completed) {
+ pending = BLK_MIG_BLOCK_SIZE;
}
trace_migration_block_save_pending(pending);
--
2.38.1
- [PATCH 20/30] migration: Yield bitmap_mutex properly when sending/sleeping, (continued)
- [PATCH 20/30] migration: Yield bitmap_mutex properly when sending/sleeping, Juan Quintela, 2022/11/15
- [PATCH 22/30] migration: Teach PSS about host page, Juan Quintela, 2022/11/15
- [PATCH 21/30] migration: Use atomic ops properly for page accountings, Juan Quintela, 2022/11/15
- [PATCH 25/30] migration: Make PageSearchStatus part of RAMState, Juan Quintela, 2022/11/15
- [PATCH 23/30] migration: Introduce pss_channel, Juan Quintela, 2022/11/15
- [PATCH 24/30] migration: Add pss_init(), Juan Quintela, 2022/11/15
- [PATCH 27/30] migration: Send requested page directly in rp-return thread, Juan Quintela, 2022/11/15
- [PATCH 28/30] migration: Remove old preempt code around state maintainance, Juan Quintela, 2022/11/15
- [PATCH 26/30] migration: Move last_sent_block into PageSearchStatus, Juan Quintela, 2022/11/15
- [PATCH 29/30] migration: Drop rs->f, Juan Quintela, 2022/11/15
- [PATCH 30/30] migration: Block migration comment or code is wrong,
Juan Quintela <=
- Re: [PATCH 00/30] Migration PULL request, Stefan Hajnoczi, 2022/11/15