Quoting Mika Kuoppala (2019-04-10 15:40:13) > Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > > /* Rerun the request; its payload has been neutered (if guilty). */ > > - rq->ring->head = intel_ring_wrap(rq->ring, rq->head); > > - intel_ring_update_space(rq->ring); > > +out_replay: > > + ce->ring->head = > > + rq ? intel_ring_wrap(ce->ring, rq->head) : ce->ring->tail; > > The ce and rq ring should be same with the rq set. I guess > you had a reasons to keep it as ce, perhaps because it is > the culprit. Yes, by this point we know that rq->hw_context == ce, and so rq->ring == ce->ring. I decided that execlists_reset() was now all about the active context, and the request just a hint as how far along that context we had completed -- hence trying to using ce as the primary throughout. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
