[libvirt] [PATCH] qemu: snapshot: restart CPUs when recover from interrupted snapshot job

[Jiri Denemark]

On Thu, Jan 05, 2017 at 09:34:17 +0000, Wangjing (King, Euler) wrote:
> If we restart libvirtd while VM was doing external memory snapshot, VM's
> state be updated to paused as a result of running a migration-to-file
> operation, and then VM will be left as paused state. In this case we must
> restart the VM's CPUs to resume it.
> 
> Signed-off-by: Wang King <king.wang at huawei.com>
> ---
>  src/qemu/qemu_process.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> index afe3cac..7d283fb 100644
> --- a/src/qemu/qemu_process.c
> +++ b/src/qemu/qemu_process.c
> @@ -3093,7 +3093,8 @@ qemuProcessRecoverJob(virQEMUDriverPtr driver,
>                (job->asyncJob == QEMU_ASYNC_JOB_SAVE &&
>                 reason == VIR_DOMAIN_PAUSED_SAVE) ||
>                (job->asyncJob == QEMU_ASYNC_JOB_SNAPSHOT &&
> -               reason == VIR_DOMAIN_PAUSED_SNAPSHOT) ||
> +               (reason == VIR_DOMAIN_PAUSED_SNAPSHOT ||
> +                reason == VIR_DOMAIN_PAUSED_MIGRATION)) ||
>                reason == VIR_DOMAIN_PAUSED_UNKNOWN)) {
>               if (qemuProcessStartCPUs(driver, vm, conn,
>                                        VIR_DOMAIN_RUNNING_UNPAUSED,

ACK and pushed. Thanks.

Jirka