drm/i915: Remove the unsightly "optimisation" from flush_fence()
authorChris Wilson <chris@chris-wilson.co.uk>
Tue, 17 Apr 2012 14:31:29 +0000 (15:31 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 18 Apr 2012 11:23:17 +0000 (13:23 +0200)
As i915_wait_request() will first check for an already passed seqno,
doing it also in the caller is a waste of space for a cold path.

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.c

index f7cd3461dc4f42892638cbbe67f9a47757bf9c6a..bac3570d2c471ff3ec6cdd77c414ef189cde6f9d 100644 (file)
@@ -2280,11 +2280,6 @@ static int i830_write_fence_reg(struct drm_i915_gem_object *obj)
        return 0;
 }
 
-static bool ring_passed_seqno(struct intel_ring_buffer *ring, u32 seqno)
-{
-       return i915_seqno_passed(ring->get_seqno(ring), seqno);
-}
-
 static int
 i915_gem_object_flush_fence(struct drm_i915_gem_object *obj)
 {
@@ -2302,14 +2297,11 @@ i915_gem_object_flush_fence(struct drm_i915_gem_object *obj)
        }
 
        if (obj->last_fenced_seqno) {
-               if (!ring_passed_seqno(obj->ring,
-                                      obj->last_fenced_seqno)) {
-                       ret = i915_wait_request(obj->ring,
-                                               obj->last_fenced_seqno,
-                                               true);
-                       if (ret)
-                               return ret;
-               }
+               ret = i915_wait_request(obj->ring,
+                                       obj->last_fenced_seqno,
+                                       true);
+               if (ret)
+                       return ret;
 
                obj->last_fenced_seqno = 0;
        }