drm/i915: Use LRI to update the semaphore registers
authorChris Wilson <chris@chris-wilson.co.uk>
Wed, 14 Nov 2012 09:15:14 +0000 (09:15 +0000)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 21 Nov 2012 16:45:00 +0000 (17:45 +0100)
The bspec was recently updated to remove the ability to update the
semaphore using the MI_SEMAPHORE_BOX command, the ability to wait upon
the semaphore value remained. Instead the advice is to update the
register using the MI_LOAD_REGISTER_IMM command. In cursory testing,
semaphores continue to function - the question is whether this fixes
some of the deadlocks where the semaphore registers contained stale
values?

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Daniel J Blueman <daniel@quora.org>
Reviewed-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_ringbuffer.c

index 1aa76892a830076df97df2102aa0316c8ee961f2..987eb5fdaf3946f546322b8eb58a07d7980a071a 100644 (file)
@@ -558,12 +558,9 @@ update_mboxes(struct intel_ring_buffer *ring,
            u32 seqno,
            u32 mmio_offset)
 {
-       intel_ring_emit(ring, MI_SEMAPHORE_MBOX |
-                             MI_SEMAPHORE_GLOBAL_GTT |
-                             MI_SEMAPHORE_REGISTER |
-                             MI_SEMAPHORE_UPDATE);
-       intel_ring_emit(ring, seqno);
+       intel_ring_emit(ring, MI_LOAD_REGISTER_IMM(1));
        intel_ring_emit(ring, mmio_offset);
+       intel_ring_emit(ring, seqno);
 }
 
 /**