On Wed, Aug 10, 2016 at 04:27:58PM +0200, Boris Fiuczynski wrote:
On 08/08/2016 06:35 PM, Ján Tomko wrote:https://bugzilla.redhat.com/show_bug.cgi?id=1227354 Translate the optional <virtio revision> attribute to disable-legacy=on/off and disable-modern=on/off options for the following devices: <memballoon> virtio-balloon-pci <disk> virtio-blk-pci <controller> virtio-scsi-pci virtio-serial-pci <filesystem> virtio-9p-pci <interface> virtio-net-pci <rng> virtio-rng-pci <video> virtio-gpu-pci <input> virtio-input-host-pci virtio-keyboard-pci virtio-mouse-pci virtio-tablet-pci --- src/qemu/qemu_command.c | 56 +++++++++++++++++++ .../qemuxml2argv-virtio-revision.args | 62 ++++++++++++++++++++++ tests/qemuxml2argvtest.c | 11 ++++ 3 files changed, 129 insertions(+)Although you mention in all above devices the type of bus in the code that differentiation does not really exist. It originates from the way the capability QEMU_CAPS_DEVICE_VIRTIO_REVISION is sensed and set, e.g. virQEMUCapsObjectPropsVirtioBalloon[] is also used for virtio-balloon-ccw devices. I would suggest to create a QEMU_CAPS_DEVICE_VIRTIO_PCI_REVISION and sense the capability for pci only and also generate the command line parameters for pci only since the virtio pci revision handling does not work at all for virtio ccw.
I have sent a new version that only probes the capability on PCI devices: https://www.redhat.com/archives/libvir-list/2016-August/msg00592.html Thanks for catching that. Jan
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list