drm/i915: Use discardable buffers for rings
authorChris Wilson <chris@chris-wilson.co.uk>
Thu, 20 Apr 2017 10:17:09 +0000 (11:17 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Thu, 20 Apr 2017 11:00:32 +0000 (12:00 +0100)
The contents of a ring are only valid between HEAD and TAIL, when the
ring is idle (HEAD == TAIL) we can simply let the pages go under memory
pressure if they are not pinned by an active context. Any new content
will be written after HEAD and so the ring will again be valid between
HEAD and TAIL, everything outside can be discarded.

Note that we take care of ensuring that we do not reset the HEAD
backwards following a GPU hang on an idle ring.

The same precautions are what enable us to use stolen memory for rings.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: http://patchwork.freedesktop.org/patch/msgid/20170420101709.27250-1-chris@chris-wilson.co.uk
Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
drivers/gpu/drm/i915/intel_ringbuffer.c

index 833740bf49c99ec6f90073e2c476fa4a787304b0..32afac6c754fed915ac8cb05b30be5db9c78191b 100644 (file)
@@ -1341,7 +1341,7 @@ intel_ring_create_vma(struct drm_i915_private *dev_priv, int size)
 
        obj = i915_gem_object_create_stolen(dev_priv, size);
        if (!obj)
-               obj = i915_gem_object_create(dev_priv, size);
+               obj = i915_gem_object_create_internal(dev_priv, size);
        if (IS_ERR(obj))
                return ERR_CAST(obj);