<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<font face="Times New Roman, Times, serif">Alex,<br>
<br>
Change looks good. Reviewed.<br>
<br>
Jon<br>
</font><br>
<div class="moz-cite-prefix">On 01/27/2016 12:11 PM, Alexander
Harlap wrote:<br>
</div>
<blockquote cite="mid:56A9247D.40106@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
Use os::active_processor_count() in G1 code instead of
os::processor_count()<br>
<br>
Here is David Holmes comment to the JDK-8147095:<br>
<br>
<i>os::processor_count() is set to the number of logical
processors configured on a system (exact definition varies by
platform). With the use of CPU sets, resource pools, and other
affinity based mechanisms this can be a much larger value than
the number of CPUs available for the VM to use. </i><br>
<i>The number of CPUs currently available for the VM to use should
be reported through os::active_processor_count(). <br>
<br>
</i>Proposed change:<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Eaharlap/8147906/webrev.00/">http://cr.openjdk.java.net/~aharlap/8147906/webrev.00/</a><br>
<br>
Testing: JPRT<br>
</blockquote>
<br>
</body>
</html>