[igt-dev] [PATCH i-g-t] lib/igt_pm: Adjust pm status check timeout and log timing

Hiatt, Don don.hiatt at intel.com
Tue Mar 10 17:11:21 UTC 2020


> Quoting don.hiatt at intel.com (2020-03-10 17:09:28)
> > From: Don Hiatt <don.hiatt at intel.com>
> >
> > Increase the overall time we wait to achieve the desired runtime
> > status to 15 seconds to fix an occasional timeout and do so
> > in 1 second increments to speed up exiting. Also log the
> > amount of time we waited to get a handle on how much variation
> > we run into.
> 
> Ahem. But it is still an error, whereas this is now hiding severe
> failures.
> -Chris

Hi Chris,

I was hoping to get a handle on if the timeout was just borderline
around the 10 seconds and then try to figure out how to catch
just why the timeout even gets that high in the first place. Local
testing with the logging in this patch shows the vast majority
of the runs are 1 second timeouts, with an odd 2 or 3.

I can remove the 15 second increment and just keep the logging,
or just abandon the patch and work on a different approach?

Thanks for your comments!

don



More information about the igt-dev mailing list