[Intel-gfx] [PATCH 1/2] drm/i915/lrc: Clear context restore/save inhibit flags for new contexts

Chris Wilson chris at chris-wilson.co.uk
Thu Jan 25 18:03:06 UTC 2018


Quoting Michel Thierry (2018-01-25 17:49:49)
> On 1/25/2018 3:24 AM, Chris Wilson wrote:
> > CTX_CONTEXT_CONTROL (CTX_SR_CTL) operates as a masked register and so
> > will only apply the bits that are selected by the upper half. In the
> > case of selectively enabling sr inhibit, this may mean the context keeps
> > the current setting (so forgetting to save the context later, eventually
> > leading to a very upset GPU!).
> 
> Oops, true no one would clear Context Save Inhibit once it's set.
> 
> > 
> > Fixes: d2b4b97933f5 ("drm/i915: Record the default hw state after reset upon load")
> Does it really fixes this one ^^^?

Actually, having added the BUG_ON, I had better not recommend back
porting to the earlier fixes.
-Chris


More information about the Intel-gfx mailing list