Quoting Tvrtko Ursulin (2019-02-11 19:11:08) > > On 06/02/2019 13:03, Chris Wilson wrote: > > @@ -1361,6 +1362,7 @@ __execlists_context_pin(struct intel_engine_cs > > *engine, > > > > ce->state->obj->pin_global++; > > i915_gem_context_get(ctx); > > + list_add(&ce->active_link, &ctx->active_engines); > > Why is it called active_engines if it lists active_contexts? :)
active_context_engines doesn't quite have the same ring to it. It's the list of engines on which this GEM context has been pinned, which is my approximation for active and is the list I need to flush if we change logical state. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
