diff options
author | Tejun Heo <tj@kernel.org> | 2013-02-27 20:03:39 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-02-27 22:10:13 -0500 |
commit | 4d53233a36fdda567cd4d080e27e1ee4b669ddd1 (patch) | |
tree | ea2db52bd4aea72c20eb7630e4f350c54772e0d6 /drivers/gpu/drm/drm_context.c | |
parent | 748689d40c2554b91197349a98ef084dc0fa70c8 (diff) |
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>
Diffstat (limited to 'drivers/gpu/drm/drm_context.c')
-rw-r--r-- | drivers/gpu/drm/drm_context.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/drm_context.c b/drivers/gpu/drm/drm_context.c index 45adf97e678f..75f62c5e2a6b 100644 --- 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) | |||
118 | void drm_ctxbitmap_cleanup(struct drm_device * dev) | 118 | void drm_ctxbitmap_cleanup(struct drm_device * dev) |
119 | { | 119 | { |
120 | mutex_lock(&dev->struct_mutex); | 120 | mutex_lock(&dev->struct_mutex); |
121 | idr_remove_all(&dev->ctx_idr); | 121 | idr_destroy(&dev->ctx_idr); |
122 | mutex_unlock(&dev->struct_mutex); | 122 | mutex_unlock(&dev->struct_mutex); |
123 | } | 123 | } |
124 | 124 | ||