From nobody Mon Feb 9 17:37:05 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=intel.com Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1580300355457353.7389463370912; Wed, 29 Jan 2020 04:19:15 -0800 (PST) Received: from localhost ([::1]:45472 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iwmJO-000318-3O for importer@patchew.org; Wed, 29 Jan 2020 07:19:14 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:52241) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iwmCI-0007d3-H5 for qemu-devel@nongnu.org; Wed, 29 Jan 2020 07:11:55 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iwmCG-0001a1-TY for qemu-devel@nongnu.org; Wed, 29 Jan 2020 07:11:54 -0500 Received: from mga09.intel.com ([134.134.136.24]:58727) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iwmCG-0001Rn-KU for qemu-devel@nongnu.org; Wed, 29 Jan 2020 07:11:52 -0500 Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 29 Jan 2020 04:11:48 -0800 Received: from jacob-builder.jf.intel.com ([10.7.199.155]) by fmsmga001.fm.intel.com with ESMTP; 29 Jan 2020 04:11:47 -0800 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.70,377,1574150400"; d="scan'208";a="314070985" From: "Liu, Yi L" To: qemu-devel@nongnu.org, david@gibson.dropbear.id.au, pbonzini@redhat.com, alex.williamson@redhat.com, peterx@redhat.com Subject: [RFC v3 10/25] vfio: register DualStageIOMMUObject to vIOMMU Date: Wed, 29 Jan 2020 04:16:41 -0800 Message-Id: <1580300216-86172-11-git-send-email-yi.l.liu@intel.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1580300216-86172-1-git-send-email-yi.l.liu@intel.com> References: <1580300216-86172-1-git-send-email-yi.l.liu@intel.com> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 134.134.136.24 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kevin.tian@intel.com, yi.l.liu@intel.com, Yi Sun , kvm@vger.kernel.org, mst@redhat.com, jun.j.tian@intel.com, eric.auger@redhat.com, yi.y.sun@intel.com, Jacob Pan , hao.wu@intel.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" From: Liu Yi L After confirming dual stage DMA translation support with kernel by checking VFIO_TYPE1_NESTING_IOMMU, VFIO could register DualStageIOMMUObject instance to vIOMMU, thus that vIOMMU is aware of such hardware capability. vIOMMU may make use of such capability by leveraging the ops provided by DualStageIOMMUObject. Cc: Kevin Tian Cc: Jacob Pan Cc: Peter Xu Cc: Eric Auger Cc: Yi Sun Cc: David Gibson Cc: Alex Williamson Signed-off-by: Liu Yi L --- hw/vfio/common.c | 30 ++++++++++++++++++++++++++++-- include/hw/vfio/vfio-common.h | 2 ++ 2 files changed, 30 insertions(+), 2 deletions(-) diff --git a/hw/vfio/common.c b/hw/vfio/common.c index a5e70b1..fc1723d 100644 --- a/hw/vfio/common.c +++ b/hw/vfio/common.c @@ -1179,6 +1179,9 @@ static int vfio_get_iommu_type(VFIOContainer *contain= er, return -EINVAL; } =20 +static struct DualStageIOMMUOps vfio_ds_iommu_ops =3D { +}; + static int vfio_init_container(VFIOContainer *container, int group_fd, Error **errp) { @@ -1210,12 +1213,29 @@ static int vfio_init_container(VFIOContainer *conta= iner, int group_fd, return -errno; } =20 + if (iommu_type =3D=3D VFIO_TYPE1_NESTING_IOMMU) { + ds_iommu_object_init(&container->dsi_obj, &vfio_ds_iommu_ops); + if (iommu_context_register_ds_iommu(container->iommu_ctx, + &container->dsi_obj)) { + /* + * Here just need an info to indicate that there is no + * DualStageIOMMUObject instance registered to vIOMMU + * due to either no IOMMUContext support in vIOMMU or + * vIOMMU internal failure. Neither is fatal error to + * VFIO as it is not mandatory requirement to use such + * capability in vIOMMU. + */ + printf("No Dual Stage IOMMU for container(0x%p)\n", container); + ds_iommu_object_destroy(&container->dsi_obj); + } + } + container->iommu_type =3D iommu_type; return 0; } =20 static int vfio_connect_container(VFIOGroup *group, AddressSpace *as, - Error **errp) + IOMMUContext *iommu_ctx, Error **errp) { VFIOContainer *container; int ret, fd; @@ -1277,6 +1297,7 @@ static int vfio_connect_container(VFIOGroup *group, A= ddressSpace *as, container =3D g_malloc0(sizeof(*container)); container->space =3D space; container->fd =3D fd; + container->iommu_ctx =3D iommu_ctx; container->error =3D NULL; QLIST_INIT(&container->giommu_list); QLIST_INIT(&container->hostwin_list); @@ -1457,6 +1478,11 @@ static void vfio_disconnect_container(VFIOGroup *gro= up) =20 trace_vfio_disconnect_container(container->fd); close(container->fd); + if (container->iommu_ctx) { + iommu_context_unregister_ds_iommu(container->iommu_ctx, + &container->dsi_obj); + } + ds_iommu_object_destroy(&container->dsi_obj); g_free(container); =20 vfio_put_address_space(space); @@ -1508,7 +1534,7 @@ VFIOGroup *vfio_get_group(int groupid, AddressSpace *= as, group->groupid =3D groupid; QLIST_INIT(&group->device_list); =20 - if (vfio_connect_container(group, as, errp)) { + if (vfio_connect_container(group, as, iommu_ctx, errp)) { error_prepend(errp, "failed to setup container for group %d: ", groupid); goto close_fd_exit; diff --git a/include/hw/vfio/vfio-common.h b/include/hw/vfio/vfio-common.h index 8ab68fa..dc68552 100644 --- a/include/hw/vfio/vfio-common.h +++ b/include/hw/vfio/vfio-common.h @@ -72,6 +72,8 @@ typedef struct VFIOContainer { MemoryListener listener; MemoryListener prereg_listener; unsigned iommu_type; + IOMMUContext *iommu_ctx; + DualStageIOMMUObject dsi_obj; Error *error; bool initialized; unsigned long pgsizes; --=20 2.7.4