Re: [Intel-gfx] [PATCH 10/12] drm/i915: Merge legacy+execlists context structs

2016-05-23 Thread Chris Wilson
On Mon, May 23, 2016 at 11:26:13AM +0100, Tvrtko Ursulin wrote: > >@@ -400,7 +412,7 @@ int i915_gem_context_init(struct drm_device *dev) > > return PTR_ERR(ctx); > > } > > > >-if (ctx->legacy_hw_ctx.rcs_state) { > >+if (ctx->engine[RCS].state) { > > Maybe now put a comment

Re: [Intel-gfx] [PATCH 10/12] drm/i915: Merge legacy+execlists context structs

2016-05-23 Thread Tvrtko Ursulin
On 22/05/16 14:02, Chris Wilson wrote: struct intel_context contains two substructs, one for the legacy RCS and one for every execlists engine. Since legacy RCS is a subset of the execlists engine support, just combine the two substructs. Signed-off-by: Chris Wilson Cc: Tvrtko Ursulin Cc: Joo

[Intel-gfx] [PATCH 10/12] drm/i915: Merge legacy+execlists context structs

2016-05-22 Thread Chris Wilson
struct intel_context contains two substructs, one for the legacy RCS and one for every execlists engine. Since legacy RCS is a subset of the execlists engine support, just combine the two substructs. Signed-off-by: Chris Wilson Cc: Tvrtko Ursulin Cc: Joonas Lahtinen --- drivers/gpu/drm/i915/i9