[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250515153903.GI382960@nvidia.com>
Date: Thu, 15 May 2025 12:39:03 -0300
From: Jason Gunthorpe <jgg@...dia.com>
To: Nicolin Chen <nicolinc@...dia.com>
Cc: kevin.tian@...el.com, corbet@....net, will@...nel.org,
bagasdotme@...il.com, robin.murphy@....com, joro@...tes.org,
thierry.reding@...il.com, vdumpa@...dia.com, jonathanh@...dia.com,
shuah@...nel.org, jsnitsel@...hat.com, nathan@...nel.org,
peterz@...radead.org, yi.l.liu@...el.com, mshavit@...gle.com,
praan@...gle.com, zhangzekun11@...wei.com, iommu@...ts.linux.dev,
linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-tegra@...r.kernel.org,
linux-kselftest@...r.kernel.org, patches@...ts.linux.dev,
mochs@...dia.com, alok.a.tiwari@...cle.com, vasant.hegde@....com
Subject: Re: [PATCH v4 10/23] iommufd/viommu: Introduce IOMMUFD_OBJ_HW_QUEUE
and its related struct
On Thu, May 08, 2025 at 08:02:31PM -0700, Nicolin Chen wrote:
> +#define iommufd_hw_queue_alloc(viommu, drv_struct, member) \
> + ({ \
> + drv_struct *ret; \
> + \
> + static_assert(__same_type(struct iommufd_viommu, *viommu)); \
> + static_assert(__same_type(struct iommufd_hw_queue, \
> + ((drv_struct *)NULL)->member)); \
> + static_assert(offsetof(drv_struct, member.obj) == 0); \
> + ret = (drv_struct *)_iommufd_object_alloc( \
> + viommu->ictx, sizeof(drv_struct), \
> + IOMMUFD_OBJ_HW_QUEUE); \
> + if (!IS_ERR(ret)) { \
> + ret->member.viommu = viommu; \
> + ret->member.ictx = viommu->ictx; \
> + } \
> + ret; \
> + })
This should just call
__iommufd_object_alloc(viommu->ictx, ret, IOMMUFD_OBJ_HW_QUEUE member.obj)
And all the casting and asserts should be in that macro, move it to
this header
> /* Helper for IOMMU driver to destroy structures created by allocators above */
> #define iommufd_struct_destroy(drv_struct, member) \
> ({ \
This is abort not destroy, the names should be consistent. But looking
more at the series I think it will be better to do the little rework I
suggested and not give this function to the driver in the first place.
Jason
Powered by blists - more mailing lists