Re: [igt-dev] [PATCH i-g-t 2/2] test/i915: gem_exec_parallel: Fix for using context engine map

[Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>]

On 19/11/2019 11:01, Venkata Sandeep Dhanalakota wrote:
> Since engines are now looked up based on context engine map,
> context should be setup with engines before using in execbuf.
> 
> Fixes: a9998c23 igt("gem_exec_parallel: use the gem_engine_topology")
> 
> Cc: Andi Shyti <andi.shyti@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
> Signed-off-by: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
> ---
>   tests/i915/gem_exec_parallel.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/i915/gem_exec_parallel.c b/tests/i915/gem_exec_parallel.c
> index 8a4482e5..56b26cf4 100644
> --- a/tests/i915/gem_exec_parallel.c
> +++ b/tests/i915/gem_exec_parallel.c
> @@ -126,8 +126,10 @@ static void *thread(void *data)
>   	execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
>   	if (t->gen < 6)
>   		execbuf.flags |= I915_EXEC_SECURE;
> -	if (t->flags & CONTEXTS)
> +	if (t->flags & CONTEXTS) {
>   		execbuf.rsvd1 = gem_context_create(fd);
> +		gem_context_set_all_engines(fd, execbuf.rsvd1);
> +	}
>   
>   	for (i = 0; i < 16; i++) {
>   		obj[0].handle = t->scratch[i];
> 

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

Regards,

Tvrtko
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev