From: "Aneesh Kumar K.V (Arm)" <aneesh.kumar@kernel.org>
To: kvm@vger.kernel.org
Cc: Suzuki K Poulose <Suzuki.Poulose@arm.com>,
Steven Price <steven.price@arm.com>,
Will Deacon <will@kernel.org>,
Julien Thierry <julien.thierry.kdev@gmail.com>,
"Aneesh Kumar K.V (Arm)" <aneesh.kumar@kernel.org>
Subject: [RFC PATCH kvmtool 09/10] vfio/iommufd: Add viommu and vdevice objects
Date: Sun, 25 May 2025 13:19:15 +0530 [thread overview]
Message-ID: <20250525074917.150332-9-aneesh.kumar@kernel.org> (raw)
In-Reply-To: <20250525074917.150332-1-aneesh.kumar@kernel.org>
This also allocates a stage1 bypass and stage2 translate table.
Signed-off-by: Aneesh Kumar K.V (Arm) <aneesh.kumar@kernel.org>
---
builtin-run.c | 2 +
include/kvm/kvm-config.h | 1 +
vfio/core.c | 4 +-
vfio/iommufd.c | 115 ++++++++++++++++++++++++++++++++++++++-
4 files changed, 119 insertions(+), 3 deletions(-)
diff --git a/builtin-run.c b/builtin-run.c
index 39198f9bc0d6..bfa3e8b09f82 100644
--- a/builtin-run.c
+++ b/builtin-run.c
@@ -263,6 +263,8 @@ static int loglevel_parser(const struct option *opt, const char *arg, int unset)
"Assign a PCI device to the virtual machine", \
vfio_device_parser, kvm), \
OPT_BOOLEAN('\0', "iommufd", &(cfg)->iommufd, "Use iommufd interface"), \
+ OPT_BOOLEAN('\0', "iommufd-vdevice", &(cfg)->iommufd_vdevice, \
+ "Use iommufd vdevice interface"), \
\
OPT_GROUP("Debug options:"), \
OPT_CALLBACK_NOOPT('\0', "debug", kvm, NULL, \
diff --git a/include/kvm/kvm-config.h b/include/kvm/kvm-config.h
index 632eaf84b7eb..d80be6826469 100644
--- a/include/kvm/kvm-config.h
+++ b/include/kvm/kvm-config.h
@@ -66,6 +66,7 @@ struct kvm_config {
bool mmio_debug;
int virtio_transport;
bool iommufd;
+ bool iommufd_vdevice;
};
#endif
diff --git a/vfio/core.c b/vfio/core.c
index 0b1796c54ffd..8dfcf3ca35c1 100644
--- a/vfio/core.c
+++ b/vfio/core.c
@@ -373,7 +373,7 @@ static int vfio__init(struct kvm *kvm)
}
kvm_vfio_device = device.fd;
- if (kvm->cfg.iommufd)
+ if (kvm->cfg.iommufd || kvm->cfg.iommufd_vdevice)
return iommufd__init(kvm);
return legacy_vfio__init(kvm);
}
@@ -395,7 +395,7 @@ static int vfio__exit(struct kvm *kvm)
free(kvm->cfg.vfio_devices);
- if (kvm->cfg.iommufd)
+ if (kvm->cfg.iommufd || kvm->cfg.iommufd_vdevice)
return iommufd__exit(kvm);
return legacy_vfio__exit(kvm);
diff --git a/vfio/iommufd.c b/vfio/iommufd.c
index 742550705746..39870320e4ac 100644
--- a/vfio/iommufd.c
+++ b/vfio/iommufd.c
@@ -108,6 +108,116 @@ err_out:
return ret;
}
+static int iommufd_alloc_s1bypass_hwpt(struct vfio_device *vdev)
+{
+ int ret;
+ unsigned long dev_num;
+ unsigned long guest_bdf;
+ struct vfio_device_bind_iommufd bind;
+ struct vfio_device_attach_iommufd_pt attach_data;
+ struct iommu_hwpt_alloc alloc_hwpt;
+ struct iommu_viommu_alloc alloc_viommu;
+ struct iommu_hwpt_arm_smmuv3 bypass_ste;
+ struct iommu_vdevice_alloc alloc_vdev;
+
+ bind.argsz = sizeof(bind);
+ bind.flags = 0;
+ bind.iommufd = iommu_fd;
+
+ /* now bind the iommufd */
+ if (ioctl(vdev->fd, VFIO_DEVICE_BIND_IOMMUFD, &bind)) {
+ ret = -errno;
+ vfio_dev_err(vdev, "failed to get info");
+ goto err_out;
+ }
+
+ alloc_hwpt.size = sizeof(struct iommu_hwpt_alloc);
+ alloc_hwpt.flags = IOMMU_HWPT_ALLOC_NEST_PARENT;
+ alloc_hwpt.dev_id = bind.out_devid;
+ alloc_hwpt.pt_id = ioas_id;
+ alloc_hwpt.data_type = IOMMU_HWPT_DATA_NONE;
+ alloc_hwpt.data_len = 0;
+ alloc_hwpt.data_uptr = 0;
+
+ if (ioctl(iommu_fd, IOMMU_HWPT_ALLOC, &alloc_hwpt)) {
+ ret = -errno;
+ pr_err("Failed to allocate HWPT");
+ goto err_out;
+ }
+
+ attach_data.argsz = sizeof(attach_data);
+ attach_data.flags = 0;
+ attach_data.pt_id = alloc_hwpt.out_hwpt_id;
+
+ if (ioctl(vdev->fd, VFIO_DEVICE_ATTACH_IOMMUFD_PT, &attach_data)) {
+ ret = -errno;
+ vfio_dev_err(vdev, "failed to attach to IOAS ");
+ goto err_out;
+ }
+
+ alloc_viommu.size = sizeof(alloc_viommu);
+ alloc_viommu.flags = 0;
+ alloc_viommu.type = IOMMU_VIOMMU_TYPE_ARM_SMMUV3;
+ alloc_viommu.dev_id = bind.out_devid;
+ alloc_viommu.hwpt_id = alloc_hwpt.out_hwpt_id;
+
+ if (ioctl(iommu_fd, IOMMU_VIOMMU_ALLOC, &alloc_viommu)) {
+ ret = -errno;
+ vfio_dev_err(vdev, "failed to allocate VIOMMU %d", ret);
+ goto err_out;
+ }
+#define STRTAB_STE_0_V (1UL << 0)
+#define STRTAB_STE_0_CFG_S2_TRANS 6
+#define STRTAB_STE_0_CFG_S1_TRANS 5
+#define STRTAB_STE_0_CFG_BYPASS 4
+
+ /* set up virtual ste as bypass ste */
+ bypass_ste.ste[0] = STRTAB_STE_0_V | (STRTAB_STE_0_CFG_BYPASS << 1);
+ bypass_ste.ste[1] = 0x0UL;
+
+ alloc_hwpt.size = sizeof(struct iommu_hwpt_alloc);
+ alloc_hwpt.flags = 0;
+ alloc_hwpt.dev_id = bind.out_devid;
+ alloc_hwpt.pt_id = alloc_viommu.out_viommu_id;
+ alloc_hwpt.data_type = IOMMU_HWPT_DATA_ARM_SMMUV3;
+ alloc_hwpt.data_len = sizeof(bypass_ste);
+ alloc_hwpt.data_uptr = (unsigned long)&bypass_ste;
+
+ if (ioctl(iommu_fd, IOMMU_HWPT_ALLOC, &alloc_hwpt)) {
+ ret = -errno;
+ pr_err("Failed to allocate S1 bypass HWPT %d", ret);
+ goto err_out;
+ }
+
+ alloc_vdev.size = sizeof(alloc_vdev),
+ alloc_vdev.viommu_id = alloc_viommu.out_viommu_id;
+ alloc_vdev.dev_id = bind.out_devid;
+
+ dev_num = vdev->dev_hdr.dev_num;
+ /* kvmtool only do 0 domain, 0 bus and 0 function devices. */
+ guest_bdf = (0ULL << 32) | (0 << 16) | dev_num << 11 | (0 << 8);
+ alloc_vdev.virt_id = guest_bdf;
+ if (ioctl(iommu_fd, IOMMU_VDEVICE_ALLOC, &alloc_vdev)) {
+ ret = -errno;
+ pr_err("Failed to allocate vdevice %d", ret);
+ goto err_out;
+ }
+
+ /* Now attach to the nested domain */
+ attach_data.argsz = sizeof(attach_data);
+ attach_data.flags = 0;
+ attach_data.pt_id = alloc_hwpt.out_hwpt_id;
+ if (ioctl(vdev->fd, VFIO_DEVICE_ATTACH_IOMMUFD_PT, &attach_data)) {
+ ret = -errno;
+ vfio_dev_err(vdev, "failed to attach Nested config to IOAS %d ", ret);
+ goto err_out;
+ }
+
+ return 0;
+err_out:
+ return ret;
+}
+
static int iommufd_configure_device(struct kvm *kvm, struct vfio_device *vdev)
{
int ret;
@@ -160,7 +270,10 @@ static int iommufd_configure_device(struct kvm *kvm, struct vfio_device *vdev)
goto err_close_device;
}
- ret = iommufd_alloc_s2bypass_hwpt(vdev);
+ if (kvm->cfg.iommufd_vdevice)
+ ret = iommufd_alloc_s1bypass_hwpt(vdev);
+ else
+ ret = iommufd_alloc_s2bypass_hwpt(vdev);
if (ret)
goto err_close_device;
--
2.43.0
next prev parent reply other threads:[~2025-05-25 7:50 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-25 7:49 [RFC PATCH kvmtool 01/10] vfio: Associate vm instance with vfio fd Aneesh Kumar K.V (Arm)
2025-05-25 7:49 ` [RFC PATCH kvmtool 02/10] vfio: Rename some functions Aneesh Kumar K.V (Arm)
2025-07-27 18:20 ` Mostafa Saleh
2025-07-29 4:53 ` Aneesh Kumar K.V
2025-05-25 7:49 ` [RFC PATCH kvmtool 03/10] vfio: Create new file legacy.c Aneesh Kumar K.V (Arm)
2025-07-27 18:23 ` Mostafa Saleh
2025-07-29 4:59 ` Aneesh Kumar K.V
2025-05-25 7:49 ` [RFC PATCH kvmtool 04/10] vfio: Update vfio header from linux kernel Aneesh Kumar K.V (Arm)
2025-07-27 18:23 ` Mostafa Saleh
2025-05-25 7:49 ` [RFC PATCH kvmtool 05/10] vfio: Add dma map/unmap handlers Aneesh Kumar K.V (Arm)
2025-07-27 18:25 ` Mostafa Saleh
2025-07-29 5:03 ` Aneesh Kumar K.V
2025-05-25 7:49 ` [RFC PATCH kvmtool 06/10] vfio/iommufd: Import iommufd header from kernel Aneesh Kumar K.V (Arm)
2025-07-27 18:25 ` Mostafa Saleh
2025-05-25 7:49 ` [RFC PATCH kvmtool 07/10] vfio/iommufd: Add basic iommufd support Aneesh Kumar K.V (Arm)
2025-07-27 18:31 ` Mostafa Saleh
2025-07-29 5:12 ` Aneesh Kumar K.V
2025-07-29 9:38 ` Mostafa Saleh
2025-05-25 7:49 ` [RFC PATCH kvmtool 08/10] vfio/iommufd: Move the hwpt allocation to helper Aneesh Kumar K.V (Arm)
2025-07-27 18:32 ` Mostafa Saleh
2025-07-29 5:14 ` Aneesh Kumar K.V
2025-07-29 9:43 ` Mostafa Saleh
2025-05-25 7:49 ` Aneesh Kumar K.V (Arm) [this message]
2025-07-21 12:27 ` [RFC PATCH kvmtool 09/10] vfio/iommufd: Add viommu and vdevice objects Will Deacon
2025-07-24 14:09 ` Aneesh Kumar K.V
2025-08-04 22:33 ` Suzuki K Poulose
2025-08-08 13:00 ` Will Deacon
2025-08-11 6:16 ` Aneesh Kumar K.V
2025-07-27 18:35 ` Mostafa Saleh
2025-07-29 5:19 ` Aneesh Kumar K.V
2025-07-29 9:41 ` Mostafa Saleh
2025-07-30 8:13 ` Aneesh Kumar K.V
2025-07-30 14:15 ` Mostafa Saleh
2025-07-31 4:39 ` Aneesh Kumar K.V
2025-08-04 15:07 ` Mostafa Saleh
2025-05-25 7:49 ` [RFC PATCH kvmtool 10/10] util/update_headers: Add vfio related header files to update list Aneesh Kumar K.V (Arm)
2025-07-27 18:35 ` Mostafa Saleh
2025-07-27 18:19 ` [RFC PATCH kvmtool 01/10] vfio: Associate vm instance with vfio fd Mostafa Saleh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250525074917.150332-9-aneesh.kumar@kernel.org \
--to=aneesh.kumar@kernel.org \
--cc=Suzuki.Poulose@arm.com \
--cc=julien.thierry.kdev@gmail.com \
--cc=kvm@vger.kernel.org \
--cc=steven.price@arm.com \
--cc=will@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.