Re: [Intel-gfx] [PATCH 04/17] drm/i915/gt: Make intel_ring_unpin() safe for concurrent pint

2019-11-19 Thread Mika Kuoppala
Chris Wilson writes: concurrent pint? Sure, thirsty already. But I am afraid you just mean concurrent pin :O > In order to avoid some nasty mutex inversions, commit 09c5ab384f6f > ("drm/i915: Keep rings pinned while the context is active") allowed the > intel_ring unpinning to be run

[Intel-gfx] [PATCH 04/17] drm/i915/gt: Make intel_ring_unpin() safe for concurrent pint

2019-11-19 Thread Chris Wilson
In order to avoid some nasty mutex inversions, commit 09c5ab384f6f ("drm/i915: Keep rings pinned while the context is active") allowed the intel_ring unpinning to be run concurrently with the next context pinning it. Thus each step in intel_ring_unpin() needed to be atomic and ordered in a nice