drm/i915: No need to search again after retiring requests
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 24 Feb 2012 00:27:22 +0000 (00:27 +0000)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 27 Feb 2012 16:37:13 +0000 (17:37 +0100)
Retiring requests does not typically free up space in the aperture,
so the additional search is pointless.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_gem_evict.c

index 6ba4be52936bfbb42285784602fe644183f1ea9a..21a82710f4b280eaf2dd48acb84cb4469cd4bec2 100644 (file)
@@ -48,21 +48,6 @@ i915_gem_evict_something(struct drm_device *dev, int min_size,
        struct drm_i915_gem_object *obj;
        int ret = 0;
 
-       i915_gem_retire_requests(dev);
-
-       /* Re-check for free space after retiring requests */
-       if (mappable) {
-               if (drm_mm_search_free_in_range(&dev_priv->mm.gtt_space,
-                                               min_size, alignment, 0,
-                                               dev_priv->mm.gtt_mappable_end,
-                                               0))
-                       return 0;
-       } else {
-               if (drm_mm_search_free(&dev_priv->mm.gtt_space,
-                                      min_size, alignment, 0))
-                       return 0;
-       }
-
        trace_i915_gem_evict(dev, min_size, alignment, mappable);
 
        /*