<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Mar 18, 2016, at 5:02 AM, Nils Eliasson <<a href="mailto:nils.eliasson@oracle.com" class="">nils.eliasson@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Hi,<br class=""><br class="">Please review this test fix.<br class=""><br class="">Summary:<br class="">This test tests the locking of the compilers - make sure no compiles can be completed while the lock is in place. When running with -XX:-TieredCompilation and -XX:CompileThreshold=100 we got a fairly long queue of compiles waiting, and when only C2 is available it can take longer to complete than the test wait time.<br class=""><br class="">Solution:<br class="">Only allow compile of the test method - make sure we have no contention on the compile queue. With only a single method in the queue we can also reduce the wait time.<br class=""><br class="">Testing:<br class="">Test run in failing configuration.<br class=""><br class="">Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8152169" class="">https://bugs.openjdk.java.net/browse/JDK-8152169</a><br class="">Webrev: <a href="http://cr.openjdk.java.net/~neliasso/8152169/webrev.01/" class="">http://cr.openjdk.java.net/~neliasso/8152169/webrev.01/</a><br class=""></div></div></blockquote><div><br class=""></div><pre style="background-color: rgb(238, 238, 238);" class=""><b class=""><font color="blue" class="">+</font><font color="black" class=""></font><font color="blue" class="">             // Sleep a while and then make sure the compile is still waiting</font><font color="black" class=""></font></b>
<font color="blue" class=""><b class="">+             try {</b></font>
<font color="blue" class=""><b class="">+                 Thread.sleep(5000);</b></font>
<font color="blue" class=""><b class="">+             } catch (Exception e) {</b></font>
<font color="blue" class=""><b class="">+                 System.out.println("Thread sleep interrupted");</b></font>
<font color="blue" class=""><b class="">+             }</b></font></pre>Should we propagate this exception instead of just issuing a warning on stdout?</div><div><br class=""><blockquote type="cite" class=""><div class=""><div class=""><br class="">Regards,<br class="">Nils<br class=""></div></div></blockquote></div><br class=""></body></html>