On Thu, Dec 17, 2015 at 10:45:15AM +0000, Tvrtko Ursulin wrote:
>
> Hi,
>
> On 14/12/15 05:46, ankitprasad.r.sharma@xxxxxxxxx wrote:
> >From: Ankitprasad Sharma <ankitprasad.r.sharma@xxxxxxxxx>
> >
> >In pwrite_fast, map an object page by page if obj_ggtt_pin fails. First,
> >we try a nonblocking pin for the whole object (since that is fastest if
> >reused), then failing that we try to grab one page in the mappable
> >aperture. It also allows us to handle objects larger than the mappable
> >aperture (e.g. if we need to pwrite with vGPU restricting the aperture
> >to a measely 8MiB or something like that).
> >
> >v2: Pin pages before starting pwrite, Combined duplicate loops (Chris)
> >
> >v3: Combined loops based on local patch by Chris (Chris)
> >
> >v4: Added i915 wrapper function for drm_mm_insert_node_in_range (Chris)
> >
> >Signed-off-by: Ankitprasad Sharma <ankitprasad.r.sharma@xxxxxxxxx>
> >Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> >---
> > drivers/gpu/drm/i915/i915_gem.c | 86 ++++++++++++++++++++++++++++++-----------
> > 1 file changed, 64 insertions(+), 22 deletions(-)
> >
> >diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
> >index bf7f203..46c1e75 100644
> >--- a/drivers/gpu/drm/i915/i915_gem.c
> >+++ b/drivers/gpu/drm/i915/i915_gem.c
> >@@ -61,6 +61,21 @@ static bool cpu_write_needs_clflush(struct drm_i915_gem_object *obj)
> > return obj->pin_display;
> > }
> >
> >+static int
> >+i915_gem_insert_node_in_range(struct drm_i915_private *i915,
> >+ struct drm_mm_node *node, u64 size,
> >+ unsigned alignment, u64 start, u64 end)
> >+{
> >+ int ret;
> >+
> >+ ret = drm_mm_insert_node_in_range_generic(&i915->gtt.base.mm, node,
> >+ size, alignment, 0, start,
> >+ end, DRM_MM_SEARCH_DEFAULT,
> >+ DRM_MM_SEARCH_DEFAULT);
> >+
> >+ return ret;
> >+}
> >+
> > /* some bookkeeping */
> > static void i915_gem_info_add_obj(struct drm_i915_private *dev_priv,
> > size_t size)
> >@@ -760,20 +775,29 @@ fast_user_write(struct io_mapping *mapping,
> > * user into the GTT, uncached.
> > */
> > static int
> >-i915_gem_gtt_pwrite_fast(struct drm_device *dev,
> >+i915_gem_gtt_pwrite_fast(struct drm_i915_private *i915,
> > struct drm_i915_gem_object *obj,
> > struct drm_i915_gem_pwrite *args,
> > struct drm_file *file)
> > {
> >- struct drm_i915_private *dev_priv = dev->dev_private;
> >- ssize_t remain;
> >- loff_t offset, page_base;
> >+ struct drm_mm_node node;
> >+ uint64_t remain, offset;
> > char __user *user_data;
> >- int page_offset, page_length, ret;
> >+ int ret;
> >
> > ret = i915_gem_obj_ggtt_pin(obj, 0, PIN_MAPPABLE | PIN_NONBLOCK);
> >- if (ret)
> >- goto out;
> >+ if (ret) {
> >+ memset(&node, 0, sizeof(node));
> >+ ret = i915_gem_insert_node_in_range(i915, &node, 4096, 0,
> >+ 0, i915->gtt.mappable_end);
>
> Suggest PAGE_SIZE instead of 4096 to match the main loop below.
>
> >+ if (ret)
> >+ goto out;
> >+
> >+ i915_gem_object_pin_pages(obj);
>
> i915_gem_object_get_pages is missing again before pin pages I think.
That's due to rebasing my patch where I merge the get_pages call into
pin_pages, sorry.
> If true it means we need an IGT to exercise this path. Should be
> easy with a huge object and just pwrite a small chunk?
Hmm, it should be hit by gem_pwrite/big-gtt + huge-gtt. If not, then we
do indeed more testing.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
