From 3fe3bde4f2c8813e13fa2992212ca46a894e68b6 Mon Sep 17 00:00:00 2001 From: Jonathan Gray Date: Thu, 28 Nov 2019 16:56:30 +1100 Subject: [PATCH] winsys/amdgpu: avoid double simple_mtx_unlock() MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit pthread_mutex_unlock() when unlocked is documented by posix as being undefined behaviour. On OpenBSD pthread_mutex_unlock() will call abort(3) if this happens. This occurs in amdgpu_winsys_create() after cb446dc0fa5c68f681108f4613560543aa4cf553 winsys/amdgpu: Add amdgpu_screen_winsys Signed-off-by: Jonathan Gray Cc: 19.2 19.3 Signed-off-by: Marek Olšák --- src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c b/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c index 568d3852893..2a341413033 100644 --- a/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c +++ b/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c @@ -326,7 +326,6 @@ amdgpu_winsys_create(int fd, const struct pipe_screen_config *config, aws = util_hash_table_get(dev_tab, dev); if (aws) { pipe_reference(NULL, &aws->reference); - simple_mtx_unlock(&dev_tab_mutex); /* Release the device handle, because we don't need it anymore. * This function is returning an existing winsys instance, which -- 2.30.2