Re: [Intel-gfx] [PATCH v2 1/6] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-24 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-04-24 15:48:10) > > On 24/04/2018 15:04, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2018-04-24 14:55:51) > >> > >> On 24/04/2018 12:28, Chris Wilson wrote: > >>> Quoting Tvrtko Ursulin (2018-04-24 12:17:15) > > On 24/04/2018 11:40, Chris Wilson wrote: >

Re: [Intel-gfx] [PATCH v2 1/6] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-24 Thread Tvrtko Ursulin
On 24/04/2018 15:04, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-04-24 14:55:51) On 24/04/2018 12:28, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-04-24 12:17:15) On 24/04/2018 11:40, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-04-24 11:14:21) On 23/04/2018 19:08, Chris Wil

Re: [Intel-gfx] [PATCH v2 1/6] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-24 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-04-24 14:55:51) > > On 24/04/2018 12:28, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2018-04-24 12:17:15) > >> > >> On 24/04/2018 11:40, Chris Wilson wrote: > >>> Quoting Tvrtko Ursulin (2018-04-24 11:14:21) > > On 23/04/2018 19:08, Chris Wilson wrote: >

Re: [Intel-gfx] [PATCH v2 1/6] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-24 Thread Tvrtko Ursulin
On 24/04/2018 12:28, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-04-24 12:17:15) On 24/04/2018 11:40, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-04-24 11:14:21) On 23/04/2018 19:08, Chris Wilson wrote: -static int reserve_engine(struct intel_engine_cs *engine) +static int reserv

Re: [Intel-gfx] [PATCH v2 1/6] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-24 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-04-24 12:17:15) > > On 24/04/2018 11:40, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2018-04-24 11:14:21) > >> > >> On 23/04/2018 19:08, Chris Wilson wrote: > >>> -static int reserve_engine(struct intel_engine_cs *engine) > >>> +static int reserve_gt(struct drm_i91

Re: [Intel-gfx] [PATCH v2 1/6] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-24 Thread Tvrtko Ursulin
On 24/04/2018 11:40, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-04-24 11:14:21) On 23/04/2018 19:08, Chris Wilson wrote: -static int reserve_engine(struct intel_engine_cs *engine) +static int reserve_gt(struct drm_i915_private *i915) { - struct drm_i915_private *i915 = engine->i9

Re: [Intel-gfx] [PATCH v2 1/6] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-24 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-04-24 11:14:21) > > On 23/04/2018 19:08, Chris Wilson wrote: > > -static int reserve_engine(struct intel_engine_cs *engine) > > +static int reserve_gt(struct drm_i915_private *i915) > > { > > - struct drm_i915_private *i915 = engine->i915; > > - u32 active =

Re: [Intel-gfx] [PATCH v2 1/6] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-24 Thread Tvrtko Ursulin
On 23/04/2018 19:08, Chris Wilson wrote: In commit 9b6586ae9f6b ("drm/i915: Keep a global seqno per-engine"), we moved from a global inflight counter to per-engine counters in the hope that will be easy to run concurrently in future. However, with the advent of the desire to move requests betwee

[Intel-gfx] [PATCH v2 1/6] drm/i915: Stop tracking timeline->inflight_seqnos

2018-04-23 Thread Chris Wilson
In commit 9b6586ae9f6b ("drm/i915: Keep a global seqno per-engine"), we moved from a global inflight counter to per-engine counters in the hope that will be easy to run concurrently in future. However, with the advent of the desire to move requests between engines, we do need a global counter to pr