intel: Drop aub dumping functionality
[platform/upstream/libdrm.git] / intel / intel_bufmgr_gem.c
index f5d6130..41de396 100644 (file)
 #ifndef ETIME
 #define ETIME ETIMEDOUT
 #endif
-#include "libdrm.h"
+#include "libdrm_macros.h"
 #include "libdrm_lists.h"
 #include "intel_bufmgr.h"
 #include "intel_bufmgr_priv.h"
 #include "intel_chipset.h"
-#include "intel_aub.h"
 #include "string.h"
 
 #include "i915_drm.h"
@@ -82,6 +81,7 @@
 } while (0)
 
 #define ARRAY_SIZE(x) (sizeof(x) / sizeof((x)[0]))
+#define MAX2(A, B) ((A) > (B) ? (A) : (B))
 
 typedef struct _drm_intel_bo_gem drm_intel_bo_gem;
 
@@ -132,9 +132,11 @@ typedef struct _drm_intel_bufmgr_gem {
        unsigned int has_vebox : 1;
        bool fenced_relocs;
 
-       char *aub_filename;
-       FILE *aub_file;
-       uint32_t aub_offset;
+       struct {
+               void *ptr;
+               uint32_t handle;
+       } userptr_active;
+
 } drm_intel_bufmgr_gem;
 
 #define DRM_INTEL_RELOC_FENCE (1<<0)
@@ -250,11 +252,6 @@ struct _drm_intel_bo_gem {
 
        /** Flags that we may need to do the SW_FINSIH ioctl on unmap. */
        bool mapped_cpu_write;
-
-       uint32_t aub_offset;
-
-       drm_intel_aub_annotation *aub_annotations;
-       unsigned aub_annotation_count;
 };
 
 static unsigned int
@@ -459,7 +456,7 @@ drm_intel_add_validate_buffer(drm_intel_bo *bo)
        bufmgr_gem->exec_objects[index].handle = bo_gem->gem_handle;
        bufmgr_gem->exec_objects[index].relocation_count = bo_gem->reloc_count;
        bufmgr_gem->exec_objects[index].relocs_ptr = (uintptr_t) bo_gem->relocs;
-       bufmgr_gem->exec_objects[index].alignment = 0;
+       bufmgr_gem->exec_objects[index].alignment = bo->align;
        bufmgr_gem->exec_objects[index].offset = 0;
        bufmgr_gem->exec_bos[index] = bo;
        bufmgr_gem->exec_count++;
@@ -501,7 +498,7 @@ drm_intel_add_validate_buffer2(drm_intel_bo *bo, int need_fence)
        bufmgr_gem->exec2_objects[index].handle = bo_gem->gem_handle;
        bufmgr_gem->exec2_objects[index].relocation_count = bo_gem->reloc_count;
        bufmgr_gem->exec2_objects[index].relocs_ptr = (uintptr_t)bo_gem->relocs;
-       bufmgr_gem->exec2_objects[index].alignment = 0;
+       bufmgr_gem->exec2_objects[index].alignment = bo->align;
        bufmgr_gem->exec2_objects[index].offset = 0;
        bufmgr_gem->exec_bos[index] = bo;
        bufmgr_gem->exec2_objects[index].flags = 0;
@@ -519,9 +516,10 @@ drm_intel_add_validate_buffer2(drm_intel_bo *bo, int need_fence)
 
 static void
 drm_intel_bo_gem_set_in_aperture_size(drm_intel_bufmgr_gem *bufmgr_gem,
-                                     drm_intel_bo_gem *bo_gem)
+                                     drm_intel_bo_gem *bo_gem,
+                                     unsigned int alignment)
 {
-       int size;
+       unsigned int size;
 
        assert(!bo_gem->used_as_reloc_target);
 
@@ -533,7 +531,7 @@ drm_intel_bo_gem_set_in_aperture_size(drm_intel_bufmgr_gem *bufmgr_gem,
         */
        size = bo_gem->bo.size;
        if (bufmgr_gem->gen < 4 && bo_gem->tiling_mode != I915_TILING_NONE) {
-               int min_size;
+               unsigned int min_size;
 
                if (bufmgr_gem->has_relaxed_fencing) {
                        if (bufmgr_gem->gen == 3)
@@ -547,10 +545,10 @@ drm_intel_bo_gem_set_in_aperture_size(drm_intel_bufmgr_gem *bufmgr_gem,
                        min_size = size;
 
                /* Account for worst-case alignment. */
-               size = 2 * min_size;
+               alignment = MAX2(alignment, min_size);
        }
 
-       bo_gem->reloc_tree_size = size;
+       bo_gem->reloc_tree_size = size + alignment;
 }
 
 static int
@@ -655,7 +653,8 @@ drm_intel_gem_bo_alloc_internal(drm_intel_bufmgr *bufmgr,
                                unsigned long size,
                                unsigned long flags,
                                uint32_t tiling_mode,
-                               unsigned long stride)
+                               unsigned long stride,
+                               unsigned int alignment)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bufmgr;
        drm_intel_bo_gem *bo_gem;
@@ -697,7 +696,9 @@ retry:
                                              bucket->head.prev, head);
                        DRMLISTDEL(&bo_gem->head);
                        alloc_from_cache = true;
+                       bo_gem->bo.align = alignment;
                } else {
+                       assert(alignment == 0);
                        /* For non-render-target BOs (where we're probably
                         * going to map it first thing in order to fill it
                         * with data), check if the last BO in the cache is
@@ -754,6 +755,7 @@ retry:
                        return NULL;
                }
                bo_gem->bo.bufmgr = bufmgr;
+               bo_gem->bo.align = alignment;
 
                bo_gem->tiling_mode = I915_TILING_NONE;
                bo_gem->swizzle_mode = I915_BIT_6_SWIZZLE_NONE;
@@ -778,10 +780,8 @@ retry:
        bo_gem->used_as_reloc_target = false;
        bo_gem->has_error = false;
        bo_gem->reusable = true;
-       bo_gem->aub_annotations = NULL;
-       bo_gem->aub_annotation_count = 0;
 
-       drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem);
+       drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem, alignment);
 
        DBG("bo_create: buf %d (%s) %ldb\n",
            bo_gem->gem_handle, bo_gem->name, size);
@@ -797,7 +797,8 @@ drm_intel_gem_bo_alloc_for_render(drm_intel_bufmgr *bufmgr,
 {
        return drm_intel_gem_bo_alloc_internal(bufmgr, name, size,
                                               BO_ALLOC_FOR_RENDER,
-                                              I915_TILING_NONE, 0);
+                                              I915_TILING_NONE, 0,
+                                              alignment);
 }
 
 static drm_intel_bo *
@@ -807,7 +808,7 @@ drm_intel_gem_bo_alloc(drm_intel_bufmgr *bufmgr,
                       unsigned int alignment)
 {
        return drm_intel_gem_bo_alloc_internal(bufmgr, name, size, 0,
-                                              I915_TILING_NONE, 0);
+                                              I915_TILING_NONE, 0, 0);
 }
 
 static drm_intel_bo *
@@ -859,7 +860,7 @@ drm_intel_gem_bo_alloc_tiled(drm_intel_bufmgr *bufmgr, const char *name,
                stride = 0;
 
        return drm_intel_gem_bo_alloc_internal(bufmgr, name, size, flags,
-                                              tiling, stride);
+                                              tiling, stride, 0);
 }
 
 static drm_intel_bo *
@@ -926,7 +927,7 @@ drm_intel_gem_bo_alloc_userptr(drm_intel_bufmgr *bufmgr,
        bo_gem->has_error = false;
        bo_gem->reusable = false;
 
-       drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem);
+       drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem, 0);
 
        DBG("bo_create_userptr: "
            "ptr %p buf %d (%s) size %ldb, stride 0x%x, tile mode %d\n",
@@ -943,7 +944,6 @@ has_userptr(drm_intel_bufmgr_gem *bufmgr_gem)
        void *ptr;
        long pgsz;
        struct drm_i915_gem_userptr userptr;
-       struct drm_gem_close close_bo;
 
        pgsz = sysconf(_SC_PAGESIZE);
        assert(pgsz > 0);
@@ -970,15 +970,15 @@ retry:
                return false;
        }
 
-       memclear(close_bo);
-       close_bo.handle = userptr.handle;
-       ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_GEM_CLOSE, &close_bo);
-       free(ptr);
-       if (ret) {
-               fprintf(stderr, "Failed to release test userptr object! (%d) "
-                               "i915 kernel driver may not be sane!\n", errno);
-               return false;
-       }
+       /* We don't release the userptr bo here as we want to keep the
+        * kernel mm tracking alive for our lifetime. The first time we
+        * create a userptr object the kernel has to install a mmu_notifer
+        * which is a heavyweight operation (e.g. it requires taking all
+        * mm_locks and stop_machine()).
+        */
+
+       bufmgr_gem->userptr_active.ptr = ptr;
+       bufmgr_gem->userptr_active.handle = userptr.handle;
 
        return true;
 }
@@ -1007,7 +1007,7 @@ check_bo_alloc_userptr(drm_intel_bufmgr *bufmgr,
  * This can be used when one application needs to pass a buffer object
  * to another.
  */
-drm_public drm_intel_bo *
+drm_intel_bo *
 drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
                                  const char *name,
                                  unsigned int handle)
@@ -1095,7 +1095,7 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
        bo_gem->tiling_mode = get_tiling.tiling_mode;
        bo_gem->swizzle_mode = get_tiling.swizzle_mode;
        /* XXX stride is unknown */
-       drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem);
+       drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem, 0);
 
        DRMINITLISTHEAD(&bo_gem->vma_list);
        DRMLISTADDTAIL(&bo_gem->name_list, &bufmgr_gem->named);
@@ -1132,7 +1132,6 @@ drm_intel_gem_bo_free(drm_intel_bo *bo)
                DBG("DRM_IOCTL_GEM_CLOSE %d failed (%s): %s\n",
                    bo_gem->gem_handle, bo_gem->name, strerror(errno));
        }
-       free(bo_gem->aub_annotations);
        free(bo);
 }
 
@@ -1469,7 +1468,7 @@ map_gtt(drm_intel_bo *bo)
        return 0;
 }
 
-drm_public int
+int
 drm_intel_gem_bo_map_gtt(drm_intel_bo *bo)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
@@ -1528,7 +1527,7 @@ drm_intel_gem_bo_map_gtt(drm_intel_bo *bo)
  * undefined).
  */
 
-drm_public int
+int
 drm_intel_gem_bo_map_unsynchronized(drm_intel_bo *bo)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
@@ -1617,7 +1616,7 @@ static int drm_intel_gem_bo_unmap(drm_intel_bo *bo)
        return ret;
 }
 
-drm_public int
+int
 drm_intel_gem_bo_unmap_gtt(drm_intel_bo *bo)
 {
        return drm_intel_gem_bo_unmap(bo);
@@ -1742,7 +1741,7 @@ drm_intel_gem_bo_wait_rendering(drm_intel_bo *bo)
  * Note that some kernels have broken the inifite wait for negative values
  * promise, upgrade to latest stable kernels if this is the case.
  */
-drm_public int
+int
 drm_intel_gem_bo_wait(drm_intel_bo *bo, int64_t timeout_ns)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
@@ -1778,7 +1777,7 @@ drm_intel_gem_bo_wait(drm_intel_bo *bo, int64_t timeout_ns)
  * In combination with drm_intel_gem_bo_pin() and manual fence management, we
  * can do tiled pixmaps this way.
  */
-drm_public void
+void
 drm_intel_gem_bo_start_gtt_access(drm_intel_bo *bo, int write_enable)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
@@ -1805,12 +1804,12 @@ static void
 drm_intel_bufmgr_gem_destroy(drm_intel_bufmgr *bufmgr)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bufmgr;
-       int i;
+       struct drm_gem_close close_bo;
+       int i, ret;
 
        free(bufmgr_gem->exec2_objects);
        free(bufmgr_gem->exec_objects);
        free(bufmgr_gem->exec_bos);
-       free(bufmgr_gem->aub_filename);
 
        pthread_mutex_destroy(&bufmgr_gem->lock);
 
@@ -1829,6 +1828,18 @@ drm_intel_bufmgr_gem_destroy(drm_intel_bufmgr *bufmgr)
                }
        }
 
+       /* Release userptr bo kept hanging around for optimisation. */
+       if (bufmgr_gem->userptr_active.ptr) {
+               memclear(close_bo);
+               close_bo.handle = bufmgr_gem->userptr_active.handle;
+               ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_GEM_CLOSE, &close_bo);
+               free(bufmgr_gem->userptr_active.ptr);
+               if (ret)
+                       fprintf(stderr,
+                               "Failed to release test userptr object! (%d) "
+                               "i915 kernel driver may not be sane!\n", errno);
+       }
+
        free(bufmgr);
 }
 
@@ -1941,7 +1952,7 @@ drm_intel_gem_bo_emit_reloc_fence(drm_intel_bo *bo, uint32_t offset,
                                read_domains, write_domain, true);
 }
 
-drm_public int
+int
 drm_intel_gem_bo_get_reloc_count(drm_intel_bo *bo)
 {
        drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
@@ -1962,7 +1973,7 @@ drm_intel_gem_bo_get_reloc_count(drm_intel_bo *bo)
  * Any further drm_intel_bufmgr_check_aperture_space() queries
  * involving this buffer in the tree are undefined after this call.
  */
-drm_public void
+void
 drm_intel_gem_bo_clear_relocs(drm_intel_bo *bo, int start)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
@@ -2092,297 +2103,12 @@ drm_intel_update_buffer_offsets2 (drm_intel_bufmgr_gem *bufmgr_gem)
        }
 }
 
-static void
-aub_out(drm_intel_bufmgr_gem *bufmgr_gem, uint32_t data)
-{
-       fwrite(&data, 1, 4, bufmgr_gem->aub_file);
-}
-
-static void
-aub_out_data(drm_intel_bufmgr_gem *bufmgr_gem, void *data, size_t size)
-{
-       fwrite(data, 1, size, bufmgr_gem->aub_file);
-}
-
-static void
-aub_write_bo_data(drm_intel_bo *bo, uint32_t offset, uint32_t size)
-{
-       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
-       drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
-       uint32_t *data;
-       unsigned int i;
-
-       data = malloc(bo->size);
-       drm_intel_bo_get_subdata(bo, offset, size, data);
-
-       /* Easy mode: write out bo with no relocations */
-       if (!bo_gem->reloc_count) {
-               aub_out_data(bufmgr_gem, data, size);
-               free(data);
-               return;
-       }
-
-       /* Otherwise, handle the relocations while writing. */
-       for (i = 0; i < size / 4; i++) {
-               int r;
-               for (r = 0; r < bo_gem->reloc_count; r++) {
-                       struct drm_i915_gem_relocation_entry *reloc;
-                       drm_intel_reloc_target *info;
-
-                       reloc = &bo_gem->relocs[r];
-                       info = &bo_gem->reloc_target_info[r];
-
-                       if (reloc->offset == offset + i * 4) {
-                               drm_intel_bo_gem *target_gem;
-                               uint32_t val;
-
-                               target_gem = (drm_intel_bo_gem *)info->bo;
-
-                               val = reloc->delta;
-                               val += target_gem->aub_offset;
-
-                               aub_out(bufmgr_gem, val);
-                               data[i] = val;
-                               break;
-                       }
-               }
-               if (r == bo_gem->reloc_count) {
-                       /* no relocation, just the data */
-                       aub_out(bufmgr_gem, data[i]);
-               }
-       }
-
-       free(data);
-}
-
-static void
-aub_bo_get_address(drm_intel_bo *bo)
-{
-       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
-       drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
-
-       /* Give the object a graphics address in the AUB file.  We
-        * don't just use the GEM object address because we do AUB
-        * dumping before execution -- we want to successfully log
-        * when the hardware might hang, and we might even want to aub
-        * capture for a driver trying to execute on a different
-        * generation of hardware by disabling the actual kernel exec
-        * call.
-        */
-       bo_gem->aub_offset = bufmgr_gem->aub_offset;
-       bufmgr_gem->aub_offset += bo->size;
-       /* XXX: Handle aperture overflow. */
-       assert(bufmgr_gem->aub_offset < 256 * 1024 * 1024);
-}
-
-static void
-aub_write_trace_block(drm_intel_bo *bo, uint32_t type, uint32_t subtype,
-                     uint32_t offset, uint32_t size)
-{
-       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
-       drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
-
-       aub_out(bufmgr_gem,
-               CMD_AUB_TRACE_HEADER_BLOCK |
-               ((bufmgr_gem->gen >= 8 ? 6 : 5) - 2));
-       aub_out(bufmgr_gem,
-               AUB_TRACE_MEMTYPE_GTT | type | AUB_TRACE_OP_DATA_WRITE);
-       aub_out(bufmgr_gem, subtype);
-       aub_out(bufmgr_gem, bo_gem->aub_offset + offset);
-       aub_out(bufmgr_gem, size);
-       if (bufmgr_gem->gen >= 8)
-               aub_out(bufmgr_gem, 0);
-       aub_write_bo_data(bo, offset, size);
-}
-
-/**
- * Break up large objects into multiple writes.  Otherwise a 128kb VBO
- * would overflow the 16 bits of size field in the packet header and
- * everything goes badly after that.
- */
-static void
-aub_write_large_trace_block(drm_intel_bo *bo, uint32_t type, uint32_t subtype,
-                           uint32_t offset, uint32_t size)
-{
-       uint32_t block_size;
-       uint32_t sub_offset;
-
-       for (sub_offset = 0; sub_offset < size; sub_offset += block_size) {
-               block_size = size - sub_offset;
-
-               if (block_size > 8 * 4096)
-                       block_size = 8 * 4096;
-
-               aub_write_trace_block(bo, type, subtype, offset + sub_offset,
-                                     block_size);
-       }
-}
-
-static void
-aub_write_bo(drm_intel_bo *bo)
-{
-       drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
-       uint32_t offset = 0;
-       unsigned i;
-
-       aub_bo_get_address(bo);
-
-       /* Write out each annotated section separately. */
-       for (i = 0; i < bo_gem->aub_annotation_count; ++i) {
-               drm_intel_aub_annotation *annotation =
-                       &bo_gem->aub_annotations[i];
-               uint32_t ending_offset = annotation->ending_offset;
-               if (ending_offset > bo->size)
-                       ending_offset = bo->size;
-               if (ending_offset > offset) {
-                       aub_write_large_trace_block(bo, annotation->type,
-                                                   annotation->subtype,
-                                                   offset,
-                                                   ending_offset - offset);
-                       offset = ending_offset;
-               }
-       }
-
-       /* Write out any remaining unannotated data */
-       if (offset < bo->size) {
-               aub_write_large_trace_block(bo, AUB_TRACE_TYPE_NOTYPE, 0,
-                                           offset, bo->size - offset);
-       }
-}
-
-/*
- * Make a ringbuffer on fly and dump it
- */
-static void
-aub_build_dump_ringbuffer(drm_intel_bufmgr_gem *bufmgr_gem,
-                         uint32_t batch_buffer, int ring_flag)
-{
-       uint32_t ringbuffer[4096];
-       int ring = AUB_TRACE_TYPE_RING_PRB0; /* The default ring */
-       int ring_count = 0;
-
-       if (ring_flag == I915_EXEC_BSD)
-               ring = AUB_TRACE_TYPE_RING_PRB1;
-       else if (ring_flag == I915_EXEC_BLT)
-               ring = AUB_TRACE_TYPE_RING_PRB2;
-
-       /* Make a ring buffer to execute our batchbuffer. */
-       memset(ringbuffer, 0, sizeof(ringbuffer));
-       if (bufmgr_gem->gen >= 8) {
-               ringbuffer[ring_count++] = AUB_MI_BATCH_BUFFER_START | (3 - 2);
-               ringbuffer[ring_count++] = batch_buffer;
-               ringbuffer[ring_count++] = 0;
-       } else {
-               ringbuffer[ring_count++] = AUB_MI_BATCH_BUFFER_START;
-               ringbuffer[ring_count++] = batch_buffer;
-       }
-
-       /* Write out the ring.  This appears to trigger execution of
-        * the ring in the simulator.
-        */
-       aub_out(bufmgr_gem,
-               CMD_AUB_TRACE_HEADER_BLOCK |
-               ((bufmgr_gem->gen >= 8 ? 6 : 5) - 2));
-       aub_out(bufmgr_gem,
-               AUB_TRACE_MEMTYPE_GTT | ring | AUB_TRACE_OP_COMMAND_WRITE);
-       aub_out(bufmgr_gem, 0); /* general/surface subtype */
-       aub_out(bufmgr_gem, bufmgr_gem->aub_offset);
-       aub_out(bufmgr_gem, ring_count * 4);
-       if (bufmgr_gem->gen >= 8)
-               aub_out(bufmgr_gem, 0);
-
-       /* FIXME: Need some flush operations here? */
-       aub_out_data(bufmgr_gem, ringbuffer, ring_count * 4);
-
-       /* Update offset pointer */
-       bufmgr_gem->aub_offset += 4096;
-}
-
-drm_public void
+void
 drm_intel_gem_bo_aub_dump_bmp(drm_intel_bo *bo,
                              int x1, int y1, int width, int height,
                              enum aub_dump_bmp_format format,
                              int pitch, int offset)
 {
-       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
-       drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *)bo;
-       uint32_t cpp;
-
-       switch (format) {
-       case AUB_DUMP_BMP_FORMAT_8BIT:
-               cpp = 1;
-               break;
-       case AUB_DUMP_BMP_FORMAT_ARGB_4444:
-               cpp = 2;
-               break;
-       case AUB_DUMP_BMP_FORMAT_ARGB_0888:
-       case AUB_DUMP_BMP_FORMAT_ARGB_8888:
-               cpp = 4;
-               break;
-       default:
-               printf("Unknown AUB dump format %d\n", format);
-               return;
-       }
-
-       if (!bufmgr_gem->aub_file)
-               return;
-
-       aub_out(bufmgr_gem, CMD_AUB_DUMP_BMP | 4);
-       aub_out(bufmgr_gem, (y1 << 16) | x1);
-       aub_out(bufmgr_gem,
-               (format << 24) |
-               (cpp << 19) |
-               pitch / 4);
-       aub_out(bufmgr_gem, (height << 16) | width);
-       aub_out(bufmgr_gem, bo_gem->aub_offset + offset);
-       aub_out(bufmgr_gem,
-               ((bo_gem->tiling_mode != I915_TILING_NONE) ? (1 << 2) : 0) |
-               ((bo_gem->tiling_mode == I915_TILING_Y) ? (1 << 3) : 0));
-}
-
-static void
-aub_exec(drm_intel_bo *bo, int ring_flag, int used)
-{
-       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
-       drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
-       int i;
-       bool batch_buffer_needs_annotations;
-
-       if (!bufmgr_gem->aub_file)
-               return;
-
-       /* If batch buffer is not annotated, annotate it the best we
-        * can.
-        */
-       batch_buffer_needs_annotations = bo_gem->aub_annotation_count == 0;
-       if (batch_buffer_needs_annotations) {
-               drm_intel_aub_annotation annotations[2] = {
-                       { AUB_TRACE_TYPE_BATCH, 0, used },
-                       { AUB_TRACE_TYPE_NOTYPE, 0, bo->size }
-               };
-               drm_intel_bufmgr_gem_set_aub_annotations(bo, annotations, 2);
-       }
-
-       /* Write out all buffers to AUB memory */
-       for (i = 0; i < bufmgr_gem->exec_count; i++) {
-               aub_write_bo(bufmgr_gem->exec_bos[i]);
-       }
-
-       /* Remove any annotations we added */
-       if (batch_buffer_needs_annotations)
-               drm_intel_bufmgr_gem_set_aub_annotations(bo, NULL, 0);
-
-       /* Dump ring buffer */
-       aub_build_dump_ringbuffer(bufmgr_gem, bo_gem->aub_offset, ring_flag);
-
-       fflush(bufmgr_gem->aub_file);
-
-       /*
-        * One frame has been dumped. So reset the aub_offset for the next frame.
-        *
-        * FIXME: Can we do this?
-        */
-       bufmgr_gem->aub_offset = 0x10000;
 }
 
 static int
@@ -2509,8 +2235,6 @@ do_exec2(drm_intel_bo *bo, int used, drm_intel_context *ctx,
                i915_execbuffer2_set_context_id(execbuf, ctx->ctx_id);
        execbuf.rsvd2 = 0;
 
-       aub_exec(bo, flags, used);
-
        if (bufmgr_gem->no_exec)
                goto skip_execution;
 
@@ -2569,7 +2293,7 @@ drm_intel_gem_bo_mrb_exec2(drm_intel_bo *bo, int used,
                        flags);
 }
 
-drm_public int
+int
 drm_intel_gem_bo_context_exec(drm_intel_bo *bo, drm_intel_context *ctx,
                              int used, unsigned int flags)
 {
@@ -2677,7 +2401,7 @@ drm_intel_gem_bo_set_tiling(drm_intel_bo *bo, uint32_t * tiling_mode,
 
        ret = drm_intel_gem_bo_set_tiling_internal(bo, *tiling_mode, stride);
        if (ret == 0)
-               drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem);
+               drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem, 0);
 
        *tiling_mode = bo_gem->tiling_mode;
        return ret;
@@ -2694,7 +2418,7 @@ drm_intel_gem_bo_get_tiling(drm_intel_bo *bo, uint32_t * tiling_mode,
        return 0;
 }
 
-drm_public drm_intel_bo *
+drm_intel_bo *
 drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int size)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bufmgr;
@@ -2775,12 +2499,12 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
        bo_gem->tiling_mode = get_tiling.tiling_mode;
        bo_gem->swizzle_mode = get_tiling.swizzle_mode;
        /* XXX stride is unknown */
-       drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem);
+       drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem, 0);
 
        return &bo_gem->bo;
 }
 
-drm_public int
+int
 drm_intel_bo_gem_export_to_prime(drm_intel_bo *bo, int *prime_fd)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
@@ -2840,7 +2564,7 @@ drm_intel_gem_bo_flink(drm_intel_bo *bo, uint32_t * name)
  * size is only bounded by how many buffers of that size we've managed to have
  * in flight at once.
  */
-drm_public void
+void
 drm_intel_bufmgr_gem_enable_reuse(drm_intel_bufmgr *bufmgr)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bufmgr;
@@ -2855,7 +2579,7 @@ drm_intel_bufmgr_gem_enable_reuse(drm_intel_bufmgr *bufmgr)
  * allocation.  If this option is not enabled, all relocs will have fence
  * register allocated.
  */
-drm_public void
+void
 drm_intel_bufmgr_gem_enable_fenced_relocs(drm_intel_bufmgr *bufmgr)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *)bufmgr;
@@ -3127,7 +2851,7 @@ init_cache_buckets(drm_intel_bufmgr_gem *bufmgr_gem)
        }
 }
 
-drm_public void
+void
 drm_intel_bufmgr_gem_set_vma_cache_size(drm_intel_bufmgr *bufmgr, int limit)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *)bufmgr;
@@ -3168,7 +2892,7 @@ get_pci_device_id(drm_intel_bufmgr_gem *bufmgr_gem)
        return devid;
 }
 
-drm_public int
+int
 drm_intel_bufmgr_gem_get_devid(drm_intel_bufmgr *bufmgr)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *)bufmgr;
@@ -3182,15 +2906,10 @@ drm_intel_bufmgr_gem_get_devid(drm_intel_bufmgr *bufmgr)
  * This function has to be called before drm_intel_bufmgr_gem_set_aub_dump()
  * for it to have any effect.
  */
-drm_public void
+void
 drm_intel_bufmgr_gem_set_aub_filename(drm_intel_bufmgr *bufmgr,
                                      const char *filename)
 {
-       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *)bufmgr;
-
-       free(bufmgr_gem->aub_filename);
-       if (filename)
-               bufmgr_gem->aub_filename = strdup(filename);
 }
 
 /**
@@ -3201,64 +2920,17 @@ drm_intel_bufmgr_gem_set_aub_filename(drm_intel_bufmgr *bufmgr,
  * You can set up a GTT and upload your objects into the referenced
  * space, then send off batchbuffers and get BMPs out the other end.
  */
-drm_public void
+void
 drm_intel_bufmgr_gem_set_aub_dump(drm_intel_bufmgr *bufmgr, int enable)
 {
-       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *)bufmgr;
-       int entry = 0x200003;
-       int i;
-       int gtt_size = 0x10000;
-       const char *filename;
-
-       if (!enable) {
-               if (bufmgr_gem->aub_file) {
-                       fclose(bufmgr_gem->aub_file);
-                       bufmgr_gem->aub_file = NULL;
-               }
-               return;
-       }
-
-       if (geteuid() != getuid())
-               return;
-
-       if (bufmgr_gem->aub_filename)
-               filename = bufmgr_gem->aub_filename;
-       else
-               filename = "intel.aub";
-       bufmgr_gem->aub_file = fopen(filename, "w+");
-       if (!bufmgr_gem->aub_file)
-               return;
-
-       /* Start allocating objects from just after the GTT. */
-       bufmgr_gem->aub_offset = gtt_size;
-
-       /* Start with a (required) version packet. */
-       aub_out(bufmgr_gem, CMD_AUB_HEADER | (13 - 2));
-       aub_out(bufmgr_gem,
-               (4 << AUB_HEADER_MAJOR_SHIFT) |
-               (0 << AUB_HEADER_MINOR_SHIFT));
-       for (i = 0; i < 8; i++) {
-               aub_out(bufmgr_gem, 0); /* app name */
-       }
-       aub_out(bufmgr_gem, 0); /* timestamp */
-       aub_out(bufmgr_gem, 0); /* timestamp */
-       aub_out(bufmgr_gem, 0); /* comment len */
-
-       /* Set up the GTT. The max we can handle is 256M */
-       aub_out(bufmgr_gem, CMD_AUB_TRACE_HEADER_BLOCK | ((bufmgr_gem->gen >= 8 ? 6 : 5) - 2));
-       /* Need to use GTT_ENTRY type for recent emulator */
-       aub_out(bufmgr_gem, AUB_TRACE_MEMTYPE_GTT_ENTRY | 0 | AUB_TRACE_OP_DATA_WRITE);
-       aub_out(bufmgr_gem, 0); /* subtype */
-       aub_out(bufmgr_gem, 0); /* offset */
-       aub_out(bufmgr_gem, gtt_size); /* size */
-       if (bufmgr_gem->gen >= 8)
-               aub_out(bufmgr_gem, 0);
-       for (i = 0x000; i < gtt_size; i += 4, entry += 0x1000) {
-               aub_out(bufmgr_gem, entry);
-       }
+       fprintf(stderr, "libdrm aub dumping is deprecated.\n\n"
+               "Use intel_aubdump from intel-gpu-tools instead.  Install intel-gpu-tools,\n"
+               "then run (for example)\n\n"
+               "\t$ intel_aubdump --output=trace.aub glxgears -geometry 500x500\n\n"
+               "See the intel_aubdump man page for more details.\n");
 }
 
-drm_public drm_intel_context *
+drm_intel_context *
 drm_intel_gem_context_create(drm_intel_bufmgr *bufmgr)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *)bufmgr;
@@ -3285,7 +2957,7 @@ drm_intel_gem_context_create(drm_intel_bufmgr *bufmgr)
        return context;
 }
 
-drm_public void
+void
 drm_intel_gem_context_destroy(drm_intel_context *ctx)
 {
        drm_intel_bufmgr_gem *bufmgr_gem;
@@ -3308,7 +2980,7 @@ drm_intel_gem_context_destroy(drm_intel_context *ctx)
        free(ctx);
 }
 
-drm_public int
+int
 drm_intel_get_reset_stats(drm_intel_context *ctx,
                          uint32_t *reset_count,
                          uint32_t *active,
@@ -3342,7 +3014,7 @@ drm_intel_get_reset_stats(drm_intel_context *ctx,
        return ret;
 }
 
-drm_public int
+int
 drm_intel_reg_read(drm_intel_bufmgr *bufmgr,
                   uint32_t offset,
                   uint64_t *result)
@@ -3360,7 +3032,7 @@ drm_intel_reg_read(drm_intel_bufmgr *bufmgr,
        return ret;
 }
 
-drm_public int
+int
 drm_intel_get_subslice_total(int fd, unsigned int *subslice_total)
 {
        drm_i915_getparam_t gp;
@@ -3376,7 +3048,7 @@ drm_intel_get_subslice_total(int fd, unsigned int *subslice_total)
        return 0;
 }
 
-drm_public int
+int
 drm_intel_get_eu_total(int fd, unsigned int *eu_total)
 {
        drm_i915_getparam_t gp;
@@ -3413,24 +3085,11 @@ drm_intel_get_eu_total(int fd, unsigned int *eu_total)
  * default state (no annotations), call this function with a \c count
  * of zero.
  */
-drm_public void
+void
 drm_intel_bufmgr_gem_set_aub_annotations(drm_intel_bo *bo,
                                         drm_intel_aub_annotation *annotations,
                                         unsigned count)
 {
-       drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
-       unsigned size = sizeof(*annotations) * count;
-       drm_intel_aub_annotation *new_annotations =
-               count > 0 ? realloc(bo_gem->aub_annotations, size) : NULL;
-       if (new_annotations == NULL) {
-               free(bo_gem->aub_annotations);
-               bo_gem->aub_annotations = NULL;
-               bo_gem->aub_annotation_count = 0;
-               return;
-       }
-       memcpy(new_annotations, annotations, size);
-       bo_gem->aub_annotations = new_annotations;
-       bo_gem->aub_annotation_count = count;
 }
 
 static pthread_mutex_t bufmgr_list_mutex = PTHREAD_MUTEX_INITIALIZER;
@@ -3474,7 +3133,7 @@ drm_intel_bufmgr_gem_unref(drm_intel_bufmgr *bufmgr)
  *
  * \param fd File descriptor of the opened DRM device.
  */
-drm_public drm_intel_bufmgr *
+drm_intel_bufmgr *
 drm_intel_bufmgr_gem_init(int fd, int batch_size)
 {
        drm_intel_bufmgr_gem *bufmgr_gem;