On Mon, 29 Sep 2025 14:36:41 +0100
Shameer Kolothum <skolothum...@nvidia.com> wrote:
> Subsequent patch will make use of this to add a PASID CAP for assigned
> devices.
>
> Signed-off-by: Shameer Kolothum <skolothum...@nvidia.com>
Trivial stuff.
> ---
> backends/iommufd.c | 9 +++++++++
> include/system/host_iommu_device.h | 12 ++++++++++++
> 2 files changed, 21 insertions(+)
>
> diff --git a/backends/iommufd.c b/backends/iommufd.c
> index 023e67bc46..0ff46a5747 100644
> --- a/backends/iommufd.c
> +++ b/backends/iommufd.c
> @@ -523,6 +523,14 @@ bool
> host_iommu_device_iommufd_detach_hwpt(HostIOMMUDeviceIOMMUFD *idev,
> return idevc->detach_hwpt(idev, errp);
> }
>
> +static uint8_t hiod_iommufd_get_pasid(HostIOMMUDevice *hiod, uint64_t
> *hw_caps)
> +{
> + HostIOMMUDeviceCaps *caps = &hiod->caps;
> +
> + *hw_caps = caps->hw_caps;
> + return caps->max_pasid_log2;
> +}
> +
> static int hiod_iommufd_get_cap(HostIOMMUDevice *hiod, int cap, Error **errp)
> {
> HostIOMMUDeviceCaps *caps = &hiod->caps;
> @@ -543,6 +551,7 @@ static void hiod_iommufd_class_init(ObjectClass *oc,
> const void *data)
> HostIOMMUDeviceClass *hioc = HOST_IOMMU_DEVICE_CLASS(oc);
>
> hioc->get_cap = hiod_iommufd_get_cap;
> + hioc->get_pasid = hiod_iommufd_get_pasid;
> };
>
> static const TypeInfo types[] = {
> diff --git a/include/system/host_iommu_device.h
> b/include/system/host_iommu_device.h
> index c6a2a3899a..3773c54977 100644
> --- a/include/system/host_iommu_device.h
> +++ b/include/system/host_iommu_device.h
> @@ -115,6 +115,18 @@ struct HostIOMMUDeviceClass {
> * @hiod: handle to the host IOMMU device
> */
> uint64_t (*get_page_size_mask)(HostIOMMUDevice *hiod);
> + /**
> + * @get_pasid: Get PASID support information along this
> + * @hiod Host IOMMU device
> + * Optional callback. If not implemented, PASID not supported
> + *
> + * @hiod: handle to the host IOMMU device
> + *
> + * @out_hw_caps: Output the generic iommu capability info which includes
> + * device PASID CAP info
Blank line here to match local style.
> + * Returns the width of PASIDs. Zero means no PASID support
* Returns: width of PASIDs. ...
to match other comments in this file. I only checked a few of them
so maybe this style is there somewhere as well.
> + */
> + uint8_t (*get_pasid)(HostIOMMUDevice *hiod, uint64_t *out_hw_caps);
> };
>
> /*
