[Intel-gfx] [PATCH] drm/i915/execlists: Use a local lock for dfs_link access

David Weinehall david.weinehall at linux.intel.com
Thu Nov 17 09:16:58 UTC 2016


On Wed, Nov 16, 2016 at 03:27:21PM +0000, Chris Wilson wrote:
> Avoid requiring struct_mutex for exclusive access to the temporary
> dfs_link inside the i915_dependency as not all callers may want to touch
> struct_mutex. So rather than force them to take a highly contended
> lock, introduce a local lock for the execlists schedule operation.
> 
> Reported-by: David Weinehall <david.weinehall at linux.intel.com>
> Fixes: 9a151987d709 ("drm/i915: Add execution priority boosting for mmioflips")
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> Cc: David Weinehall <david.weinehall at linux.intel.com>

I'll provide a retroactive:

Tested-by: David Weinehall <david.weinehall at linux.intel.com>

Seems to work fine; no flooded logs on my test machine with this patch.


Kind regards, David


More information about the Intel-gfx mailing list