<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p>Good change.</p>
<p>Alex<br>
</p>
<br>
<div class="moz-cite-prefix">On 8/18/2017 4:17 PM, sangheon.kim
wrote:<br>
</div>
<blockquote
cite="mid:ebf3953c-b952-5335-4799-6c446457c3b7@oracle.com"
type="cite">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
Hi all,<br>
<br>
Could I have some reviews for this tiny change of adding
ParallelGCThreads=2 in TestPrintReferences.java?<br>
<br>
The test is assuming always to have 'Balance queues:' log but it
is only generated when the VM did queue balancing. Queue balancing
happens when MT process is enabled(with
+ParallelRefProcBalancingEnabled) or MT discovery is enabled. And
MT discovery is enabled if we have multiple ParallelGCThreads
(> 1).<br>
<br>
Adding explicit thread setting of 'ParallelGCThreads=1' is easiest
way to avoid this test failure on limited environments. i.e.
machines which determine to use 1 ParallelGCThreads.<br>
<br>
The test failure log also includes:<br>
"[Java HotSpot(TM) 64-Bit Server VM warning: If the number of
processors is expected to increase from one, then you should
configure the number of parallel GC threads appropriately using
-XX:ParallelGCThreads=N"<br>
<br>
CR: <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8186402">https://bugs.openjdk.java.net/browse/JDK-8186402</a><br>
Webrev: <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Esangheki/8186402/webrev.0/">http://cr.openjdk.java.net/~sangheki/8186402/webrev.0/</a><br>
Testing: JPRT<br>
<br>
Thanks,<br>
Sangheon<br>
</blockquote>
<br>
</body>
</html>