diff options
| author | Jan Vesely <jan.vesely@rutgers.edu> | 2018-05-10 18:29:13 -0400 |
|---|---|---|
| committer | Dylan Baker <dylan@pnwbakers.com> | 2018-05-11 08:35:13 -0700 |
| commit | 36c24a608e81307e7abbc60f2e397bd1fe514fb4 (patch) | |
| tree | 96b51952c6c2ac9545d82cb8f137f34d47613bce | |
| parent | 1def4eaa5c29a7b25d626a37d55f1541273cf336 (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>
(cherry picked from commit 58272c1ad771802a6f15a482ae552649e9085042)
| -rw-r--r-- | src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 7 |
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; |