Commit cb0aeaa8 authored by Chris Wilson's avatar Chris Wilson

drm/i915: Only free the oldest stale context before allocating

Currently, we move all unreferenced contexts to an RCU free list and
then onto a worker for eventual reaping. To compensate against this
growing into a long list with frequent allocations starving the system
of available memory, before we allocate a new context we reap all the
stale contexts. This puts all the cost of destroying the context into
the next allocator, which is presumably more sensitive to syscall
latency and unfair. We can limit the number of contexts being freed by
the new allocator to both keep the list trimmed and to allow the
allocator to be reasonably fast.
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170705142634.18554-4-chris@chris-wilson.co.ukReviewed-by: default avatarTvrtko Ursulin <tvrtko.ursulin@intel.com>
parent 6b6573d1
...@@ -201,6 +201,21 @@ static void contexts_free(struct drm_i915_private *i915) ...@@ -201,6 +201,21 @@ static void contexts_free(struct drm_i915_private *i915)
i915_gem_context_free(ctx); i915_gem_context_free(ctx);
} }
static void contexts_free_first(struct drm_i915_private *i915)
{
struct i915_gem_context *ctx;
struct llist_node *freed;
lockdep_assert_held(&i915->drm.struct_mutex);
freed = llist_del_first(&i915->contexts.free_list);
if (!freed)
return;
ctx = container_of(freed, typeof(*ctx), free_link);
i915_gem_context_free(ctx);
}
static void contexts_free_worker(struct work_struct *work) static void contexts_free_worker(struct work_struct *work)
{ {
struct drm_i915_private *i915 = struct drm_i915_private *i915 =
...@@ -383,8 +398,8 @@ i915_gem_create_context(struct drm_i915_private *dev_priv, ...@@ -383,8 +398,8 @@ i915_gem_create_context(struct drm_i915_private *dev_priv,
lockdep_assert_held(&dev_priv->drm.struct_mutex); lockdep_assert_held(&dev_priv->drm.struct_mutex);
/* Reap stale contexts */ /* Reap the most stale context */
contexts_free(dev_priv); contexts_free_first(dev_priv);
ctx = __create_hw_context(dev_priv, file_priv); ctx = __create_hw_context(dev_priv, file_priv);
if (IS_ERR(ctx)) if (IS_ERR(ctx))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment