1
0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-03-07 17:28:15 +00:00

conf: Introduce extension flag and zPCI member for PCI address

This patch introduces PCI address extension flag for virDomainDeviceInfo
and virPCIDeviceAddress. The extension flag in virDomainDeviceInfo is
used internally during calculating PCI extension flag. The one in
virPCIDeviceAddress is the duplicate to indicate extension address is
being used. Currently only zPCI extension address is introduced to deal
with 'uid' and 'fid' on the S390 platform.

Signed-off-by: Yi Min Zhao <zyimin@linux.ibm.com>
Reviewed-by: Boris Fiuczynski <fiuczy@linux.ibm.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Andrea Bolognani <abologna@redhat.com>
This commit is contained in:
Yi Min Zhao 2018-11-08 19:00:21 +08:00 committed by Andrea Bolognani
parent f49a5e3bf4
commit 478e5f90fd
4 changed files with 149 additions and 2 deletions

View File

@ -166,6 +166,10 @@ struct _virDomainDeviceInfo {
* assignment, never saved and never reported.
*/
int pciConnectFlags; /* enum virDomainPCIConnectFlags */
/* pciAddrExtFlags is only used internally to calculate PCI
* address extension flags during address assignment.
*/
int pciAddrExtFlags; /* enum virDomainPCIAddressExtensionFlags */
char *loadparm;
/* PCI devices will only be automatically placed on a PCI bus

View File

@ -29,6 +29,11 @@
# define VIR_PCI_ADDRESS_SLOT_LAST 31
# define VIR_PCI_ADDRESS_FUNCTION_LAST 7
typedef enum {
VIR_PCI_ADDRESS_EXTENSION_NONE = 0, /* no extension */
VIR_PCI_ADDRESS_EXTENSION_ZPCI = 1 << 0, /* zPCI support */
} virPCIDeviceAddressExtensionFlags;
typedef enum {
VIR_PCI_CONNECT_HOTPLUGGABLE = 1 << 0, /* is hotplug needed/supported */

View File

@ -514,6 +514,64 @@ qemuDomainAssignVirtioMMIOAddresses(virDomainDefPtr def,
}
static bool
qemuDomainDeviceSupportZPCI(virDomainDeviceDefPtr device)
{
switch ((virDomainDeviceType)device->type) {
case VIR_DOMAIN_DEVICE_CHR:
return false;
case VIR_DOMAIN_DEVICE_CONTROLLER:
case VIR_DOMAIN_DEVICE_DISK:
case VIR_DOMAIN_DEVICE_LEASE:
case VIR_DOMAIN_DEVICE_FS:
case VIR_DOMAIN_DEVICE_NET:
case VIR_DOMAIN_DEVICE_INPUT:
case VIR_DOMAIN_DEVICE_SOUND:
case VIR_DOMAIN_DEVICE_VIDEO:
case VIR_DOMAIN_DEVICE_HOSTDEV:
case VIR_DOMAIN_DEVICE_WATCHDOG:
case VIR_DOMAIN_DEVICE_GRAPHICS:
case VIR_DOMAIN_DEVICE_HUB:
case VIR_DOMAIN_DEVICE_REDIRDEV:
case VIR_DOMAIN_DEVICE_SMARTCARD:
case VIR_DOMAIN_DEVICE_MEMBALLOON:
case VIR_DOMAIN_DEVICE_NVRAM:
case VIR_DOMAIN_DEVICE_RNG:
case VIR_DOMAIN_DEVICE_SHMEM:
case VIR_DOMAIN_DEVICE_TPM:
case VIR_DOMAIN_DEVICE_PANIC:
case VIR_DOMAIN_DEVICE_MEMORY:
case VIR_DOMAIN_DEVICE_IOMMU:
case VIR_DOMAIN_DEVICE_VSOCK:
break;
case VIR_DOMAIN_DEVICE_NONE:
case VIR_DOMAIN_DEVICE_LAST:
default:
virReportEnumRangeError(virDomainDeviceType, device->type);
return false;
}
return true;
}
static virPCIDeviceAddressExtensionFlags
qemuDomainDeviceCalculatePCIAddressExtensionFlags(virQEMUCapsPtr qemuCaps,
virDomainDeviceDefPtr dev)
{
virPCIDeviceAddressExtensionFlags extFlags = 0;
if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE_ZPCI) &&
qemuDomainDeviceSupportZPCI(dev)) {
extFlags |= VIR_PCI_ADDRESS_EXTENSION_ZPCI;
}
return extFlags;
}
/**
* qemuDomainDeviceCalculatePCIConnectFlags:
*
@ -995,6 +1053,56 @@ qemuDomainFillAllPCIConnectFlags(virDomainDefPtr def,
}
/**
* qemuDomainFillDevicePCIExtensionFlagsIter:
*
* @def: the entire DomainDef
* @dev: The device to be checked
* @info: virDomainDeviceInfo within the device
* @opaque: qemu capabilities
*
* Sets the pciAddressExtFlags for a single device's info. Has properly
* formatted arguments to be called by virDomainDeviceInfoIterate().
*
* Always returns 0 - there is no failure.
*/
static int
qemuDomainFillDevicePCIExtensionFlagsIter(virDomainDefPtr def ATTRIBUTE_UNUSED,
virDomainDeviceDefPtr dev,
virDomainDeviceInfoPtr info,
void *opaque)
{
virQEMUCapsPtr qemuCaps = opaque;
info->pciAddrExtFlags =
qemuDomainDeviceCalculatePCIAddressExtensionFlags(qemuCaps, dev);
return 0;
}
/**
* qemuDomainFillAllPCIExtensionFlags:
*
* @def: the entire DomainDef
* @qemuCaps: as you'd expect
*
* Set the info->pciAddressExtFlags for all devices in the domain.
*
* Returns 0 on success or -1 on failure (the only possibility of
* failure would be some internal problem with
* virDomainDeviceInfoIterate())
*/
static int
qemuDomainFillAllPCIExtensionFlags(virDomainDefPtr def,
virQEMUCapsPtr qemuCaps)
{
return virDomainDeviceInfoIterate(def,
qemuDomainFillDevicePCIExtensionFlagsIter,
qemuCaps);
}
/**
* qemuDomainFindUnusedIsolationGroupIter:
* @def: domain definition
@ -1269,6 +1377,27 @@ qemuDomainFillDevicePCIConnectFlags(virDomainDefPtr def,
}
/**
* qemuDomainFillDevicePCIExtensionFlags:
*
* @dev: The device to be checked
* @info: virDomainDeviceInfo within the device
* @qemuCaps: as you'd expect
*
* Set the info->pciAddressExtFlags for a single device.
*
* No return value.
*/
static void
qemuDomainFillDevicePCIExtensionFlags(virDomainDeviceDefPtr dev,
virDomainDeviceInfoPtr info,
virQEMUCapsPtr qemuCaps)
{
info->pciAddrExtFlags =
qemuDomainDeviceCalculatePCIAddressExtensionFlags(qemuCaps, dev);
}
static int
qemuDomainPCIAddressReserveNextAddr(virDomainPCIAddressSetPtr addrs,
virDomainDeviceInfoPtr dev)
@ -2405,6 +2534,9 @@ qemuDomainAssignPCIAddresses(virDomainDefPtr def,
if (qemuDomainFillAllPCIConnectFlags(def, qemuCaps, driver) < 0)
goto cleanup;
if (qemuDomainFillAllPCIExtensionFlags(def, qemuCaps) < 0)
goto cleanup;
if (qemuDomainSetupIsolationGroups(def) < 0)
goto cleanup;
@ -2440,7 +2572,8 @@ qemuDomainAssignPCIAddresses(virDomainDefPtr def,
*/
virDomainDeviceInfo info = {
.pciConnectFlags = (VIR_PCI_CONNECT_HOTPLUGGABLE |
VIR_PCI_CONNECT_TYPE_PCI_DEVICE)
VIR_PCI_CONNECT_TYPE_PCI_DEVICE),
.pciAddrExtFlags = VIR_PCI_ADDRESS_EXTENSION_NONE
};
bool buses_reserved = true;
@ -2477,7 +2610,8 @@ qemuDomainAssignPCIAddresses(virDomainDefPtr def,
qemuDomainHasPCIeRoot(def)) {
virDomainDeviceInfo info = {
.pciConnectFlags = (VIR_PCI_CONNECT_HOTPLUGGABLE |
VIR_PCI_CONNECT_TYPE_PCIE_DEVICE)
VIR_PCI_CONNECT_TYPE_PCIE_DEVICE),
.pciAddrExtFlags = VIR_PCI_ADDRESS_EXTENSION_NONE
};
/* if there isn't an empty pcie-root-port, this will
@ -2994,6 +3128,8 @@ qemuDomainEnsurePCIAddress(virDomainObjPtr obj,
qemuDomainFillDevicePCIConnectFlags(obj->def, dev, priv->qemuCaps, driver);
qemuDomainFillDevicePCIExtensionFlags(dev, info, priv->qemuCaps);
return virDomainPCIAddressEnsureAddr(priv->pciaddrs, info,
info->pciConnectFlags);
}

View File

@ -50,6 +50,8 @@ struct _virPCIDeviceAddress {
unsigned int slot;
unsigned int function;
int multi; /* virTristateSwitch */
int extFlags; /* enum virPCIDeviceAddressExtensionFlags */
virZPCIDeviceAddress zpci;
};
typedef enum {