[Intel-gfx] [PATCH 03/12 v2] drm/i915: move vbetool invoked ier stuff

Jesse Barnes jbarnes at virtuousgeek.org
Fri May 4 21:40:18 CEST 2012


On Thu, 26 Apr 2012 16:03:00 -0700
Ben Widawsky <ben at bwidawsk.net> wrote:

> This extra bit of interrupt enabling code doesn't belong in the wait
> seqno function. If anything we should pull it out to a helper so the
> throttle code can also use it. The history is a bit vague, but I am
> going to attempt to just dump it, unless someone can argue otherwise.
> 
> Removing this allows for a shared lock free wait seqno function. To
> keep tabs on this issue though, the IER value is stored on error
> capture (recommended by Chris Wilson)
> 
> v2: fixed typo EIR->IER (Ben)
> Fix some white space (Ben)
> Move IER capture to globally instead of per ring (Ben)
> 

It may be safe to remove this now.  Distros used to run vbetool at
random times, shutting things off and messing with hw.  But these days
they use the driver with a bit less molestation, so the IER check/fix
shouldn't be needed.

Jesse



More information about the Intel-gfx mailing list