We need to add support for pwrite'ing an LMEM object.

Signed-off-by: Matthew Auld <matthew.a...@intel.com>
Signed-off-by: Steve Hampson <steven.t.hamp...@intel.com>
Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
Cc: Abdiel Janulgue <abdiel.janul...@linux.intel.com>
---
 drivers/gpu/drm/i915/gem/i915_gem_lmem.c | 87 ++++++++++++++++++++++++
 1 file changed, 87 insertions(+)

diff --git a/drivers/gpu/drm/i915/gem/i915_gem_lmem.c 
b/drivers/gpu/drm/i915/gem/i915_gem_lmem.c
index f5a13994dc2a..f00078ac331e 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_lmem.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_lmem.c
@@ -94,6 +94,92 @@ static int lmem_pread(struct drm_i915_gem_object *obj,
        return ret;
 }
 
+static int lmem_pwrite(struct drm_i915_gem_object *obj,
+                      const struct drm_i915_gem_pwrite *arg)
+{
+       struct drm_i915_private *i915 = to_i915(obj->base.dev);
+       struct intel_runtime_pm *rpm = &i915->runtime_pm;
+       intel_wakeref_t wakeref;
+       struct dma_fence *fence;
+       char __user *user_data;
+       unsigned int offset;
+       unsigned long idx;
+       u64 remain;
+       int ret;
+
+       ret = i915_gem_object_wait(obj,
+                                  I915_WAIT_INTERRUPTIBLE,
+                                  MAX_SCHEDULE_TIMEOUT);
+       if (ret)
+               return ret;
+
+       ret = i915_gem_object_pin_pages(obj);
+       if (ret)
+               return ret;
+
+       i915_gem_object_lock(obj);
+       ret = i915_gem_object_set_to_wc_domain(obj, true);
+       if (ret) {
+               i915_gem_object_unlock(obj);
+               goto out_unpin;
+       }
+
+       fence = i915_gem_object_lock_fence(obj);
+       i915_gem_object_unlock(obj);
+       if (!fence) {
+               ret = -ENOMEM;
+               goto out_unpin;
+       }
+
+       wakeref = intel_runtime_pm_get(rpm);
+
+       remain = arg->size;
+       user_data = u64_to_user_ptr(arg->data_ptr);
+       offset = offset_in_page(arg->offset);
+       for (idx = arg->offset >> PAGE_SHIFT; remain; idx++) {
+               unsigned long unwritten;
+               void __iomem *vaddr;
+               int length;
+
+               length = remain;
+               if (offset + length > PAGE_SIZE)
+                       length = PAGE_SIZE - offset;
+
+               vaddr = i915_gem_object_lmem_io_map_page_atomic(obj, idx);
+               if (!vaddr) {
+                       ret = -ENOMEM;
+                       goto out_put;
+               }
+
+               unwritten = __copy_from_user_inatomic_nocache((void 
__force*)vaddr + offset,
+                                                             user_data, 
length);
+               io_mapping_unmap_atomic(vaddr);
+               if (unwritten) {
+                       vaddr = i915_gem_object_lmem_io_map_page(obj, idx);
+                       unwritten = copy_from_user((void __force*)vaddr + 
offset,
+                                                  user_data, length);
+                       io_mapping_unmap(vaddr);
+               }
+               if (unwritten) {
+                       ret = -EFAULT;
+                       goto out_put;
+               }
+
+               remain -= length;
+               user_data += length;
+               offset = 0;
+       }
+
+out_put:
+       intel_runtime_pm_put(rpm, wakeref);
+       i915_gem_object_unlock_fence(obj, fence);
+out_unpin:
+       i915_gem_object_unpin_pages(obj);
+
+       return ret;
+}
+
+
 const struct drm_i915_gem_object_ops i915_gem_lmem_obj_ops = {
        .flags = I915_GEM_OBJECT_IS_MAPPABLE,
 
@@ -102,6 +188,7 @@ const struct drm_i915_gem_object_ops i915_gem_lmem_obj_ops 
= {
        .release = i915_gem_object_release_memory_region,
 
        .pread = lmem_pread,
+       .pwrite = lmem_pwrite,
 };
 
 /* XXX: Time to vfunc your life up? */
-- 
2.20.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to