drm: don't use idr_remove_all()

idr_destroy() can destroy idr by itself and idr_remove_all() is being
deprecated.  Drop its usage.

* drm_ctxbitmap_cleanup() was calling idr_remove_all() but forgetting
  idr_destroy() thus leaking all buffered free idr_layers.  Replace it
  with idr_destroy().

Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: David Airlie <airlied@linux.ie>
Cc: Inki Dae <inki.dae@samsung.com>
Cc: Joonyoung Shim <jy0922.shim@samsung.com>
Cc: Seung-Woo Kim <sw0312.kim@samsung.com>
Cc: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

diff --git a/drivers/gpu/drm/drm_context.c b/drivers/gpu/drm/drm_context.c
index 45adf97e678f9ea887fa46e51eb64c1c0dd93bbc..75f62c5e2a6b4f9e5d9e98cd8259ac027006640d 100644 (file)
--- a/drivers/gpu/drm/drm_context.c
+++ b/drivers/gpu/drm/drm_context.c
@@ -118,7 +118,7 @@ int drm_ctxbitmap_init(struct drm_device * dev)
 void drm_ctxbitmap_cleanup(struct drm_device * dev)
 {
        mutex_lock(&dev->struct_mutex);
-       idr_remove_all(&dev->ctx_idr);
+       idr_destroy(&dev->ctx_idr);
        mutex_unlock(&dev->struct_mutex);
 }
 

diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
index 3bdf2a650d9c4c4ec8f65d1c8f0e1ebd3bcc507e..99928b933b16ade8854dd1fdcb95a45b310812f1 100644 (file)
--- a/drivers/gpu/drm/drm_crtc.c
+++ b/drivers/gpu/drm/drm_crtc.c
@@ -1272,7 +1272,6 @@ void drm_mode_config_cleanup(struct drm_device *dev)
                crtc->funcs->destroy(crtc);
        }
 
-       idr_remove_all(&dev->mode_config.crtc_idr);
        idr_destroy(&dev->mode_config.crtc_idr);
 }
 EXPORT_SYMBOL(drm_mode_config_cleanup);

diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
index be174cab105a7ccc27b127f307a07652fdc8678a..25f91cd23e60932ec9246840de26f650c49047f1 100644 (file)
--- a/drivers/gpu/drm/drm_drv.c
+++ b/drivers/gpu/drm/drm_drv.c
@@ -297,7 +297,6 @@ static void __exit drm_core_exit(void)
 
        unregister_chrdev(DRM_MAJOR, "drm");
 
-       idr_remove_all(&drm_minors_idr);
        idr_destroy(&drm_minors_idr);
 }
 

diff --git a/drivers/gpu/drm/drm_gem.c b/drivers/gpu/drm/drm_gem.c
index 24efae464e2c4660a99daabbc7735e8b9d2ec2d6..e775859f0a830f4694e47d29bc968e3956f8f1d1 100644 (file)
--- a/drivers/gpu/drm/drm_gem.c
+++ b/drivers/gpu/drm/drm_gem.c
@@ -561,8 +561,6 @@ drm_gem_release(struct drm_device *dev, struct drm_file *file_private)
 {
        idr_for_each(&file_private->object_idr,
                     &drm_gem_object_release_handle, file_private);
-
-       idr_remove_all(&file_private->object_idr);
        idr_destroy(&file_private->object_idr);
 }
 

diff --git a/drivers/gpu/drm/exynos/exynos_drm_ipp.c b/drivers/gpu/drm/exynos/exynos_drm_ipp.c
index 1a556354e92fa15ee5b051600662a9f0d98a868b..90398dfbfd75ed9fac8ad459196abb2e8fe4f960 100644 (file)
--- a/drivers/gpu/drm/exynos/exynos_drm_ipp.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_ipp.c
@@ -1786,8 +1786,6 @@ err_iommu:
                        drm_iommu_detach_device(drm_dev, ippdrv->dev);
 
 err_idr:
-       idr_remove_all(&ctx->ipp_idr);
-       idr_remove_all(&ctx->prop_idr);
        idr_destroy(&ctx->ipp_idr);
        idr_destroy(&ctx->prop_idr);
        return ret;
@@ -1965,8 +1963,6 @@ static int ipp_remove(struct platform_device *pdev)
        exynos_drm_subdrv_unregister(&ctx->subdrv);
 
        /* remove,destroy ipp idr */
-       idr_remove_all(&ctx->ipp_idr);
-       idr_remove_all(&ctx->prop_idr);
        idr_destroy(&ctx->ipp_idr);
        idr_destroy(&ctx->prop_idr);
 

diff --git a/drivers/gpu/drm/sis/sis_drv.c b/drivers/gpu/drm/sis/sis_drv.c
index 841065b998a14e2601719f7b9494b1050e380194..5a5325e6b75999863e236a3a85102cd35423ad09 100644 (file)
--- a/drivers/gpu/drm/sis/sis_drv.c
+++ b/drivers/gpu/drm/sis/sis_drv.c
@@ -58,7 +58,6 @@ static int sis_driver_unload(struct drm_device *dev)
 {
        drm_sis_private_t *dev_priv = dev->dev_private;
 
-       idr_remove_all(&dev_priv->object_idr);
        idr_destroy(&dev_priv->object_idr);
 
        kfree(dev_priv);

diff --git a/drivers/gpu/drm/via/via_map.c b/drivers/gpu/drm/via/via_map.c
index c0f1cc7f5ca93404ef52677d7ef2982e570cdc59..d0ab3fb32acd7228cf6494223309d1d94e30e086 100644 (file)
--- a/drivers/gpu/drm/via/via_map.c
+++ b/drivers/gpu/drm/via/via_map.c
@@ -120,7 +120,6 @@ int via_driver_unload(struct drm_device *dev)
 {
        drm_via_private_t *dev_priv = dev->dev_private;
 
-       idr_remove_all(&dev_priv->object_idr);
        idr_destroy(&dev_priv->object_idr);
 
        kfree(dev_priv);