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

[Chris Wilson]

Quoting Tvrtko Ursulin (2019-11-20 09:13:48)
> 
> On 19/11/2019 11:36, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2019-11-19 11:22:30)
> >>
> >> On 19/11/2019 11:11, Chris Wilson wrote:
> >>> Quoting Venkata Sandeep Dhanalakota (2019-11-19 11:01:17)
> >>>> Since engines are now looked up based on context engine map,
> >>>> context should be setup with engines before using in execbuf.
> >>>
> >>> Stop changing fundamental tests of legacy behaviour. If you want to
> >>> exercise new API, do so in parallel.
> >>
> >> The test is already using the new api so as it stands there's a mismatch
> >> on engine flags and context - since it passes the engine flags to the
> >> thread, derived from a context with engine map, but the thread uses a
> >> legacy context. It has to be aligned for the intended engine to get
> >> exercised.
> > 
> > Then consider it a retrospective complaint for mucking up a basic test :-p
> 
> He who works makes mistakes, or something like that. :)
> 
> Ack then to merge this one?
> 
> And engine dropping flavour of gem_ctx_exec/invalid-context to be sent 
> later.

Ack.
-Chris