Re: [PATCH 3/5] drm/i915: Suppress EIO during i915_gem_idle()

Chris Wilson
2012-04-18T09:03:32Z

On Wed, 18 Apr 2012 10:51:29 +0200, Daniel Vetter <daniel@ffwll.ch> wrote:
> On Sat, Apr 14, 2012 at 09:55:49AM +0100, Chris Wilson wrote:
> > i915_gem_idle() is called when we need to suspend the GPU. If the GPU is
> > already suspended by this point due to being wedged, we can safely
> > continue.
> >
> > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>
> Imo the right way to fix this case here is by checking this in gpu_idle
> and just not bothering to emit any flushes and request if the gpu is
> wedged already.

Updated to fix i915_gpu_idle() instead. I don't want to add any more
checks to dev->wedged though, as that I agree will become a nightmare.
-Chris

--
Chris Wilson, Intel Open Source Technology Centre