Re: [Intel-gfx] [PATCH 07/12] drm/i915: Move pinning of dev_priv->kernel_context into its creator

2016-05-23 Thread Chris Wilson
On Mon, May 23, 2016 at 10:33:24AM +0100, Tvrtko Ursulin wrote: > >@@ -313,30 +311,14 @@ i915_gem_create_context(struct drm_device *dev, > > if (IS_ERR(ctx)) > > return ctx; > > > >-if (is_global_default_ctx && ctx->legacy_hw_ctx.rcs_state) { > >-/* We may need to

Re: [Intel-gfx] [PATCH 07/12] drm/i915: Move pinning of dev_priv->kernel_context into its creator

2016-05-23 Thread Tvrtko Ursulin
On 22/05/16 14:02, Chris Wilson wrote: Rather than have every context ask "am I owned by the kernel? pin!", move that logic into the creator of the kernel context, in order to improve code comprehension. Makes sense. Signed-off-by: Chris Wilson Cc: Tvrtko Ursulin

[Intel-gfx] [PATCH 07/12] drm/i915: Move pinning of dev_priv->kernel_context into its creator

2016-05-22 Thread Chris Wilson
Rather than have every context ask "am I owned by the kernel? pin!", move that logic into the creator of the kernel context, in order to improve code comprehension. Signed-off-by: Chris Wilson Cc: Tvrtko Ursulin Cc: Joonas Lahtinen