[PATCH 05/26] dma-buf: use new iterator in dma_resv_wait_timeout

[Christian König]

This makes the function much simpler since the complex
retry logic is now handled elsewhere.

Signed-off-by: Christian König <christian.koe...@amd.com>
---
 drivers/dma-buf/dma-resv.c | 64 +++++---------------------------------
 1 file changed, 7 insertions(+), 57 deletions(-)

diff --git a/drivers/dma-buf/dma-resv.c b/drivers/dma-buf/dma-resv.c
index 0a0e3ee62648..3ce929b770ea 100644
--- a/drivers/dma-buf/dma-resv.c
+++ b/drivers/dma-buf/dma-resv.c
@@ -571,74 +571,24 @@ long dma_resv_wait_timeout(struct dma_resv *obj, bool 
wait_all, bool intr,
                           unsigned long timeout)
 {
        long ret = timeout ? timeout : 1;
-       unsigned int seq, shared_count;
+       struct dma_resv_cursor cursor;
        struct dma_fence *fence;
-       int i;
 
-retry:
-       shared_count = 0;
-       seq = read_seqcount_begin(&obj->seq);
        rcu_read_lock();
-       i = -1;
-
-       fence = dma_resv_excl_fence(obj);
-       if (fence && !test_bit(DMA_FENCE_FLAG_SIGNALED_BIT, &fence->flags)) {
-               if (!dma_fence_get_rcu(fence))
-                       goto unlock_retry;
+       dma_resv_for_each_fence_unlocked(obj, &cursor, wait_all, fence) {
+               rcu_read_unlock();
 
-               if (dma_fence_is_signaled(fence)) {
+               ret = dma_fence_wait_timeout(fence, intr, ret);
+               if (ret <= 0) {
                        dma_fence_put(fence);
-                       fence = NULL;
+                       return ret;
                }
 
-       } else {
-               fence = NULL;
-       }
-
-       if (wait_all) {
-               struct dma_resv_list *fobj = dma_resv_shared_list(obj);
-
-               if (fobj)
-                       shared_count = fobj->shared_count;
-
-               for (i = 0; !fence && i < shared_count; ++i) {
-                       struct dma_fence *lfence;
-
-                       lfence = rcu_dereference(fobj->shared[i]);
-                       if (test_bit(DMA_FENCE_FLAG_SIGNALED_BIT,
-                                    &lfence->flags))
-                               continue;
-
-                       if (!dma_fence_get_rcu(lfence))
-                               goto unlock_retry;
-
-                       if (dma_fence_is_signaled(lfence)) {
-                               dma_fence_put(lfence);
-                               continue;
-                       }
-
-                       fence = lfence;
-                       break;
-               }
+               rcu_read_lock();
        }
-
        rcu_read_unlock();
-       if (fence) {
-               if (read_seqcount_retry(&obj->seq, seq)) {
-                       dma_fence_put(fence);
-                       goto retry;
-               }
 
-               ret = dma_fence_wait_timeout(fence, intr, ret);
-               dma_fence_put(fence);
-               if (ret > 0 && wait_all && (i + 1 < shared_count))
-                       goto retry;
-       }
        return ret;
-
-unlock_retry:
-       rcu_read_unlock();
-       goto retry;
 }
 EXPORT_SYMBOL_GPL(dma_resv_wait_timeout);
 
-- 
2.25.1