[Intel-gfx] [PATCH i-g-t 4/7] lib/igt_chamelium: Add support for configurable DUT suspend/resume delay
Lyude Paul
lyude at redhat.com
Mon Jun 26 21:14:07 UTC 2017
> >
> > Fair enough, but at this point, only lib/igt_chamelium.c is using
> > igtrc and all
> > the parsing is implemented there.
> >
> > So one course of action here would be to move igtrc parsing outside
> > of
> > igt_chamelium (maybe to igt_core?) and use some global variable for
> > storing this
> > value. What do you think?
> >
>
> That seems like a plan, except you can just call a function that will
> do
> the parsing for you and return a structure with all the parameters.
> If
> it is called a second time, just return the cached value.
>
> What do you think, Lyude?
Yep, the reason it's called .igtrc is mainly because we were eventually
expecting someone else to start using it, and suspend/resume delay is
definitely something that will be useful for the rest of the tests
(I've got quite a few machines myself that i had to increase the delay
on just because the firmware was too slow when suspending/resuming). So
I'm all for it.
More information about the Intel-gfx
mailing list