[igt-dev] [PATCH v21 2/6] lib/i915: add gem_engine_topology library and for_each loop definition

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Tue Apr 16 17:09:20 UTC 2019


On 16/04/2019 16:28, Andi Shyti wrote:
>>> +void gem_unset_context_engines(int fd, uint32_t ctx)
>>> +{
>>> +       gem_context_destroy(fd, ctx);
>>> +}
>>
>> That's a little surprising. I certainly would not expect my context to
>> be destroyed just to reset the engines back to default.
>>
>> On the Rusty scale that would be
>> 	-7. The obvious use is wrong.
> 
> if I have a context creator 'gem_make_context_set_all_engines()'
> (which you already didn't like at the previous version), I should
> have a context destroyer.
> 
> I understand your concern, I will remove the context creation and
> destruction from gem_engine_topology.

Yes I only asked for a helper to create a default/full engine map on a 
context. :) (So one call site does not have to abuse the 
intel_init_engines or so.)

Regards,

Tvrtko


More information about the igt-dev mailing list