From 5e12641e0ea49fa1f0b4338f999ec63f41efec4c Mon Sep 17 00:00:00 2001 From: Laine Stump Date: Tue, 3 Dec 2013 12:40:38 +0200 Subject: [PATCH] qemu: report error on attempt to live change virtio-net queues This resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1029732 The BZ asked for the capability to change the number of queues used by a virtio-net device while the device is in use. Because the number of queues can only be set at the time the device is created, that isn't possible. However, libvirt also shouldn't be silently reporting success when someone tries to change the number of queues. So this patch flags that as an error (just as attempts to change any of the other virtio-specific parameters already do). --- src/qemu/qemu_hotplug.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c index 649c884445..c136232eaf 100644 --- a/src/qemu/qemu_hotplug.c +++ b/src/qemu/qemu_hotplug.c @@ -1899,7 +1899,8 @@ qemuDomainChangeNet(virQEMUDriverPtr driver, (olddev->driver.virtio.name != newdev->driver.virtio.name || olddev->driver.virtio.txmode != newdev->driver.virtio.txmode || olddev->driver.virtio.ioeventfd != newdev->driver.virtio.ioeventfd || - olddev->driver.virtio.event_idx != newdev->driver.virtio.event_idx)) { + olddev->driver.virtio.event_idx != newdev->driver.virtio.event_idx || + olddev->driver.virtio.queues != newdev->driver.virtio.queues)) { virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s", _("cannot modify virtio network device driver attributes")); goto cleanup; -- GitLab