diff options
| author | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2015-09-08 07:52:48 -0400 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-09-10 12:27:17 -0400 |
| commit | f0fea8dd362c77a06e7b8744d63f429fe065c6b4 (patch) | |
| tree | ac99dd4be569dbfdbbcb9c31ac64ced064724062 | |
| parent | f2a066f3dee8a0e37015d15298e14e9ad6c0a53d (diff) | |
drm/i915: Remove one very outdated comment
Comment disagrees with the code which has changed a lot since
it was documented.
Note that the logic to remove -EIO handling was dropped in
commit 1488fc08c1706288616c602416654fd38c773deb
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date: Tue Apr 24 15:47:31 2012 +0100
drm/i915: Remove the deferred-free list
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
| -rw-r--r-- | drivers/gpu/drm/i915/i915_gem.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index 41263cd4170c..760cfba22e71 100644 --- a/drivers/gpu/drm/i915/i915_gem.c +++ b/drivers/gpu/drm/i915/i915_gem.c | |||
| @@ -3230,10 +3230,6 @@ int i915_vma_unbind(struct i915_vma *vma) | |||
| 3230 | ret = i915_gem_object_wait_rendering(obj, false); | 3230 | ret = i915_gem_object_wait_rendering(obj, false); |
| 3231 | if (ret) | 3231 | if (ret) |
| 3232 | return ret; | 3232 | return ret; |
| 3233 | /* Continue on if we fail due to EIO, the GPU is hung so we | ||
| 3234 | * should be safe and we need to cleanup or else we might | ||
| 3235 | * cause memory corruption through use-after-free. | ||
| 3236 | */ | ||
| 3237 | 3233 | ||
| 3238 | if (i915_is_ggtt(vma->vm) && | 3234 | if (i915_is_ggtt(vma->vm) && |
| 3239 | vma->ggtt_view.type == I915_GGTT_VIEW_NORMAL) { | 3235 | vma->ggtt_view.type == I915_GGTT_VIEW_NORMAL) { |