<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>Ok. If this occurs again, please update the threshold.&nbsp;<br></div><div><br>On May 17, 2017, at 12:03 PM, David P Grove &lt;<a href="mailto:groved@us.ibm.com">groved@us.ibm.com</a>&gt; wrote:<br><br></div><blockquote type="cite"><div><p><tt><font size="2"><a href="mailto:swift-dev-bounces@swift.org">swift-dev-bounces@swift.org</a> wrote on 05/17/2017 02:04:23 PM:</font></tt><br><tt><font size="2">&gt; <br>&gt; Failure in dispatch-starfish. This test looks timing-sensitive. <br>&gt; Boris, is this unlikely to be related to your changes? Does anyone <br>&gt; familiar with dispatch have suggestions, such as increasing the <br>&gt; latency threshold?</font></tt><br><tt><font size="2">&gt; <br>&gt; [BEGIN] Latency</font></tt><br><tt><font size="2">&gt; Actual: 1553</font></tt><br><tt><font size="2">&gt; Expected: &lt;1000</font></tt><br><tt><font size="2">&gt; [FAIL] Latency (dispatch_starfish.c:82)</font></tt><br><tt><font size="2">&gt; dispatch_starfish.c:82</font></tt><br><tt><font size="2">&gt; lap: 9</font></tt><br><tt><font size="2">&gt; count: 1000</font></tt><br><tt><font size="2">&gt; delta: 6756400000 ns</font></tt><br><tt><font size="2">&gt; math: 3374.825175 ns / lap</font></tt><br><tt><font size="2">&gt; <br>&gt; [BEGIN] Latency</font></tt><br><tt><font size="2">&gt; Actual: 3374</font></tt><br><tt><font size="2">&gt; Expected: &lt;1000</font></tt><br><tt><font size="2">&gt; [FAIL] Latency (dispatch_starfish.c:82)</font></tt><br><tt><font size="2">&gt; dispatch_starfish.c:82</font></tt><br><tt><font size="2">&gt; <br></font></tt><br><tt><font size="2">The test is subject to failing spuriously if the machine is badly overloaded. &nbsp;One could increase the threshold by changing the #define on line 43 of dispatch_starfish.c</font></tt><br><br><tt><font size="2">There's a trade-off between spurious failures due to overloads on build machines vs. missing performance regressions by making the latency thresholds much too lenient. &nbsp;</font></tt><br><br><tt><font size="2">We've been running with the current threshold in dispatch_starfish since we began running the dispatch tests in the CI system last summer. &nbsp;So if this isn't a one-off failure caused by CI load, it might be pointing to an underlying performance regression.</font></tt><br><br><tt><font size="2">--dave</font></tt><br><br>

</p></div></blockquote></body></html>