[Mesa-dev] [PATCH mesa 06/10] egl+glx: turn LIBGL_DRI3_DISABLE into a boolean
Eric Engestrom
eric.engestrom at imgtec.com
Fri Sep 8 12:40:52 UTC 2017
Instead of setting based on set/unset, allow users to use boolean values.
In the docs, use `DISABLE=true` instead of `DISABLE=1` as it's clearer IMO.
Signed-off-by: Eric Engestrom <eric.engestrom at imgtec.com>
---
docs/envvars.html | 2 +-
src/egl/drivers/dri2/platform_x11.c | 2 +-
src/glx/glxext.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/docs/envvars.html b/docs/envvars.html
index aeb5ff2847..89563e3636 100644
--- a/docs/envvars.html
+++ b/docs/envvars.html
@@ -34,7 +34,7 @@
<li>LIBGL_NO_DRAWARRAYS - if set do not use DrawArrays GLX protocol (for debugging)
<li>LIBGL_SHOW_FPS - print framerate to stdout based on the number of glXSwapBuffers
calls per second.
-<li>LIBGL_DRI3_DISABLE - disable DRI3 if set (the value does not matter)
+<li>LIBGL_DRI3_DISABLE - disable DRI3 if set to `true`.
</ul>
diff --git a/src/egl/drivers/dri2/platform_x11.c b/src/egl/drivers/dri2/platform_x11.c
index 5d5eea3773..d80eee2969 100644
--- a/src/egl/drivers/dri2/platform_x11.c
+++ b/src/egl/drivers/dri2/platform_x11.c
@@ -1461,7 +1461,7 @@ dri2_initialize_x11(_EGLDriver *drv, _EGLDisplay *disp)
if (!env_var_as_boolean("LIBGL_ALWAYS_SOFTWARE", false)) {
#ifdef HAVE_DRI3
- if (!getenv("LIBGL_DRI3_DISABLE"))
+ if (!env_var_as_boolean("LIBGL_DRI3_DISABLE", false))
initialized = dri2_initialize_x11_dri3(drv, disp);
#endif
diff --git a/src/glx/glxext.c b/src/glx/glxext.c
index 35e582cb22..20b5be5987 100644
--- a/src/glx/glxext.c
+++ b/src/glx/glxext.c
@@ -920,7 +920,7 @@ __glXInitialize(Display * dpy)
#if defined(GLX_USE_DRM)
if (glx_direct && glx_accel) {
#if defined(HAVE_DRI3)
- if (!getenv("LIBGL_DRI3_DISABLE"))
+ if (!env_var_as_boolean("LIBGL_DRI3_DISABLE", false))
dpyPriv->dri3Display = dri3_create_display(dpy);
#endif /* HAVE_DRI3 */
dpyPriv->dri2Display = dri2CreateDisplay(dpy);
--
Cheers,
Eric
More information about the mesa-dev
mailing list