[PATCH 09/26] drm/amdgpu: use new iterator in amdgpu_ttm_bo_eviction_valuable

[Christian König]

Simplifying the code a bit.

Signed-off-by: Christian König <christian.koe...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 14 ++++----------
 1 file changed, 4 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
index 1129e17e9f09..b3859c8ded85 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
@@ -1332,10 +1332,9 @@ static bool amdgpu_ttm_bo_eviction_valuable(struct 
ttm_buffer_object *bo,
                                            const struct ttm_place *place)
 {
        unsigned long num_pages = bo->resource->num_pages;
+       struct dma_resv_iter resv_cursor;
        struct amdgpu_res_cursor cursor;
-       struct dma_resv_list *flist;
        struct dma_fence *f;
-       int i;
 
        /* Swapout? */
        if (bo->resource->mem_type == TTM_PL_SYSTEM)
@@ -1349,14 +1348,9 @@ static bool amdgpu_ttm_bo_eviction_valuable(struct 
ttm_buffer_object *bo,
         * If true, then return false as any KFD process needs all its BOs to
         * be resident to run successfully
         */
-       flist = dma_resv_shared_list(bo->base.resv);
-       if (flist) {
-               for (i = 0; i < flist->shared_count; ++i) {
-                       f = rcu_dereference_protected(flist->shared[i],
-                               dma_resv_held(bo->base.resv));
-                       if (amdkfd_fence_check_mm(f, current->mm))
-                               return false;
-               }
+       dma_resv_for_each_fence(bo->base.resv, &resv_cursor, true, f) {
+               if (amdkfd_fence_check_mm(f, current->mm))
+                       return false;
        }
 
        switch (bo->resource->mem_type) {
-- 
2.25.1