On 07/02/2022 06:41, Lu Baolu wrote:
diff --git a/drivers/iommu/intel/iommu.c b/drivers/iommu/intel/iommu.c
index 583ec0fa4ac1..e8d58654361c 100644
--- a/drivers/iommu/intel/iommu.c
+++ b/drivers/iommu/intel/iommu.c
@@ -3348,9 +3348,6 @@ static inline int iommu_devinfo_cache_init(void)
  static int __init iommu_init_mempool(void)
  {
        int ret;
-       ret = iova_cache_get();
-       if (ret)
-               return ret;
ret = iommu_domain_cache_init();
        if (ret)
@@ -3362,7 +3359,6 @@ static int __init iommu_init_mempool(void)
kmem_cache_destroy(iommu_domain_cache);
  domain_error:

nit: is this label still really required? only failures in iommu_domain_cache_init() jump to it, and that can return directly now.

Thanks,
John

-       iova_cache_put();

_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu

Reply via email to