Re: [Intel-gfx] [PATCH 3/7] drm/i915: Retire requests along rings

2018-04-27 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-04-27 13:50:44) > > On 26/04/2018 18:49, Chris Wilson wrote: > > +static void __retire_engine_request(struct intel_engine_cs *engine, > > + struct i915_request *rq) > > +{ > > + GEM_TRACE("%s(%s) fence %llx:%d, global=%d, current %d\

Re: [Intel-gfx] [PATCH 3/7] drm/i915: Retire requests along rings

2018-04-27 Thread Tvrtko Ursulin
On 26/04/2018 18:49, Chris Wilson wrote: In the next patch, rings are the central timeline as requests may jump between engines. Therefore in the future as we retire in order along the engine timeline, we may retire out-of-order within a ring (as the ring now occurs along multiple engines), lead

[Intel-gfx] [PATCH 3/7] drm/i915: Retire requests along rings

2018-04-26 Thread Chris Wilson
In the next patch, rings are the central timeline as requests may jump between engines. Therefore in the future as we retire in order along the engine timeline, we may retire out-of-order within a ring (as the ring now occurs along multiple engines), leading to much hilarity in miscomputing the pos

[Intel-gfx] [PATCH 3/7] drm/i915: Retire requests along rings

2018-04-25 Thread Chris Wilson
In the next patch, rings are the central timeline as requests may jump between engines. Therefore in the future as we retire in order along the engine timeline, we may retire out-of-order within a ring (as the ring now occurs along multiple engines), leading to much hilarity in miscomputing the pos