i965: Delete the intel_regions.c code.
authorEric Anholt <eric@anholt.net>
Fri, 25 Apr 2014 20:50:10 +0000 (13:50 -0700)
committerEric Anholt <eric@anholt.net>
Thu, 1 May 2014 22:12:27 +0000 (15:12 -0700)
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
22 files changed:
src/mesa/drivers/dri/i965/Makefile.sources
src/mesa/drivers/dri/i965/brw_clear.c
src/mesa/drivers/dri/i965/brw_curbe.c
src/mesa/drivers/dri/i965/brw_draw.c
src/mesa/drivers/dri/i965/brw_misc_state.c
src/mesa/drivers/dri/i965/brw_tex.c
src/mesa/drivers/dri/i965/gen7_misc_state.c
src/mesa/drivers/dri/i965/gen8_depth_state.c
src/mesa/drivers/dri/i965/intel_blit.c
src/mesa/drivers/dri/i965/intel_debug.c
src/mesa/drivers/dri/i965/intel_debug.h
src/mesa/drivers/dri/i965/intel_mipmap_tree.c
src/mesa/drivers/dri/i965/intel_mipmap_tree.h
src/mesa/drivers/dri/i965/intel_pixel.c
src/mesa/drivers/dri/i965/intel_pixel_copy.c
src/mesa/drivers/dri/i965/intel_pixel_draw.c
src/mesa/drivers/dri/i965/intel_pixel_read.c
src/mesa/drivers/dri/i965/intel_regions.c [deleted file]
src/mesa/drivers/dri/i965/intel_regions.h
src/mesa/drivers/dri/i965/intel_tex_copy.c
src/mesa/drivers/dri/i965/intel_tex_image.c
src/mesa/drivers/dri/i965/intel_upload.c

index 836c62b7e7569bfb54bfac303a337144c371ce03..87c73baeca06a2b4c09b79fb09a62ff6b915cc34 100644 (file)
@@ -11,7 +11,6 @@ i965_FILES = \
        intel_extensions.c \
        intel_fbo.c \
        intel_mipmap_tree.c \
-       intel_regions.c \
        intel_resolve_map.c \
        intel_screen.c \
        intel_pixel.c \
index ee8f54fec54088b85afc1c301caa3378a1ca3eaa..a1c1d0131b208889bb0795a5346139dd8f9389ba 100644 (file)
@@ -36,7 +36,6 @@
 #include "intel_blit.h"
 #include "intel_fbo.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 
 #include "brw_context.h"
 #include "brw_blorp.h"
index e3514ea0905ecfb654d2b7d830e519d17dcbf8f2..e5ca2ce6b666bb76040c1a1e5bcf882f391a95f4 100644 (file)
@@ -39,7 +39,6 @@
 #include "program/prog_print.h"
 #include "program/prog_statevars.h"
 #include "intel_batchbuffer.h"
-#include "intel_regions.h"
 #include "brw_context.h"
 #include "brw_defines.h"
 #include "brw_state.h"
index d16106c40492844fd09ae755b77aba4e62e2b7f7..fef1c063e156ae72110fa5e37373ae55b3a4eb01 100644 (file)
@@ -51,7 +51,6 @@
 #include "intel_buffers.h"
 #include "intel_fbo.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 #include "intel_buffer_objects.h"
 
 #define FILE_DEBUG_FLAG DEBUG_PRIMS
index ebe77882c5cfe11c642ec8e512b79164c77dfce0..dcf40d036e736c4c3582a81c3b0716186e44eff5 100644 (file)
@@ -34,7 +34,6 @@
 #include "intel_batchbuffer.h"
 #include "intel_fbo.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 
 #include "brw_context.h"
 #include "brw_state.h"
index 13578b83e30aa03effc44d1cd71e95a818405d7d..2cb9aa6503adbf2c7434a40deaca3d41d465d678 100644 (file)
@@ -34,7 +34,6 @@
 #include "main/mtypes.h"
 #include "main/teximage.h"
 
-#include "intel_regions.h"
 #include "intel_tex.h"
 #include "brw_context.h"
 
index 731325e2f35c3be685bcf534bf7e0bb5f34c4d0d..b6759f15ffe1c20fc8f7e91818c95d8070627008 100644 (file)
@@ -24,7 +24,6 @@
 #include "main/mtypes.h"
 #include "intel_batchbuffer.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 #include "intel_fbo.h"
 #include "brw_context.h"
 #include "brw_state.h"
index 2de30eea6374f683db851b1b01a2ec2defcbfbdd..f6031e97231acacddaf9e12012fc271ba3d85019 100644 (file)
@@ -23,7 +23,6 @@
 
 #include "intel_batchbuffer.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 #include "intel_fbo.h"
 #include "intel_resolve_map.h"
 #include "brw_context.h"
index 2ebaf73768ffcd76abcf1921ebf56ba6416fc96d..b5c3408b935c330be0fcd86c7f05154b8c5cfaf8 100644 (file)
@@ -38,7 +38,6 @@
 #include "intel_buffers.h"
 #include "intel_fbo.h"
 #include "intel_reg.h"
-#include "intel_regions.h"
 #include "intel_batchbuffer.h"
 #include "intel_mipmap_tree.h"
 
index dc74202f491b1b54859217503b5576c88991a73c..621a571c3d8fe5effaec53eb292b5364908eab09 100644 (file)
@@ -46,7 +46,6 @@ static const struct dri_debug_control debug_control[] = {
    { "bat",   DEBUG_BATCH},
    { "pix",   DEBUG_PIXEL},
    { "buf",   DEBUG_BUFMGR},
-   { "reg",   DEBUG_REGION},
    { "fbo",   DEBUG_FBO},
    { "fs",    DEBUG_WM },
    { "gs",    DEBUG_GS},
index d4f1666261b766cd7b822d798e193375e79c94b8..6402cec80d8f0d56d24f8884e24e70a4a4b2ff66 100644 (file)
@@ -42,7 +42,6 @@ extern uint64_t INTEL_DEBUG;
 #define DEBUG_BATCH       0x80
 #define DEBUG_PIXEL       0x100
 #define DEBUG_BUFMGR      0x200
-#define DEBUG_REGION      0x400
 #define DEBUG_FBO         0x800
 #define DEBUG_GS          0x1000
 #define DEBUG_SYNC       0x2000
index 5f241eff03403f80a6add69afffb1b3e9d13164f..b23fddf7b0fc36caa0b58d664216f9ae06bbe80a 100644 (file)
@@ -31,7 +31,6 @@
 #include "intel_batchbuffer.h"
 #include "intel_chipset.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 #include "intel_resolve_map.h"
 #include "intel_tex.h"
 #include "intel_blit.h"
index 502a9bf421884e1790deff19b416989fc410b12d..8181f82f3b3ae9e1030ace1cd10b5399415f701d 100644 (file)
@@ -48,7 +48,8 @@
 
 #include <assert.h>
 
-#include "intel_regions.h"
+#include "main/mtypes.h"
+#include "intel_bufmgr.h"
 #include "intel_resolve_map.h"
 #include <GL/internal/dri_interface.h>
 
@@ -56,6 +57,7 @@
 extern "C" {
 #endif
 
+struct brw_context;
 struct intel_renderbuffer;
 
 struct intel_resolve_map;
index a6db7fa36d014e96f609b881a49fe354ad59fbc1..34543334cdee11def598d884359fcb6efcf4295e 100644 (file)
@@ -34,7 +34,6 @@
 
 #include "brw_context.h"
 #include "intel_pixel.h"
-#include "intel_regions.h"
 
 #define FILE_DEBUG_FLAG DEBUG_PIXEL
 
index 5687c2837e9540f8fbd44c17aeed898f0e786091..d928e1dfd70c01e82c96cfe84949c709bc493227 100644 (file)
@@ -36,7 +36,6 @@
 #include "brw_context.h"
 #include "intel_buffers.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 #include "intel_pixel.h"
 #include "intel_fbo.h"
 #include "intel_blit.h"
index b46bf2616824abde6dac9c34591441a70c9b7ffd..8222f33f4ae89624f36137dee473d8c680208650 100644 (file)
@@ -44,7 +44,6 @@
 #include "intel_buffers.h"
 #include "intel_fbo.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 #include "intel_pixel.h"
 #include "intel_buffer_objects.h"
 
index 24d2800670ce236ec448badef48e9ef7aab78cc1..beb31522c13e57af1760926e9550190aa7dc9728 100644 (file)
@@ -42,7 +42,6 @@
 #include "intel_buffers.h"
 #include "intel_fbo.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 #include "intel_pixel.h"
 #include "intel_buffer_objects.h"
 
diff --git a/src/mesa/drivers/dri/i965/intel_regions.c b/src/mesa/drivers/dri/i965/intel_regions.c
deleted file mode 100644 (file)
index 8e693fd..0000000
+++ /dev/null
@@ -1,198 +0,0 @@
-/**************************************************************************
- *
- * Copyright 2006 VMware, Inc.
- * All Rights Reserved.
- *
- * 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, sub license, 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 NON-INFRINGEMENT.
- * IN NO EVENT SHALL VMWARE AND/OR ITS SUPPLIERS 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.
- *
- **************************************************************************/
-
-/* Provide additional functionality on top of bufmgr buffers:
- *   - 2d semantics and blit operations
- *   - refcounting of buffers for multiple images in a buffer.
- *   - refcounting of buffer mappings.
- *   - some logic for moving the buffers to the best memory pools for
- *     given operations.
- *
- * Most of this is to make it easier to implement the fixed-layout
- * mipmap tree required by intel hardware in the face of GL's
- * programming interface where each image can be specifed in random
- * order and it isn't clear what layout the tree should have until the
- * last moment.
- */
-
-#include <sys/ioctl.h>
-#include <errno.h>
-
-#include "main/hash.h"
-#include "brw_context.h"
-#include "intel_regions.h"
-#include "intel_blit.h"
-#include "intel_buffer_objects.h"
-#include "intel_bufmgr.h"
-#include "intel_batchbuffer.h"
-
-#define FILE_DEBUG_FLAG DEBUG_REGION
-
-/* This should be set to the maximum backtrace size desired.
- * Set it to 0 to disable backtrace debugging.
- */
-#define DEBUG_BACKTRACE_SIZE 0
-
-#if DEBUG_BACKTRACE_SIZE == 0
-/* Use the standard debug output */
-#define _DBG(...) DBG(__VA_ARGS__)
-#else
-/* Use backtracing debug output */
-#define _DBG(...) {debug_backtrace(); DBG(__VA_ARGS__);}
-
-/* Backtracing debug support */
-#include <execinfo.h>
-
-static void
-debug_backtrace(void)
-{
-   void *trace[DEBUG_BACKTRACE_SIZE];
-   char **strings = NULL;
-   int traceSize;
-   register int i;
-
-   traceSize = backtrace(trace, DEBUG_BACKTRACE_SIZE);
-   strings = backtrace_symbols(trace, traceSize);
-   if (strings == NULL) {
-      DBG("no backtrace:");
-      return;
-   }
-
-   /* Spit out all the strings with a colon separator.  Ignore
-    * the first, since we don't really care about the call
-    * to debug_backtrace() itself.  Skip until the final "/" in
-    * the trace to avoid really long lines.
-    */
-   for (i = 1; i < traceSize; i++) {
-      char *p = strings[i], *slash = strings[i];
-      while (*p) {
-         if (*p++ == '/') {
-            slash = p;
-         }
-      }
-
-      DBG("%s:", slash);
-   }
-
-   /* Free up the memory, and we're done */
-   free(strings);
-}
-
-#endif
-
-static struct intel_region *
-intel_region_alloc_internal(struct intel_screen *screen,
-                           GLuint cpp,
-                           GLuint width, GLuint height, GLuint pitch,
-                           uint32_t tiling, drm_intel_bo *buffer)
-{
-   struct intel_region *region;
-
-   region = calloc(sizeof(*region), 1);
-   if (region == NULL)
-      return region;
-
-   region->cpp = cpp;
-   region->width = width;
-   region->height = height;
-   region->pitch = pitch;
-   region->refcount = 1;
-   region->bo = buffer;
-   region->tiling = tiling;
-
-   _DBG("%s <-- %p\n", __FUNCTION__, region);
-   return region;
-}
-
-struct intel_region *
-intel_region_alloc(struct intel_screen *screen,
-                  uint32_t tiling,
-                   GLuint cpp, GLuint width, GLuint height,
-                  bool expect_accelerated_upload)
-{
-   drm_intel_bo *buffer;
-   unsigned long flags = 0;
-   unsigned long aligned_pitch;
-   struct intel_region *region;
-
-   if (expect_accelerated_upload)
-      flags |= BO_ALLOC_FOR_RENDER;
-
-   buffer = drm_intel_bo_alloc_tiled(screen->bufmgr, "region",
-                                    width, height, cpp,
-                                    &tiling, &aligned_pitch, flags);
-   if (buffer == NULL)
-      return NULL;
-
-   region = intel_region_alloc_internal(screen, cpp, width, height,
-                                        aligned_pitch, tiling, buffer);
-   if (region == NULL) {
-      drm_intel_bo_unreference(buffer);
-      return NULL;
-   }
-
-   return region;
-}
-
-void
-intel_region_reference(struct intel_region **dst, struct intel_region *src)
-{
-   _DBG("%s: %p(%d) -> %p(%d)\n", __FUNCTION__,
-       *dst, *dst ? (*dst)->refcount : 0, src, src ? src->refcount : 0);
-
-   if (src != *dst) {
-      if (*dst)
-        intel_region_release(dst);
-
-      if (src)
-         src->refcount++;
-      *dst = src;
-   }
-}
-
-void
-intel_region_release(struct intel_region **region_handle)
-{
-   struct intel_region *region = *region_handle;
-
-   if (region == NULL) {
-      _DBG("%s NULL\n", __FUNCTION__);
-      return;
-   }
-
-   _DBG("%s %p %d\n", __FUNCTION__, region, region->refcount - 1);
-
-   ASSERT(region->refcount > 0);
-   region->refcount--;
-
-   if (region->refcount == 0) {
-      drm_intel_bo_unreference(region->bo);
-
-      free(region);
-   }
-   *region_handle = NULL;
-}
index 13ed03ce584ce49e59d4b4d281aede8320a36dec..a34f0b0e3f8401b74fb007f3e674c78d1d6b5381 100644 (file)
 extern "C" {
 #endif
 
-struct brw_context;
-struct intel_screen;
-struct intel_buffer_object;
-
-/**
- * A layer on top of the bufmgr buffers that adds a few useful things:
- *
- * - Refcounting for local buffer references.
- * - Refcounting for buffer maps
- * - Buffer dimensions - pitch and height.
- * - Blitter commands for copying 2D regions between buffers. (really???)
- */
-struct intel_region
-{
-   drm_intel_bo *bo;  /**< buffer manager's buffer */
-   GLuint refcount; /**< Reference count for region */
-   GLuint cpp;      /**< bytes per pixel */
-   GLuint width;    /**< in pixels */
-   GLuint height;   /**< in pixels */
-   GLuint pitch;    /**< in bytes */
-
-   uint32_t tiling; /**< Which tiling mode the region is in */
-};
-
-
-/* Allocate a refcounted region.  Pointers to regions should only be
- * copied by calling intel_reference_region().
- */
-struct intel_region *intel_region_alloc(struct intel_screen *screen,
-                                        uint32_t tiling,
-                                       GLuint cpp, GLuint width,
-                                        GLuint height,
-                                       bool expect_accelerated_upload);
-
-void intel_region_reference(struct intel_region **dst,
-                            struct intel_region *src);
-
-void intel_region_release(struct intel_region **ib);
-
 /**
  * Used with images created with image_from_names
  * to help support planar images.
index c2b6c350c1b07851f66d309d76c7042fff95a1ec..97f1569330d1702853ea0373c7877eafb64f98cb 100644 (file)
@@ -36,7 +36,6 @@
 
 #include "intel_screen.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 #include "intel_fbo.h"
 #include "intel_tex.h"
 #include "intel_blit.h"
index bbb66026f23dabd1f7e51461bb912bc4786b44c9..42fdee32a328b62bd7fa8d606e8c47950488d56f 100644 (file)
@@ -21,6 +21,7 @@
 #include "intel_tex.h"
 #include "intel_blit.h"
 #include "intel_fbo.h"
+#include "intel_regions.h"
 
 #include "brw_context.h"
 
index bb3f615d987587d657b52a7029fe6b051384f6f8..870aabc88631cf9d3148f8d5c7a79645a5153f5c 100644 (file)
@@ -39,7 +39,6 @@
 #include "intel_batchbuffer.h"
 #include "intel_fbo.h"
 #include "intel_mipmap_tree.h"
-#include "intel_regions.h"
 
 #include "brw_context.h"