<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<font face="Times New Roman, Times, serif">Thomas,<br>
</font><br>
<div class="moz-cite-prefix">On 05/02/2016 05:35 AM, Thomas Schatzl
wrote:<br>
</div>
<blockquote cite="mid:1462192550.6903.36.camel@oracle.com"
type="cite">
<pre wrap="">Hi all,
can I have reviews for this change that originated from the
discussion about JDK-8150393 where it has been found
that G1ConcurrentMark::scan_root_regions() always starts all threads
for scanning the root regions independent of the number of root regions
available. This potentially leads to starting (and needing to
synchronize for) to too many threads on huge machines (and on smaller
ones if the conditions are met of course).
CR:
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8155810">https://bugs.openjdk.java.net/browse/JDK-8155810</a>
Webrev:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~tschatzl/8155810/webrev/">http://cr.openjdk.java.net/~tschatzl/8155810/webrev/</a></pre>
</blockquote>
<br>
Change looks good.<br>
<br>
Jon<br>
<br>
<blockquote cite="mid:1462192550.6903.36.camel@oracle.com"
type="cite">
<pre wrap="">
Testing:
jprt
Thanks,
Thomas
</pre>
</blockquote>
<br>
</body>
</html>