Quoting Tvrtko Ursulin (2021-02-04 11:19:00)
>
> On 01/02/2021 08:56, Chris Wilson wrote:
> > bool intel_engine_is_idle(struct intel_engine_cs *engine)
> > {
> > + struct i915_sched *se = intel_engine_get_scheduler(engine);
>
> What do you have 'se' stand for?
Originally i915_sched_engine, then working backwards there was just a
single i915_sched struct (but may have a few working together). So it's
CPU runlist with a bit more speciality logic.
This is akin to drm_sched_entity, se could work for that. Other ideas
for the name would be i915_sched_queue.
Anyway se stuck around from the first pass. It was easy to type!
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
