mirror of
https://github.com/izzy2lost/xemu.git
synced 2026-07-06 00:20:22 -07:00
migration/ram: One less indent for ram_find_and_save_block()
The check over PAGE_DIRTY_FOUND isn't necessary. We could indent one less and assert that instead. Reviewed-by: Juraj Marcin <jmarcin@redhat.com> Link: https://lore.kernel.org/r/20250613140801.474264-9-peterx@redhat.com Signed-off-by: Peter Xu <peterx@redhat.com> Signed-off-by: Fabiano Rosas <farosas@suse.de>
This commit is contained in:
+11
-9
@@ -2286,16 +2286,18 @@ static int ram_find_and_save_block(RAMState *rs)
|
||||
if (!get_queued_page(rs, pss)) {
|
||||
/* priority queue empty, so just search for something dirty */
|
||||
int res = find_dirty_block(rs, pss);
|
||||
if (res != PAGE_DIRTY_FOUND) {
|
||||
if (res == PAGE_ALL_CLEAN) {
|
||||
break;
|
||||
} else if (res == PAGE_TRY_AGAIN) {
|
||||
continue;
|
||||
} else if (res < 0) {
|
||||
pages = res;
|
||||
break;
|
||||
}
|
||||
|
||||
if (res == PAGE_ALL_CLEAN) {
|
||||
break;
|
||||
} else if (res == PAGE_TRY_AGAIN) {
|
||||
continue;
|
||||
} else if (res < 0) {
|
||||
pages = res;
|
||||
break;
|
||||
}
|
||||
|
||||
/* Otherwise we must have a dirty page to move */
|
||||
assert(res == PAGE_DIRTY_FOUND);
|
||||
}
|
||||
pages = ram_save_host_page(rs, pss);
|
||||
if (pages) {
|
||||
|
||||
Reference in New Issue
Block a user