[igt-dev] [PATCH i-g-t v2 5/5] test/prime_vgem: use the gem_engine_topology library

[Tvrtko Ursulin]

On 06/06/2019 15:17, Andi Shyti wrote:
> Hi Ram,
> 
> just a nitpick
> 
>> -	execbuf.flags = ring | flags;
>> +	execbuf.flags = e ? e->flags : I915_EXEC_DEFAULT;
> 
> Because we are moving to the new engine naming, the
> "I915_EXEC_DEFAULT" engine, conceptually, should not be used.
> 
> Politically, with Tvrtko, we agreed that he default engine now
> is "I915_ENGINE_CLASS_RENDER", it's not the best as solution, but
> works.

I totally don't remember I ever suggested to use class or even it was 
discussed.  :) Didn't we say to just use zero?

Regards,

Tvrtko

> The rest looks good.
> 
> Reviewed-by: Andi Shyti <andi.shyti at intel.com>
> 
> Thanks,
> Andi
>