From: Wei Yang <richardw.y...@linux.intel.com> In case we gets a queued page, the order of block is interrupted. We may not rely on the complete_round flag to say we have already searched the whole blocks on the list.
Signed-off-by: Wei Yang <richardw.y...@linux.intel.com> Reviewed-by: Juan Quintela <quint...@redhat.com> Message-Id: <20190605010828.6969-1-richardw.y...@linux.intel.com> Signed-off-by: Juan Quintela <quint...@redhat.com> --- migration/ram.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/migration/ram.c b/migration/ram.c index 96c84f770a..89eec7ee9d 100644 --- a/migration/ram.c +++ b/migration/ram.c @@ -2286,6 +2286,12 @@ static bool get_queued_page(RAMState *rs, PageSearchStatus *pss) */ pss->block = block; pss->page = offset >> TARGET_PAGE_BITS; + + /* + * This unqueued page would break the "one round" check, even is + * really rare. + */ + pss->complete_round = false; } return !!block; -- 2.21.0