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@xxxxxxxxxxxxxxx>
> Fixes: 9a151987d709 ("drm/i915: Add execution priority boosting for mmioflips")
> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
> Cc: David Weinehall <david.weinehall@xxxxxxxxxxxxxxx>
I'll provide a retroactive:
Tested-by: David Weinehall <david.weinehall@xxxxxxxxxxxxxxx>
Seems to work fine; no flooded logs on my test machine with this patch.
Kind regards, David
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
