Quoting Chris Wilson (2019-10-11 11:15:58) > Quoting Tvrtko Ursulin (2019-10-11 10:47:26) > > > + if (unlikely(i915_gem_context_is_banned(ce->gem_context))) > > > + cancel_active(rq, engine); > > > > Or you are counting this is already the last runnable request from this > > context due coalescing? It wouldn't work if for any reason coalescing > > would be prevented. Either with GVT, or I had some ideas to prevent > > coalescing for contexts where watchdog is enabled in the future. In > > which case this would be a hidden gotcha. Maybe all that's needed in > > mark_complete is also to look towards the end of the list? > > I'm not following. We are looking at the context here, which is track by > the last request submitted for that context. Oh I see, you were pointing out that I had not walked back along the context to find the incomplete request for correct patching. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
