[libvirt PATCH 01/80] qemu: Add debug messages to job recovery code

Fangge Jin fjin at redhat.com
Tue May 17 03:59:42 UTC 2022


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 at redhat.com> wrote:

> Signed-off-by: Jiri Denemark <jdenemar at 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20220517/e7b878f5/attachment-0001.htm>


More information about the libvir-list mailing list