Re: [Intel-gfx] [PATCH v2 3/6] drm/i915: Only track live rings for retiring

2018-04-24 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-04-24 10:37:03) > > On 23/04/2018 19:08, Chris Wilson wrote: > > @@ -1403,14 +1406,17 @@ static void ring_retire_requests(struct intel_ring > > *ring) > > > > void i915_retire_requests(struct drm_i915_private *i915) > > { > > - struct intel_ring *ring, *nex

Re: [Intel-gfx] [PATCH v2 3/6] drm/i915: Only track live rings for retiring

2018-04-24 Thread Tvrtko Ursulin
On 23/04/2018 19:08, Chris Wilson wrote: We don't need to track every ring for its lifetime as they are managed by the contexts/engines. What we do want to track are the live rings so that we can sporadically clean up requests if userspace falls behind. We can simply restrict the gt->rings list

[Intel-gfx] [PATCH v2 3/6] drm/i915: Only track live rings for retiring

2018-04-23 Thread Chris Wilson
We don't need to track every ring for its lifetime as they are managed by the contexts/engines. What we do want to track are the live rings so that we can sporadically clean up requests if userspace falls behind. We can simply restrict the gt->rings list to being only gt->live_rings. v2: s/live/ac