|
 |
a41c76 |
From ec2f52e9e935136d8517567bbe0e56c6e8c40af1 Mon Sep 17 00:00:00 2001
|
|
|
a41c76 |
Message-Id: <ec2f52e9e935136d8517567bbe0e56c6e8c40af1@dist-git>
|
|
|
a41c76 |
From: Peter Krempa <pkrempa@redhat.com>
|
|
|
a41c76 |
Date: Wed, 19 Feb 2020 15:10:05 +0100
|
|
|
a41c76 |
Subject: [PATCH] qemu: hotplug: Fix handling of the 'copy-on-read' layer with
|
|
|
a41c76 |
blockdev
|
|
|
a41c76 |
MIME-Version: 1.0
|
|
|
a41c76 |
Content-Type: text/plain; charset=UTF-8
|
|
|
a41c76 |
Content-Transfer-Encoding: 8bit
|
|
|
a41c76 |
|
|
|
a41c76 |
My original implementation was completely broken because it attempted to
|
|
|
a41c76 |
use object-add/del instead of blockdev-add/del.
|
|
|
a41c76 |
|
|
|
a41c76 |
https://bugzilla.redhat.com/show_bug.cgi?id=1798366
|
|
|
a41c76 |
|
|
|
a41c76 |
Signed-off-by: Peter Krempa <pkrempa@redhat.com>
|
|
|
a41c76 |
Reviewed-by: Ján Tomko <jtomko@redhat.com>
|
|
|
a41c76 |
Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>
|
|
|
a41c76 |
(cherry picked from commit b71cf8726c25ea135ac81051e332544e6b886ad2)
|
|
|
a41c76 |
|
|
|
a41c76 |
https://bugzilla.redhat.com/show_bug.cgi?id=1798366
|
|
|
a41c76 |
Message-Id: <170358b3722e56fc308e7cf3b0a2625349346ae9.1582120424.git.pkrempa@redhat.com>
|
|
|
a41c76 |
Reviewed-by: Ján Tomko <jtomko@redhat.com>
|
|
|
a41c76 |
---
|
|
|
a41c76 |
src/qemu/qemu_hotplug.c | 25 ++++++++++++++++---------
|
|
|
a41c76 |
1 file changed, 16 insertions(+), 9 deletions(-)
|
|
|
a41c76 |
|
|
|
a41c76 |
diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c
|
|
|
a41c76 |
index 31d455505b..12bc1f9dd5 100644
|
|
|
a41c76 |
--- a/src/qemu/qemu_hotplug.c
|
|
|
a41c76 |
+++ b/src/qemu/qemu_hotplug.c
|
|
|
a41c76 |
@@ -680,6 +680,7 @@ qemuDomainAttachDiskGeneric(virQEMUDriverPtr driver,
|
|
|
a41c76 |
g_autoptr(virQEMUDriverConfig) cfg = virQEMUDriverGetConfig(driver);
|
|
|
a41c76 |
g_autoptr(virJSONValue) corProps = NULL;
|
|
|
a41c76 |
g_autofree char *corAlias = NULL;
|
|
|
a41c76 |
+ bool corAdded = false;
|
|
|
a41c76 |
bool blockdev = virQEMUCapsGet(priv->qemuCaps, QEMU_CAPS_BLOCKDEV);
|
|
|
a41c76 |
|
|
|
a41c76 |
if (qemuDomainStorageSourceChainAccessAllow(driver, vm, disk->src) < 0)
|
|
|
a41c76 |
@@ -692,9 +693,12 @@ qemuDomainAttachDiskGeneric(virQEMUDriverPtr driver,
|
|
|
a41c76 |
goto cleanup;
|
|
|
a41c76 |
|
|
|
a41c76 |
if (blockdev) {
|
|
|
a41c76 |
- if (disk->copy_on_read == VIR_TRISTATE_SWITCH_ON &&
|
|
|
a41c76 |
- !(corProps = qemuBlockStorageGetCopyOnReadProps(disk)))
|
|
|
a41c76 |
- goto cleanup;
|
|
|
a41c76 |
+ if (disk->copy_on_read == VIR_TRISTATE_SWITCH_ON) {
|
|
|
a41c76 |
+ if (!(corProps = qemuBlockStorageGetCopyOnReadProps(disk)))
|
|
|
a41c76 |
+ goto cleanup;
|
|
|
a41c76 |
+
|
|
|
a41c76 |
+ corAlias = g_strdup(QEMU_DOMAIN_DISK_PRIVATE(disk)->nodeCopyOnRead);
|
|
|
a41c76 |
+ }
|
|
|
a41c76 |
|
|
|
a41c76 |
if (!(data = qemuBuildStorageSourceChainAttachPrepareBlockdev(disk->src,
|
|
|
a41c76 |
priv->qemuCaps)))
|
|
|
a41c76 |
@@ -719,9 +723,12 @@ qemuDomainAttachDiskGeneric(virQEMUDriverPtr driver,
|
|
|
a41c76 |
if (qemuBlockStorageSourceChainAttach(priv->mon, data) < 0)
|
|
|
a41c76 |
goto exit_monitor;
|
|
|
a41c76 |
|
|
|
a41c76 |
- if (corProps &&
|
|
|
a41c76 |
- qemuMonitorAddObject(priv->mon, &corProps, &corAlias) < 0)
|
|
|
a41c76 |
- goto exit_monitor;
|
|
|
a41c76 |
+ if (corProps) {
|
|
|
a41c76 |
+ if (qemuMonitorBlockdevAdd(priv->mon, &corProps) < 0)
|
|
|
a41c76 |
+ goto exit_monitor;
|
|
|
a41c76 |
+
|
|
|
a41c76 |
+ corAdded = true;
|
|
|
a41c76 |
+ }
|
|
|
a41c76 |
|
|
|
a41c76 |
if (qemuDomainAttachExtensionDevice(priv->mon, &disk->info) < 0)
|
|
|
a41c76 |
goto exit_monitor;
|
|
|
a41c76 |
@@ -763,8 +770,8 @@ qemuDomainAttachDiskGeneric(virQEMUDriverPtr driver,
|
|
|
a41c76 |
return ret;
|
|
|
a41c76 |
|
|
|
a41c76 |
exit_monitor:
|
|
|
a41c76 |
- if (corAlias)
|
|
|
a41c76 |
- ignore_value(qemuMonitorDelObject(priv->mon, corAlias));
|
|
|
a41c76 |
+ if (corAdded)
|
|
|
a41c76 |
+ ignore_value(qemuMonitorBlockdevDel(priv->mon, corAlias));
|
|
|
a41c76 |
qemuBlockStorageSourceChainDetach(priv->mon, data);
|
|
|
a41c76 |
|
|
|
a41c76 |
if (qemuDomainObjExitMonitor(driver, vm) < 0)
|
|
|
a41c76 |
@@ -4252,7 +4259,7 @@ qemuDomainRemoveDiskDevice(virQEMUDriverPtr driver,
|
|
|
a41c76 |
qemuDomainObjEnterMonitor(driver, vm);
|
|
|
a41c76 |
|
|
|
a41c76 |
if (corAlias)
|
|
|
a41c76 |
- ignore_value(qemuMonitorDelObject(priv->mon, corAlias));
|
|
|
a41c76 |
+ ignore_value(qemuMonitorBlockdevDel(priv->mon, corAlias));
|
|
|
a41c76 |
|
|
|
a41c76 |
if (diskBackend)
|
|
|
a41c76 |
qemuBlockStorageSourceChainDetach(priv->mon, diskBackend);
|
|
|
a41c76 |
--
|
|
|
a41c76 |
2.25.0
|
|
|
a41c76 |
|