diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2016-09-09 09:11:50 -0400 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2016-09-09 09:23:03 -0400 |
commit | 22dd3bb9190566c7c3b80edb7dea67d1e21d0f91 (patch) | |
tree | 01898a53077a19a7c4608716a7c638924bc0992b /drivers/gpu/drm/i915/i915_gem_gtt.c | |
parent | ea746f3659232b3104d9534d5a7ebd9934ae1dd6 (diff) |
drm/i915: Mark up all locked waiters
In the next patch we want to handle reset directly by a locked waiter in
order to avoid issues with returning before the reset is handled. To
handle the reset, we must first know whether we hold the struct_mutex.
If we do not hold the struct_mtuex we can not perform the reset, but we do
not block the reset worker either (and so we can just continue to wait for
request completion) - otherwise we must relinquish the mutex.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20160909131201.16673-10-chris@chris-wilson.co.uk
Diffstat (limited to 'drivers/gpu/drm/i915/i915_gem_gtt.c')
-rw-r--r-- | drivers/gpu/drm/i915/i915_gem_gtt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c index 9bcac52b8268..f3c6876da521 100644 --- a/drivers/gpu/drm/i915/i915_gem_gtt.c +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c | |||
@@ -2683,7 +2683,7 @@ void i915_gem_gtt_finish_object(struct drm_i915_gem_object *obj) | |||
2683 | struct i915_ggtt *ggtt = &dev_priv->ggtt; | 2683 | struct i915_ggtt *ggtt = &dev_priv->ggtt; |
2684 | 2684 | ||
2685 | if (unlikely(ggtt->do_idle_maps)) { | 2685 | if (unlikely(ggtt->do_idle_maps)) { |
2686 | if (i915_gem_wait_for_idle(dev_priv, 0)) { | 2686 | if (i915_gem_wait_for_idle(dev_priv, I915_WAIT_LOCKED)) { |
2687 | DRM_ERROR("Failed to wait for idle; VT'd may hang.\n"); | 2687 | DRM_ERROR("Failed to wait for idle; VT'd may hang.\n"); |
2688 | /* Wait a bit, in hopes it avoids the hang */ | 2688 | /* Wait a bit, in hopes it avoids the hang */ |
2689 | udelay(10); | 2689 | udelay(10); |