intel: Drop aub dumping functionality
[platform/upstream/libdrm.git] / intel / intel_bufmgr_gem.c
index eae2199..41de396 100644 (file)
 #include <assert.h>
 #include <pthread.h>
 #include <sys/ioctl.h>
-#include <sys/mman.h>
 #include <sys/stat.h>
 #include <sys/types.h>
 #include <stdbool.h>
 
 #include "errno.h"
+#ifndef ETIME
+#define ETIME ETIMEDOUT
+#endif
+#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"
@@ -71,7 +73,7 @@
 #define VG(x)
 #endif
 
-#define VG_CLEAR(s) VG(memset(&s, 0, sizeof(s)))
+#define memclear(s) memset(&s, 0, sizeof(s))
 
 #define DBG(...) do {                                  \
        if (bufmgr_gem->bufmgr.debug)                   \
@@ -79,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;
 
@@ -90,6 +93,8 @@ struct drm_intel_gem_bo_bucket {
 typedef struct _drm_intel_bufmgr_gem {
        drm_intel_bufmgr bufmgr;
 
+       atomic_t refcount;
+
        int fd;
 
        int max_relocs;
@@ -107,6 +112,8 @@ typedef struct _drm_intel_bufmgr_gem {
        int num_buckets;
        time_t time;
 
+       drmMMListHead managers;
+
        drmMMListHead named;
        drmMMListHead vma_cache;
        int vma_count, vma_open, vma_max;
@@ -122,10 +129,14 @@ typedef struct _drm_intel_bufmgr_gem {
        unsigned int has_wait_timeout : 1;
        unsigned int bo_reuse : 1;
        unsigned int no_exec : 1;
+       unsigned int has_vebox : 1;
        bool fenced_relocs;
 
-       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)
@@ -144,6 +155,8 @@ struct _drm_intel_bo_gem {
 
        /**
         * Kenel-assigned global name for this object
+         *
+         * List contains both flink named and prime fd'd objects
         */
        unsigned int global_name;
        drmMMListHead name_list;
@@ -175,6 +188,11 @@ struct _drm_intel_bo_gem {
        void *mem_virtual;
        /** GTT virtual address for the buffer, saved across map/unmap cycles */
        void *gtt_virtual;
+       /**
+        * Virtual address of the buffer allocated by user, used for userptr
+        * objects only.
+        */
+       void *user_virtual;
        int map_count;
        drmMMListHead vma_list;
 
@@ -205,6 +223,20 @@ struct _drm_intel_bo_gem {
        bool reusable;
 
        /**
+        * Boolean of whether the GPU is definitely not accessing the buffer.
+        *
+        * This is only valid when reusable, since non-reusable
+        * buffers are those that have been shared wth other
+        * processes, so we don't know their state.
+        */
+       bool idle;
+
+       /**
+        * Boolean of whether this buffer was allocated with userptr
+        */
+       bool is_userptr;
+
+       /**
         * Size in bytes of this buffer and its relocation descendents.
         *
         * Used to avoid costly tree walking in
@@ -220,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
@@ -375,7 +402,7 @@ drm_intel_gem_dump_validation_list(drm_intel_bufmgr_gem *bufmgr_gem)
                            (unsigned long long)bo_gem->relocs[j].offset,
                            target_gem->gem_handle,
                            target_gem->name,
-                           target_bo->offset,
+                           target_bo->offset64,
                            bo_gem->relocs[j].delta);
                }
        }
@@ -429,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++;
@@ -471,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;
@@ -489,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);
 
@@ -503,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)
@@ -517,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
@@ -560,11 +588,19 @@ drm_intel_gem_bo_busy(drm_intel_bo *bo)
        struct drm_i915_gem_busy busy;
        int ret;
 
-       VG_CLEAR(busy);
+       if (bo_gem->reusable && bo_gem->idle)
+               return false;
+
+       memclear(busy);
        busy.handle = bo_gem->gem_handle;
 
        ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_GEM_BUSY, &busy);
-
+       if (ret == 0) {
+               bo_gem->idle = !busy.busy;
+               return busy.busy;
+       } else {
+               return false;
+       }
        return (ret == 0 && busy.busy);
 }
 
@@ -574,7 +610,7 @@ drm_intel_gem_bo_madvise_internal(drm_intel_bufmgr_gem *bufmgr_gem,
 {
        struct drm_i915_gem_madvise madv;
 
-       VG_CLEAR(madv);
+       memclear(madv);
        madv.handle = bo_gem->gem_handle;
        madv.madv = state;
        madv.retained = 1;
@@ -617,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;
@@ -659,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
@@ -703,7 +742,7 @@ retry:
 
                bo_gem->bo.size = bo_size;
 
-               VG_CLEAR(create);
+               memclear(create);
                create.size = bo_size;
 
                ret = drmIoctl(bufmgr_gem->fd,
@@ -716,20 +755,22 @@ 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;
                bo_gem->stride = 0;
 
+               /* drm_intel_gem_bo_free calls DRMLISTDEL() for an uninitialized
+                  list (vma_list), so better set the list head here */
+               DRMINITLISTHEAD(&bo_gem->name_list);
+               DRMINITLISTHEAD(&bo_gem->vma_list);
                if (drm_intel_gem_bo_set_tiling_internal(&bo_gem->bo,
                                                         tiling_mode,
                                                         stride)) {
                    drm_intel_gem_bo_free(&bo_gem->bo);
                    return NULL;
                }
-
-               DRMINITLISTHEAD(&bo_gem->name_list);
-               DRMINITLISTHEAD(&bo_gem->vma_list);
        }
 
        bo_gem->name = name;
@@ -739,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);
@@ -758,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 *
@@ -768,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 *
@@ -820,7 +860,145 @@ 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 *
+drm_intel_gem_bo_alloc_userptr(drm_intel_bufmgr *bufmgr,
+                               const char *name,
+                               void *addr,
+                               uint32_t tiling_mode,
+                               uint32_t stride,
+                               unsigned long size,
+                               unsigned long flags)
+{
+       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bufmgr;
+       drm_intel_bo_gem *bo_gem;
+       int ret;
+       struct drm_i915_gem_userptr userptr;
+
+       /* Tiling with userptr surfaces is not supported
+        * on all hardware so refuse it for time being.
+        */
+       if (tiling_mode != I915_TILING_NONE)
+               return NULL;
+
+       bo_gem = calloc(1, sizeof(*bo_gem));
+       if (!bo_gem)
+               return NULL;
+
+       bo_gem->bo.size = size;
+
+       memclear(userptr);
+       userptr.user_ptr = (__u64)((unsigned long)addr);
+       userptr.user_size = size;
+       userptr.flags = flags;
+
+       ret = drmIoctl(bufmgr_gem->fd,
+                       DRM_IOCTL_I915_GEM_USERPTR,
+                       &userptr);
+       if (ret != 0) {
+               DBG("bo_create_userptr: "
+                   "ioctl failed with user ptr %p size 0x%lx, "
+                   "user flags 0x%lx\n", addr, size, flags);
+               free(bo_gem);
+               return NULL;
+       }
+
+       bo_gem->gem_handle = userptr.handle;
+       bo_gem->bo.handle = bo_gem->gem_handle;
+       bo_gem->bo.bufmgr    = bufmgr;
+       bo_gem->is_userptr   = true;
+       bo_gem->bo.virtual   = addr;
+       /* Save the address provided by user */
+       bo_gem->user_virtual = addr;
+       bo_gem->tiling_mode  = I915_TILING_NONE;
+       bo_gem->swizzle_mode = I915_BIT_6_SWIZZLE_NONE;
+       bo_gem->stride       = 0;
+
+       DRMINITLISTHEAD(&bo_gem->name_list);
+       DRMINITLISTHEAD(&bo_gem->vma_list);
+
+       bo_gem->name = name;
+       atomic_set(&bo_gem->refcount, 1);
+       bo_gem->validate_index = -1;
+       bo_gem->reloc_tree_fences = 0;
+       bo_gem->used_as_reloc_target = false;
+       bo_gem->has_error = false;
+       bo_gem->reusable = false;
+
+       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",
+               addr, bo_gem->gem_handle, bo_gem->name,
+               size, stride, tiling_mode);
+
+       return &bo_gem->bo;
+}
+
+static bool
+has_userptr(drm_intel_bufmgr_gem *bufmgr_gem)
+{
+       int ret;
+       void *ptr;
+       long pgsz;
+       struct drm_i915_gem_userptr userptr;
+
+       pgsz = sysconf(_SC_PAGESIZE);
+       assert(pgsz > 0);
+
+       ret = posix_memalign(&ptr, pgsz, pgsz);
+       if (ret) {
+               DBG("Failed to get a page (%ld) for userptr detection!\n",
+                       pgsz);
+               return false;
+       }
+
+       memclear(userptr);
+       userptr.user_ptr = (__u64)(unsigned long)ptr;
+       userptr.user_size = pgsz;
+
+retry:
+       ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_GEM_USERPTR, &userptr);
+       if (ret) {
+               if (errno == ENODEV && userptr.flags == 0) {
+                       userptr.flags = I915_USERPTR_UNSYNCHRONIZED;
+                       goto retry;
+               }
+               free(ptr);
+               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;
+}
+
+static drm_intel_bo *
+check_bo_alloc_userptr(drm_intel_bufmgr *bufmgr,
+                      const char *name,
+                      void *addr,
+                      uint32_t tiling_mode,
+                      uint32_t stride,
+                      unsigned long size,
+                      unsigned long flags)
+{
+       if (has_userptr((drm_intel_bufmgr_gem *)bufmgr))
+               bufmgr->bo_alloc_userptr = drm_intel_gem_bo_alloc_userptr;
+       else
+               bufmgr->bo_alloc_userptr = NULL;
+
+       return drm_intel_bo_alloc_userptr(bufmgr, name, addr,
+                                         tiling_mode, stride, size, flags);
 }
 
 /**
@@ -847,21 +1025,19 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
         * alternating names for the front/back buffer a linear search
         * provides a sufficiently fast match.
         */
+       pthread_mutex_lock(&bufmgr_gem->lock);
        for (list = bufmgr_gem->named.next;
             list != &bufmgr_gem->named;
             list = list->next) {
                bo_gem = DRMLISTENTRY(drm_intel_bo_gem, list, name_list);
                if (bo_gem->global_name == handle) {
                        drm_intel_gem_bo_reference(&bo_gem->bo);
+                       pthread_mutex_unlock(&bufmgr_gem->lock);
                        return &bo_gem->bo;
                }
        }
 
-       bo_gem = calloc(1, sizeof(*bo_gem));
-       if (!bo_gem)
-               return NULL;
-
-       VG_CLEAR(open_arg);
+       memclear(open_arg);
        open_arg.name = handle;
        ret = drmIoctl(bufmgr_gem->fd,
                       DRM_IOCTL_GEM_OPEN,
@@ -869,11 +1045,33 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
        if (ret != 0) {
                DBG("Couldn't reference %s handle 0x%08x: %s\n",
                    name, handle, strerror(errno));
-               free(bo_gem);
+               pthread_mutex_unlock(&bufmgr_gem->lock);
                return NULL;
        }
+        /* Now see if someone has used a prime handle to get this
+         * object from the kernel before by looking through the list
+         * again for a matching gem_handle
+         */
+       for (list = bufmgr_gem->named.next;
+            list != &bufmgr_gem->named;
+            list = list->next) {
+               bo_gem = DRMLISTENTRY(drm_intel_bo_gem, list, name_list);
+               if (bo_gem->gem_handle == open_arg.handle) {
+                       drm_intel_gem_bo_reference(&bo_gem->bo);
+                       pthread_mutex_unlock(&bufmgr_gem->lock);
+                       return &bo_gem->bo;
+               }
+       }
+
+       bo_gem = calloc(1, sizeof(*bo_gem));
+       if (!bo_gem) {
+               pthread_mutex_unlock(&bufmgr_gem->lock);
+               return NULL;
+       }
+
        bo_gem->bo.size = open_arg.size;
        bo_gem->bo.offset = 0;
+       bo_gem->bo.offset64 = 0;
        bo_gem->bo.virtual = NULL;
        bo_gem->bo.bufmgr = bufmgr;
        bo_gem->name = name;
@@ -884,22 +1082,24 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
        bo_gem->global_name = handle;
        bo_gem->reusable = false;
 
-       VG_CLEAR(get_tiling);
+       memclear(get_tiling);
        get_tiling.handle = bo_gem->gem_handle;
        ret = drmIoctl(bufmgr_gem->fd,
                       DRM_IOCTL_I915_GEM_GET_TILING,
                       &get_tiling);
        if (ret != 0) {
                drm_intel_gem_bo_unreference(&bo_gem->bo);
+               pthread_mutex_unlock(&bufmgr_gem->lock);
                return NULL;
        }
        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);
+       pthread_mutex_unlock(&bufmgr_gem->lock);
        DBG("bo_create_from_handle: %d (%s)\n", handle, bo_gem->name);
 
        return &bo_gem->bo;
@@ -916,23 +1116,22 @@ drm_intel_gem_bo_free(drm_intel_bo *bo)
        DRMLISTDEL(&bo_gem->vma_list);
        if (bo_gem->mem_virtual) {
                VG(VALGRIND_FREELIKE_BLOCK(bo_gem->mem_virtual, 0));
-               munmap(bo_gem->mem_virtual, bo_gem->bo.size);
+               drm_munmap(bo_gem->mem_virtual, bo_gem->bo.size);
                bufmgr_gem->vma_count--;
        }
        if (bo_gem->gtt_virtual) {
-               munmap(bo_gem->gtt_virtual, bo_gem->bo.size);
+               drm_munmap(bo_gem->gtt_virtual, bo_gem->bo.size);
                bufmgr_gem->vma_count--;
        }
 
        /* Close this object */
-       VG_CLEAR(close);
+       memclear(close);
        close.handle = bo_gem->gem_handle;
        ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_GEM_CLOSE, &close);
        if (ret != 0) {
                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);
 }
 
@@ -1005,12 +1204,12 @@ static void drm_intel_gem_bo_purge_vma_cache(drm_intel_bufmgr_gem *bufmgr_gem)
                DRMLISTDELINIT(&bo_gem->vma_list);
 
                if (bo_gem->mem_virtual) {
-                       munmap(bo_gem->mem_virtual, bo_gem->bo.size);
+                       drm_munmap(bo_gem->mem_virtual, bo_gem->bo.size);
                        bo_gem->mem_virtual = NULL;
                        bufmgr_gem->vma_count--;
                }
                if (bo_gem->gtt_virtual) {
-                       munmap(bo_gem->gtt_virtual, bo_gem->bo.size);
+                       drm_munmap(bo_gem->gtt_virtual, bo_gem->bo.size);
                        bo_gem->gtt_virtual = NULL;
                        bufmgr_gem->vma_count--;
                }
@@ -1114,7 +1313,8 @@ static void drm_intel_gem_bo_unreference(drm_intel_bo *bo)
        drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
 
        assert(atomic_read(&bo_gem->refcount) > 0);
-       if (atomic_dec_and_test(&bo_gem->refcount)) {
+
+       if (atomic_add_unless(&bo_gem->refcount, -1, 1)) {
                drm_intel_bufmgr_gem *bufmgr_gem =
                    (drm_intel_bufmgr_gem *) bo->bufmgr;
                struct timespec time;
@@ -1122,8 +1322,12 @@ static void drm_intel_gem_bo_unreference(drm_intel_bo *bo)
                clock_gettime(CLOCK_MONOTONIC, &time);
 
                pthread_mutex_lock(&bufmgr_gem->lock);
-               drm_intel_gem_bo_unreference_final(bo, time.tv_sec);
-               drm_intel_gem_cleanup_bo_cache(bufmgr_gem, time.tv_sec);
+
+               if (atomic_dec_and_test(&bo_gem->refcount)) {
+                       drm_intel_gem_bo_unreference_final(bo, time.tv_sec);
+                       drm_intel_gem_cleanup_bo_cache(bufmgr_gem, time.tv_sec);
+               }
+
                pthread_mutex_unlock(&bufmgr_gem->lock);
        }
 }
@@ -1135,6 +1339,12 @@ static int drm_intel_gem_bo_map(drm_intel_bo *bo, int write_enable)
        struct drm_i915_gem_set_domain set_domain;
        int ret;
 
+       if (bo_gem->is_userptr) {
+               /* Return the same user ptr */
+               bo->virtual = bo_gem->user_virtual;
+               return 0;
+       }
+
        pthread_mutex_lock(&bufmgr_gem->lock);
 
        if (bo_gem->map_count++ == 0)
@@ -1146,9 +1356,8 @@ static int drm_intel_gem_bo_map(drm_intel_bo *bo, int write_enable)
                DBG("bo_map: %d (%s), map_count=%d\n",
                    bo_gem->gem_handle, bo_gem->name, bo_gem->map_count);
 
-               VG_CLEAR(mmap_arg);
+               memclear(mmap_arg);
                mmap_arg.handle = bo_gem->gem_handle;
-               mmap_arg.offset = 0;
                mmap_arg.size = bo->size;
                ret = drmIoctl(bufmgr_gem->fd,
                               DRM_IOCTL_I915_GEM_MMAP,
@@ -1170,7 +1379,7 @@ static int drm_intel_gem_bo_map(drm_intel_bo *bo, int write_enable)
            bo_gem->mem_virtual);
        bo->virtual = bo_gem->mem_virtual;
 
-       VG_CLEAR(set_domain);
+       memclear(set_domain);
        set_domain.handle = bo_gem->gem_handle;
        set_domain.read_domains = I915_GEM_DOMAIN_CPU;
        if (write_enable)
@@ -1203,6 +1412,9 @@ map_gtt(drm_intel_bo *bo)
        drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
        int ret;
 
+       if (bo_gem->is_userptr)
+               return -EINVAL;
+
        if (bo_gem->map_count++ == 0)
                drm_intel_gem_bo_open_vma(bufmgr_gem, bo_gem);
 
@@ -1213,7 +1425,7 @@ map_gtt(drm_intel_bo *bo)
                DBG("bo_map_gtt: mmap %d (%s), map_count=%d\n",
                    bo_gem->gem_handle, bo_gem->name, bo_gem->map_count);
 
-               VG_CLEAR(mmap_arg);
+               memclear(mmap_arg);
                mmap_arg.handle = bo_gem->gem_handle;
 
                /* Get the fake offset back... */
@@ -1232,9 +1444,9 @@ map_gtt(drm_intel_bo *bo)
                }
 
                /* and mmap it */
-               bo_gem->gtt_virtual = mmap(0, bo->size, PROT_READ | PROT_WRITE,
-                                          MAP_SHARED, bufmgr_gem->fd,
-                                          mmap_arg.offset);
+               bo_gem->gtt_virtual = drm_mmap(0, bo->size, PROT_READ | PROT_WRITE,
+                                              MAP_SHARED, bufmgr_gem->fd,
+                                              mmap_arg.offset);
                if (bo_gem->gtt_virtual == MAP_FAILED) {
                        bo_gem->gtt_virtual = NULL;
                        ret = -errno;
@@ -1256,7 +1468,8 @@ map_gtt(drm_intel_bo *bo)
        return 0;
 }
 
-int drm_intel_gem_bo_map_gtt(drm_intel_bo *bo)
+int
+drm_intel_gem_bo_map_gtt(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;
@@ -1280,7 +1493,7 @@ int drm_intel_gem_bo_map_gtt(drm_intel_bo *bo)
         * tell it when we're about to use things if we had done
         * rendering and it still happens to be bound to the GTT.
         */
-       VG_CLEAR(set_domain);
+       memclear(set_domain);
        set_domain.handle = bo_gem->gem_handle;
        set_domain.read_domains = I915_GEM_DOMAIN_GTT;
        set_domain.write_domain = I915_GEM_DOMAIN_GTT;
@@ -1314,9 +1527,13 @@ int drm_intel_gem_bo_map_gtt(drm_intel_bo *bo)
  * undefined).
  */
 
-int drm_intel_gem_bo_map_unsynchronized(drm_intel_bo *bo)
+int
+drm_intel_gem_bo_map_unsynchronized(drm_intel_bo *bo)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
+#ifdef HAVE_VALGRIND
+       drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
+#endif
        int ret;
 
        /* If the CPU cache isn't coherent with the GTT, then use a
@@ -1330,7 +1547,13 @@ int drm_intel_gem_bo_map_unsynchronized(drm_intel_bo *bo)
                return drm_intel_gem_bo_map_gtt(bo);
 
        pthread_mutex_lock(&bufmgr_gem->lock);
+
        ret = map_gtt(bo);
+       if (ret == 0) {
+               drm_intel_gem_bo_mark_mmaps_incoherent(bo);
+               VG(VALGRIND_MAKE_MEM_DEFINED(bo_gem->gtt_virtual, bo->size));
+       }
+
        pthread_mutex_unlock(&bufmgr_gem->lock);
 
        return ret;
@@ -1338,13 +1561,18 @@ int drm_intel_gem_bo_map_unsynchronized(drm_intel_bo *bo)
 
 static int drm_intel_gem_bo_unmap(drm_intel_bo *bo)
 {
-       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
+       drm_intel_bufmgr_gem *bufmgr_gem;
        drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
        int ret = 0;
 
        if (bo == NULL)
                return 0;
 
+       if (bo_gem->is_userptr)
+               return 0;
+
+       bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
+
        pthread_mutex_lock(&bufmgr_gem->lock);
 
        if (bo_gem->map_count <= 0) {
@@ -1364,7 +1592,7 @@ static int drm_intel_gem_bo_unmap(drm_intel_bo *bo)
                 * Unlike GTT set domains, this only does work if the
                 * buffer should be scanout-related.
                 */
-               VG_CLEAR(sw_finish);
+               memclear(sw_finish);
                sw_finish.handle = bo_gem->gem_handle;
                ret = drmIoctl(bufmgr_gem->fd,
                               DRM_IOCTL_I915_GEM_SW_FINISH,
@@ -1388,7 +1616,8 @@ static int drm_intel_gem_bo_unmap(drm_intel_bo *bo)
        return ret;
 }
 
-int drm_intel_gem_bo_unmap_gtt(drm_intel_bo *bo)
+int
+drm_intel_gem_bo_unmap_gtt(drm_intel_bo *bo)
 {
        return drm_intel_gem_bo_unmap(bo);
 }
@@ -1402,7 +1631,10 @@ drm_intel_gem_bo_subdata(drm_intel_bo *bo, unsigned long offset,
        struct drm_i915_gem_pwrite pwrite;
        int ret;
 
-       VG_CLEAR(pwrite);
+       if (bo_gem->is_userptr)
+               return -EINVAL;
+
+       memclear(pwrite);
        pwrite.handle = bo_gem->gem_handle;
        pwrite.offset = offset;
        pwrite.size = size;
@@ -1427,7 +1659,7 @@ drm_intel_gem_get_pipe_from_crtc_id(drm_intel_bufmgr *bufmgr, int crtc_id)
        struct drm_i915_get_pipe_from_crtc_id get_pipe_from_crtc_id;
        int ret;
 
-       VG_CLEAR(get_pipe_from_crtc_id);
+       memclear(get_pipe_from_crtc_id);
        get_pipe_from_crtc_id.crtc_id = crtc_id;
        ret = drmIoctl(bufmgr_gem->fd,
                       DRM_IOCTL_I915_GET_PIPE_FROM_CRTC_ID,
@@ -1454,7 +1686,10 @@ drm_intel_gem_bo_get_subdata(drm_intel_bo *bo, unsigned long offset,
        struct drm_i915_gem_pread pread;
        int ret;
 
-       VG_CLEAR(pread);
+       if (bo_gem->is_userptr)
+               return -EINVAL;
+
+       memclear(pread);
        pread.handle = bo_gem->gem_handle;
        pread.offset = offset;
        pread.size = size;
@@ -1502,8 +1737,12 @@ drm_intel_gem_bo_wait_rendering(drm_intel_bo *bo)
  * not guarantee that the buffer is re-issued via another thread, or an flinked
  * handle. Userspace must make sure this race does not occur if such precision
  * is important.
+ *
+ * Note that some kernels have broken the inifite wait for negative values
+ * promise, upgrade to latest stable kernels if this is the case.
  */
-int drm_intel_gem_bo_wait(drm_intel_bo *bo, int64_t timeout_ns)
+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;
        drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
@@ -1521,9 +1760,9 @@ int drm_intel_gem_bo_wait(drm_intel_bo *bo, int64_t timeout_ns)
                }
        }
 
+       memclear(wait);
        wait.bo_handle = bo_gem->gem_handle;
        wait.timeout_ns = timeout_ns;
-       wait.flags = 0;
        ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_GEM_WAIT, &wait);
        if (ret == -1)
                return -errno;
@@ -1546,7 +1785,7 @@ drm_intel_gem_bo_start_gtt_access(drm_intel_bo *bo, int write_enable)
        struct drm_i915_gem_set_domain set_domain;
        int ret;
 
-       VG_CLEAR(set_domain);
+       memclear(set_domain);
        set_domain.handle = bo_gem->gem_handle;
        set_domain.read_domains = I915_GEM_DOMAIN_GTT;
        set_domain.write_domain = write_enable ? I915_GEM_DOMAIN_GTT : 0;
@@ -1565,7 +1804,8 @@ 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);
@@ -1588,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);
 }
 
@@ -1638,6 +1890,14 @@ do_bo_emit_reloc(drm_intel_bo *bo, uint32_t offset,
        assert(offset <= bo->size - 4);
        assert((write_domain & (write_domain - 1)) == 0);
 
+       /* An object needing a fence is a tiled buffer, so it won't have
+        * relocs to other buffers.
+        */
+       if (need_fence) {
+               assert(target_bo_gem->reloc_count == 0);
+               target_bo_gem->reloc_tree_fences = 1;
+       }
+
        /* Make sure that we're not adding a reloc to something whose size has
         * already been accounted for.
         */
@@ -1645,13 +1905,8 @@ do_bo_emit_reloc(drm_intel_bo *bo, uint32_t offset,
        if (target_bo_gem != bo_gem) {
                target_bo_gem->used_as_reloc_target = true;
                bo_gem->reloc_tree_size += target_bo_gem->reloc_tree_size;
+               bo_gem->reloc_tree_fences += target_bo_gem->reloc_tree_fences;
        }
-       /* An object needing a fence is a tiled buffer, so it won't have
-        * relocs to other buffers.
-        */
-       if (need_fence)
-               target_bo_gem->reloc_tree_fences = 1;
-       bo_gem->reloc_tree_fences += target_bo_gem->reloc_tree_fences;
 
        bo_gem->relocs[bo_gem->reloc_count].offset = offset;
        bo_gem->relocs[bo_gem->reloc_count].delta = target_offset;
@@ -1659,7 +1914,7 @@ do_bo_emit_reloc(drm_intel_bo *bo, uint32_t offset,
            target_bo_gem->gem_handle;
        bo_gem->relocs[bo_gem->reloc_count].read_domains = read_domains;
        bo_gem->relocs[bo_gem->reloc_count].write_domain = write_domain;
-       bo_gem->relocs[bo_gem->reloc_count].presumed_offset = target_bo->offset;
+       bo_gem->relocs[bo_gem->reloc_count].presumed_offset = target_bo->offset64;
 
        bo_gem->reloc_target_info[bo_gem->reloc_count].bo = target_bo;
        if (target_bo != bo)
@@ -1721,6 +1976,7 @@ drm_intel_gem_bo_get_reloc_count(drm_intel_bo *bo)
 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;
        drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
        int i;
        struct timespec time;
@@ -1728,15 +1984,22 @@ drm_intel_gem_bo_clear_relocs(drm_intel_bo *bo, int start)
        clock_gettime(CLOCK_MONOTONIC, &time);
 
        assert(bo_gem->reloc_count >= start);
+
        /* Unreference the cleared target buffers */
+       pthread_mutex_lock(&bufmgr_gem->lock);
+
        for (i = start; i < bo_gem->reloc_count; i++) {
-               if (bo_gem->reloc_target_info[i].bo != bo) {
-                       drm_intel_gem_bo_unreference_locked_timed(bo_gem->
-                                                                 reloc_target_info[i].bo,
+               drm_intel_bo_gem *target_bo_gem = (drm_intel_bo_gem *) bo_gem->reloc_target_info[i].bo;
+               if (&target_bo_gem->bo != bo) {
+                       bo_gem->reloc_tree_fences -= target_bo_gem->reloc_tree_fences;
+                       drm_intel_gem_bo_unreference_locked_timed(&target_bo_gem->bo,
                                                                  time.tv_sec);
                }
        }
        bo_gem->reloc_count = start;
+
+       pthread_mutex_unlock(&bufmgr_gem->lock);
+
 }
 
 /**
@@ -1809,11 +2072,12 @@ drm_intel_update_buffer_offsets(drm_intel_bufmgr_gem *bufmgr_gem)
                drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
 
                /* Update the buffer offset */
-               if (bufmgr_gem->exec_objects[i].offset != bo->offset) {
+               if (bufmgr_gem->exec_objects[i].offset != bo->offset64) {
                        DBG("BO %d (%s) migrated: 0x%08lx -> 0x%08llx\n",
-                           bo_gem->gem_handle, bo_gem->name, bo->offset,
+                           bo_gem->gem_handle, bo_gem->name, bo->offset64,
                            (unsigned long long)bufmgr_gem->exec_objects[i].
                            offset);
+                       bo->offset64 = bufmgr_gem->exec_objects[i].offset;
                        bo->offset = bufmgr_gem->exec_objects[i].offset;
                }
        }
@@ -1829,294 +2093,22 @@ drm_intel_update_buffer_offsets2 (drm_intel_bufmgr_gem *bufmgr_gem)
                drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *)bo;
 
                /* Update the buffer offset */
-               if (bufmgr_gem->exec2_objects[i].offset != bo->offset) {
+               if (bufmgr_gem->exec2_objects[i].offset != bo->offset64) {
                        DBG("BO %d (%s) migrated: 0x%08lx -> 0x%08llx\n",
-                           bo_gem->gem_handle, bo_gem->name, bo->offset,
+                           bo_gem->gem_handle, bo_gem->name, bo->offset64,
                            (unsigned long long)bufmgr_gem->exec2_objects[i].offset);
+                       bo->offset64 = bufmgr_gem->exec2_objects[i].offset;
                        bo->offset = bufmgr_gem->exec2_objects[i].offset;
                }
        }
 }
 
-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 |
-               (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);
-       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;
-
-       /* Make a ring buffer to execute our batchbuffer. */
-       memset(ringbuffer, 0, sizeof(ringbuffer));
-       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 |
-               (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);
-
-       /* FIXME: Need some flush operations here? */
-       aub_out_data(bufmgr_gem, ringbuffer, ring_count * 4);
-
-       /* Update offset pointer */
-       bufmgr_gem->aub_offset += 4096;
-}
-
 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
@@ -2140,7 +2132,7 @@ drm_intel_gem_bo_exec(drm_intel_bo *bo, int used,
         */
        drm_intel_add_validate_buffer(bo);
 
-       VG_CLEAR(execbuf);
+       memclear(execbuf);
        execbuf.buffers_ptr = (uintptr_t) bufmgr_gem->exec_objects;
        execbuf.buffer_count = bufmgr_gem->exec_count;
        execbuf.batch_start_offset = 0;
@@ -2176,6 +2168,8 @@ drm_intel_gem_bo_exec(drm_intel_bo *bo, int used,
                drm_intel_bo *bo = bufmgr_gem->exec_bos[i];
                drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
 
+               bo_gem->idle = false;
+
                /* Disconnect the buffer from the validate list */
                bo_gem->validate_index = -1;
                bufmgr_gem->exec_bos[i] = NULL;
@@ -2207,6 +2201,10 @@ do_exec2(drm_intel_bo *bo, int used, drm_intel_context *ctx,
                if (!bufmgr_gem->has_bsd)
                        return -EINVAL;
                break;
+       case I915_EXEC_VEBOX:
+               if (!bufmgr_gem->has_vebox)
+                       return -EINVAL;
+               break;
        case I915_EXEC_RENDER:
        case I915_EXEC_DEFAULT:
                break;
@@ -2221,7 +2219,7 @@ do_exec2(drm_intel_bo *bo, int used, drm_intel_context *ctx,
         */
        drm_intel_add_validate_buffer2(bo, 0);
 
-       VG_CLEAR(execbuf);
+       memclear(execbuf);
        execbuf.buffers_ptr = (uintptr_t)bufmgr_gem->exec2_objects;
        execbuf.buffer_count = bufmgr_gem->exec_count;
        execbuf.batch_start_offset = 0;
@@ -2237,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;
 
@@ -2267,6 +2263,8 @@ skip_execution:
                drm_intel_bo *bo = bufmgr_gem->exec_bos[i];
                drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *)bo;
 
+               bo_gem->idle = false;
+
                /* Disconnect the buffer from the validate list */
                bo_gem->validate_index = -1;
                bufmgr_gem->exec_bos[i] = NULL;
@@ -2310,7 +2308,7 @@ drm_intel_gem_bo_pin(drm_intel_bo *bo, uint32_t alignment)
        struct drm_i915_gem_pin pin;
        int ret;
 
-       VG_CLEAR(pin);
+       memclear(pin);
        pin.handle = bo_gem->gem_handle;
        pin.alignment = alignment;
 
@@ -2320,6 +2318,7 @@ drm_intel_gem_bo_pin(drm_intel_bo *bo, uint32_t alignment)
        if (ret != 0)
                return -errno;
 
+       bo->offset64 = pin.offset;
        bo->offset = pin.offset;
        return 0;
 }
@@ -2332,7 +2331,7 @@ drm_intel_gem_bo_unpin(drm_intel_bo *bo)
        struct drm_i915_gem_unpin unpin;
        int ret;
 
-       VG_CLEAR(unpin);
+       memclear(unpin);
        unpin.handle = bo_gem->gem_handle;
 
        ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_GEM_UNPIN, &unpin);
@@ -2388,6 +2387,12 @@ drm_intel_gem_bo_set_tiling(drm_intel_bo *bo, uint32_t * tiling_mode,
        drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
        int ret;
 
+       /* Tiling with userptr surfaces is not supported
+        * on all hardware so refuse it for time being.
+        */
+       if (bo_gem->is_userptr)
+               return -EINVAL;
+
        /* Linear buffers have no stride. By ensuring that we only ever use
         * stride 0 with linear buffers, we simplify our code.
         */
@@ -2396,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;
@@ -2421,18 +2426,49 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
        uint32_t handle;
        drm_intel_bo_gem *bo_gem;
        struct drm_i915_gem_get_tiling get_tiling;
+       drmMMListHead *list;
 
        ret = drmPrimeFDToHandle(bufmgr_gem->fd, prime_fd, &handle);
+
+       /*
+        * See if the kernel has already returned this buffer to us. Just as
+        * for named buffers, we must not create two bo's pointing at the same
+        * kernel object
+        */
+       pthread_mutex_lock(&bufmgr_gem->lock);
+       for (list = bufmgr_gem->named.next;
+            list != &bufmgr_gem->named;
+            list = list->next) {
+               bo_gem = DRMLISTENTRY(drm_intel_bo_gem, list, name_list);
+               if (bo_gem->gem_handle == handle) {
+                       drm_intel_gem_bo_reference(&bo_gem->bo);
+                       pthread_mutex_unlock(&bufmgr_gem->lock);
+                       return &bo_gem->bo;
+               }
+       }
+
        if (ret) {
          fprintf(stderr,"ret is %d %d\n", ret, errno);
+         pthread_mutex_unlock(&bufmgr_gem->lock);
                return NULL;
        }
 
        bo_gem = calloc(1, sizeof(*bo_gem));
-       if (!bo_gem)
+       if (!bo_gem) {
+               pthread_mutex_unlock(&bufmgr_gem->lock);
                return NULL;
+       }
+       /* Determine size of bo.  The fd-to-handle ioctl really should
+        * return the size, but it doesn't.  If we have kernel 3.12 or
+        * later, we can lseek on the prime fd to get the size.  Older
+        * kernels will just fail, in which case we fall back to the
+        * provided (estimated or guess size). */
+       ret = lseek(prime_fd, 0, SEEK_END);
+       if (ret != -1)
+               bo_gem->bo.size = ret;
+       else
+               bo_gem->bo.size = size;
 
-       bo_gem->bo.size = size;
        bo_gem->bo.handle = handle;
        bo_gem->bo.bufmgr = bufmgr;
 
@@ -2447,10 +2483,11 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
        bo_gem->has_error = false;
        bo_gem->reusable = false;
 
-       DRMINITLISTHEAD(&bo_gem->name_list);
        DRMINITLISTHEAD(&bo_gem->vma_list);
+       DRMLISTADDTAIL(&bo_gem->name_list, &bufmgr_gem->named);
+       pthread_mutex_unlock(&bufmgr_gem->lock);
 
-       VG_CLEAR(get_tiling);
+       memclear(get_tiling);
        get_tiling.handle = bo_gem->gem_handle;
        ret = drmIoctl(bufmgr_gem->fd,
                       DRM_IOCTL_I915_GEM_GET_TILING,
@@ -2462,7 +2499,7 @@ 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;
 }
@@ -2473,7 +2510,18 @@ 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;
        drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
 
-       return drmPrimeHandleToFD(bufmgr_gem->fd, bo_gem->gem_handle, DRM_CLOEXEC, prime_fd);
+       pthread_mutex_lock(&bufmgr_gem->lock);
+        if (DRMLISTEMPTY(&bo_gem->name_list))
+                DRMLISTADDTAIL(&bo_gem->name_list, &bufmgr_gem->named);
+       pthread_mutex_unlock(&bufmgr_gem->lock);
+
+       if (drmPrimeHandleToFD(bufmgr_gem->fd, bo_gem->gem_handle,
+                              DRM_CLOEXEC, prime_fd) != 0)
+               return -errno;
+
+       bo_gem->reusable = false;
+
+       return 0;
 }
 
 static int
@@ -2486,17 +2534,23 @@ drm_intel_gem_bo_flink(drm_intel_bo *bo, uint32_t * name)
        if (!bo_gem->global_name) {
                struct drm_gem_flink flink;
 
-               VG_CLEAR(flink);
+               memclear(flink);
                flink.handle = bo_gem->gem_handle;
 
+               pthread_mutex_lock(&bufmgr_gem->lock);
+
                ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_GEM_FLINK, &flink);
-               if (ret != 0)
+               if (ret != 0) {
+                       pthread_mutex_unlock(&bufmgr_gem->lock);
                        return -errno;
+               }
 
                bo_gem->global_name = flink.name;
                bo_gem->reusable = false;
 
-               DRMLISTADDTAIL(&bo_gem->name_list, &bufmgr_gem->named);
+                if (DRMLISTEMPTY(&bo_gem->name_list))
+                        DRMLISTADDTAIL(&bo_gem->name_list, &bufmgr_gem->named);
+               pthread_mutex_unlock(&bufmgr_gem->lock);
        }
 
        *name = bo_gem->global_name;
@@ -2815,7 +2869,7 @@ static int
 get_pci_device_id(drm_intel_bufmgr_gem *bufmgr_gem)
 {
        char *devid_override;
-       int devid;
+       int devid = 0;
        int ret;
        drm_i915_getparam_t gp;
 
@@ -2827,8 +2881,7 @@ get_pci_device_id(drm_intel_bufmgr_gem *bufmgr_gem)
                }
        }
 
-       VG_CLEAR(devid);
-       VG_CLEAR(gp);
+       memclear(gp);
        gp.param = I915_PARAM_CHIPSET_ID;
        gp.value = &devid;
        ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_GETPARAM, &gp);
@@ -2848,6 +2901,18 @@ drm_intel_bufmgr_gem_get_devid(drm_intel_bufmgr *bufmgr)
 }
 
 /**
+ * Sets the AUB filename.
+ *
+ * This function has to be called before drm_intel_bufmgr_gem_set_aub_dump()
+ * for it to have any effect.
+ */
+void
+drm_intel_bufmgr_gem_set_aub_filename(drm_intel_bufmgr *bufmgr,
+                                     const char *filename)
+{
+}
+
+/**
  * Sets up AUB dumping.
  *
  * This is a trace file format that can be used with the simulator.
@@ -2858,49 +2923,11 @@ drm_intel_bufmgr_gem_get_devid(drm_intel_bufmgr *bufmgr)
 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;
-
-       if (!enable) {
-               if (bufmgr_gem->aub_file) {
-                       fclose(bufmgr_gem->aub_file);
-                       bufmgr_gem->aub_file = NULL;
-               }
-       }
-
-       if (geteuid() != getuid())
-               return;
-
-       bufmgr_gem->aub_file = fopen("intel.aub", "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 | (5 - 2));
-       aub_out(bufmgr_gem, AUB_TRACE_MEMTYPE_NONLOCAL | 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 */
-       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_intel_context *
@@ -2908,18 +2935,22 @@ drm_intel_gem_context_create(drm_intel_bufmgr *bufmgr)
 {
        drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *)bufmgr;
        struct drm_i915_gem_context_create create;
-       drm_i915_getparam_t gp;
        drm_intel_context *context = NULL;
-       int tmp = 0, ret;
+       int ret;
 
+       context = calloc(1, sizeof(*context));
+       if (!context)
+               return NULL;
+
+       memclear(create);
        ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_GEM_CONTEXT_CREATE, &create);
        if (ret != 0) {
                DBG("DRM_IOCTL_I915_GEM_CONTEXT_CREATE failed: %s\n",
                    strerror(errno));
+               free(context);
                return NULL;
        }
 
-       context = calloc(1, sizeof(*context));
        context->ctx_id = create.ctx_id;
        context->bufmgr = bufmgr;
 
@@ -2936,6 +2967,8 @@ drm_intel_gem_context_destroy(drm_intel_context *ctx)
        if (ctx == NULL)
                return;
 
+       memclear(destroy);
+
        bufmgr_gem = (drm_intel_bufmgr_gem *)ctx->bufmgr;
        destroy.ctx_id = ctx->ctx_id;
        ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_GEM_CONTEXT_DESTROY,
@@ -2947,6 +2980,89 @@ drm_intel_gem_context_destroy(drm_intel_context *ctx)
        free(ctx);
 }
 
+int
+drm_intel_get_reset_stats(drm_intel_context *ctx,
+                         uint32_t *reset_count,
+                         uint32_t *active,
+                         uint32_t *pending)
+{
+       drm_intel_bufmgr_gem *bufmgr_gem;
+       struct drm_i915_reset_stats stats;
+       int ret;
+
+       if (ctx == NULL)
+               return -EINVAL;
+
+       memclear(stats);
+
+       bufmgr_gem = (drm_intel_bufmgr_gem *)ctx->bufmgr;
+       stats.ctx_id = ctx->ctx_id;
+       ret = drmIoctl(bufmgr_gem->fd,
+                      DRM_IOCTL_I915_GET_RESET_STATS,
+                      &stats);
+       if (ret == 0) {
+               if (reset_count != NULL)
+                       *reset_count = stats.reset_count;
+
+               if (active != NULL)
+                       *active = stats.batch_active;
+
+               if (pending != NULL)
+                       *pending = stats.batch_pending;
+       }
+
+       return ret;
+}
+
+int
+drm_intel_reg_read(drm_intel_bufmgr *bufmgr,
+                  uint32_t offset,
+                  uint64_t *result)
+{
+       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *)bufmgr;
+       struct drm_i915_reg_read reg_read;
+       int ret;
+
+       memclear(reg_read);
+       reg_read.offset = offset;
+
+       ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_REG_READ, &reg_read);
+
+       *result = reg_read.val;
+       return ret;
+}
+
+int
+drm_intel_get_subslice_total(int fd, unsigned int *subslice_total)
+{
+       drm_i915_getparam_t gp;
+       int ret;
+
+       memclear(gp);
+       gp.value = (int*)subslice_total;
+       gp.param = I915_PARAM_SUBSLICE_TOTAL;
+       ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
+       if (ret)
+               return -errno;
+
+       return 0;
+}
+
+int
+drm_intel_get_eu_total(int fd, unsigned int *eu_total)
+{
+       drm_i915_getparam_t gp;
+       int ret;
+
+       memclear(gp);
+       gp.value = (int*)eu_total;
+       gp.param = I915_PARAM_EU_TOTAL;
+       ret = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
+       if (ret)
+               return -errno;
+
+       return 0;
+}
 
 /**
  * Annotate the given bo for use in aub dumping.
@@ -2974,19 +3090,41 @@ 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;
+}
+
+static pthread_mutex_t bufmgr_list_mutex = PTHREAD_MUTEX_INITIALIZER;
+static drmMMListHead bufmgr_list = { &bufmgr_list, &bufmgr_list };
+
+static drm_intel_bufmgr_gem *
+drm_intel_bufmgr_gem_find(int fd)
+{
+       drm_intel_bufmgr_gem *bufmgr_gem;
+
+       DRMLISTFOREACHENTRY(bufmgr_gem, &bufmgr_list, managers) {
+               if (bufmgr_gem->fd == fd) {
+                       atomic_inc(&bufmgr_gem->refcount);
+                       return bufmgr_gem;
+               }
+       }
+
+       return NULL;
+}
+
+static void
+drm_intel_bufmgr_gem_unref(drm_intel_bufmgr *bufmgr)
+{
+       drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *)bufmgr;
+
+       if (atomic_add_unless(&bufmgr_gem->refcount, -1, 1)) {
+               pthread_mutex_lock(&bufmgr_list_mutex);
+
+               if (atomic_dec_and_test(&bufmgr_gem->refcount)) {
+                       DRMLISTDEL(&bufmgr_gem->managers);
+                       drm_intel_bufmgr_gem_destroy(bufmgr);
+               }
+
+               pthread_mutex_unlock(&bufmgr_list_mutex);
        }
-       memcpy(new_annotations, annotations, size);
-       bo_gem->aub_annotations = new_annotations;
-       bo_gem->aub_annotation_count = count;
 }
 
 /**
@@ -3004,17 +3142,26 @@ drm_intel_bufmgr_gem_init(int fd, int batch_size)
        int ret, tmp;
        bool exec2 = false;
 
+       pthread_mutex_lock(&bufmgr_list_mutex);
+
+       bufmgr_gem = drm_intel_bufmgr_gem_find(fd);
+       if (bufmgr_gem)
+               goto exit;
+
        bufmgr_gem = calloc(1, sizeof(*bufmgr_gem));
        if (bufmgr_gem == NULL)
-               return NULL;
+               goto exit;
 
        bufmgr_gem->fd = fd;
+       atomic_set(&bufmgr_gem->refcount, 1);
 
        if (pthread_mutex_init(&bufmgr_gem->lock, NULL) != 0) {
                free(bufmgr_gem);
-               return NULL;
+               bufmgr_gem = NULL;
+               goto exit;
        }
 
+       memclear(aperture);
        ret = drmIoctl(bufmgr_gem->fd,
                       DRM_IOCTL_I915_GEM_GET_APERTURE,
                       &aperture);
@@ -3044,9 +3191,16 @@ drm_intel_bufmgr_gem_init(int fd, int batch_size)
        else if (IS_GEN6(bufmgr_gem->pci_device))
                bufmgr_gem->gen = 6;
        else if (IS_GEN7(bufmgr_gem->pci_device))
-               bufmgr_gem->gen = 7;
-       else
-               assert(0);
+               bufmgr_gem->gen = 7;
+       else if (IS_GEN8(bufmgr_gem->pci_device))
+               bufmgr_gem->gen = 8;
+       else if (IS_GEN9(bufmgr_gem->pci_device))
+               bufmgr_gem->gen = 9;
+       else {
+               free(bufmgr_gem);
+               bufmgr_gem = NULL;
+               goto exit;
+       }
 
        if (IS_GEN3(bufmgr_gem->pci_device) &&
            bufmgr_gem->gtt_size > 256*1024*1024) {
@@ -3057,7 +3211,7 @@ drm_intel_bufmgr_gem_init(int fd, int batch_size)
                bufmgr_gem->gtt_size -= 256*1024*1024;
        }
 
-       VG_CLEAR(gp);
+       memclear(gp);
        gp.value = &tmp;
 
        gp.param = I915_PARAM_HAS_EXECBUF2;
@@ -3077,6 +3231,8 @@ drm_intel_bufmgr_gem_init(int fd, int batch_size)
        ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_GETPARAM, &gp);
        bufmgr_gem->has_relaxed_fencing = ret == 0;
 
+       bufmgr_gem->bufmgr.bo_alloc_userptr = check_bo_alloc_userptr;
+
        gp.param = I915_PARAM_HAS_WAIT_TIMEOUT;
        ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_GETPARAM, &gp);
        bufmgr_gem->has_wait_timeout = ret == 0;
@@ -3090,7 +3246,11 @@ drm_intel_bufmgr_gem_init(int fd, int batch_size)
                bufmgr_gem->has_llc = (IS_GEN6(bufmgr_gem->pci_device) |
                                IS_GEN7(bufmgr_gem->pci_device));
        } else
-               bufmgr_gem->has_llc = ret == 0;
+               bufmgr_gem->has_llc = *gp.value;
+
+       gp.param = I915_PARAM_HAS_VEBOX;
+       ret = drmIoctl(bufmgr_gem->fd, DRM_IOCTL_I915_GETPARAM, &gp);
+       bufmgr_gem->has_vebox = (ret == 0) & (*gp.value > 0);
 
        if (bufmgr_gem->gen < 4) {
                gp.param = I915_PARAM_NUM_FENCES_AVAIL;
@@ -3152,7 +3312,7 @@ drm_intel_bufmgr_gem_init(int fd, int batch_size)
                bufmgr_gem->bufmgr.bo_exec = drm_intel_gem_bo_exec;
        bufmgr_gem->bufmgr.bo_busy = drm_intel_gem_bo_busy;
        bufmgr_gem->bufmgr.bo_madvise = drm_intel_gem_bo_madvise;
-       bufmgr_gem->bufmgr.destroy = drm_intel_bufmgr_gem_destroy;
+       bufmgr_gem->bufmgr.destroy = drm_intel_bufmgr_gem_unref;
        bufmgr_gem->bufmgr.debug = 0;
        bufmgr_gem->bufmgr.check_aperture_space =
            drm_intel_gem_check_aperture_space;
@@ -3168,5 +3328,10 @@ drm_intel_bufmgr_gem_init(int fd, int batch_size)
        DRMINITLISTHEAD(&bufmgr_gem->vma_cache);
        bufmgr_gem->vma_max = -1; /* unlimited by default */
 
-       return &bufmgr_gem->bufmgr;
+       DRMLISTADD(&bufmgr_gem->managers, &bufmgr_list);
+
+exit:
+       pthread_mutex_unlock(&bufmgr_list_mutex);
+
+       return bufmgr_gem != NULL ? &bufmgr_gem->bufmgr : NULL;
 }