The function is always called with deleted BOs.

While at it cleanup the indentation as well.

Signed-off-by: Christian König <christian.koe...@amd.com>
---
 drivers/gpu/drm/ttm/ttm_bo.c | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
index 1fbc36f05d89..c1104c8857b7 100644
--- a/drivers/gpu/drm/ttm/ttm_bo.c
+++ b/drivers/gpu/drm/ttm/ttm_bo.c
@@ -522,14 +522,9 @@ static int ttm_bo_cleanup_refs(struct ttm_buffer_object 
*bo,
                               bool interruptible, bool no_wait_gpu,
                               bool unlock_resv)
 {
-       struct dma_resv *resv;
+       struct dma_resv *resv = &bo->base._resv;
        int ret;
 
-       if (unlikely(list_empty(&bo->ddestroy)))
-               resv = bo->base.resv;
-       else
-               resv = &bo->base._resv;
-
        if (dma_resv_test_signaled_rcu(resv, true))
                ret = 0;
        else
@@ -542,9 +537,8 @@ static int ttm_bo_cleanup_refs(struct ttm_buffer_object *bo,
                        dma_resv_unlock(bo->base.resv);
                spin_unlock(&ttm_bo_glob.lru_lock);
 
-               lret = dma_resv_wait_timeout_rcu(resv, true,
-                                                          interruptible,
-                                                          30 * HZ);
+               lret = dma_resv_wait_timeout_rcu(resv, true, interruptible,
+                                                30 * HZ);
 
                if (lret < 0)
                        return lret;
-- 
2.17.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to