From nobody Mon Feb 9 11:28:42 2026 Delivered-To: importer@patchew.org Received-SPF: none (zohomail.com: 8.43.85.245 is neither permitted nor denied by domain of lists.libvirt.org) client-ip=8.43.85.245; envelope-from=devel-bounces@lists.libvirt.org; helo=lists.libvirt.org; Authentication-Results: mx.zohomail.com; spf=none (zohomail.com: 8.43.85.245 is neither permitted nor denied by domain of lists.libvirt.org) smtp.mailfrom=devel-bounces@lists.libvirt.org; dmarc=fail(p=none dis=none) header.from=redhat.com Return-Path: Received: from lists.libvirt.org (lists.libvirt.org [8.43.85.245]) by mx.zohomail.com with SMTPS id 1704444275351881.6226099702527; Fri, 5 Jan 2024 00:44:35 -0800 (PST) Received: by lists.libvirt.org (Postfix, from userid 996) id 401B019D3; Fri, 5 Jan 2024 03:44:34 -0500 (EST) Received: from lists.libvirt.org (localhost [IPv6:::1]) by lists.libvirt.org (Postfix) with ESMTP id 74F3A188C; Fri, 5 Jan 2024 03:21:34 -0500 (EST) Received: by lists.libvirt.org (Postfix, from userid 996) id E191A17FB; Fri, 5 Jan 2024 03:20:28 -0500 (EST) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.libvirt.org (Postfix) with ESMTPS id 5B18A1815 for ; Fri, 5 Jan 2024 03:20:20 -0500 (EST) Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-518-9Xoxb7SgPkyXsG8nnGoDRg-1; Fri, 05 Jan 2024 03:20:18 -0500 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.rdu2.redhat.com [10.11.54.4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 1971083BA85 for ; Fri, 5 Jan 2024 08:20:18 +0000 (UTC) Received: from vhost3.router.laine.org (unknown [10.22.16.86]) by smtp.corp.redhat.com (Postfix) with ESMTP id F215A2026D6F; Fri, 5 Jan 2024 08:20:17 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on lists.libvirt.org X-Spam-Level: X-Spam-Status: No, score=-0.8 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.4 X-MC-Unique: 9Xoxb7SgPkyXsG8nnGoDRg-1 From: Laine Stump To: devel@lists.libvirt.org Subject: [PATCH v3 05/13] conf: put hostdev PCI backend into a struct Date: Fri, 5 Jan 2024 03:20:08 -0500 Message-ID: <20240105082016.281442-6-laine@redhat.com> In-Reply-To: <20240105082016.281442-1-laine@redhat.com> References: <20240105082016.281442-1-laine@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.4 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Message-ID-Hash: MPJ6OBURCW3GZ6RC3OJM5VSCQC3GBSS3 X-Message-ID-Hash: MPJ6OBURCW3GZ6RC3OJM5VSCQC3GBSS3 X-MailFrom: laine@redhat.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-config-1; header-match-config-2; header-match-config-3; header-match-devel.lists.libvirt.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; suspicious-header CC: Peter Krempa X-Mailman-Version: 3.2.2 Precedence: list List-Id: Development discussions about the libvirt library & tools Archived-At: List-Archive: List-Help: List-Post: List-Subscribe: List-Unsubscribe: Content-Type: text/plain; charset="utf-8"; x-default="true" Content-Transfer-Encoding: quoted-printable X-ZM-MESSAGEID: 1704444275652100001 The new struct is virDeviceHostdevPCIDriverInfo, and the "backend" enum in the hostdevDef will be replaced with a virDeviceHostdevPCIDriverInfo named "driver'. Since the enum value in this new struct is called "name", it means that all references to "backend" will become "driver.name". This will allow easily adding other items for new attributes in the element / C struct, which will be useful once we are using this new struct in multiple places. Signed-off-by: Laine Stump Reviewed-by: Peter Krempa --- Change from V2: names have changed, extra line breaks removed src/conf/device_conf.h | 4 ++++ src/conf/domain_conf.c | 30 +++++++++++++----------------- src/conf/domain_conf.h | 2 +- src/conf/virconftypes.h | 2 ++ src/hypervisor/virhostdev.c | 6 +++--- src/libxl/libxl_domain.c | 6 +++--- src/libxl/libxl_driver.c | 6 +++--- src/qemu/qemu_command.c | 6 +++--- src/qemu/qemu_domain.c | 4 ++-- src/qemu/qemu_validate.c | 6 +++--- src/security/security_apparmor.c | 2 +- src/security/security_dac.c | 4 ++-- src/security/security_selinux.c | 4 ++-- src/security/virt-aa-helper.c | 2 +- src/test/test_driver.c | 6 +++--- tests/virhostdevtest.c | 2 +- 16 files changed, 47 insertions(+), 45 deletions(-) diff --git a/src/conf/device_conf.h b/src/conf/device_conf.h index 18aa4125fa..1f7e066142 100644 --- a/src/conf/device_conf.h +++ b/src/conf/device_conf.h @@ -43,6 +43,10 @@ typedef enum { =20 VIR_ENUM_DECL(virDeviceHostdevPCIDriverName); =20 +struct _virDeviceHostdevPCIDriverInfo { + virDeviceHostdevPCIDriverName name; +}; + typedef enum { VIR_DOMAIN_DEVICE_ADDRESS_TYPE_NONE =3D 0, VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI, diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 35a9138a95..887b871589 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -6285,7 +6285,7 @@ virDomainHostdevDefParseXMLSubsys(xmlNodePtr node, if (virXMLPropEnum(driver_node, "name", virDeviceHostdevPCIDriverNameTypeFromString, VIR_XML_PROP_NONZERO, - &pcisrc->backend) < 0) { + &pcisrc->driver.name) < 0) { return -1; } } @@ -23378,17 +23378,17 @@ virDomainHostdevDefFormatSubsysPCI(virBuffer *buf, g_auto(virBuffer) sourceChildBuf =3D VIR_BUFFER_INIT_CHILD(buf); virDomainHostdevSubsysPCI *pcisrc =3D &def->source.subsys.u.pci; =20 - if (pcisrc->backend !=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_DEFAULT) { - const char *backend =3D virDeviceHostdevPCIDriverNameTypeToString(= pcisrc->backend); + if (pcisrc->driver.name !=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_DEFAUL= T) { + const char *driverName =3D virDeviceHostdevPCIDriverNameTypeToStri= ng(pcisrc->driver.name); =20 - if (!backend) { + if (!driverName) { virReportError(VIR_ERR_INTERNAL_ERROR, - _("unexpected pci hostdev driver name type %1$d= "), - pcisrc->backend); + _("unexpected pci hostdev driver type %1$d"), + pcisrc->driver.name); return -1; } =20 - virBufferAsprintf(&driverAttrBuf, " name=3D'%s'", backend); + virBufferAsprintf(&driverAttrBuf, " name=3D'%s'", driverName); } =20 virXMLFormatElement(buf, "driver", &driverAttrBuf, NULL); @@ -29916,15 +29916,15 @@ virDomainNetDefActualFromNetworkPort(virDomainNet= Def *iface, actual->data.hostdev.def.source.subsys.u.pci.addr =3D port->plug.h= ostdevpci.addr; switch ((virNetworkForwardDriverNameType)port->plug.hostdevpci.dri= ver) { case VIR_NETWORK_FORWARD_DRIVER_NAME_DEFAULT: - actual->data.hostdev.def.source.subsys.u.pci.backend =3D VIR_D= EVICE_HOSTDEV_PCI_DRIVER_NAME_DEFAULT; + actual->data.hostdev.def.source.subsys.u.pci.driver.name =3D V= IR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_DEFAULT; break; =20 case VIR_NETWORK_FORWARD_DRIVER_NAME_KVM: - actual->data.hostdev.def.source.subsys.u.pci.backend =3D VIR_D= EVICE_HOSTDEV_PCI_DRIVER_NAME_KVM; + actual->data.hostdev.def.source.subsys.u.pci.driver.name =3D V= IR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_KVM; break; =20 case VIR_NETWORK_FORWARD_DRIVER_NAME_VFIO: - actual->data.hostdev.def.source.subsys.u.pci.backend =3D VIR_D= EVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO; + actual->data.hostdev.def.source.subsys.u.pci.driver.name =3D V= IR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO; break; =20 case VIR_NETWORK_FORWARD_DRIVER_NAME_LAST: @@ -30034,7 +30034,7 @@ virDomainNetDefActualToNetworkPort(virDomainDef *do= m, } port->plug.hostdevpci.managed =3D virTristateBoolFromBool(actual->= data.hostdev.def.managed); port->plug.hostdevpci.addr =3D actual->data.hostdev.def.source.sub= sys.u.pci.addr; - switch (actual->data.hostdev.def.source.subsys.u.pci.backend) { + switch (actual->data.hostdev.def.source.subsys.u.pci.driver.name) { case VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_DEFAULT: port->plug.hostdevpci.driver =3D VIR_NETWORK_FORWARD_DRIVER_NA= ME_DEFAULT; break; @@ -30048,14 +30048,10 @@ virDomainNetDefActualToNetworkPort(virDomainDef *= dom, break; =20 case VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_XEN: - virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", - _("Unexpected PCI backend 'xen'")); - break; - case VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_LAST: default: virReportEnumRangeError(virDeviceHostdevPCIDriverName, - actual->data.hostdev.def.source.subsys= .u.pci.backend); + actual->data.hostdev.def.source.subsys= .u.pci.driver.name); return NULL; } =20 @@ -30999,7 +30995,7 @@ virHostdevIsVFIODevice(const virDomainHostdevDef *h= ostdev) { return hostdev->mode =3D=3D VIR_DOMAIN_HOSTDEV_MODE_SUBSYS && hostdev->source.subsys.type =3D=3D VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_= PCI && - hostdev->source.subsys.u.pci.backend =3D=3D VIR_DEVICE_HOSTDEV_PCI= _DRIVER_NAME_VFIO; + hostdev->source.subsys.u.pci.driver.name =3D=3D VIR_DEVICE_HOSTDEV= _PCI_DRIVER_NAME_VFIO; } =20 =20 diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h index 0582cc928d..ee8bcbc6b3 100644 --- a/src/conf/domain_conf.h +++ b/src/conf/domain_conf.h @@ -235,7 +235,7 @@ struct _virDomainHostdevSubsysUSB { =20 struct _virDomainHostdevSubsysPCI { virPCIDeviceAddress addr; /* host address */ - virDeviceHostdevPCIDriverName backend; + virDeviceHostdevPCIDriverInfo driver; =20 virBitmap *origstates; }; diff --git a/src/conf/virconftypes.h b/src/conf/virconftypes.h index 26cb966194..0779bc224b 100644 --- a/src/conf/virconftypes.h +++ b/src/conf/virconftypes.h @@ -66,6 +66,8 @@ typedef struct _virCapsHostSecModelLabel virCapsHostSecMo= delLabel; =20 typedef struct _virCapsStoragePool virCapsStoragePool; =20 +typedef struct _virDeviceHostdevPCIDriverInfo virDeviceHostdevPCIDriverInf= o; + typedef struct _virDomainABIStability virDomainABIStability; =20 typedef struct _virDomainActualNetDef virDomainActualNetDef; diff --git a/src/hypervisor/virhostdev.c b/src/hypervisor/virhostdev.c index 26c9ac3c28..27b0acdc91 100644 --- a/src/hypervisor/virhostdev.c +++ b/src/hypervisor/virhostdev.c @@ -243,14 +243,14 @@ virHostdevGetPCIHostDevice(const virDomainHostdevDef = *hostdev, =20 virPCIDeviceSetManaged(actual, hostdev->managed); =20 - if (pcisrc->backend =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO) { + if (pcisrc->driver.name =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO= ) { virPCIDeviceSetStubDriverType(actual, VIR_PCI_STUB_DRIVER_VFIO); - } else if (pcisrc->backend =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_X= EN) { + } else if (pcisrc->driver.name =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NA= ME_XEN) { virPCIDeviceSetStubDriverType(actual, VIR_PCI_STUB_DRIVER_XEN); } else { virReportError(VIR_ERR_CONFIG_UNSUPPORTED, _("pci backend driver type '%1$s' is not supported"= ), - virDeviceHostdevPCIDriverNameTypeToString(pcisrc->b= ackend)); + virDeviceHostdevPCIDriverNameTypeToString(pcisrc->d= river.name)); return -1; } =20 diff --git a/src/libxl/libxl_domain.c b/src/libxl/libxl_domain.c index 88d977e79b..1a7de00704 100644 --- a/src/libxl/libxl_domain.c +++ b/src/libxl/libxl_domain.c @@ -160,8 +160,8 @@ libxlDomainDeviceDefPostParse(virDomainDeviceDef *dev, =20 if (hostdev->mode =3D=3D VIR_DOMAIN_HOSTDEV_MODE_SUBSYS && hostdev->source.subsys.type =3D=3D VIR_DOMAIN_HOSTDEV_SUBSYS_T= YPE_PCI && - pcisrc->backend =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_DEFA= ULT) - pcisrc->backend =3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_XEN; + pcisrc->driver.name =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_= DEFAULT) + pcisrc->driver.name =3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_XEN; } =20 if (dev->type =3D=3D VIR_DOMAIN_DEVICE_VIDEO) { @@ -997,7 +997,7 @@ libxlNetworkPrepareDevices(virDomainDef *def) =20 if (hostdev->mode =3D=3D VIR_DOMAIN_HOSTDEV_MODE_SUBSYS && hostdev->source.subsys.type =3D=3D VIR_DOMAIN_HOSTDEV_SUBS= YS_TYPE_PCI) - pcisrc->backend =3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_XEN; + pcisrc->driver.name =3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME= _XEN; =20 if (virDomainHostdevInsert(def, hostdev) < 0) return -1; diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c index d7f79089c3..29fbc823dd 100644 --- a/src/libxl/libxl_driver.c +++ b/src/libxl/libxl_driver.c @@ -3397,13 +3397,13 @@ libxlDomainAttachNetDevice(libxlDriverPrivate *driv= er, virDomainHostdevDef *hostdev =3D virDomainNetGetActualHostdev(net); virDomainHostdevSubsysPCI *pcisrc =3D &hostdev->source.subsys.u.pc= i; =20 - /* For those just allocated from a network pool whose backend is + /* For those just allocated from a network pool whose driver type = is * still VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_DEFAULT, we need to set - * backend correctly. + * driver name correctly. */ if (hostdev->mode =3D=3D VIR_DOMAIN_HOSTDEV_MODE_SUBSYS && hostdev->source.subsys.type =3D=3D VIR_DOMAIN_HOSTDEV_SUBSYS_T= YPE_PCI) - pcisrc->backend =3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_XEN; + pcisrc->driver.name =3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_XEN; =20 /* This is really a "smart hostdev", so it should be attached * as a hostdev (the hostdev code will reach over into the diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 2f0b1d0acd..b3ebf711e5 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_command.c @@ -4707,8 +4707,8 @@ qemuBuildPCIHostdevDevProps(const virDomainDef *def, g_autofree char *host =3D virPCIDeviceAddressAsString(&pcisrc->addr); const char *failover_pair_id =3D NULL; =20 - /* caller has to assign proper passthrough backend type */ - switch (pcisrc->backend) { + /* caller has to assign proper passthrough driver name */ + switch (pcisrc->driver.name) { case VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO: break; =20 @@ -4718,7 +4718,7 @@ qemuBuildPCIHostdevDevProps(const virDomainDef *def, case VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_LAST: virReportError(VIR_ERR_INTERNAL_ERROR, _("invalid PCI passthrough type '%1$s'"), - virDeviceHostdevPCIDriverNameTypeToString(pcisrc->b= ackend)); + virDeviceHostdevPCIDriverNameTypeToString(pcisrc->d= river.name)); return NULL; } =20 diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index 1e2b74af02..1aeeb74503 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -10531,7 +10531,7 @@ qemuDomainGetHostdevPath(virDomainHostdevDef *dev, case VIR_DOMAIN_HOSTDEV_MODE_SUBSYS: switch (dev->source.subsys.type) { case VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_PCI: - if (pcisrc->backend =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_= VFIO) { + if (pcisrc->driver.name =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_N= AME_VFIO) { if (!(tmpPath =3D virPCIDeviceAddressGetIOMMUGroupDev(&pci= src->addr))) return -1; =20 @@ -11302,7 +11302,7 @@ qemuDomainPrepareHostdevPCI(virDomainHostdevDef *ho= stdev, virQEMUCaps *qemuCaps) { bool supportsPassthroughVFIO =3D qemuHostdevHostSupportsPassthroughVFI= O(); - virDeviceHostdevPCIDriverName *driverName =3D &hostdev->source.subsys.= u.pci.backend; + virDeviceHostdevPCIDriverName *driverName =3D &hostdev->source.subsys.= u.pci.driver.name; =20 /* assign defaults for hostdev passthrough */ switch (*driverName) { diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c index 2aab0d9db2..4d388bbcc4 100644 --- a/src/qemu/qemu_validate.c +++ b/src/qemu/qemu_validate.c @@ -2437,7 +2437,7 @@ qemuValidateDomainDeviceDefHostdev(const virDomainHos= tdevDef *hostdev, const virDomainDef *def, virQEMUCaps *qemuCaps) { - int backend; + virDeviceHostdevPCIDriverName driverName; =20 /* forbid capabilities mode hostdev in this kind of hypervisor */ if (hostdev->mode =3D=3D VIR_DOMAIN_HOSTDEV_MODE_CAPABILITIES) { @@ -2462,9 +2462,9 @@ qemuValidateDomainDeviceDefHostdev(const virDomainHos= tdevDef *hostdev, break; =20 case VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_PCI: - backend =3D hostdev->source.subsys.u.pci.backend; + driverName =3D hostdev->source.subsys.u.pci.driver.name; =20 - if (backend =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO) { + if (driverName =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO)= { if (!virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE_VFIO_PCI)) { virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", _("VFIO PCI device assignment is not su= pported by this version of qemu")); diff --git a/src/security/security_apparmor.c b/src/security/security_appar= mor.c index a1cbbc77e9..c1dc859751 100644 --- a/src/security/security_apparmor.c +++ b/src/security/security_apparmor.c @@ -869,7 +869,7 @@ AppArmorSetSecurityHostdevLabel(virSecurityManager *mgr, if (!pci) goto done; =20 - if (pcisrc->backend =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO= ) { + if (pcisrc->driver.name =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_= VFIO) { char *vfioGroupDev =3D virPCIDeviceGetIOMMUGroupDev(pci); =20 if (!vfioGroupDev) { diff --git a/src/security/security_dac.c b/src/security/security_dac.c index cb8b72b289..4b8130630f 100644 --- a/src/security/security_dac.c +++ b/src/security/security_dac.c @@ -1257,7 +1257,7 @@ virSecurityDACSetHostdevLabel(virSecurityManager *mgr, if (!pci) return -1; =20 - if (pcisrc->backend =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO= ) { + if (pcisrc->driver.name =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_= VFIO) { g_autofree char *vfioGroupDev =3D virPCIDeviceGetIOMMUGroupDev= (pci); =20 if (!vfioGroupDev) @@ -1418,7 +1418,7 @@ virSecurityDACRestoreHostdevLabel(virSecurityManager = *mgr, if (!pci) return -1; =20 - if (pcisrc->backend =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO= ) { + if (pcisrc->driver.name =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_= VFIO) { g_autofree char *vfioGroupDev =3D virPCIDeviceGetIOMMUGroupDev= (pci); =20 if (!vfioGroupDev) diff --git a/src/security/security_selinux.c b/src/security/security_selinu= x.c index 2ee542f6a4..ffad058d9a 100644 --- a/src/security/security_selinux.c +++ b/src/security/security_selinux.c @@ -2201,7 +2201,7 @@ virSecuritySELinuxSetHostdevSubsysLabel(virSecurityMa= nager *mgr, if (!pci) return -1; =20 - if (pcisrc->backend =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO= ) { + if (pcisrc->driver.name =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_= VFIO) { g_autofree char *vfioGroupDev =3D virPCIDeviceGetIOMMUGroupDev= (pci); =20 if (!vfioGroupDev) @@ -2437,7 +2437,7 @@ virSecuritySELinuxRestoreHostdevSubsysLabel(virSecuri= tyManager *mgr, if (!pci) return -1; =20 - if (pcisrc->backend =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO= ) { + if (pcisrc->driver.name =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_= VFIO) { g_autofree char *vfioGroupDev =3D virPCIDeviceGetIOMMUGroupDev= (pci); =20 if (!vfioGroupDev) diff --git a/src/security/virt-aa-helper.c b/src/security/virt-aa-helper.c index d01df8710d..0374581f07 100644 --- a/src/security/virt-aa-helper.c +++ b/src/security/virt-aa-helper.c @@ -1092,7 +1092,7 @@ get_files(vahControl * ctl) case VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_PCI: { virPCIDevice *pci =3D virPCIDeviceNew(&dev->source.subsys.= u.pci.addr); =20 - virDeviceHostdevPCIDriverName driverName =3D dev->source.s= ubsys.u.pci.backend; + virDeviceHostdevPCIDriverName driverName =3D dev->source.s= ubsys.u.pci.driver.name; =20 if (driverName =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_V= FIO || driverName =3D=3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_D= EFAULT) { diff --git a/src/test/test_driver.c b/src/test/test_driver.c index 63e1f743c3..ed545848af 100644 --- a/src/test/test_driver.c +++ b/src/test/test_driver.c @@ -10042,9 +10042,9 @@ testDomainAttachHostPCIDevice(testDriver *driver G_= GNUC_UNUSED, virDomainObj *vm, virDomainHostdevDef *hostdev) { - int backend =3D hostdev->source.subsys.u.pci.backend; + int driverName =3D hostdev->source.subsys.u.pci.driver.name; =20 - switch (backend) { + switch (driverName) { case VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO: case VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_DEFAULT: case VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_KVM: @@ -10054,7 +10054,7 @@ testDomainAttachHostPCIDevice(testDriver *driver G_= GNUC_UNUSED, case VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_LAST: virReportError(VIR_ERR_CONFIG_UNSUPPORTED, _("test hypervisor does not support device assignme= nt mode '%1$s'"), - virDeviceHostdevPCIDriverNameTypeToString(backend)); + virDeviceHostdevPCIDriverNameTypeToString(driverNam= e)); return -1; } =20 diff --git a/tests/virhostdevtest.c b/tests/virhostdevtest.c index 89b4fb5e7c..aec474a148 100644 --- a/tests/virhostdevtest.c +++ b/tests/virhostdevtest.c @@ -134,7 +134,7 @@ myInit(void) subsys->u.pci.addr.bus =3D 0; subsys->u.pci.addr.slot =3D i + 1; subsys->u.pci.addr.function =3D 0; - subsys->u.pci.backend =3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_VFIO; + subsys->u.pci.driver.name =3D VIR_DEVICE_HOSTDEV_PCI_DRIVER_NAME_V= FIO; } =20 for (i =3D 0; i < nhostdevs; i++) { --=20 2.43.0 _______________________________________________ Devel mailing list -- devel@lists.libvirt.org To unsubscribe send an email to devel-leave@lists.libvirt.org