Re: [Intel-gfx] [PATCH] drm/i915: Kill the active list spinlock

2010-08-06 Thread Eric Anholt
On Wed, 4 Aug 2010 14:09:45 +0100, Chris Wilson ch...@chris-wilson.co.uk wrote: This spinlock only served debugging purposes in a time when we could not be sure of the mutex ever being released upon a GPU hang. As we now should be able rely on hangcheck to do the job for us (and that error

Re: [Intel-gfx] [PATCH] drm/i915: Kill the active list spinlock

2010-08-06 Thread Chris Wilson
On Fri, 06 Aug 2010 14:41:02 -0700, Eric Anholt e...@anholt.net wrote: On Wed, 4 Aug 2010 14:09:45 +0100, Chris Wilson ch...@chris-wilson.co.uk wrote: This spinlock only served debugging purposes in a time when we could not be sure of the mutex ever being released upon a GPU hang. As we

[Intel-gfx] [PATCH] drm/i915: Kill the active list spinlock

2010-08-04 Thread Chris Wilson
This spinlock only served debugging purposes in a time when we could not be sure of the mutex ever being released upon a GPU hang. As we now should be able rely on hangcheck to do the job for us (and that error reporting should not itself require the struct mutex) we can kill the incomplete and