mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-01-11 07:17:44 +00:00
qemu: domain: Move hostdev validation into separate function
This commit is contained in:
parent
577ccd07c3
commit
a6936ec098
@ -3684,6 +3684,23 @@ qemuDomainDeviceDefValidateNetwork(const virDomainNetDef *net)
|
|||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
static int
|
||||||
|
qemuDomainDeviceDefValidateHostdev(const virDomainHostdevDef *hostdev,
|
||||||
|
const virDomainDef *def)
|
||||||
|
{
|
||||||
|
/* forbid capabilities mode hostdev in this kind of hypervisor */
|
||||||
|
if (hostdev->mode == VIR_DOMAIN_HOSTDEV_MODE_CAPABILITIES) {
|
||||||
|
virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
|
||||||
|
_("hostdev mode 'capabilities' is not "
|
||||||
|
"supported in %s"),
|
||||||
|
virDomainVirtTypeToString(def->virtType));
|
||||||
|
return -1;
|
||||||
|
}
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
static int
|
static int
|
||||||
qemuDomainDeviceDefValidate(const virDomainDeviceDef *dev,
|
qemuDomainDeviceDefValidate(const virDomainDeviceDef *dev,
|
||||||
const virDomainDef *def,
|
const virDomainDef *def,
|
||||||
@ -3709,16 +3726,9 @@ qemuDomainDeviceDefValidate(const virDomainDeviceDef *dev,
|
|||||||
} else if (dev->type == VIR_DOMAIN_DEVICE_WATCHDOG) {
|
} else if (dev->type == VIR_DOMAIN_DEVICE_WATCHDOG) {
|
||||||
if (qemuDomainWatchdogDefValidate(dev->data.watchdog, def) < 0)
|
if (qemuDomainWatchdogDefValidate(dev->data.watchdog, def) < 0)
|
||||||
goto cleanup;
|
goto cleanup;
|
||||||
}
|
} else if (dev->type == VIR_DOMAIN_DEVICE_HOSTDEV) {
|
||||||
|
if (qemuDomainDeviceDefValidateHostdev(dev->data.hostdev, def) < 0)
|
||||||
/* forbid capabilities mode hostdev in this kind of hypervisor */
|
goto cleanup;
|
||||||
if (dev->type == VIR_DOMAIN_DEVICE_HOSTDEV &&
|
|
||||||
dev->data.hostdev->mode == VIR_DOMAIN_HOSTDEV_MODE_CAPABILITIES) {
|
|
||||||
virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
|
|
||||||
_("hostdev mode 'capabilities' is not "
|
|
||||||
"supported in %s"),
|
|
||||||
virDomainVirtTypeToString(def->virtType));
|
|
||||||
goto cleanup;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if (dev->type == VIR_DOMAIN_DEVICE_VIDEO) {
|
if (dev->type == VIR_DOMAIN_DEVICE_VIDEO) {
|
||||||
|
Loading…
Reference in New Issue
Block a user