On Wed, 18 Apr 2012 10:51:29 +0200, Daniel Vetter <daniel at 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 at 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
