drm/ttm: optionally move duplicates to a separate list
[firefly-linux-kernel-4.4.55.git] / drivers / gpu / drm / qxl / qxl_release.c
index 14e776f1d14e3bf099259d8d6b8f94ac9896f63b..d9b25684ac989d73341b7cddd70bb181b28eee2b 100644 (file)
@@ -21,6 +21,7 @@
  */
 #include "qxl_drv.h"
 #include "qxl_object.h"
+#include <trace/events/fence.h>
 
 /*
  * drawable cmd cache - allocate a bunch of VRAM pages, suballocate
 static const int release_size_per_bo[] = { RELEASE_SIZE, SURFACE_RELEASE_SIZE, RELEASE_SIZE };
 static const int releases_per_bo[] = { RELEASES_PER_BO, SURFACE_RELEASES_PER_BO, RELEASES_PER_BO };
 
+static const char *qxl_get_driver_name(struct fence *fence)
+{
+       return "qxl";
+}
+
+static const char *qxl_get_timeline_name(struct fence *fence)
+{
+       return "release";
+}
+
+static bool qxl_nop_signaling(struct fence *fence)
+{
+       /* fences are always automatically signaled, so just pretend we did this.. */
+       return true;
+}
+
+static long qxl_fence_wait(struct fence *fence, bool intr, signed long timeout)
+{
+       struct qxl_device *qdev;
+       struct qxl_release *release;
+       int count = 0, sc = 0;
+       bool have_drawable_releases;
+       unsigned long cur, end = jiffies + timeout;
+
+       qdev = container_of(fence->lock, struct qxl_device, release_lock);
+       release = container_of(fence, struct qxl_release, base);
+       have_drawable_releases = release->type == QXL_RELEASE_DRAWABLE;
+
+retry:
+       sc++;
+
+       if (fence_is_signaled(fence))
+               goto signaled;
+
+       qxl_io_notify_oom(qdev);
+
+       for (count = 0; count < 11; count++) {
+               if (!qxl_queue_garbage_collect(qdev, true))
+                       break;
+
+               if (fence_is_signaled(fence))
+                       goto signaled;
+       }
+
+       if (fence_is_signaled(fence))
+               goto signaled;
+
+       if (have_drawable_releases || sc < 4) {
+               if (sc > 2)
+                       /* back off */
+                       usleep_range(500, 1000);
+
+               if (time_after(jiffies, end))
+                       return 0;
+
+               if (have_drawable_releases && sc > 300) {
+                       FENCE_WARN(fence, "failed to wait on release %d "
+                                         "after spincount %d\n",
+                                         fence->context & ~0xf0000000, sc);
+                       goto signaled;
+               }
+               goto retry;
+       }
+       /*
+        * yeah, original sync_obj_wait gave up after 3 spins when
+        * have_drawable_releases is not set.
+        */
+
+signaled:
+       cur = jiffies;
+       if (time_after(cur, end))
+               return 0;
+       return end - cur;
+}
+
+static const struct fence_ops qxl_fence_ops = {
+       .get_driver_name = qxl_get_driver_name,
+       .get_timeline_name = qxl_get_timeline_name,
+       .enable_signaling = qxl_nop_signaling,
+       .wait = qxl_fence_wait,
+};
+
 static uint64_t
 qxl_release_alloc(struct qxl_device *qdev, int type,
                  struct qxl_release **ret)
@@ -46,13 +129,13 @@ qxl_release_alloc(struct qxl_device *qdev, int type,
        struct qxl_release *release;
        int handle;
        size_t size = sizeof(*release);
-       int idr_ret;
 
        release = kmalloc(size, GFP_KERNEL);
        if (!release) {
                DRM_ERROR("Out of memory\n");
                return 0;
        }
+       release->base.ops = NULL;
        release->type = type;
        release->release_offset = 0;
        release->surface_release_id = 0;
@@ -60,44 +143,61 @@ qxl_release_alloc(struct qxl_device *qdev, int type,
 
        idr_preload(GFP_KERNEL);
        spin_lock(&qdev->release_idr_lock);
-       idr_ret = idr_alloc(&qdev->release_idr, release, 1, 0, GFP_NOWAIT);
+       handle = idr_alloc(&qdev->release_idr, release, 1, 0, GFP_NOWAIT);
+       release->base.seqno = ++qdev->release_seqno;
        spin_unlock(&qdev->release_idr_lock);
        idr_preload_end();
-       handle = idr_ret;
-       if (idr_ret < 0)
-               goto release_fail;
+       if (handle < 0) {
+               kfree(release);
+               *ret = NULL;
+               return handle;
+       }
        *ret = release;
        QXL_INFO(qdev, "allocated release %lld\n", handle);
        release->id = handle;
-release_fail:
-
        return handle;
 }
 
+static void
+qxl_release_free_list(struct qxl_release *release)
+{
+       while (!list_empty(&release->bos)) {
+               struct qxl_bo_list *entry;
+               struct qxl_bo *bo;
+
+               entry = container_of(release->bos.next,
+                                    struct qxl_bo_list, tv.head);
+               bo = to_qxl_bo(entry->tv.bo);
+               qxl_bo_unref(&bo);
+               list_del(&entry->tv.head);
+               kfree(entry);
+       }
+}
+
 void
 qxl_release_free(struct qxl_device *qdev,
                 struct qxl_release *release)
 {
-       struct qxl_bo_list *entry, *tmp;
        QXL_INFO(qdev, "release %d, type %d\n", release->id,
                 release->type);
 
        if (release->surface_release_id)
                qxl_surface_id_dealloc(qdev, release->surface_release_id);
 
-       list_for_each_entry_safe(entry, tmp, &release->bos, tv.head) {
-               struct qxl_bo *bo = to_qxl_bo(entry->tv.bo);
-               QXL_INFO(qdev, "release %llx\n",
-                       drm_vma_node_offset_addr(&entry->tv.bo->vma_node)
-                                               - DRM_FILE_OFFSET);
-               qxl_fence_remove_release(&bo->fence, release->id);
-               qxl_bo_unref(&bo);
-               kfree(entry);
-       }
        spin_lock(&qdev->release_idr_lock);
        idr_remove(&qdev->release_idr, release->id);
        spin_unlock(&qdev->release_idr_lock);
-       kfree(release);
+
+       if (release->base.ops) {
+               WARN_ON(list_empty(&release->bos));
+               qxl_release_free_list(release);
+
+               fence_signal(&release->base);
+               fence_put(&release->base);
+       } else {
+               qxl_release_free_list(release);
+               kfree(release);
+       }
 }
 
 static int qxl_release_bo_alloc(struct qxl_device *qdev,
@@ -126,6 +226,7 @@ int qxl_release_list_add(struct qxl_release *release, struct qxl_bo *bo)
 
        qxl_bo_ref(bo);
        entry->tv.bo = &bo->tbo;
+       entry->tv.shared = false;
        list_add_tail(&entry->tv.head, &release->bos);
        return 0;
 }
@@ -142,6 +243,10 @@ static int qxl_release_validate_bo(struct qxl_bo *bo)
                        return ret;
        }
 
+       ret = reservation_object_reserve_shared(bo->tbo.resv);
+       if (ret)
+               return ret;
+
        /* allocate a surface for reserved + validated buffers */
        ret = qxl_bo_check_id(bo->gem_base.dev->dev_private, bo);
        if (ret)
@@ -159,7 +264,8 @@ int qxl_release_reserve_list(struct qxl_release *release, bool no_intr)
        if (list_is_singular(&release->bos))
                return 0;
 
-       ret = ttm_eu_reserve_buffers(&release->ticket, &release->bos);
+       ret = ttm_eu_reserve_buffers(&release->ticket, &release->bos,
+                                    !no_intr, NULL);
        if (ret)
                return ret;
 
@@ -199,6 +305,8 @@ int qxl_alloc_surface_release_reserved(struct qxl_device *qdev,
 
                /* stash the release after the create command */
                idr_ret = qxl_release_alloc(qdev, QXL_RELEASE_SURFACE_CMD, release);
+               if (idr_ret < 0)
+                       return idr_ret;
                bo = qxl_bo_ref(to_qxl_bo(entry->tv.bo));
 
                (*release)->release_offset = create_rel->release_offset + 64;
@@ -239,6 +347,11 @@ int qxl_alloc_release_reserved(struct qxl_device *qdev, unsigned long size,
        }
 
        idr_ret = qxl_release_alloc(qdev, type, release);
+       if (idr_ret < 0) {
+               if (rbo)
+                       *rbo = NULL;
+               return idr_ret;
+       }
 
        mutex_lock(&qdev->release_mutex);
        if (qdev->current_release_bo_offset[cur_idx] + 1 >= releases_per_bo[cur_idx]) {
@@ -319,40 +432,44 @@ void qxl_release_unmap(struct qxl_device *qdev,
 
 void qxl_release_fence_buffer_objects(struct qxl_release *release)
 {
-       struct ttm_validate_buffer *entry;
        struct ttm_buffer_object *bo;
        struct ttm_bo_global *glob;
        struct ttm_bo_device *bdev;
        struct ttm_bo_driver *driver;
        struct qxl_bo *qbo;
+       struct ttm_validate_buffer *entry;
+       struct qxl_device *qdev;
 
        /* if only one object on the release its the release itself
           since these objects are pinned no need to reserve */
-       if (list_is_singular(&release->bos))
+       if (list_is_singular(&release->bos) || list_empty(&release->bos))
                return;
 
        bo = list_first_entry(&release->bos, struct ttm_validate_buffer, head)->bo;
        bdev = bo->bdev;
+       qdev = container_of(bdev, struct qxl_device, mman.bdev);
+
+       /*
+        * Since we never really allocated a context and we don't want to conflict,
+        * set the highest bits. This will break if we really allow exporting of dma-bufs.
+        */
+       fence_init(&release->base, &qxl_fence_ops, &qdev->release_lock,
+                  release->id | 0xf0000000, release->base.seqno);
+       trace_fence_emit(&release->base);
+
        driver = bdev->driver;
        glob = bo->glob;
 
        spin_lock(&glob->lru_lock);
-       spin_lock(&bdev->fence_lock);
 
        list_for_each_entry(entry, &release->bos, head) {
                bo = entry->bo;
                qbo = to_qxl_bo(bo);
 
-               if (!entry->bo->sync_obj)
-                       entry->bo->sync_obj = &qbo->fence;
-
-               qxl_fence_add_release_locked(&qbo->fence, release->id);
-
+               reservation_object_add_shared_fence(bo->resv, &release->base);
                ttm_bo_add_to_lru(bo);
                __ttm_bo_unreserve(bo);
-               entry->reserved = false;
        }
-       spin_unlock(&bdev->fence_lock);
        spin_unlock(&glob->lru_lock);
        ww_acquire_fini(&release->ticket);
 }