dri: Merge drisw_util.c into dri_util.c
authorEric Anholt <eric@anholt.net>
Thu, 26 Sep 2013 18:35:31 +0000 (11:35 -0700)
committerEric Anholt <eric@anholt.net>
Thu, 10 Oct 2013 23:34:30 +0000 (16:34 -0700)
The only important difference was not calling drmGetVersion, and making
the swrast extension vtable.  That doesn't justify duplicating the other
330 lines of code.

v2: fix the scons build (code by Emil Velikov)
v3: fix scons build with swrast-only (code by Emil Velikov)
v4: Drop the new define I added, when we already have __NOT_HAVE_DRM_H.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
src/gallium/targets/SConscript.dri
src/gallium/targets/dri-swrast/Makefile.am
src/gallium/targets/dri-swrast/SConscript
src/mesa/drivers/SConscript
src/mesa/drivers/dri/common/dri_util.c
src/mesa/drivers/dri/common/drisw_util.c [deleted file]
src/mesa/drivers/dri/swrast/Makefile.sources

index 04c8ec789f83b762b2c93bf469785b0f5f0d780f..8b15532e3c0649684ea84e6a1f6c2f9e7e3ce1ec 100644 (file)
@@ -3,9 +3,6 @@
 
 Import('*')
 
-if not env['dri']:
-    Return()
-
 drienv = env.Clone()
 
 drienv.Replace(CPPPATH = [
@@ -30,6 +27,11 @@ drienv.Replace(CPPPATH = [
     '#src/egl/drivers/dri',
 ])
 
+driswenv = drienv.Clone()
+driswenv.Append(CPPDEFINES = [
+    '__NOT_HAVE_DRM_H',
+])
+
 drienv.PkgUseModules('DRM')
 
 dri_common_utils = drienv.SharedObject(
@@ -47,9 +49,9 @@ dri_common_dri_util = drienv.SharedObject(
     source = '#src/mesa/drivers/dri/common/dri_util.c'
 )
 
-dri_common_drisw_util = drienv.SharedObject(
+dri_common_drisw_util = driswenv.SharedObject(
     target = 'drisw_util.o',
-    source = '#src/mesa/drivers/dri/common/drisw_util.c'
+    source = '#src/mesa/drivers/dri/common/dri_util.c'
 )
 
 
@@ -69,8 +71,13 @@ drienv.AppendUnique(LIBS = [
     'expat',
 ])
 
+driswenv.AppendUnique(LIBS = [
+    'expat',
+])
+
 Export([
     'drienv',
+    'driswenv',
     'COMMON_DRI_SW_OBJECTS',
     'COMMON_DRI_DRM_OBJECTS',
 ])
index cdbda976457adc0585b78da52d4b105cc3d7dd87..5d2f146386854cafaf6a8bca4ff6aa23c9d654bc 100644 (file)
@@ -44,7 +44,7 @@ dri_LTLIBRARIES = swrast_dri.la
 swrast_dri_la_SOURCES = \
        swrast_drm_api.c \
        $(top_srcdir)/src/mesa/drivers/dri/common/utils.c \
-       $(top_srcdir)/src/mesa/drivers/dri/common/drisw_util.c \
+       $(top_srcdir)/src/mesa/drivers/dri/common/dri_util.c \
        $(top_srcdir)/src/mesa/drivers/dri/common/xmlconfig.c
 
 swrast_dri_la_LDFLAGS = $(DRI_DRIVER_LDFLAGS)
index 6b64c56413cf7b658de4da4da8d05b30229d316e..8ddd14ea5a7858a49382e7640e95e731dd7b614c 100644 (file)
@@ -1,6 +1,6 @@
 Import('*')
 
-env = drienv.Clone()
+env = driswenv.Clone()
 
 env.Append(CPPPATH = [
     '#/src/gallium/winsys/sw/dri',
index 6dcc50655016439e6cbe27724d340c15511a1adb..355e6807a78b0c972f97767089e3d801f76d9f90 100644 (file)
@@ -5,7 +5,7 @@ SConscript('osmesa/SConscript')
 if env['x11']:
     SConscript('x11/SConscript')
 
-if env['drm']:
+if env['dri']:
     SConscript('dri/common/xmlpool/SConscript')
 
 if env['platform'] == 'windows':
index fec05a0c20e93f2d8e396ae4b83c8b7eefc300c6..424b65d7a2fc10ff48db199146bd81f46ba0a7da 100644 (file)
@@ -39,7 +39,9 @@
  */
 
 
+#ifndef __NOT_HAVE_DRM_H
 #include <xf86drm.h>
+#endif
 #include "dri_util.h"
 #include "utils.h"
 #include "xmlpool.h"
@@ -70,6 +72,8 @@ setupLoaderExtensions(__DRIscreen *psp,
            psp->dri2.image = (__DRIimageLookupExtension *) extensions[i];
        if (strcmp(extensions[i]->name, __DRI_USE_INVALIDATE) == 0)
            psp->dri2.useInvalidate = (__DRIuseInvalidateExtension *) extensions[i];
+       if (strcmp(extensions[i]->name, __DRI_SWRAST_LOADER) == 0)
+           psp->swrast_loader = (__DRIswrastLoaderExtension *) extensions[i];
     }
 }
 
@@ -87,7 +91,6 @@ dri2CreateNewScreen(int scrn, int fd,
 {
     static const __DRIextension *emptyExtensionList[] = { NULL };
     __DRIscreen *psp;
-    drmVersionPtr version;
 
     psp = calloc(1, sizeof(*psp));
     if (!psp)
@@ -95,13 +98,17 @@ dri2CreateNewScreen(int scrn, int fd,
 
     setupLoaderExtensions(psp, extensions);
 
-    version = drmGetVersion(fd);
-    if (version) {
-       psp->drm_version.major = version->version_major;
-       psp->drm_version.minor = version->version_minor;
-       psp->drm_version.patch = version->version_patchlevel;
-       drmFreeVersion(version);
+#ifndef __NOT_HAVE_DRM_H
+    if (fd != -1) {
+       drmVersionPtr version = drmGetVersion(fd);
+       if (version) {
+          psp->drm_version.major = version->version_major;
+          psp->drm_version.minor = version->version_minor;
+          psp->drm_version.patch = version->version_patchlevel;
+          drmFreeVersion(version);
+       }
     }
+#endif
 
     psp->loaderPrivate = data;
 
@@ -123,6 +130,14 @@ dri2CreateNewScreen(int scrn, int fd,
     return psp;
 }
 
+/** swrast driver createNewScreen entrypoint. */
+static __DRIscreen *
+driCreateNewScreen(int scrn, const __DRIextension **extensions,
+                  const __DRIconfig ***driver_configs, void *data)
+{
+   return dri2CreateNewScreen(scrn, -1, extensions, driver_configs, data);
+}
+
 /**
  * Destroy the per-screen private information.
  * 
@@ -562,6 +577,19 @@ dri2GetAPIMask(__DRIscreen *screen)
     return screen->api_mask;
 }
 
+/**
+ * swrast swapbuffers entrypoint.
+ *
+ * DRI2 implements this inside the loader with only flushes handled by the
+ * driver.
+ */
+static void
+driSwapBuffers(__DRIdrawable *pdp)
+{
+    assert(pdp->driScreenPriv->swrast_loader);
+
+    driDriverAPI.SwapBuffers(pdp);
+}
 
 /** Core interface */
 const __DRIcoreExtension driCoreExtension = {
@@ -574,8 +602,8 @@ const __DRIcoreExtension driCoreExtension = {
     .indexConfigAttrib          = driIndexConfigAttrib,
     .createNewDrawable          = NULL,
     .destroyDrawable            = driDestroyDrawable,
-    .swapBuffers                = NULL,
-    .createNewContext           = NULL,
+    .swapBuffers                = driSwapBuffers, /* swrast */
+    .createNewContext           = dri2CreateNewContext, /* swrast */
     .copyContext                = driCopyContext,
     .destroyContext             = driDestroyContext,
     .bindContext                = driBindContext,
@@ -596,6 +624,14 @@ const __DRIdri2Extension driDRI2Extension = {
     .createContextAttribs       = dri2CreateContextAttribs
 };
 
+const __DRIswrastExtension driSWRastExtension = {
+    { __DRI_SWRAST, __DRI_SWRAST_VERSION },
+    driCreateNewScreen,
+    dri2CreateNewDrawable,
+    dri2CreateNewContextForAPI,
+    dri2CreateContextAttribs
+};
+
 const __DRI2configQueryExtension dri2ConfigQueryExtension = {
    .base = { __DRI2_CONFIG_QUERY, __DRI2_CONFIG_QUERY_VERSION },
 
diff --git a/src/mesa/drivers/dri/common/drisw_util.c b/src/mesa/drivers/dri/common/drisw_util.c
deleted file mode 100644 (file)
index 89f03c3..0000000
+++ /dev/null
@@ -1,375 +0,0 @@
-/*
- * Copyright 1998-1999 Precision Insight, Inc., Cedar Park, Texas.
- * All Rights Reserved.
- * Copyright 2010 George Sapountzis <gsapountzis@gmail.com>
- *
- * 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 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.
- */
-
-/**
- * \file drisw_util.c
- *
- * DRISW utility functions, i.e. dri_util.c stripped from drm-specific bits.
- */
-
-#include "dri_util.h"
-#include "utils.h"
-
-
-/**
- * Screen functions
- */
-
-static void
-setupLoaderExtensions(__DRIscreen *psp,
-                     const __DRIextension **extensions)
-{
-    int i;
-
-    for (i = 0; extensions[i]; i++) {
-       if (strcmp(extensions[i]->name, __DRI_SWRAST_LOADER) == 0)
-           psp->swrast_loader = (__DRIswrastLoaderExtension *) extensions[i];
-    }
-}
-
-static __DRIscreen *
-driCreateNewScreen(int scrn, const __DRIextension **extensions,
-                  const __DRIconfig ***driver_configs, void *data)
-{
-    static const __DRIextension *emptyExtensionList[] = { NULL };
-    __DRIscreen *psp;
-
-    psp = CALLOC_STRUCT(__DRIscreenRec);
-    if (!psp)
-       return NULL;
-
-    setupLoaderExtensions(psp, extensions);
-
-    psp->loaderPrivate = data;
-
-    psp->extensions = emptyExtensionList;
-    psp->fd = -1;
-    psp->myNum = scrn;
-
-    *driver_configs = driDriverAPI.InitScreen(psp);
-    if (*driver_configs == NULL) {
-       free(psp);
-       return NULL;
-    }
-
-    return psp;
-}
-
-static void driDestroyScreen(__DRIscreen *psp)
-{
-    if (psp) {
-       driDriverAPI.DestroyScreen(psp);
-       free(psp);
-    }
-}
-
-static const __DRIextension **driGetExtensions(__DRIscreen *psp)
-{
-    return psp->extensions;
-}
-
-
-/**
- * Context functions
- */
-
-static __DRIcontext *
-driCreateContextAttribs(__DRIscreen *screen, int api,
-                       const __DRIconfig *config,
-                       __DRIcontext *shared,
-                       unsigned num_attribs,
-                       const uint32_t *attribs,
-                       unsigned *error,
-                       void *data)
-{
-    __DRIcontext *pcp;
-    const struct gl_config *modes = (config != NULL) ? &config->modes : NULL;
-    void * const shareCtx = (shared != NULL) ? shared->driverPrivate : NULL;
-    gl_api mesa_api;
-    unsigned major_version = 1;
-    unsigned minor_version = 0;
-    uint32_t flags = 0;
-
-    /* Either num_attribs is zero and attribs is NULL, or num_attribs is not
-     * zero and attribs is not NULL.
-     */
-    assert((num_attribs == 0) == (attribs == NULL));
-
-    switch (api) {
-    case __DRI_API_OPENGL:
-            mesa_api = API_OPENGL_COMPAT;
-            break;
-    case __DRI_API_GLES:
-            mesa_api = API_OPENGLES;
-            break;
-    case __DRI_API_GLES2:
-    case __DRI_API_GLES3:
-            mesa_api = API_OPENGLES2;
-            break;
-    case __DRI_API_OPENGL_CORE:
-            mesa_api = API_OPENGL_CORE;
-            break;
-    default:
-            *error = __DRI_CTX_ERROR_BAD_API;
-            return NULL;
-    }
-
-    for (unsigned i = 0; i < num_attribs; i++) {
-       switch (attribs[i * 2]) {
-       case __DRI_CTX_ATTRIB_MAJOR_VERSION:
-           major_version = attribs[i * 2 + 1];
-           break;
-       case __DRI_CTX_ATTRIB_MINOR_VERSION:
-           minor_version = attribs[i * 2 + 1];
-           break;
-       case __DRI_CTX_ATTRIB_FLAGS:
-           flags = attribs[i * 2 + 1];
-           break;
-       default:
-           /* We can't create a context that satisfies the requirements of an
-            * attribute that we don't understand.  Return failure.
-            */
-           return NULL;
-       }
-    }
-
-    /* Mesa does not support the GL_ARB_compatibilty extension or the
-     * compatibility profile.  This means that we treat a API_OPENGL_COMPAT 3.1 as
-     * API_OPENGL_CORE and reject API_OPENGL_COMPAT 3.2+.
-     */
-    if (mesa_api == API_OPENGL_COMPAT && major_version == 3 && minor_version == 1)
-       mesa_api = API_OPENGL_CORE;
-
-    if (mesa_api == API_OPENGL_COMPAT
-        && ((major_version > 3)
-            || (major_version == 3 && minor_version >= 2))) {
-       *error = __DRI_CTX_ERROR_BAD_API;
-       return NULL;
-    }
-    /* There are no forward-compatible contexts before OpenGL 3.0.  The
-     * GLX_ARB_create_context spec says:
-     *
-     *     "Forward-compatible contexts are defined only for OpenGL versions
-     *     3.0 and later."
-     *
-     * Moreover, Mesa can't fulfill the requirements of a forward-looking
-     * context.  Return failure if a forward-looking context is requested.
-     *
-     * In Mesa, a debug context is the same as a regular context.
-     */
-    if (major_version >= 3) {
-       if ((flags & ~__DRI_CTX_FLAG_DEBUG) != 0)
-           return NULL;
-    }
-
-    pcp = CALLOC_STRUCT(__DRIcontextRec);
-    if (!pcp)
-        return NULL;
-
-    pcp->loaderPrivate = data;
-
-    pcp->driScreenPriv = screen;
-    pcp->driDrawablePriv = NULL;
-    pcp->driReadablePriv = NULL;
-
-    if (!driDriverAPI.CreateContext(mesa_api, modes, pcp,
-                                   major_version, minor_version,
-                                   flags, error, shareCtx)) {
-        free(pcp);
-        return NULL;
-    }
-
-    return pcp;
-}
-
-static __DRIcontext *
-driCreateNewContextForAPI(__DRIscreen *psp, int api,
-                          const __DRIconfig *config,
-                          __DRIcontext *shared, void *data)
-{
-    unsigned error;
-
-    return driCreateContextAttribs(psp, api, config, shared, 0, NULL,
-                                  &error, data);
-}
-
-static __DRIcontext *
-driCreateNewContext(__DRIscreen *psp, const __DRIconfig *config,
-                   __DRIcontext *shared, void *data)
-{
-    return driCreateNewContextForAPI(psp, __DRI_API_OPENGL,
-                                    config, shared, data);
-}
-
-static void
-driDestroyContext(__DRIcontext *pcp)
-{
-    if (pcp) {
-       driDriverAPI.DestroyContext(pcp);
-       free(pcp);
-    }
-}
-
-static int
-driCopyContext(__DRIcontext *dst, __DRIcontext *src, unsigned long mask)
-{
-    return GL_FALSE;
-}
-
-static void dri_get_drawable(__DRIdrawable *pdp);
-static void dri_put_drawable(__DRIdrawable *pdp);
-
-static int driBindContext(__DRIcontext *pcp,
-                         __DRIdrawable *pdp,
-                         __DRIdrawable *prp)
-{
-    /* Bind the drawable to the context */
-    if (pcp) {
-       pcp->driDrawablePriv = pdp;
-       pcp->driReadablePriv = prp;
-       if (pdp) {
-           pdp->driContextPriv = pcp;
-           dri_get_drawable(pdp);
-       }
-       if (prp && pdp != prp) {
-           dri_get_drawable(prp);
-       }
-    }
-
-    return driDriverAPI.MakeCurrent(pcp, pdp, prp);
-}
-
-static int driUnbindContext(__DRIcontext *pcp)
-{
-    __DRIdrawable *pdp;
-    __DRIdrawable *prp;
-
-    if (pcp == NULL)
-       return GL_FALSE;
-
-    pdp = pcp->driDrawablePriv;
-    prp = pcp->driReadablePriv;
-
-    /* already unbound */
-    if (!pdp && !prp)
-       return GL_TRUE;
-
-    driDriverAPI.UnbindContext(pcp);
-
-    dri_put_drawable(pdp);
-
-    if (prp != pdp) {
-       dri_put_drawable(prp);
-    }
-
-    pcp->driDrawablePriv = NULL;
-    pcp->driReadablePriv = NULL;
-
-    return GL_TRUE;
-}
-
-
-/**
- * Drawable functions
- */
-
-static void dri_get_drawable(__DRIdrawable *pdp)
-{
-    pdp->refcount++;
-}
-
-static void dri_put_drawable(__DRIdrawable *pdp)
-{
-    if (pdp) {
-       pdp->refcount--;
-       if (pdp->refcount)
-           return;
-
-       driDriverAPI.DestroyBuffer(pdp);
-       free(pdp);
-    }
-}
-
-static __DRIdrawable *
-driCreateNewDrawable(__DRIscreen *psp,
-                    const __DRIconfig *config, void *data)
-{
-    __DRIdrawable *pdp;
-
-    pdp = CALLOC_STRUCT(__DRIdrawableRec);
-    if (!pdp)
-       return NULL;
-
-    pdp->loaderPrivate = data;
-
-    pdp->driScreenPriv = psp;
-    pdp->driContextPriv = NULL;
-
-    dri_get_drawable(pdp);
-
-    if (!driDriverAPI.CreateBuffer(psp, pdp, &config->modes, GL_FALSE)) {
-       free(pdp);
-       return NULL;
-    }
-
-    pdp->lastStamp = 1; /* const */
-
-    return pdp;
-}
-
-static void
-driDestroyDrawable(__DRIdrawable *pdp)
-{
-    dri_put_drawable(pdp);
-}
-
-static void driSwapBuffers(__DRIdrawable *pdp)
-{
-    driDriverAPI.SwapBuffers(pdp);
-}
-
-const __DRIcoreExtension driCoreExtension = {
-    { __DRI_CORE, __DRI_CORE_VERSION },
-    NULL, /* driCreateNewScreen */
-    driDestroyScreen,
-    driGetExtensions,
-    driGetConfigAttrib,
-    driIndexConfigAttrib,
-    NULL, /* driCreateNewDrawable */
-    driDestroyDrawable,
-    driSwapBuffers,
-    driCreateNewContext,
-    driCopyContext,
-    driDestroyContext,
-    driBindContext,
-    driUnbindContext
-};
-
-const __DRIswrastExtension driSWRastExtension = {
-    { __DRI_SWRAST, __DRI_SWRAST_VERSION },
-    driCreateNewScreen,
-    driCreateNewDrawable,
-    driCreateNewContextForAPI,
-    driCreateContextAttribs
-};
index 7630dba4b0e8cea921452bfac8ebd2c6ec3f4ff0..fc7ef32dbd55f56f1d2a4815b0047ef361cd090e 100644 (file)
@@ -3,7 +3,8 @@ SWRAST_DRIVER_FILES = \
 
 SWRAST_COMMON_FILES = \
        ../common/utils.c \
-       ../common/drisw_util.c
+       ../common/dri_util.c \
+       ../common/xmlconfig.c
 
 SWRAST_C_FILES = \
        $(SWRAST_COMMON_FILES) \