<p>Theo and I preferred this approach as it allows each test to determine what a reasonable timeout should be, using dynamic information if desired.</p>
<p>Also, we understood that getting signal-safety right in the Python framework was hard, so we figured this was less likely to break.</p>
<p>That said, we'd be happy to have these merged without timeout support if that's what it takes. :-)</p>
<p>Thanks for reviewing these!<br>
Jamey</p>
<div class="gmail_quote">On May 8, 2014 10:49 AM, "Eric Anholt" <<a href="mailto:eric@anholt.net">eric@anholt.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Jamey Sharp <<a href="mailto:jamey@minilop.net">jamey@minilop.net</a>> writes:<br>
<br>
> From: TheoH <<a href="mailto:Theo0x48@gmail.com">Theo0x48@gmail.com</a>><br>
><br>
> We've observed hangs on some drivers in these calls, which make it<br>
> harder to run the rest of the Piglit test suite.<br>
<br>
I know people have been working on having timeout support in the<br>
framework, which seems much better than having it ad-hoc in tests that<br>
we've discovered already hang on some systems.  Any framework folks have<br>
status on that?<br>
</blockquote></div>