On Wed, Jun 06, 2018 at 08:21:26PM -0400, John Ferlan wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1583623 When attaching a virtio-scsi with IOThreads for the config of a live domain, allow the <address> to not be defined thus allowing post parse processing to fill in the address. Make the error a bit clearer that virtio-scsi IOThreads require virtio pci or ccw controller address types.
So two different changes.
Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>
---
src/qemu/qemu_domain.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 1fb1ef1deb..97f91ec593 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -4736,11 +4736,12 @@ qemuDomainCheckSCSIControllerIOThreads(const virDomainControllerDef *controller,
if (!controller->iothread)
return true;
- if (controller->info.type != VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI &&
+ if (controller->info.type != VIR_DOMAIN_DEVICE_ADDRESS_TYPE_NONE &&
+ controller->info.type != VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI &&
controller->info.type != VIR_DOMAIN_DEVICE_ADDRESS_TYPE_CCW) {
So this check relies on the assignAddresses callback being called before validation, which is only done after parsing the domain, not parsing the individual device. Also, it looks like this change would let the user bypass the check for S390 "address" type by not specifying an address, but that's a corner case. Maybe the proper fix would be to stop relying on postParse in attach config - especially since sometimes we need to make sure the device is suitable for both the persistent definition and the live definition. But in the meantime, this should do. If you split the functional change from the error message change and fix the typo pointed out by Katerina: Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx> Jano
virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
- _("IOThreads only available for virtio pci and "
- "virtio ccw controllers"));
+ _("virtio-scsi IOThreads only available for virtio "
+ "pci and virtio ccw controllers"));
return false;
}
--
2.14.4
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
