Oops sorry, forget about this one. This does not work ok :/ Need to take a closer look why.

On 06/16/2017 10:21 AM, Tapani Pälli wrote:
From: Randy Xu <[email protected]>

This was causing crashes in Vulkan and GLES stress testing.

Currently, both libEGL and the DRI module call dup when the function
is called. Remove the former since it's ultimately a driver decision
how to manage the file descriptor.

v2: remove dup from the EGL layer (Emil Velikov)

Fixes: 0201f01dc4e ("egl: add EGL_ANDROID_native_fence_sync")
Signed-off-by: Randy Xu <[email protected]>
---
  src/egl/drivers/dri2/egl_dri2.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
index 020a0bc..0caccd4 100644
--- a/src/egl/drivers/dri2/egl_dri2.c
+++ b/src/egl/drivers/dri2/egl_dri2.c
@@ -2885,7 +2885,7 @@ dri2_dup_native_fence_fd(_EGLDriver *drv, _EGLDisplay 
*dpy, _EGLSync *sync)
        return EGL_NO_NATIVE_FENCE_FD_ANDROID;
     }
- return dup(sync->SyncFd);
+   return sync->SyncFd;
  }
static EGLint

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to