[PATCH] iommu/exynos: Don't unconditionally steal bus ops
Robin Murphy
robin.murphy at arm.com
Tue Jan 9 03:58:14 PST 2018
On 09/01/18 09:59, Marek Szyprowski wrote:
> Hi Robin,
>
> On 2018-01-08 20:27, Robin Murphy wrote:
>> Removing the early device registration hook overlooked the fact that
>> it only ran conditionally on a compatible device being present in the
>> DT. With exynos_iommu_init() now running as an unconditional initcall,
>> problems arise on non-Exynos systems when other IOMMU drivers find
>> themselves unable to install their ops on the platform bus, or at worst
>> the Exynos ops get called with someone else's domain and all hell breaks
>> loose.
>>
>> Fix this by delaying the setting of bus ops until an Exynos IOMMU is
>> actually found, to replicate the previous order of events.
>>
>> Fixes: 928055a01b3f ("iommu/exynos: Remove custom platform device
>> registration code")
>> Signed-off-by: Robin Murphy <robin.murphy at arm.com>
>
> Right, my fault. However I will prefer to resurrect code added initially
> by commit a7b67cd5d9af "iommu/exynos: Play nice in multi-platform builds".
> There is no need to do all the things done in the exynos_iommu_init on
> non-Exynos platforms.
Yeah; I had a moment of doubt and left the rest as-is, but I guess all
of that global setup could in fact be delayed until the first probe, as
with dma_dev. Anyway, I'll respin just the minimal fix to un-break
multiplatform, and leave any further refactoring up to you.
Thanks,
Robin.
>> ---
>> drivers/iommu/exynos-iommu.c | 16 +++++++---------
>> 1 file changed, 7 insertions(+), 9 deletions(-)
>>
>> diff --git a/drivers/iommu/exynos-iommu.c b/drivers/iommu/exynos-iommu.c
>> index 6a96a4c42153..e9e756156429 100644
>> --- a/drivers/iommu/exynos-iommu.c
>> +++ b/drivers/iommu/exynos-iommu.c
>> @@ -574,6 +574,12 @@ static int __init exynos_sysmmu_probe(struct
>> platform_device *pdev)
>> struct sysmmu_drvdata *data;
>> struct resource *res;
>> + if (platform_bus_type->iommu_ops != &exynos_iommu_ops) {
>> + ret = bus_set_iommu(&platform_bus_type, &exynos_iommu_ops);
>> + if (ret)
>> + return ret;
>> + }
>> +
>> data = devm_kzalloc(dev, sizeof(*data), GFP_KERNEL);
>> if (!data)
>> return -ENOMEM;
>> @@ -1367,16 +1373,8 @@ static int __init exynos_iommu_init(void)
>> goto err_zero_lv2;
>> }
>> - ret = bus_set_iommu(&platform_bus_type, &exynos_iommu_ops);
>> - if (ret) {
>> - pr_err("%s: Failed to register exynos-iommu driver.\n",
>> - __func__);
>> - goto err_set_iommu;
>> - }
>> -
>> return 0;
>> -err_set_iommu:
>> - kmem_cache_free(lv2table_kmem_cache, zero_lv2_table);
>> +
>> err_zero_lv2:
>> platform_driver_unregister(&exynos_sysmmu_driver);
>> err_reg_driver:
>
> Best regards
More information about the linux-arm-kernel
mailing list