Quoting Tvrtko Ursulin (2020-01-14 11:13:06) > > > On 13/01/2020 10:44, Chris Wilson wrote: > > @@ -2473,6 +2475,7 @@ static void execlists_submit_request(struct > > i915_request *request) > > spin_lock_irqsave(&engine->active.lock, flags); > > > > queue_request(engine, &request->sched, rq_prio(request)); > > + set_bit(I915_FENCE_FLAG_PQUEUE, &request->fence.flags); > > Move into queue_request so it is closer to priolist management, just like at > other call sites?
Sure. When going through them, it felt natural to treat this as the list operation. (That's what I was thinking :) -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx