Since libvirt now tries to interpret network device models (unless an unknow model is used) the documentation didn't make a good job specifying what is supported. Rewrite the docs to explicitly list the models which we do parse. Signed-off-by: Peter Krempa <pkrempa(a)redhat.com&gt; --- docs/formatdomain.rst | 29 ++++++++++++++++------------- 1 file changed, 16 insertions(+), 13 deletions(-) diff --git a/docs/formatdomain.rst b/docs/formatdomain.rst index e2f66b982c..a68d90c4a4 100644 --- a/docs/formatdomain.rst +++ b/docs/formatdomain.rst @@ -5592,23 +5592,26 @@ Setting the NIC model ... For hypervisors which support this, you can set the model of emulated network -interface card. +interface card via the ``model`` element. -The values for ``type`` aren't defined specifically by libvirt, but by what the -underlying hypervisor supports (if any). For QEMU and KVM you can get a list of -supported models with these commands: +While libvirt accepts any value as the ``type`` and passes it to the +hypervisor to preserve compatibility, most devices nowadays have additional +handling and address allocation which may not work properly unless the model +is known by libvirt. -:: +Libvirt supports natively the following network device models: +``virtio``, ``virtio-transitional`` (:since:`Since 5.2.0`), +``virtio-non-transitional`` (:since:`Since 5.2.0`), ``e1000``, ``e1000e``, +``igb`` (:since:`Since 9.3.0`), ``rtl8139``, ``netfront``, +``usb-net`` (:since:`Since 10.3.0`), ``spapr-vlan``, ``lan9118``, ``scm91c111``, +``vlance``, ``vmxnet``, ``vmxnet2``, ``vmxnet3``, ``Am79C970A``, ``Am79C973``, +``82540EM``, ``82545EM``, ``82543GC``. + +For QEMU you can get a list of supported models with this commands:

- qemu -net nic,model=? /dev/null - qemu-kvm -net nic,model=? /dev/null +:: -Typical values for QEMU and KVM include: ne2k_isa i82551 i82557b i82559er -ne2k_pci pcnet rtl8139 e1000 virtio. :since:`Since 5.2.0`, -``virtio-transitional`` and ``virtio-non-transitional`` values are supported. -See `Virtio transitional devices`_ for more details. -:since:`Since 9.3.0` igb is also supported. -:since:`Since 10.3.0` usb-net is supported. + qemu-system-x86_64 -net nic,model=? Setting NIC driver-specific options ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Rework 'virDomainUSBDeviceDefForeach' to use virDomainDeviceInfoIterate instead of open-coding all iterators. To achieve this 'virDomainDeviceIsUSB' needs to be fixed as it didn't properly handle 'sound', 'fs', 'chr', 'ccid', and 'net usb devices.

Signed-off-by: Peter Krempa <pkrempa(a)redhat.com&gt; --- src/conf/domain_conf.c | 228 +++++++++++++++++++---------------------- 1 file changed, 106 insertions(+), 122 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 48c5d546da..457bee08b1 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -28262,28 +28262,117 @@ virDomainObjFormat(virDomainObj *obj, return virBufferContentAndReset(&buf); } + static bool -virDomainDeviceIsUSB(virDomainDeviceDef *dev) -{ - int t = dev->type; - if ((t == VIR_DOMAIN_DEVICE_DISK && - dev->data.disk->bus == VIR_DOMAIN_DISK_BUS_USB) || - (t == VIR_DOMAIN_DEVICE_INPUT && - dev->data.input->bus == VIR_DOMAIN_INPUT_BUS_USB) || - (t == VIR_DOMAIN_DEVICE_HOSTDEV && - dev->data.hostdev->mode == VIR_DOMAIN_HOSTDEV_MODE_SUBSYS && - dev->data.hostdev->source.subsys.type == - VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_USB) || - (t == VIR_DOMAIN_DEVICE_HUB && - dev->data.hub->type == VIR_DOMAIN_HUB_TYPE_USB) || - (t == VIR_DOMAIN_DEVICE_REDIRDEV && - dev->data.redirdev->bus == VIR_DOMAIN_REDIRDEV_BUS_USB)) - return true; +virDomainDeviceIsUSB(virDomainDeviceDef *dev, + bool skipHubs) +{ + switch (dev->type) { + case VIR_DOMAIN_DEVICE_HUB: + if (skipHubs) + return false; + + return dev->data.hub->type == VIR_DOMAIN_HUB_TYPE_USB; + + case VIR_DOMAIN_DEVICE_HOSTDEV: + return dev->data.hostdev->source.subsys.type == VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_USB; + + case VIR_DOMAIN_DEVICE_DISK: + return dev->data.disk->bus == VIR_DOMAIN_DISK_BUS_USB; + + case VIR_DOMAIN_DEVICE_NET: + return dev->data.net->model == VIR_DOMAIN_NET_MODEL_USB_NET; + + case VIR_DOMAIN_DEVICE_CONTROLLER: + return dev->data.controller->type == VIR_DOMAIN_CONTROLLER_TYPE_CCID; + + case VIR_DOMAIN_DEVICE_INPUT: + return dev->data.input->bus == VIR_DOMAIN_INPUT_BUS_USB; + + case VIR_DOMAIN_DEVICE_CHR: + if (dev->data.chr->deviceType == VIR_DOMAIN_CHR_DEVICE_TYPE_SERIAL) + return dev->data.chr->targetType == VIR_DOMAIN_CHR_SERIAL_TARGET_TYPE_USB; + + return false; + + case VIR_DOMAIN_DEVICE_SOUND: + return dev->data.sound->model == VIR_DOMAIN_SOUND_MODEL_USB; + + case VIR_DOMAIN_DEVICE_REDIRDEV: + return dev->data.redirdev->bus == VIR_DOMAIN_REDIRDEV_BUS_USB; + + case VIR_DOMAIN_DEVICE_FS: + return dev->data.fs->fsdriver == VIR_DOMAIN_FS_DRIVER_TYPE_MTP; + + case VIR_DOMAIN_DEVICE_LEASE: + case VIR_DOMAIN_DEVICE_VIDEO: + case VIR_DOMAIN_DEVICE_WATCHDOG: + case VIR_DOMAIN_DEVICE_GRAPHICS: + case VIR_DOMAIN_DEVICE_NONE: + case VIR_DOMAIN_DEVICE_SMARTCARD: + case VIR_DOMAIN_DEVICE_MEMBALLOON: + case VIR_DOMAIN_DEVICE_NVRAM: + case VIR_DOMAIN_DEVICE_SHMEM: + case VIR_DOMAIN_DEVICE_TPM: + case VIR_DOMAIN_DEVICE_PANIC: + case VIR_DOMAIN_DEVICE_LAST: + case VIR_DOMAIN_DEVICE_RNG: + case VIR_DOMAIN_DEVICE_MEMORY: + case VIR_DOMAIN_DEVICE_IOMMU: + case VIR_DOMAIN_DEVICE_VSOCK: + case VIR_DOMAIN_DEVICE_AUDIO: + case VIR_DOMAIN_DEVICE_CRYPTO: + break; + } return false; } +struct virDomainUSBDeviceDefForeachIteratorData { + virDomainUSBDeviceDefIterator iter; + void *iterdata; + bool skipHubs; +}; + + +static int +virDomainUSBDeviceDefForeachIterator(virDomainDef *def G_GNUC_UNUSED, + virDomainDeviceDef *device, + virDomainDeviceInfo *info, + void *opaque) +{ + struct virDomainUSBDeviceDefForeachIteratorData *data = opaque; + + if (!virDomainDeviceIsUSB(device, data->skipHubs)) + return 0; + + if (data->iter(info, data->iterdata) < 0) + return -1; + + return 0; +} + + +int +virDomainUSBDeviceDefForeach(virDomainDef *def, + virDomainUSBDeviceDefIterator iter, + void *opaque, + bool skipHubs) +{ + struct virDomainUSBDeviceDefForeachIteratorData data = { + .iter = iter, + .iterdata = opaque, + .skipHubs = skipHubs, + }; + + return virDomainDeviceInfoIterate(def, + virDomainUSBDeviceDefForeachIterator, + &data); +} + + + typedef struct _virDomainCompatibleDeviceData virDomainCompatibleDeviceData; struct _virDomainCompatibleDeviceData { virDomainDeviceInfo *newInfo; @@ -28345,7 +28434,7 @@ virDomainDefCompatibleDevice(virDomainDef *def, if (!virDomainDefHasUSB(def) && def->os.type != VIR_DOMAIN_OSTYPE_EXE && - virDomainDeviceIsUSB(dev)) { + virDomainDeviceIsUSB(dev, false)) { virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", _("Device configuration is not compatible: Domain has no USB bus support")); return -1; @@ -28658,111 +28747,6 @@ virDomainSmartcardDefForeach(virDomainDef *def, } -int -virDomainUSBDeviceDefForeach(virDomainDef *def, - virDomainUSBDeviceDefIterator iter, - void *opaque, - bool skipHubs) -{ - size_t i; - - /* usb-hub */ - if (!skipHubs) { - for (i = 0; i < def->nhubs; i++) { - virDomainHubDef *hub = def->hubs[i]; - if (hub->type == VIR_DOMAIN_HUB_TYPE_USB) { - if (iter(&hub->info, opaque) < 0) - return -1; - } - } - } - - /* usb-host */ - for (i = 0; i < def->nhostdevs; i++) { - virDomainHostdevDef *hostdev = def->hostdevs[i]; - if (hostdev->source.subsys.type == VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_USB) { - if (iter(hostdev->info, opaque) < 0) - return -1; - } - } - - /* usb-storage */ - for (i = 0; i < def->ndisks; i++) { - virDomainDiskDef *disk = def->disks[i]; - if (disk->bus == VIR_DOMAIN_DISK_BUS_USB) { - if (iter(&disk->info, opaque) < 0) - return -1; - } - } - - /* usb-net */ - for (i = 0; i < def->nnets; i++) { - virDomainNetDef *net = def->nets[i]; - if (net->model == VIR_DOMAIN_NET_MODEL_USB_NET) { - if (iter(&net->info, opaque) < 0) - return -1; - } - } - - /* usb-ccid */ - for (i = 0; i < def->ncontrollers; i++) { - virDomainControllerDef *cont = def->controllers[i]; - if (cont->type == VIR_DOMAIN_CONTROLLER_TYPE_CCID) { - if (iter(&cont->info, opaque) < 0) - return -1; - } - } - - /* usb-kbd, usb-mouse, usb-tablet */ - for (i = 0; i < def->ninputs; i++) { - virDomainInputDef *input = def->inputs[i]; - - if (input->bus == VIR_DOMAIN_INPUT_BUS_USB) { - if (iter(&input->info, opaque) < 0) - return -1; - } - } - - /* usb-serial */ - for (i = 0; i < def->nserials; i++) { - virDomainChrDef *serial = def->serials[i]; - if (serial->targetType == VIR_DOMAIN_CHR_SERIAL_TARGET_TYPE_USB) { - if (iter(&serial->info, opaque) < 0) - return -1; - } - } - - /* usb-audio model=usb */ - for (i = 0; i < def->nsounds; i++) { - virDomainSoundDef *sound = def->sounds[i]; - if (sound->model == VIR_DOMAIN_SOUND_MODEL_USB) { - if (iter(&sound->info, opaque) < 0) - return -1; - } - } - - /* usb-redir */ - for (i = 0; i < def->nredirdevs; i++) { - virDomainRedirdevDef *redirdev = def->redirdevs[i]; - if (redirdev->bus == VIR_DOMAIN_REDIRDEV_BUS_USB) { - if (iter(&redirdev->info, opaque) < 0) - return -1; - } - } - - /* usb-mtp */ - for (i = 0; i < def->nfss; i++) { - virDomainFSDef *fsdev = def->fss[i]; - if (fsdev->fsdriver == VIR_DOMAIN_FS_DRIVER_TYPE_MTP) { - if (iter(&fsdev->info, opaque) < 0) - return -1; - } - } - - return 0; -} - - /* Copy src into a new definition; with the quality of the copy * depending on the migratable flag (false for transitions between * persistent and active, true for transitions across save files or