[igt-dev] [PATCH v13 7/9] lib/igt_gt: use for_each_engine_class_instance to loop through active engines
Chris Wilson
chris at chris-wilson.co.uk
Wed Mar 20 10:40:34 UTC 2019
Quoting Tvrtko Ursulin (2019-03-20 10:33:43)
>
> On 20/03/2019 10:09, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2019-03-20 10:04:50)
> >>
> >> On 19/03/2019 23:44, Andi Shyti wrote:
> >> If you do this, and also add vcs2 to the static table in this series,
> >
> > Can we give the static table every class with up to MAX_INSTANCE^2?
> > Shotgun approach.
>
> Problem is test enumeration is driven of it and we are not allowed to
> access the device in that phase. And I don't think we want to enumerate
> a gazillion of subtests just to make the static table future proof.
Having a few hundred unreported tests should not be an issue. And should
be a motivator for someone else to fix their problem if it was. And CI
folk, it is a problem in the static partition design.
> So I was thinking Andi would be adding an assert in the code which
> borrows the name from the static table so we notice it needs to be
> expanded for new engines.
And then we are always behind and having to keep internal forks, instead
of futureproofing as much as we can get away with.
-Chris
More information about the igt-dev
mailing list