@@ -48,21 +48,30 @@ int __intel_context_do_pin(struct intel_context *ce)
{
int err;
- if (mutex_lock_interruptible(&ce->pin_mutex))
- return -EINTR;
-
- if (likely(!atomic_read(&ce->pin_count))) {
- intel_wakeref_t wakeref;
+ if (unlikely(!test_bit(CONTEXT_ALLOC_BIT, &ce->flags))) {
+ err = mutex_lock_interruptible(&ce->alloc_mutex);
+ if (err)
+ return err;
- if (unlikely(!test_bit(CONTEXT_ALLOC_BIT, &ce->flags))) {
+ if (!test_bit(CONTEXT_ALLOC_BIT, &ce->flags)) {
err = ce->ops->alloc(ce);
if (unlikely(err))
goto err;
+ smp_mb(); /* Make sure allocation is flushed to memory */
+
__set_bit(CONTEXT_ALLOC_BIT, &ce->flags);
}
+ mutex_unlock(&ce->alloc_mutex);
+ }
+
+ err = mutex_lock_interruptible(&ce->pin_mutex);
+ if (err)
+ return err;
+
+ if (likely(!atomic_read(&ce->pin_count))) {
+ intel_wakeref_t wakeref;
- err = 0;
with_intel_runtime_pm(ce->engine->uncore->rpm, wakeref)
err = ce->ops->pin(ce);
if (err)
@@ -95,14 +104,13 @@ void intel_context_unpin(struct intel_context *ce)
/* We may be called from inside intel_context_pin() to evict another */
intel_context_get(ce);
- mutex_lock_nested(&ce->pin_mutex, SINGLE_DEPTH_NESTING);
+ mutex_lock(&ce->pin_mutex);
if (likely(atomic_dec_and_test(&ce->pin_count))) {
GEM_TRACE("%s context:%llx retire\n",
ce->engine->name, ce->timeline->fence_context);
ce->ops->unpin(ce);
-
i915_gem_context_put(ce->gem_context);
intel_context_active_release(ce);
}
@@ -246,6 +254,7 @@ intel_context_init(struct intel_context *ce,
INIT_LIST_HEAD(&ce->signals);
mutex_init(&ce->pin_mutex);
+ mutex_init(&ce->alloc_mutex);
i915_active_init(&ce->active,
__intel_context_active, __intel_context_retire);
@@ -257,6 +266,7 @@ void intel_context_fini(struct intel_context *ce)
intel_timeline_put(ce->timeline);
i915_vm_put(ce->vm);
+ mutex_destroy(&ce->alloc_mutex);
mutex_destroy(&ce->pin_mutex);
i915_active_fini(&ce->active);
}
@@ -65,6 +65,7 @@ struct intel_context {
atomic_t pin_count;
struct mutex pin_mutex; /* guards pinning and associated on-gpuing */
+ struct mutex alloc_mutex; /* guards against concurrent contex allocation */
/**
* active: Active tracker for the rq activity (inc. external) on this
Lets see what breaks? Round 2? Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> --- drivers/gpu/drm/i915/gt/intel_context.c | 28 +++++++++++++------ drivers/gpu/drm/i915/gt/intel_context_types.h | 1 + 2 files changed, 20 insertions(+), 9 deletions(-)