summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJan Vesely <jan.vesely@rutgers.edu>2018-05-10 18:29:13 -0400
committerJan Vesely <jan.vesely@rutgers.edu>2018-05-10 23:23:50 -0400
commit58272c1ad771802a6f15a482ae552649e9085042 (patch)
tree9415ae2691cf41788e3a42bad3bd43c4ad84d164
parenta2e9d9b4c199ff1b53a625a345e566200016e2d9 (diff)
winsys/amdgpu: Destroy dev_hash table when the last winsys is removed.
Fixes memory leak on module unload. CC: <mesa-stable@lists.freedesktop.org> Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu> Reviewed-by: Marek Olšák <marek.olsak@amd.com>
-rw-r--r--src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c b/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c
index f4bbd3e7324..84d8ca6fcf3 100644
--- a/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c
+++ b/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c
@@ -220,8 +220,13 @@ static bool amdgpu_winsys_unref(struct radeon_winsys *rws)
simple_mtx_lock(&dev_tab_mutex);
destroy = pipe_reference(&ws->reference, NULL);
- if (destroy && dev_tab)
+ if (destroy && dev_tab) {
util_hash_table_remove(dev_tab, ws->dev);
+ if (util_hash_table_count(dev_tab) == 0) {
+ util_hash_table_destroy(dev_tab);
+ dev_tab = NULL;
+ }
+ }
simple_mtx_unlock(&dev_tab_mutex);
return destroy;