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@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
