freedreno: Fix leak of the screen hash table.
authorEmma Anholt <emma@anholt.net>
Tue, 22 Jun 2021 19:09:44 +0000 (12:09 -0700)
committerMarge Bot <eric+marge@anholt.net>
Tue, 22 Jun 2021 20:39:18 +0000 (20:39 +0000)
Now we have no leaks detected on a run of a basic deqp test
(dEQP-GLES2.functional.shaders.constants.const_float_assignment_3_fragment).

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11537>

src/gallium/winsys/freedreno/drm/freedreno_drm_winsys.c

index 6773af4..211e165 100644 (file)
@@ -54,6 +54,11 @@ fd_drm_screen_destroy(struct pipe_screen *pscreen)
        if (destroy) {
                int fd = fd_device_fd(screen->dev);
                _mesa_hash_table_remove_key(fd_tab, intptr_to_pointer(fd));
+
+               if (!fd_tab->entries) {
+                       _mesa_hash_table_destroy(fd_tab, NULL);
+                       fd_tab = NULL;
+               }
        }
        mtx_unlock(&fd_screen_mutex);