[libvirt] [PATCH] qemu: qemuDomainChangeNet: don't overwrite device info when pci addr is missing

[Ján Tomko]

On Tue, Jul 31, 2018 at 04:34:39PM +0200, Katerina Koukiou wrote:
>When trying to update an interface's rom settings with an device XML
>that is missing the PCI addr element, all new rom settings where not applied.
>
>https://bugzilla.redhat.com/show_bug.cgi?id=1599513
>
>Signed-off-by: Katerina Koukiou <kkoukiou at redhat.com>
>---
>Not sure why we chose to overwrite the whole info before though, I hope
>that this doesn't cause side problems.
>

It allows the user to omit parts of the XML that do not need changing.

By dropping the virDomainDeviceInfoCopy call, we start validating the
attributes that weren't provided in the XML.

If we want to keep this convenience functionality, we should also autofill
other fields of virDomainDeviceInfo which weren't provided in the XML.

>
> src/qemu/qemu_hotplug.c | 13 +++++--------
> 1 file changed, 5 insertions(+), 8 deletions(-)
>
>diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c
>index 1488f0a7c2..f45192b1d3 100644
>--- a/src/qemu/qemu_hotplug.c
>+++ b/src/qemu/qemu_hotplug.c
>@@ -3445,17 +3445,14 @@ qemuDomainChangeNet(virQEMUDriverPtr driver,
>         goto cleanup;
>     }
>
>-    /* info: if newdev->info is empty, fill it in from olddev,
>-     * otherwise verify that it matches - nothing is allowed to
>-     * change. (There is no helper function to do this, so
>-     * individually check the few feidls of virDomainDeviceInfo that
>-     * are relevant in this case).
>+    /* info: if newdev->info.addr.pci is empty, fill it in from olddev,
>+     * otherwise verify that it matches.
>      */
>     if (!virDomainDeviceAddressIsValid(&newdev->info,
>-                                       VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI) &&
>-        virDomainDeviceInfoCopy(&newdev->info, &olddev->info) < 0) {
>-        goto cleanup;
>+                                       VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI)) {
>+        newdev->info.addr.pci = olddev->info.addr.pci;
>     }
>+

Unrelated whitespace change.

Jano

>     if (!virPCIDeviceAddressEqual(&olddev->info.addr.pci,
>                                   &newdev->info.addr.pci)) {
>         virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s",
>-- 
>2.17.1
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180731/e8bf6d58/attachment-0001.sig>