[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 5/6] Postcopy: Reset state to avoid cleanup assert
From: |
Juan Quintela |
Subject: |
[Qemu-devel] [PULL 5/6] Postcopy: Reset state to avoid cleanup assert |
Date: |
Mon, 6 Feb 2017 17:51:48 +0100 |
From: "Dr. David Alan Gilbert" <address@hidden>
On a destination host with no userfault support an incoming
postcopy would cause the state to enter ADVISE before
it realised there was no support, and because it was in ADVISE
state it would perform a cleanup at the end. Since there
was no support the cleanup function should be unreachable,
but ends up being called and asserting.
Reset the state when we realise we have no support, thus the
cleanup doesn't happen.
Signed-off-by: Dr. David Alan Gilbert <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>
---
migration/savevm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/migration/savevm.c b/migration/savevm.c
index 8b8c74d..0199768 100644
--- a/migration/savevm.c
+++ b/migration/savevm.c
@@ -1355,6 +1355,7 @@ static int
loadvm_postcopy_handle_advise(MigrationIncomingState *mis)
}
if (!postcopy_ram_supported_by_host()) {
+ postcopy_state_set(POSTCOPY_INCOMING_NONE);
return -1;
}
--
2.7.4
- [Qemu-devel] [PULL 0/6] Migration pull, Juan Quintela, 2017/02/06
- [Qemu-devel] [PULL 1/6] migration: create Migration Incoming State at init time, Juan Quintela, 2017/02/06
- [Qemu-devel] [PULL 2/6] vmstate_register_with_alias_id: Take an Error **, Juan Quintela, 2017/02/06
- [Qemu-devel] [PULL 3/6] migration: Check for ID length, Juan Quintela, 2017/02/06
- [Qemu-devel] [PULL 4/6] vmstate registration: check return values, Juan Quintela, 2017/02/06
- [Qemu-devel] [PULL 5/6] Postcopy: Reset state to avoid cleanup assert,
Juan Quintela <=
- [Qemu-devel] [PULL 6/6] postcopy: Recover block devices on early failure, Juan Quintela, 2017/02/06
- Re: [Qemu-devel] [PULL 0/6] Migration pull, Peter Maydell, 2017/02/06