Re: [Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-11 Thread Chris Wilson
Quoting Tvrtko Ursulin (2019-03-11 16:34:41) > > On 11/03/2019 16:22, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2019-03-11 16:16:27) > >> > >> On 11/03/2019 14:45, Chris Wilson wrote: > >>> Quoting Chris Wilson (2019-03-11 09:45:17) > Quoting Tvrtko Ursulin (2019-03-11 09:23:44) > >

Re: [Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-11 Thread Tvrtko Ursulin
On 11/03/2019 16:22, Chris Wilson wrote: Quoting Tvrtko Ursulin (2019-03-11 16:16:27) On 11/03/2019 14:45, Chris Wilson wrote: Quoting Chris Wilson (2019-03-11 09:45:17) Quoting Tvrtko Ursulin (2019-03-11 09:23:44) On 08/03/2019 16:47, Chris Wilson wrote: Quoting Tvrtko Ursulin (2019-03-0

Re: [Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-11 Thread Chris Wilson
Quoting Tvrtko Ursulin (2019-03-11 16:16:27) > > On 11/03/2019 14:45, Chris Wilson wrote: > > Quoting Chris Wilson (2019-03-11 09:45:17) > >> Quoting Tvrtko Ursulin (2019-03-11 09:23:44) > >>> > >>> On 08/03/2019 16:47, Chris Wilson wrote: > Quoting Tvrtko Ursulin (2019-03-08 16:27:22) >

Re: [Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-11 Thread Tvrtko Ursulin
On 11/03/2019 14:45, Chris Wilson wrote: Quoting Chris Wilson (2019-03-11 09:45:17) Quoting Tvrtko Ursulin (2019-03-11 09:23:44) On 08/03/2019 16:47, Chris Wilson wrote: Quoting Tvrtko Ursulin (2019-03-08 16:27:22) On 08/03/2019 14:12, Chris Wilson wrote: +static int +set_engines(struct i

Re: [Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-11 Thread Chris Wilson
Quoting Chris Wilson (2019-03-11 09:45:17) > Quoting Tvrtko Ursulin (2019-03-11 09:23:44) > > > > On 08/03/2019 16:47, Chris Wilson wrote: > > > Quoting Tvrtko Ursulin (2019-03-08 16:27:22) > > >> > > >> On 08/03/2019 14:12, Chris Wilson wrote: > > >>> +static int > > >>> +set_engines(struct i915_

Re: [Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-11 Thread Tvrtko Ursulin
On 11/03/2019 09:45, Chris Wilson wrote: Quoting Tvrtko Ursulin (2019-03-11 09:23:44) On 08/03/2019 16:47, Chris Wilson wrote: Quoting Tvrtko Ursulin (2019-03-08 16:27:22) On 08/03/2019 14:12, Chris Wilson wrote: +static int +set_engines(struct i915_gem_context *ctx, + const struct

Re: [Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-11 Thread Chris Wilson
Quoting Tvrtko Ursulin (2019-03-11 09:23:44) > > On 08/03/2019 16:47, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2019-03-08 16:27:22) > >> > >> On 08/03/2019 14:12, Chris Wilson wrote: > >>> +static int > >>> +set_engines(struct i915_gem_context *ctx, > >>> + const struct drm_i915_gem

Re: [Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-11 Thread Tvrtko Ursulin
On 08/03/2019 16:47, Chris Wilson wrote: Quoting Tvrtko Ursulin (2019-03-08 16:27:22) On 08/03/2019 14:12, Chris Wilson wrote: Over the last few years, we have debated how to extend the user API to support an increase in the number of engines, that may be sparse and even be heterogeneous with

Re: [Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-08 Thread Chris Wilson
Quoting Tvrtko Ursulin (2019-03-08 16:27:22) > > On 08/03/2019 14:12, Chris Wilson wrote: > > Over the last few years, we have debated how to extend the user API to > > support an increase in the number of engines, that may be sparse and > > even be heterogeneous within a class (not all video deco

Re: [Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-08 Thread Tvrtko Ursulin
On 08/03/2019 14:12, Chris Wilson wrote: Over the last few years, we have debated how to extend the user API to support an increase in the number of engines, that may be sparse and even be heterogeneous within a class (not all video decoders created equal). We settled on using (class, instance)

[Intel-gfx] [PATCH 08/13] drm/i915: Allow a context to define its set of engines

2019-03-08 Thread Chris Wilson
Over the last few years, we have debated how to extend the user API to support an increase in the number of engines, that may be sparse and even be heterogeneous within a class (not all video decoders created equal). We settled on using (class, instance) tuples to identify a specific engine, with a