Only acquire the various atomic references required to unbind the vma if we do need to unbind the vma.
Signed-off-by: Chris Wilson chris@chris-wilson.co.uk Acked-by: Imre Deak imre.deak@intel.com Link: https://patchwork.freedesktop.org/patch/msgid/20191222210256.2066451-1-chris... (cherry picked from commit f5af1659d809e264d619e5f483fd8f47bced3b6a) --- drivers/gpu/drm/i915/i915_gem.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index f7c52b437f6a..998b67e3466e 100644 --- a/drivers/gpu/drm/i915/i915_gem.c +++ b/drivers/gpu/drm/i915/i915_gem.c @@ -130,6 +130,10 @@ int i915_gem_object_unbind(struct drm_i915_gem_object *obj, struct i915_address_space *vm = vma->vm; bool awake = false;
+ list_move_tail(&vma->obj_link, &still_in_list); + if (!i915_vma_is_bound(vma, I915_VMA_BIND_MASK)) + continue; + ret = -EAGAIN; if (!i915_vm_tryopen(vm)) break; @@ -144,7 +148,6 @@ int i915_gem_object_unbind(struct drm_i915_gem_object *obj, } }
- list_move_tail(&vma->obj_link, &still_in_list); spin_unlock(&obj->vma.lock);
ret = -EBUSY;