Mesa (staging/19.3): winsys/amdgpu: avoid double simple_mtx_unlock()
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Tue Dec 3 18:40:29 UTC 2019
Module: Mesa
Branch: staging/19.3
Commit: a26064534501fe9857252367a8d593f0c7b8cee3
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=a26064534501fe9857252367a8d593f0c7b8cee3
Author: Jonathan Gray <jsg at jsg.id.au>
Date: Thu Nov 28 16:56:30 2019 +1100
winsys/amdgpu: avoid double simple_mtx_unlock()
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 <jsg at jsg.id.au>
Cc: 19.2 19.3 <mesa-stable at lists.freedesktop.org>
Signed-off-by: Marek Olšák <marek.olsak at amd.com>
(cherry picked from commit 3fe3bde4f2c8813e13fa2992212ca46a894e68b6)
---
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
More information about the mesa-commit
mailing list