[Intel-gfx] [PATCH] drm/i915/execlists: Relax the locked clear_bit(IRQ_EXECLIST)
Chris Wilson
chris at chris-wilson.co.uk
Thu Mar 23 22:17:15 UTC 2017
On Thu, Mar 23, 2017 at 04:51:54PM +0000, Tvrtko Ursulin wrote:
>
> On 23/03/2017 13:48, Chris Wilson wrote:
> >We only need to care about the ordering of the clearing of the bit with
> >the uncached CSB read in order to correctly detect a new interrupt
> >before the read completes. The uncached read itself acts as a full
> >memory barrier, so we do not to enforce another in the form of a locked
>
> need ^
>
> >clear_bit.
> >
> >Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> >Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> >---
> > drivers/gpu/drm/i915/intel_lrc.c | 9 ++++++++-
> > 1 file changed, 8 insertions(+), 1 deletion(-)
> >
> >diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c
> >index 59acdd3b7a12..c9010fa881b4 100644
> >--- a/drivers/gpu/drm/i915/intel_lrc.c
> >+++ b/drivers/gpu/drm/i915/intel_lrc.c
> >@@ -540,7 +540,14 @@ static void intel_lrc_irq_handler(unsigned long data)
> > dev_priv->regs + i915_mmio_reg_offset(RING_CONTEXT_STATUS_BUF_LO(engine, 0));
> > unsigned int csb, head, tail;
> >
> >- clear_bit(ENGINE_IRQ_EXECLIST, &engine->irq_posted);
> >+ /* The write will be ordered by the uncached read (itself
> >+ * a memory barrier), we do need another in the form a
>
> not ^
>
> >+ * locked instruction. That is the clear of IRQ_EXECLIST bit
> >+ * will be visible to another cpu prior to the completion
> >+ * of the CSB read. If the other cpu receives an interrupt
> >+ * before then, we will redo the CSB read.
>
> I don't think the the visibility of clearing the bit matters to
> anyone. Just the harmlessness of the race between the interrupt
> sandwiching between test bit and clear bit.
Yes, spelling out the harmlessness makes a better comment. Thanks,
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list