etnaviv: drm: Don't miscalculate timeout
[mesa.git] / src / etnaviv / drm / etnaviv_bo.c
index 31ad484da4f13f2151ceed6e08c1336acaf650be..0ec64b780f5d2fb1f4afefc8486faffc0e50a420 100644 (file)
  *    Christian Gmeiner <christian.gmeiner@gmail.com>
  */
 
+#include "os/os_mman.h"
+#include "util/hash_table.h"
+
 #include "etnaviv_priv.h"
 #include "etnaviv_drmif.h"
 
-pthread_mutex_t table_lock = PTHREAD_MUTEX_INITIALIZER;
-void bo_del(struct etna_bo *bo);
+pthread_mutex_t etna_drm_table_lock = PTHREAD_MUTEX_INITIALIZER;
+void _etna_bo_del(struct etna_bo *bo);
 
-/* set buffer name, and add to table, call w/ table_lock held: */
+/* set buffer name, and add to table, call w/ etna_drm_table_lock held: */
 static void set_name(struct etna_bo *bo, uint32_t name)
 {
        bo->name = name;
        /* add ourself into the name table: */
-       drmHashInsert(bo->dev->name_table, name, bo);
+       _mesa_hash_table_insert(bo->dev->name_table, &bo->name, bo);
 }
 
-/* Called under table_lock */
-void bo_del(struct etna_bo *bo)
+/* Called under etna_drm_table_lock */
+void _etna_bo_del(struct etna_bo *bo)
 {
-       if (bo->map)
-               drm_munmap(bo->map, bo->size);
+       VG_BO_FREE(bo);
 
-       if (bo->name)
-               drmHashDelete(bo->dev->name_table, bo->name);
+       if (bo->va)
+               util_vma_heap_free(&bo->dev->address_space, bo->va, bo->size);
+
+       if (bo->map)
+               os_munmap(bo->map, bo->size);
 
        if (bo->handle) {
                struct drm_gem_close req = {
                        .handle = bo->handle,
                };
 
-               drmHashDelete(bo->dev->handle_table, bo->handle);
+               if (bo->name)
+                       _mesa_hash_table_remove_key(bo->dev->name_table, &bo->name);
+
+               _mesa_hash_table_remove_key(bo->dev->handle_table, &bo->handle);
                drmIoctl(bo->dev->fd, DRM_IOCTL_GEM_CLOSE, &req);
        }
 
        free(bo);
 }
 
-/* lookup a buffer from it's handle, call w/ table_lock held: */
+/* lookup a buffer from it's handle, call w/ etna_drm_table_lock held: */
 static struct etna_bo *lookup_bo(void *tbl, uint32_t handle)
 {
        struct etna_bo *bo = NULL;
+       struct hash_entry *entry = _mesa_hash_table_search(tbl, &handle);
 
-       if (!drmHashLookup(tbl, handle, (void **)&bo)) {
+       if (entry) {
                /* found, incr refcnt and return: */
-               bo = etna_bo_ref(bo);
+               bo = etna_bo_ref(entry->data);
 
                /* don't break the bucket if this bo was found in one */
                list_delinit(&bo->list);
@@ -75,7 +84,7 @@ static struct etna_bo *lookup_bo(void *tbl, uint32_t handle)
        return bo;
 }
 
-/* allocate a new buffer object, call w/ table_lock held */
+/* allocate a new buffer object, call w/ etna_drm_table_lock held */
 static struct etna_bo *bo_from_handle(struct etna_device *dev,
                uint32_t size, uint32_t handle, uint32_t flags)
 {
@@ -98,7 +107,10 @@ static struct etna_bo *bo_from_handle(struct etna_device *dev,
        p_atomic_set(&bo->refcnt, 1);
        list_inithead(&bo->list);
        /* add ourselves to the handle table: */
-       drmHashInsert(dev->handle_table, handle, bo);
+       _mesa_hash_table_insert(dev->handle_table, &bo->handle, bo);
+
+       if (dev->use_softpin)
+               bo->va = util_vma_heap_alloc(&dev->address_space, bo->size, 4096);
 
        return bo;
 }
@@ -123,10 +135,12 @@ struct etna_bo *etna_bo_new(struct etna_device *dev, uint32_t size,
        if (ret)
                return NULL;
 
-       pthread_mutex_lock(&table_lock);
+       pthread_mutex_lock(&etna_drm_table_lock);
        bo = bo_from_handle(dev, size, req.handle, flags);
        bo->reuse = 1;
-       pthread_mutex_unlock(&table_lock);
+       pthread_mutex_unlock(&etna_drm_table_lock);
+
+       VG_BO_ALLOC(bo);
 
        return bo;
 }
@@ -167,7 +181,7 @@ struct etna_bo *etna_bo_from_name(struct etna_device *dev,
                .name = name,
        };
 
-       pthread_mutex_lock(&table_lock);
+       pthread_mutex_lock(&etna_drm_table_lock);
 
        /* check name table first, to see if bo is already open: */
        bo = lookup_bo(dev->name_table, name);
@@ -184,11 +198,13 @@ struct etna_bo *etna_bo_from_name(struct etna_device *dev,
                goto out_unlock;
 
        bo = bo_from_handle(dev, req.size, req.handle, 0);
-       if (bo)
+       if (bo) {
                set_name(bo, name);
+               VG_BO_ALLOC(bo);
+       }
 
 out_unlock:
-       pthread_mutex_unlock(&table_lock);
+       pthread_mutex_unlock(&etna_drm_table_lock);
 
        return bo;
 }
@@ -207,11 +223,11 @@ struct etna_bo *etna_bo_from_dmabuf(struct etna_device *dev, int fd)
         * racing against etna_bo_del, which might invalidate the
         * returned handle.
         */
-       pthread_mutex_lock(&table_lock);
+       pthread_mutex_lock(&etna_drm_table_lock);
 
        ret = drmPrimeFDToHandle(dev->fd, fd, &handle);
        if (ret) {
-               pthread_mutex_unlock(&table_lock);
+               pthread_mutex_unlock(&etna_drm_table_lock);
                return NULL;
        }
 
@@ -225,8 +241,10 @@ struct etna_bo *etna_bo_from_dmabuf(struct etna_device *dev, int fd)
 
        bo = bo_from_handle(dev, size, handle, 0);
 
+       VG_BO_ALLOC(bo);
+
 out_unlock:
-       pthread_mutex_unlock(&table_lock);
+       pthread_mutex_unlock(&etna_drm_table_lock);
 
        return bo;
 }
@@ -234,23 +252,23 @@ out_unlock:
 /* destroy a buffer object */
 void etna_bo_del(struct etna_bo *bo)
 {
-       struct etna_device *dev = bo->dev;
-
        if (!bo)
                return;
 
+       struct etna_device *dev = bo->dev;
+
        if (!p_atomic_dec_zero(&bo->refcnt))
                return;
 
-       pthread_mutex_lock(&table_lock);
+       pthread_mutex_lock(&etna_drm_table_lock);
 
        if (bo->reuse && (etna_bo_cache_free(&dev->bo_cache, bo) == 0))
                goto out;
 
-       bo_del(bo);
+       _etna_bo_del(bo);
        etna_device_del_locked(dev);
 out:
-       pthread_mutex_unlock(&table_lock);
+       pthread_mutex_unlock(&etna_drm_table_lock);
 }
 
 /* get the global flink/DRI2 buffer name */
@@ -267,9 +285,9 @@ int etna_bo_get_name(struct etna_bo *bo, uint32_t *name)
                        return ret;
                }
 
-               pthread_mutex_lock(&table_lock);
+               pthread_mutex_lock(&etna_drm_table_lock);
                set_name(bo, req.name);
-               pthread_mutex_unlock(&table_lock);
+               pthread_mutex_unlock(&etna_drm_table_lock);
                bo->reuse = 0;
        }
 
@@ -307,6 +325,11 @@ uint32_t etna_bo_size(struct etna_bo *bo)
        return bo->size;
 }
 
+uint32_t etna_bo_gpu_va(struct etna_bo *bo)
+{
+       return bo->va;
+}
+
 void *etna_bo_map(struct etna_bo *bo)
 {
        if (!bo->map) {
@@ -314,8 +337,8 @@ void *etna_bo_map(struct etna_bo *bo)
                        get_buffer_info(bo);
                }
 
-               bo->map = drm_mmap(0, bo->size, PROT_READ | PROT_WRITE,
-                               MAP_SHARED, bo->dev->fd, bo->offset);
+               bo->map = os_mmap(0, bo->size, PROT_READ | PROT_WRITE,
+                                 MAP_SHARED, bo->dev->fd, bo->offset);
                if (bo->map == MAP_FAILED) {
                        ERROR_MSG("mmap failed: %s", strerror(errno));
                        bo->map = NULL;