Quoting Christian König (2019-08-21 13:31:45)

@@ -528,20 +352,9 @@ void dma_resv_prune_fences(struct dma_resv *obj) dma_fence_put(fence); }

•   list = dma_resv_get_list(obj);
  

•   if (!list)
  

•           return;
  

•   for (i = 0; i < list->shared_count; ++i) {
  

•           fence = rcu_dereference_protected(list->shared[i],
  

•                                             dma_resv_held(obj));
  

•           if (!dma_fence_is_signaled(fence))
  

•                   continue;
  

•           RCU_INIT_POINTER(list->shared[i], dma_fence_get_stub());
  

•           dma_fence_put(fence);
  

•   }
  

•   fence = dma_resv_fences_deref(obj, &obj->readers);
  

•   if (dma_fence_is_signaled(fence))
  

•           dma_resv_fences_set(obj, &obj->readers, NULL);
  

Something to note is that a dma-fence-array is not automatically signaled and dma_fence_is_signaled() does not check the array. -Chris