[igt-dev] [PATCH v16 5/8] tests: gem_exec_basic: add engine discovery test

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Fri Mar 29 12:41:48 UTC 2019


On 29/03/2019 12:06, Andi Shyti wrote:
>>> +	igt_subtest("exec-physical-engines-ctx") {
>>> +		uint64_t ctx_id;
>>> +		struct intel_execution_engine2 *e2;
>>> +
>>> +		ctx_id = gem_context_create(fd);
>>
>> In this one you don't want a context. It creates a mismatch between execbuf
>> sent by gem_context_has_engine and a context which hasn't been configured to
>> use engine index addressing.
> 
> Yes, I forgot to remove it... BTW, is this test needed at all? I
> added it just as a proposal.

In my opinion you can drop this patch since you use both new iterators 
later in the series.

Regards,

Tvrtko


More information about the igt-dev mailing list