mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-03-07 17:28:15 +00:00
qemu: Separate protocol checks from qemuMigrationJobSetPhase
Into a new qemuMigrationCheckPhase helper, which can be reused in other places. Signed-off-by: Jiri Denemark <jdenemar@redhat.com> Reviewed-by: Peter Krempa <pkrempa@redhat.com> Reviewed-by: Pavel Hrdina <phrdina@redhat.com>
This commit is contained in:
parent
6efd671a11
commit
a5dc9bf96b
@ -139,9 +139,9 @@ qemuMigrationJobStart(virQEMUDriver *driver,
|
||||
}
|
||||
|
||||
|
||||
static void ATTRIBUTE_NONNULL(1)
|
||||
qemuMigrationJobSetPhase(virDomainObj *vm,
|
||||
qemuMigrationJobPhase phase)
|
||||
static int
|
||||
qemuMigrationCheckPhase(virDomainObj *vm,
|
||||
qemuMigrationJobPhase phase)
|
||||
{
|
||||
qemuDomainObjPrivate *priv = vm->privateData;
|
||||
|
||||
@ -150,9 +150,20 @@ qemuMigrationJobSetPhase(virDomainObj *vm,
|
||||
VIR_ERROR(_("migration protocol going backwards %s => %s"),
|
||||
qemuMigrationJobPhaseTypeToString(priv->job.phase),
|
||||
qemuMigrationJobPhaseTypeToString(phase));
|
||||
return;
|
||||
return -1;
|
||||
}
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
||||
static void ATTRIBUTE_NONNULL(1)
|
||||
qemuMigrationJobSetPhase(virDomainObj *vm,
|
||||
qemuMigrationJobPhase phase)
|
||||
{
|
||||
if (qemuMigrationCheckPhase(vm, phase) < 0)
|
||||
return;
|
||||
|
||||
qemuDomainObjSetJobPhase(vm, phase);
|
||||
}
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user