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

[Andi Shyti]

> > > > +       igt_subtest("exec-engines-ctx") {
> > > > +               uint64_t ctx_id;
> > > > +               struct intel_execution_engine2 *e2;
> > > > +
> > > > +               ctx_id = gem_context_create(fd);
> > > > +
> > > > +               for_each_context_engine(fd, ctx_id, e2)
> > > > +                       igt_assert(gem_context_has_engine(fd, ctx_id,
> > > > +                                                         e2->flags));
> > > > +
> > > > +               gem_context_destroy(fd, ctx_id);
> > > > +       }
> > > 
> > > Starting subtest: exec-engines-ctx
> > > (gem_exec_basic:17617) CRITICAL: Test assertion failure function __real_main117, file ../tests/i915/gem_exec_basic.c:146:
> > > (gem_exec_basic:17617) CRITICAL: Failed assertion: gem_context_has_engine(fd, ctx_id, e2->flags)
> > > Stack trace:
> > >   #0 ../lib/igt_core.c:1474 __igt_fail_assert()
> > >   #1 ../tests/i915/gem_exec_basic.c:144 __real_main117()
> > >   #2 ../tests/i915/gem_exec_basic.c:117 main()
> > >   #3 ../csu/libc-start.c:342 __libc_start_main()
> > >   #4 [_start+0x2a]
> > > Subtest exec-engines-ctx failed.
> > > **** DEBUG ****
> > > (gem_exec_basic:17617) CRITICAL: Test assertion failure function __real_main117, file ../tests/i915/gem_exec_basic.c:146:
> > > (gem_exec_basic:17617) CRITICAL: Failed assertion: gem_context_has_engine(fd, ctx_id, e2->flags)
> > > (gem_exec_basic:17617) igt_core-INFO: Stack trace:
> > > (gem_exec_basic:17617) igt_core-INFO:   #0 ../lib/igt_core.c:1474 __igt_fail_assert()
> > > (gem_exec_basic:17617) igt_core-INFO:   #1 ../tests/i915/gem_exec_basic.c:144 __real_main117()
> > > (gem_exec_basic:17617) igt_core-INFO:   #2 ../tests/i915/gem_exec_basic.c:117 main()
> > > (gem_exec_basic:17617) igt_core-INFO:   #3 ../csu/libc-start.c:342 __libc_start_main()
> > > (gem_exec_basic:17617) igt_core-INFO:   #4 [_start+0x2a]
> > > ****  END  ****
> > > 
> > > I'd say that is worth a little polishing on what the error might have
> > > been.
> > 
> > mmmhhh... how did you get it? I tested it on drm-tip kernel
> > (different versions), mainline kernel (different versions) and
> > Tvrtko's kernel and didn't get any failure.
> 
> I'm testing on the patches posted for upstream. The point is not that it
> failed, but that the failure message doesn't give a clue: e2->flags and
> e2->name would definitely be helpful. igt_debug of the query results may
> help, but be judicious and avoid spammy debug.

Ah... OK, I see your point, you're right. Indeed, I could
understand nothing from the log :)

Thanks,
Andi