[Intel-gfx] [PATCH] drm/i915: Always double check for a missed interrupt for new bottom halves

Chris Wilson chris at chris-wilson.co.uk
Tue Jul 5 11:55:28 UTC 2016


On Tue, Jul 05, 2016 at 12:29:05PM +0100, Chris Wilson wrote:
> After assigning ourselves as the new bottom-half, we must perform a
> cursory check to prevent a missed interrupt.  Either we miss the interrupt
> whilst programming the hardware, or if there was a previous waiter (for
> a later seqno) they may be woken instead of us (due to the inherent race
> in the unlocked read of b->tasklet in the irq handler) and so we miss the
> wake up.
> 
> Spotted-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
Fixes: 688e6c725816 ("drm/i915: Slaughter the thundering... herd")
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list