Hi

I'm testing this patch series, but it has conflict with latest code.
Could you please rebase to latest code and provide a V2?
Thanks.

BR,
Fangge Jin

On Tue, May 10, 2022 at 11:22 PM Jiri Denemark <jdenemar@redhat.com> wrote:
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
---
 src/qemu/qemu_process.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index b0b00eb0a2..1925559fad 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -3445,6 +3445,9 @@ qemuProcessRecoverMigrationIn(virQEMUDriver *driver,
                     (state == VIR_DOMAIN_RUNNING &&
                      reason == VIR_DOMAIN_RUNNING_POSTCOPY);

+    VIR_DEBUG("Active incoming migration in phase %s",
+              qemuMigrationJobPhaseTypeToString(job->phase));
+
     switch ((qemuMigrationJobPhase) job->phase) {
     case QEMU_MIGRATION_PHASE_NONE:
     case QEMU_MIGRATION_PHASE_PERFORM2:
@@ -3506,6 +3509,9 @@ qemuProcessRecoverMigrationOut(virQEMUDriver *driver,
                      reason == VIR_DOMAIN_PAUSED_POSTCOPY_FAILED);
     bool resume = false;

+    VIR_DEBUG("Active outgoing migration in phase %s",
+              qemuMigrationJobPhaseTypeToString(job->phase));
+
     switch ((qemuMigrationJobPhase) job->phase) {
     case QEMU_MIGRATION_PHASE_NONE:
     case QEMU_MIGRATION_PHASE_PREPARE:
@@ -3601,6 +3607,13 @@ qemuProcessRecoverJob(virQEMUDriver *driver,

     state = virDomainObjGetState(vm, &reason);

+    VIR_DEBUG("Recovering job for domain %s, state=%s(%s), async=%s, job=%s",
+              vm->def->name,
+              virDomainStateTypeToString(state),
+              virDomainStateReasonToString(state, reason),
+              virDomainAsyncJobTypeToString(job->asyncJob),
+              virDomainJobTypeToString(job->active));
+
     switch (job->asyncJob) {
     case VIR_ASYNC_JOB_MIGRATION_OUT:
         if (qemuProcessRecoverMigrationOut(driver, vm, job,
--
2.35.1