On Tue, Sep 15, 2015 at 02:03:27PM +0530, ankitprasad.r.sha...@intel.com wrote:
> @@ -1090,17 +1184,17 @@ i915_gem_pwrite_ioctl(struct drm_device *dev, void 
> *data,
>               goto out;
>       }
>  
> -     /* prime objects have no backing filp to GEM pread/pwrite
> -      * pages from.
> -      */
> -     if (!obj->base.filp) {
> -             ret = -EINVAL;
> -             goto out;
> -     }
> -
>       trace_i915_gem_object_pwrite(obj, args->offset, args->size);
>  
>       ret = -EFAULT;
> +
> +     /* pwrite for non shmem backed objects */
> +     if (!obj->base.filp) {
> +             ret = i915_gem_gtt_pread_pwrite(dev, obj, args->size,
> +                                             args->offset, args->data_ptr,
> +                                             true);
> +             goto out;
> +     }

There already exists a GTT write path, along with a more correct
description of its limitations.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to