Re: [Intel-gfx] [PATCH] drm/i915: Fail gpu reset if the forcewake fifo hasn't drained

2014-03-08 Thread Ben Widawsky
On Sat, Mar 08, 2014 at 08:58:24PM +0100, Daniel Vetter wrote: > On Sat, Mar 8, 2014 at 7:50 PM, Ben Widawsky wrote: > > I've seen this too. Though I think the WARN does coincide with what the > > docs state - it doesn't seem to match reality. So I totally agree this > > is the right course. > > >

Re: [Intel-gfx] [PATCH] drm/i915: Fail gpu reset if the forcewake fifo hasn't drained

2014-03-08 Thread Daniel Vetter
On Sat, Mar 8, 2014 at 7:50 PM, Ben Widawsky wrote: > I've seen this too. Though I think the WARN does coincide with what the > docs state - it doesn't seem to match reality. So I totally agree this > is the right course. > > However, for my curiosity, Chris, can you elaborate on why you think it

Re: [Intel-gfx] [PATCH] drm/i915: Fail gpu reset if the forcewake fifo hasn't drained

2014-03-08 Thread Ben Widawsky
On Fri, Mar 07, 2014 at 10:35:56PM +0100, Daniel Vetter wrote: > On Fri, Mar 07, 2014 at 09:09:03PM +0100, Daniel Vetter wrote: > > Since the gpu reset + full ppgtt merge we have a hard hang on snb when > > running the gem_reset_stat tests. Recently Mika also some more strict > > forcewake fifo war

Re: [Intel-gfx] [PATCH] drm/i915: Fail gpu reset if the forcewake fifo hasn't drained

2014-03-07 Thread Daniel Vetter
On Fri, Mar 07, 2014 at 09:09:03PM +0100, Daniel Vetter wrote: > Since the gpu reset + full ppgtt merge we have a hard hang on snb when > running the gem_reset_stat tests. Recently Mika also some more strict > forcewake fifo warnigns for gen6/7 in > > commit 20277c60ed08ab4f7237854cc6c2046649f9200

[Intel-gfx] [PATCH] drm/i915: Fail gpu reset if the forcewake fifo hasn't drained

2014-03-07 Thread Daniel Vetter
Since the gpu reset + full ppgtt merge we have a hard hang on snb when running the gem_reset_stat tests. Recently Mika also some more strict forcewake fifo warnigns for gen6/7 in commit 20277c60ed08ab4f7237854cc6c2046649f9200f Author: Mika Kuoppala Date: Wed Mar 5 18:08:19 2014 +0200 drm/i