Re: [PATCH] drm/i915: Exit cherryview_irq_handler() after one pass

Ville Syrjälä
2016-03-10T13:01:20Z

On Thu, Mar 10, 2016 at 12:42:48PM +0000, Chris Wilson wrote:
> On Thu, Mar 10, 2016 at 02:24:39PM +0200, Ville Syrjälä wrote:
> > On Thu, Mar 10, 2016 at 12:10:46PM +0000, Chris Wilson wrote:
> > > My testing only looks at the GT side, and we do stress that pretty hard
> > > because of execlists and have reasonable methods of detection if we stop
> > > processing execbuf. I'm more worried about the display and pipe interrupts.
> >
> > IIRC GT was where the problem was originally.
>
> Before execlists, the only source of GT interrupts would be
> user-interrupts. There the problem is usually not so much that we miss
> the GT interrupt, but that the seqno write is not completed by the time
> the interrupt is asserted. I hope that was the problem you saw.

IIRC those guys already had execlists in use with their Android stuff.
But we did have that whole snooping mess still unresolved at the time
so who knows what was really going on.

--
Ville Syrjälä
Intel OTC