[Beignet] [PATCH] Clarify status of cl_khr_gl_sharing

Rebecca N. Palmer rebecca_palmer at zoho.com
Sun Jan 22 22:58:34 UTC 2017


Signed-off-by: Rebecca N. Palmer <rebecca_palmer at zoho.com>
---
> -The release notes say cl_khr_gl_sharing is now supported, but docs/Beignet.mdwn still says it isn't.
It exists but is off by default, probably because it isn't finished:
clEnqueueAcquireGLObjects / clEnqueueReleaseGLObjects are no-ops,
and clGetGLObjectInfo / clGetGLTextureInfo are missing entirely.

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 59abc45..7c01488 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -195,7 +195,7 @@ ELSE(XFIXES_FOUND)
 ENDIF(XFIXES_FOUND)
 ENDIF(X11_FOUND)
 
-OPTION(ENABLE_GL_SHARING "cl_khr_gl_sharing" OFF)
+OPTION(ENABLE_GL_SHARING "Enable (incomplete) cl_khr_gl_sharing" OFF)
 
 IF(ENABLE_GL_SHARING)
   pkg_check_modules(OPENGL REQUIRED gl)
diff --git a/docs/Beignet.mdwn b/docs/Beignet.mdwn
index 5c62b4c..3577d57 100644
--- a/docs/Beignet.mdwn
+++ b/docs/Beignet.mdwn
@@ -222,10 +222,8 @@ Known Issues
   This loses some precision but gains performance.
 
 * cl\_khr\_gl\_sharing.
-  This extension highly depends on mesa support. It seems that mesa would not provide
-  such type of extensions, we may have to hack with mesa source code to support this
-  extension. This feature used to work with a previous mesa git version. But now, it's
-  simply broken.
+  This extension is disabled by default, as it is only partially implemented.
+  It can be enabled by passing -DENABLE_GL_SHARING=ON to cmake.
 
 Project repository
 ------------------



More information about the Beignet mailing list