<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Derek,<br>
<br>
Thanks for fixing this.<br>
<br>
Fix looks good.<br>
<br>
What do you think about always making testDynamicNumberOfGCThread()<br>
check for the uniprocessor case (as opposed to passing in a flag to
explicitly<br>
check it)? <br>
<br>
ForceDynamicNumberOfGCThreads is a diagnostic flag<br>
<br>
diagnostic(bool, ForceDynamicNumberOfGCThreads,
false, \<br>
"Force dynamic selection of the number of
" \<br>
"parallel threads parallel gc will use to aid
debugging") \<br>
<br>
so I think you need +UnlockDiagnosticVMOptions.<br>
<br>
Jon<br>
<br>
<div class="moz-cite-prefix">On 04/21/2015 06:53 AM, Derek White
wrote:<br>
</div>
<blockquote cite="mid:5536563F.4020806@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<tt>Hi All,</tt><tt><br>
</tt><tt><br>
</tt><tt>Please review this fix for: <br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8076995">https://bugs.openjdk.java.net/browse/JDK-8076995</a></tt><br>
<pre wrap="">Webrev:
<a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Edrwhite/8076995/webrev.00/">http://cr.openjdk.java.net/~drwhite/8076995/webrev.00/</a>
Summary:
Part 1 is a test bug that tries to run G1 on embedded SE builds. Not changed by this webrev.
Part two is assertion failure that is being fixed by this webrev.
This is a fix for bug that triggered an assert when running CMS on very
small machines - 1 core x86, or 1-4 core ARM. This may seem unlikely but
can easily happen when running virtual instances.
Failure stack traces also show bug crashing printing a stack trace, but this is being tracked in another bug.
Thanks,
- Derek
</pre>
<tt><br>
</tt><tt><br>
</tt><tt><br>
</tt>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
</blockquote>
<br>
</body>
</html>