egl/glx: Remove egl_glx driver
Mesa now has a real, feature-rich EGL implementation on X11 via xcb.
Therefore I believe there is no longer a practical need for the egl_glx
driver.
Furthermore, egl_glx appears to be unmaintained. The most recent
nontrivial commit to egl_glx was 6baa5f1
on 2011-11-25.
Tested by running weston-smoke in windowed Weston on X with i965.
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
Acked-by: Kenneth Graunke <kenneth@whitecape.org>
Acked-by: Kristian Høgsberg <krh@bitplanet.net>
This commit is contained in:
parent
0224bd20f3
commit
1340e24406
11
configure.ac
11
configure.ac
|
@ -1206,8 +1206,6 @@ if test "x$enable_egl" = xyes; then
|
||||||
AC_CHECK_FUNC(mincore, [DEFINES="$DEFINES -DHAVE_MINCORE"])
|
AC_CHECK_FUNC(mincore, [DEFINES="$DEFINES -DHAVE_MINCORE"])
|
||||||
|
|
||||||
if test "$enable_static" != yes; then
|
if test "$enable_static" != yes; then
|
||||||
# build egl_glx when libGL is built
|
|
||||||
|
|
||||||
if test "x$enable_dri" = xyes; then
|
if test "x$enable_dri" = xyes; then
|
||||||
HAVE_EGL_DRIVER_DRI2=1
|
HAVE_EGL_DRIVER_DRI2=1
|
||||||
fi
|
fi
|
||||||
|
@ -1440,10 +1438,6 @@ for plat in $egl_platforms; do
|
||||||
|
|
||||||
x11)
|
x11)
|
||||||
PKG_CHECK_MODULES([XCB_DRI2], [x11-xcb xcb-dri2 >= 1.8 xcb-xfixes])
|
PKG_CHECK_MODULES([XCB_DRI2], [x11-xcb xcb-dri2 >= 1.8 xcb-xfixes])
|
||||||
|
|
||||||
if test "x$enable_glx" = xyes; then
|
|
||||||
HAVE_EGL_DRIVER_GLX=1
|
|
||||||
fi
|
|
||||||
;;
|
;;
|
||||||
|
|
||||||
drm)
|
drm)
|
||||||
|
@ -1486,7 +1480,6 @@ AM_CONDITIONAL(HAVE_EGL_PLATFORM_FBDEV, echo "$egl_platforms" | grep 'fbdev' >/d
|
||||||
AM_CONDITIONAL(HAVE_EGL_PLATFORM_NULL, echo "$egl_platforms" | grep 'null' >/dev/null 2>&1)
|
AM_CONDITIONAL(HAVE_EGL_PLATFORM_NULL, echo "$egl_platforms" | grep 'null' >/dev/null 2>&1)
|
||||||
|
|
||||||
AM_CONDITIONAL(HAVE_EGL_DRIVER_DRI2, test "x$HAVE_EGL_DRIVER_DRI2" != "x")
|
AM_CONDITIONAL(HAVE_EGL_DRIVER_DRI2, test "x$HAVE_EGL_DRIVER_DRI2" != "x")
|
||||||
AM_CONDITIONAL(HAVE_EGL_DRIVER_GLX, test "x$HAVE_EGL_DRIVER_GLX" != "x")
|
|
||||||
|
|
||||||
AC_SUBST([EGL_NATIVE_PLATFORM])
|
AC_SUBST([EGL_NATIVE_PLATFORM])
|
||||||
AC_SUBST([EGL_CFLAGS])
|
AC_SUBST([EGL_CFLAGS])
|
||||||
|
@ -1976,7 +1969,6 @@ AC_CONFIG_FILES([Makefile
|
||||||
src/egl/Makefile
|
src/egl/Makefile
|
||||||
src/egl/drivers/Makefile
|
src/egl/drivers/Makefile
|
||||||
src/egl/drivers/dri2/Makefile
|
src/egl/drivers/dri2/Makefile
|
||||||
src/egl/drivers/glx/Makefile
|
|
||||||
src/egl/main/Makefile
|
src/egl/main/Makefile
|
||||||
src/egl/main/egl.pc
|
src/egl/main/egl.pc
|
||||||
src/egl/wayland/Makefile
|
src/egl/wayland/Makefile
|
||||||
|
@ -2158,9 +2150,6 @@ if test "$enable_egl" = yes; then
|
||||||
echo " EGL platforms: $egl_platforms"
|
echo " EGL platforms: $egl_platforms"
|
||||||
|
|
||||||
egl_drivers=""
|
egl_drivers=""
|
||||||
if test "x$HAVE_EGL_DRIVER_GLX" != "x"; then
|
|
||||||
egl_drivers="$egl_drivers builtin:egl_glx"
|
|
||||||
fi
|
|
||||||
if test "x$HAVE_EGL_DRIVER_DRI2" != "x"; then
|
if test "x$HAVE_EGL_DRIVER_DRI2" != "x"; then
|
||||||
egl_drivers="$egl_drivers builtin:egl_dri2"
|
egl_drivers="$egl_drivers builtin:egl_dri2"
|
||||||
fi
|
fi
|
||||||
|
|
|
@ -232,16 +232,6 @@ The supported platforms are X11, DRM, FBDEV, and GDI.</p>
|
||||||
|
|
||||||
</dd>
|
</dd>
|
||||||
|
|
||||||
<dt><code>egl_glx</code></dt>
|
|
||||||
<dd>
|
|
||||||
|
|
||||||
<p>This driver provides a wrapper to GLX. It uses exclusively GLX to implement
|
|
||||||
the EGL API. It supports both direct and indirect rendering when the GLX does.
|
|
||||||
It is accelerated when the GLX is. As such, it cannot provide functions that
|
|
||||||
is not available in GLX or GLX extensions.</p>
|
|
||||||
</dd>
|
|
||||||
</dl>
|
|
||||||
|
|
||||||
<h2>Packaging</h2>
|
<h2>Packaging</h2>
|
||||||
|
|
||||||
<p>The ABI between the main library and its drivers are not stable. Nor is
|
<p>The ABI between the main library and its drivers are not stable. Nor is
|
||||||
|
@ -262,10 +252,6 @@ is disabled by default.</p>
|
||||||
<code>src/egl/</code>. The sources of the <code>egl</code> state tracker can
|
<code>src/egl/</code>. The sources of the <code>egl</code> state tracker can
|
||||||
be found at <code>src/gallium/state_trackers/egl/</code>.</p>
|
be found at <code>src/gallium/state_trackers/egl/</code>.</p>
|
||||||
|
|
||||||
<p>The suggested way to learn to write a EGL driver is to see how other drivers
|
|
||||||
are written. <code>egl_glx</code> should be a good reference. It works in any
|
|
||||||
environment that has GLX support, and it is simpler than most drivers.</p>
|
|
||||||
|
|
||||||
<h3>Lifetime of Display Resources</h3>
|
<h3>Lifetime of Display Resources</h3>
|
||||||
|
|
||||||
<p>Contexts and surfaces are examples of display resources. They might live
|
<p>Contexts and surfaces are examples of display resources. They might live
|
||||||
|
|
|
@ -21,10 +21,6 @@
|
||||||
|
|
||||||
SUBDIRS =
|
SUBDIRS =
|
||||||
|
|
||||||
if HAVE_EGL_DRIVER_GLX
|
|
||||||
SUBDIRS += glx
|
|
||||||
endif
|
|
||||||
|
|
||||||
if HAVE_EGL_DRIVER_DRI2
|
if HAVE_EGL_DRIVER_DRI2
|
||||||
SUBDIRS += dri2
|
SUBDIRS += dri2
|
||||||
endif
|
endif
|
||||||
|
|
|
@ -1,34 +0,0 @@
|
||||||
# Copyright © 2012 Intel Corporation
|
|
||||||
#
|
|
||||||
# Permission is hereby granted, free of charge, to any person obtaining a
|
|
||||||
# copy of this software and associated documentation files (the "Software"),
|
|
||||||
# to deal in the Software without restriction, including without limitation
|
|
||||||
# the rights to use, copy, modify, merge, publish, distribute, sublicense,
|
|
||||||
# and/or sell copies of the Software, and to permit persons to whom the
|
|
||||||
# Software is furnished to do so, subject to the following conditions:
|
|
||||||
#
|
|
||||||
# The above copyright notice and this permission notice (including the next
|
|
||||||
# paragraph) shall be included in all copies or substantial portions of the
|
|
||||||
# Software.
|
|
||||||
#
|
|
||||||
# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
||||||
# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
||||||
# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
|
|
||||||
# THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
||||||
# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
|
|
||||||
# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
|
|
||||||
# IN THE SOFTWARE.
|
|
||||||
|
|
||||||
AM_CFLAGS = \
|
|
||||||
-I$(top_srcdir)/include \
|
|
||||||
-I$(top_srcdir)/src/egl/main \
|
|
||||||
$(VISIBILITY_CFLAGS) \
|
|
||||||
$(X11_INCLUDES) \
|
|
||||||
$(DEFINES)
|
|
||||||
|
|
||||||
noinst_LTLIBRARIES = libegl_glx.la
|
|
||||||
|
|
||||||
libegl_glx_la_SOURCES = egl_glx.c
|
|
||||||
|
|
||||||
libegl_glx_la_LIBADD = \
|
|
||||||
$(EGL_LIB_DEPS)
|
|
File diff suppressed because it is too large
Load Diff
|
@ -102,12 +102,6 @@ if HAVE_EGL_PLATFORM_NULL
|
||||||
AM_CFLAGS += -DHAVE_NULL_PLATFORM
|
AM_CFLAGS += -DHAVE_NULL_PLATFORM
|
||||||
endif
|
endif
|
||||||
|
|
||||||
if HAVE_EGL_DRIVER_GLX
|
|
||||||
AM_CFLAGS += -D_EGL_BUILT_IN_DRIVER_GLX
|
|
||||||
libEGL_la_LIBADD += ../drivers/glx/libegl_glx.la
|
|
||||||
libEGL_la_LIBADD += $(DLOPEN_LIBS)
|
|
||||||
endif
|
|
||||||
|
|
||||||
if HAVE_EGL_DRIVER_DRI2
|
if HAVE_EGL_DRIVER_DRI2
|
||||||
AM_CFLAGS += -D_EGL_BUILT_IN_DRIVER_DRI2
|
AM_CFLAGS += -D_EGL_BUILT_IN_DRIVER_DRI2
|
||||||
AM_CFLAGS += -DHAVE_XCB_DRI2
|
AM_CFLAGS += -DHAVE_XCB_DRI2
|
||||||
|
|
|
@ -72,9 +72,6 @@ const struct {
|
||||||
#endif
|
#endif
|
||||||
#ifdef _EGL_BUILT_IN_DRIVER_DRI2
|
#ifdef _EGL_BUILT_IN_DRIVER_DRI2
|
||||||
{ "egl_dri2", _eglBuiltInDriverDRI2 },
|
{ "egl_dri2", _eglBuiltInDriverDRI2 },
|
||||||
#endif
|
|
||||||
#ifdef _EGL_BUILT_IN_DRIVER_GLX
|
|
||||||
{ "egl_glx", _eglBuiltInDriverGLX },
|
|
||||||
#endif
|
#endif
|
||||||
{ NULL, NULL }
|
{ NULL, NULL }
|
||||||
};
|
};
|
||||||
|
|
Loading…
Reference in New Issue