|
 |
3e5111 |
From e88d64a75a55dcd0173ba35b6f100d2ec5fcf312 Mon Sep 17 00:00:00 2001
|
|
|
3e5111 |
Message-Id: <e88d64a75a55dcd0173ba35b6f100d2ec5fcf312@dist-git>
|
|
|
3e5111 |
From: Peter Krempa <pkrempa@redhat.com>
|
|
|
3e5111 |
Date: Wed, 7 Jun 2017 13:31:21 +0200
|
|
|
3e5111 |
Subject: [PATCH] qemu: Conditionally allow block-copy for persistent domains
|
|
|
3e5111 |
|
|
|
3e5111 |
Allow starting the block-copy job for a persistent domain if a user
|
|
|
3e5111 |
declares by using a flag that the job will not be recovered if the VM is
|
|
|
3e5111 |
switched off while the job is active.
|
|
|
3e5111 |
|
|
|
3e5111 |
This allows to use the block-copy job with persistent VMs under the same
|
|
|
3e5111 |
conditions as would apply to transient domains.
|
|
|
3e5111 |
|
|
|
3e5111 |
(cherry picked from commit b7e534c651bfa28c65f7b11661b478613c96598c)
|
|
|
3e5111 |
|
|
|
3e5111 |
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1459113
|
|
|
3e5111 |
|
|
|
3e5111 |
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
|
|
|
3e5111 |
---
|
|
|
3e5111 |
include/libvirt/libvirt-domain.h | 3 +++
|
|
|
3e5111 |
src/libvirt-domain.c | 4 ++++
|
|
|
3e5111 |
src/qemu/qemu_driver.c | 9 ++++++---
|
|
|
3e5111 |
3 files changed, 13 insertions(+), 3 deletions(-)
|
|
|
3e5111 |
|
|
|
3e5111 |
diff --git a/include/libvirt/libvirt-domain.h b/include/libvirt/libvirt-domain.h
|
|
|
3e5111 |
index 720db32f76..45f939a8cc 100644
|
|
|
3e5111 |
--- a/include/libvirt/libvirt-domain.h
|
|
|
3e5111 |
+++ b/include/libvirt/libvirt-domain.h
|
|
|
3e5111 |
@@ -2428,6 +2428,9 @@ typedef enum {
|
|
|
3e5111 |
backing chain */
|
|
|
3e5111 |
VIR_DOMAIN_BLOCK_COPY_REUSE_EXT = 1 << 1, /* Reuse existing external
|
|
|
3e5111 |
file for a copy */
|
|
|
3e5111 |
+ VIR_DOMAIN_BLOCK_COPY_TRANSIENT_JOB = 1 << 2, /* Don't force usage of
|
|
|
3e5111 |
+ recoverable job for the
|
|
|
3e5111 |
+ copy operation */
|
|
|
3e5111 |
} virDomainBlockCopyFlags;
|
|
|
3e5111 |
|
|
|
3e5111 |
/**
|
|
|
3e5111 |
diff --git a/src/libvirt-domain.c b/src/libvirt-domain.c
|
|
|
3e5111 |
index 4670c54e5e..0adc9e8cbe 100644
|
|
|
3e5111 |
--- a/src/libvirt-domain.c
|
|
|
3e5111 |
+++ b/src/libvirt-domain.c
|
|
|
3e5111 |
@@ -10026,6 +10026,10 @@ virDomainBlockRebase(virDomainPtr dom, const char *disk,
|
|
|
3e5111 |
* or virDomainDetachDevice(), while a copy job is active; they may
|
|
|
3e5111 |
* also restrict a copy job to transient domains.
|
|
|
3e5111 |
*
|
|
|
3e5111 |
+ * If @flags contains VIR_DOMAIN_BLOCK_COPY_TRANSIENT_JOB the job will not be
|
|
|
3e5111 |
+ * recoverable if the VM is turned off while job is active. This flag will
|
|
|
3e5111 |
+ * remove the restriction of copy jobs to transient domains.
|
|
|
3e5111 |
+ *
|
|
|
3e5111 |
* The @disk parameter is either an unambiguous source name of the
|
|
|
3e5111 |
* block device (the <source file='...'/> sub-element, such as
|
|
|
3e5111 |
* "/path/to/image"), or the device target shorthand (the
|
|
|
3e5111 |
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
|
|
|
3e5111 |
index a9ac06a63c..354329fe64 100644
|
|
|
3e5111 |
--- a/src/qemu/qemu_driver.c
|
|
|
3e5111 |
+++ b/src/qemu/qemu_driver.c
|
|
|
3e5111 |
@@ -16509,7 +16509,8 @@ qemuDomainBlockCopyCommon(virDomainObjPtr vm,
|
|
|
3e5111 |
|
|
|
3e5111 |
/* Preliminaries: find the disk we are editing, sanity checks */
|
|
|
3e5111 |
virCheckFlags(VIR_DOMAIN_BLOCK_COPY_SHALLOW |
|
|
|
3e5111 |
- VIR_DOMAIN_BLOCK_COPY_REUSE_EXT, -1);
|
|
|
3e5111 |
+ VIR_DOMAIN_BLOCK_COPY_REUSE_EXT |
|
|
|
3e5111 |
+ VIR_DOMAIN_BLOCK_COPY_TRANSIENT_JOB, -1);
|
|
|
3e5111 |
|
|
|
3e5111 |
priv = vm->privateData;
|
|
|
3e5111 |
cfg = virQEMUDriverGetConfig(driver);
|
|
|
3e5111 |
@@ -16548,7 +16549,8 @@ qemuDomainBlockCopyCommon(virDomainObjPtr vm,
|
|
|
3e5111 |
_("block copy is not supported with this QEMU binary"));
|
|
|
3e5111 |
goto endjob;
|
|
|
3e5111 |
}
|
|
|
3e5111 |
- if (vm->persistent) {
|
|
|
3e5111 |
+ if (!(flags & VIR_DOMAIN_BLOCK_COPY_TRANSIENT_JOB) &&
|
|
|
3e5111 |
+ vm->persistent) {
|
|
|
3e5111 |
/* XXX if qemu ever lets us start a new domain with mirroring
|
|
|
3e5111 |
* already active, we can relax this; but for now, the risk of
|
|
|
3e5111 |
* 'managedsave' due to libvirt-guests means we can't risk
|
|
|
3e5111 |
@@ -16775,7 +16777,8 @@ qemuDomainBlockCopy(virDomainPtr dom, const char *disk, const char *destxml,
|
|
|
3e5111 |
size_t i;
|
|
|
3e5111 |
|
|
|
3e5111 |
virCheckFlags(VIR_DOMAIN_BLOCK_COPY_SHALLOW |
|
|
|
3e5111 |
- VIR_DOMAIN_BLOCK_COPY_REUSE_EXT, -1);
|
|
|
3e5111 |
+ VIR_DOMAIN_BLOCK_COPY_REUSE_EXT |
|
|
|
3e5111 |
+ VIR_DOMAIN_BLOCK_COPY_TRANSIENT_JOB, -1);
|
|
|
3e5111 |
if (virTypedParamsValidate(params, nparams,
|
|
|
3e5111 |
VIR_DOMAIN_BLOCK_COPY_BANDWIDTH,
|
|
|
3e5111 |
VIR_TYPED_PARAM_ULLONG,
|
|
|
3e5111 |
--
|
|
|
3e5111 |
2.13.1
|
|
|
3e5111 |
|