External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-01-22 04:25:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix error propagation in finish method for v3 migration

Browse Source 
Even when failing to start CPUs, the finish method was returning
a success result. Fix this so that the QEMU process is killed
off when finish fails under v3 protocol. Also rename the
killOnFinish boolean to 'v3proto' to make it clearer that this
is a tunable based on the migration protocol version

* src/qemu/qemu_driver.c: Update for API change
* src/qemu/qemu_migration.c, src/qemu/qemu_migration.h: Kill
  VM in qemuMigrationFinish if failing to start CPUs
This commit is contained in:
Daniel P. Berrange 2011-05-23 08:50:11 -04:00
parent b336710383
commit 1853bd9d6e
3 changed files with 39 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/qemu/qemu_driver.c
View File

@ -5981,7 +5981,7 @@ qemudDomainMigratePerform (virDomainPtr dom,
ret = qemuMigrationPerform(driver, dom->conn, vm,
NULL, dconnuri, uri, cookie, cookielen,
NULL, NULL, /* No output cookies in v2 */
flags, dname, resource, true);
flags, dname, resource, false);
cleanup:
qemuDriverUnlock(driver);
@ -6030,7 +6030,7 @@ qemudDomainMigrateFinish2 (virConnectPtr dconn,
*/
dom = qemuMigrationFinish(driver, dconn, vm,
NULL, 0, NULL, NULL, /* No cookies */
flags, retcode);
flags, retcode, false);
cleanup:
if (orig_err) {
@ -6232,7 +6232,7 @@ qemuDomainMigratePerform3(virDomainPtr dom,
ret = qemuMigrationPerform(driver, dom->conn, vm, xmlin,
dconnuri, uri, cookiein, cookieinlen,
cookieout, cookieoutlen,
flags, dname, resource, false);
flags, dname, resource, true);
cleanup:
qemuDriverUnlock(driver);
@ -6281,7 +6281,7 @@ qemuDomainMigrateFinish3(virConnectPtr dconn,
*newdom = qemuMigrationFinish(driver, dconn, vm,
cookiein, cookieinlen,
cookieout, cookieoutlen,
flags, cancelled);
flags, cancelled, true);
ret = 0;

37
src/qemu/qemu_migration.c
View File

@ -2047,7 +2047,7 @@ int qemuMigrationPerform(struct qemud_driver *driver,
unsigned long flags,
const char *dname,
unsigned long resource,
bool killOnFinish)
bool v3proto)
{
virDomainEventPtr event = NULL;
int ret = -1;
@ -2092,8 +2092,13 @@ int qemuMigrationPerform(struct qemud_driver *driver,
goto endjob;
}
/* Clean up the source domain. */
if (killOnFinish) {
/*
* In v3 protocol, the source VM is not killed off until the
* confirm step.
*/
if (v3proto) {
resume = 0;
} else {
qemuProcessStop(driver, vm, 1, VIR_DOMAIN_SHUTOFF_MIGRATED);
qemuAuditDomainStop(vm, "migrated");
resume = 0;
@ -2193,7 +2198,8 @@ qemuMigrationFinish(struct qemud_driver *driver,
char **cookieout,
int *cookieoutlen,
unsigned long flags,
int retcode)
int retcode,
bool v3proto)
{
virDomainPtr dom = NULL;
virDomainEventPtr event = NULL;
@ -2257,7 +2263,6 @@ qemuMigrationFinish(struct qemud_driver *driver,
event = NULL;
}
dom = virGetDomain (dconn, vm->def->name, vm->def->uuid);
if (!(flags & VIR_MIGRATE_PAUSED)) {
/* run 'cont' on the destination, which allows migration on qemu
@ -2266,6 +2271,26 @@ qemuMigrationFinish(struct qemud_driver *driver,
*/
if (qemuProcessStartCPUs(driver, vm, dconn,
VIR_DOMAIN_RUNNING_MIGRATED) < 0) {
/*
* In v3 protocol, the source VM is still available to
* restart during confirm() step, so we kill it off
* now.
* In v2 protocol, the source is dead, so we leave
* target in paused state, in case admin can fix
* things up
*/
if (v3proto) {
qemuProcessStop(driver, vm, 1, VIR_DOMAIN_SHUTOFF_FAILED);
qemuAuditDomainStop(vm, "failed");
event = virDomainEventNewFromObj(vm,
VIR_DOMAIN_EVENT_STOPPED,
VIR_DOMAIN_EVENT_STOPPED_FAILED);
if (!vm->persistent) {
if (qemuDomainObjEndJob(vm) > 0)
virDomainRemoveInactive(&driver->domains, vm);
vm = NULL;
}
}
if (virGetLastError() == NULL)
qemuReportError(VIR_ERR_INTERNAL_ERROR,
"%s", _("resume operation failed"));
@ -2273,6 +2298,8 @@ qemuMigrationFinish(struct qemud_driver *driver,
}
}
dom = virGetDomain (dconn, vm->def->name, vm->def->uuid);
event = virDomainEventNewFromObj(vm,
VIR_DOMAIN_EVENT_RESUMED,
VIR_DOMAIN_EVENT_RESUMED_MIGRATED);

5
src/qemu/qemu_migration.h
View File

@ -72,7 +72,7 @@ int qemuMigrationPerform(struct qemud_driver *driver,
unsigned long flags,
const char *dname,
unsigned long resource,
bool killOnFinish);
bool v3proto);
virDomainPtr qemuMigrationFinish(struct qemud_driver *driver,
virConnectPtr dconn,
@ -82,7 +82,8 @@ virDomainPtr qemuMigrationFinish(struct qemud_driver *driver,
char **cookieout,
int *cookieoutlen,
unsigned long flags,
int retcode);
int retcode,
bool v3proto);
int qemuMigrationConfirm(struct qemud_driver *driver,
virConnectPtr conn,