[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250613134117.GD1174925@nvidia.com>
Date: Fri, 13 Jun 2025 10:41:17 -0300
From: Jason Gunthorpe <jgg@...dia.com>
To: Nicolin Chen <nicolinc@...dia.com>
Cc: kevin.tian@...el.com, will@...nel.org, robin.murphy@....com,
joro@...tes.org, ddutile@...hat.com, yi.l.liu@...el.com,
peterz@...radead.org, jsnitsel@...hat.com, praan@...gle.com,
linux-arm-kernel@...ts.infradead.org, iommu@...ts.linux.dev,
linux-kernel@...r.kernel.org, patches@...ts.linux.dev,
baolu.lu@...ux.intel.com
Subject: Re: [PATCH v1 05/12] iommu: Introduce get_viommu_size and
viommu_init ops
On Mon, Jun 09, 2025 at 10:13:28AM -0700, Nicolin Chen wrote:
> @@ -654,6 +665,10 @@ struct iommu_ops {
>
> int (*def_domain_type)(struct device *dev);
>
> + int (*get_viommu_size)(enum iommu_viommu_type viommu_type,
> + struct device *dev, size_t *viommu_size);
I'd return the size in a size_t instead of using an output
pointer. Make 0 mean not supported..
Jason
Powered by blists - more mailing lists