Messages in this thread | | | Date | Thu, 28 May 2020 08:42:16 +0200 | From | Joerg Roedel <> | Subject | Re: [PATCH 02/10] iommu/amd: Unexport get_dev_data() |
| |
On Wed, May 27, 2020 at 11:13:53PM -0700, Christoph Hellwig wrote: > On Wed, May 27, 2020 at 01:53:05PM +0200, Joerg Roedel wrote: > > From: Joerg Roedel <jroedel@suse.de> > > > > This function is internal to the AMD IOMMU driver and only exported > > because the amd_iommu_v2 modules calls it. But the reason it is called > > from there could better be handled by amd_iommu_is_attach_deferred(). > > So unexport get_dev_data() and use amd_iommu_is_attach_deferred() > > instead. > > Btw, what is the reason amd_iommu_v2 is a separate module? It is > very little code, and other drivers seem to just integrate such > functionality.
The module contains optional functionality that is only needed by the amd_kfd driver, which itself only does something useful on (newer) AMD GPUs. So I made it a separate module back in the days to save the memory when it is not needed. But this caused other problems with the amd_kfd module, when they got loaded in the wrong order. And the module is often loaded by distros anyway, as it successfully loads even when no AMD IOMMU is in the system. The reason for that was to have the symbols available for drivers which can optionally use AMD IOMMUv2 functionality.
In fact I have already thought about making it built-in, just havn't done so yet.
Regards,
Joerg
| |