Re: [Intel-gfx] [PATCH 2/2] drm/i915/execlists: Reset RING registers upon resume

2016-09-19 Thread Joonas Lahtinen
On pe, 2016-09-16 at 20:23 +0100, Chris Wilson wrote: >  void intel_lr_context_resume(struct drm_i915_private *dev_priv) >  { > > - struct i915_gem_context *ctx = dev_priv->kernel_context; > >   struct intel_engine_cs *engine; > > + struct i915_gem_context *ctx; > + > > + /* Because we

[Intel-gfx] [PATCH 2/2] drm/i915/execlists: Reset RING registers upon resume

2016-09-16 Thread Chris Wilson
There is a disparity in the context image saved to disk and our own bookkeeping - that is we presume the RING_HEAD and RING_TAIL match our stored ce->ring->tail value. However, as we emit WA_TAIL_DWORDS into the ring but may not tell the GPU about them, the GPU may be lagging behind our