drm/i915: Always pin contexts into the high GGTT
Now that we have fast top-down insertion into the drm_mm, we can use it for frequent runtime operations like insertion of the context object, whereas before we limited it to the one-off insertion of the pinned kernel context. Keeping the active context objects out of the mappable region of the global GTT (except under memory pressure) improves our ability to allocate mappable aperture region without triggering a GPU stall. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/20170210101422.1598-1-chris@chris-wilson.co.uk Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com>
This commit is contained in:
@@ -773,11 +773,9 @@ static int execlists_context_pin(struct intel_engine_cs *engine,
|
||||
}
|
||||
GEM_BUG_ON(!ce->state);
|
||||
|
||||
flags = PIN_GLOBAL;
|
||||
flags = PIN_GLOBAL | PIN_HIGH;
|
||||
if (ctx->ggtt_offset_bias)
|
||||
flags |= PIN_OFFSET_BIAS | ctx->ggtt_offset_bias;
|
||||
if (i915_gem_context_is_kernel(ctx))
|
||||
flags |= PIN_HIGH;
|
||||
|
||||
ret = i915_vma_pin(ce->state, 0, GEN8_LR_CONTEXT_ALIGN, flags);
|
||||
if (ret)
|
||||
|
Reference in New Issue
Block a user