On Thu, Apr 11, 2019 at 01:32:03PM +0100, Chris Wilson wrote: > Quoting Tvrtko Ursulin (2019-04-11 13:26:54) > > From: Andi Shyti <andi.shyti@xxxxxxxxx> > > > > Replace the legacy for_each_engine* defines with the ones > > implemented in the gem_engine_topology library. > > > > Use whenever possible gem_engine_can_store_dword() that checks > > class instead of flags. > > > > Now the __for_each_engine_class_instance and > > for_each_engine_class_instance are unused, remove them. > > > > Suggested-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > Signed-off-by: Andi Shyti <andi.shyti@xxxxxxxxx> > > Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx> > > Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > I see a lot of new gem_context_create(), but not gem_require_contexts(). gem_require_contexts() should go on top of engine_topology. I will add it there. Thanks, Andi > -Chris > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
