Re: [Intel-gfx] [PATCH v2 2/6] drm/i915: Move legacy kernel context pinning to intel_ringbuffer.c

2016-05-26 Thread Chris Wilson
On Thu, May 26, 2016 at 01:22:04PM +0300, Joonas Lahtinen wrote: > > @@ -2327,6 +2350,14 @@ void intel_cleanup_engine(struct intel_engine_cs > > *engine) > >   > >   i915_cmd_parser_fini_ring(engine); > >   i915_gem_batch_pool_fini(>batch_pool); > > + > > + kctx = dev_priv->kernel_context;

Re: [Intel-gfx] [PATCH v2 2/6] drm/i915: Move legacy kernel context pinning to intel_ringbuffer.c

2016-05-26 Thread Joonas Lahtinen
On ke, 2016-05-25 at 15:04 +0100, Chris Wilson wrote: > This is so that we have symmetry with intel_lrc.c and avoid a source of > if (i915.enable_execlists) layering violation within i915_gem_context.c - > that is we move the specific handling of the dev_priv->kernel_context > for legacy

[Intel-gfx] [PATCH v2 2/6] drm/i915: Move legacy kernel context pinning to intel_ringbuffer.c

2016-05-25 Thread Chris Wilson
This is so that we have symmetry with intel_lrc.c and avoid a source of if (i915.enable_execlists) layering violation within i915_gem_context.c - that is we move the specific handling of the dev_priv->kernel_context for legacy submission into the legacy submission code. This depends upon the