Am 01.06.22 um 14:09 schrieb Mohan Marimuthu, Yogesh:
[SNIP]
-       /* Make sure all BOs are remembered as writers */
-       amdgpu_bo_list_for_each_entry(e, p->bo_list)
+       list_for_each_entry(e, &p->validated, tv.head) {
+
+               /* Everybody except for the gang leader uses BOOKKEEP */
+               for (i = 0; i < (p->gang_size - 1); ++i) {
+                       dma_resv_add_fence(e->tv.bo->base.resv,
+                                          &p->jobs[i]->base.s_fence->finished,
+                                          DMA_RESV_USAGE_BOOKKEEP);
+               }
+
+               /* The gang leader as remembered as writer */
                e->tv.num_shared = 0;
+       }


p->jobs[i] = NULL is done in previous loop. Now when running 
&p->jobs[i]->base.s_fence->finished there is NULL pointer crash.

Ah, yes good point. Going to fix that.

Any more comments on this? Did you finished the test cases?

Thanks,
Christian.

Reply via email to