On Wed, Jan 04, 2017 at 02:15:51PM +0100, Peter Krempa wrote:
> Commit 4b951d1e38259ff5d03e9eedb65095eead8099e1 missed the fact that the
> VM needs to be resumed after a live external checkpoint (memory
> snapshot) where the cpus would be paused by the migration rather than
> libvirt.
> ---
> src/qemu/qemu_driver.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index b359e7757..675a4d0e7 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -14503,6 +14503,9 @@ qemuDomainSnapshotCreateActiveExternal(virConnectPtr conn,
> * atomic flag now says whether we need to pause, and a
> * capability bit says whether to use transaction.
> */
> + if (memory)
> + resume = true;
> +
> if ((memory && !(flags & VIR_DOMAIN_SNAPSHOT_CREATE_LIVE)) ||
> (!memory && atomic && !transaction)) {
> if (qemuProcessStopCPUs(driver, vm, VIR_DOMAIN_PAUSED_SNAPSHOT,
ACK
Pavel
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
