On Wed, Aug 17, 2016 at 10:57:34AM +0100, Tvrtko Ursulin wrote: > > On 17/08/16 10:41, Chris Wilson wrote: > >On Wed, Aug 17, 2016 at 10:34:18AM +0100, Tvrtko Ursulin wrote: > >>Or add an initialized engine array to dev_priv, in addition to the > >>existing map for best of both worlds. > > > >We have the ring_mask which already tells us that mapping, so I think > >the second array is overkill. > > Yes, I said "in addition to the existing map". In addition we could > have an array of only initialized engines to avoid any skipping at > runtime. Since iterators end with intel_engine_initialized anyway. And I'm saying we already have that information in ring_mask. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx