<div dir="ltr"><div>Pushed, thanks!</div><div><br></div><div>Marek<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Nov 28, 2019 at 12:56 AM Jonathan Gray <<a href="mailto:jsg@jsg.id.au">jsg@jsg.id.au</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">pthread_mutex_unlock() when unlocked is documented by posix as<br>
being undefined behaviour.  On OpenBSD pthread_mutex_unlock() will call<br>
abort(3) if this happens.<br>
<br>
This occurs in amdgpu_winsys_create() after<br>
cb446dc0fa5c68f681108f4613560543aa4cf553<br>
winsys/amdgpu: Add amdgpu_screen_winsys<br>
<br>
Signed-off-by: Jonathan Gray <<a href="mailto:jsg@jsg.id.au" target="_blank">jsg@jsg.id.au</a>><br>
Cc: 19.2 19.3 <<a href="mailto:mesa-stable@lists.freedesktop.org" target="_blank">mesa-stable@lists.freedesktop.org</a>><br>
---<br>
 src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 1 -<br>
 1 file changed, 1 deletion(-)<br>
<br>
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c b/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c<br>
index a89432d8594..2fe227c9213 100644<br>
--- a/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c<br>
+++ b/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c<br>
@@ -326,7 +326,6 @@ amdgpu_winsys_create(int fd, const struct pipe_screen_config *config,<br>
    aws = util_hash_table_get(dev_tab, dev);<br>
    if (aws) {<br>
       pipe_reference(NULL, &aws->reference);<br>
-      simple_mtx_unlock(&dev_tab_mutex);<br>
<br>
       /* Release the device handle, because we don't need it anymore.<br>
        * This function is returning an existing winsys instance, which<br>
-- <br>
2.24.0<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a></blockquote></div>