diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2012-04-24 22:48:47 +0200 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2012-05-03 11:18:09 +0200 |
commit | 7338aefa5cad08eb6794aa8e9a5853dedb990aa0 (patch) | |
tree | ab722239c55f3382d77675a63016a3ca46e22b87 /drivers/gpu/drm/i915/intel_ringbuffer.h | |
parent | drm/i915: implement Disable4x2SubspanOptimization w/a for ivb, too (diff) | |
download | linux-7338aefa5cad08eb6794aa8e9a5853dedb990aa0.tar.xz linux-7338aefa5cad08eb6794aa8e9a5853dedb990aa0.zip |
drm/i915: Use a global lock for modifying global irq flags
We were attempting to use a per-ring spinlock whilst modifying global
IRQ flags. A recipe for rare missed interrupts.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Acked-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_ringbuffer.h')
-rw-r--r-- | drivers/gpu/drm/i915/intel_ringbuffer.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.h b/drivers/gpu/drm/i915/intel_ringbuffer.h index 06a66adf69c2..e0b25bb2fb5d 100644 --- a/drivers/gpu/drm/i915/intel_ringbuffer.h +++ b/drivers/gpu/drm/i915/intel_ringbuffer.h @@ -56,8 +56,7 @@ struct intel_ring_buffer { */ u32 last_retired_head; - spinlock_t irq_lock; - u32 irq_refcount; + u32 irq_refcount; /* protected by dev_priv->irq_lock */ u32 irq_enable_mask; /* bitmask to enable ring interrupt */ u32 irq_seqno; /* last seq seem at irq time */ u32 trace_irq_seqno; |