On Thu, 2019-01-17 at 12:52 -0500, Cole Robinson wrote: > This generates new XML like: > > <disk> > <enum name='model'> > <value>virtio-transitional</value> > <value>virtio-non-transitional</value> > </enum> > </disk> <value>virtio</value> should appear in the output as well... [...] > @@ -5286,6 +5286,13 @@ virQEMUCapsFillDomainDeviceDiskCaps(virQEMUCapsPtr qemuCaps, > if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_ICH9_AHCI)) > VIR_DOMAIN_CAPS_ENUM_SET(disk->bus, VIR_DOMAIN_DISK_BUS_SATA); > > + if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE_VIRTIO_BLK_TRANSITIONAL)) > + VIR_DOMAIN_CAPS_ENUM_SET(disk->model, > + VIR_DOMAIN_DISK_MODEL_VIRTIO_TRANSITIONAL); > + if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE_VIRTIO_BLK_NON_TRANSITIONAL)) > + VIR_DOMAIN_CAPS_ENUM_SET(disk->model, > + VIR_DOMAIN_DISK_MODEL_VIRTIO_NON_TRANSITIONAL); ... but it seems we don't have a capability for that? Weird. The approach looks sane otherwise. -- Andrea Bolognani / Red Hat / Virtualization -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
