Request for code review - JDK-8147906 : G1 use of os::processor_count()

Jon Masamitsu jon.masamitsu at oracle.com
Thu Jan 28 19:57:30 UTC 2016


Alex,

Change looks good.  Reviewed.

Jon

On 01/27/2016 12:11 PM, Alexander Harlap wrote:
> Use os::active_processor_count() in G1 code instead of 
> os::processor_count()
>
> Here is David Holmes comment to the JDK-8147095:
>
> /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. /
> /The number of CPUs currently available for the VM to use should be 
> reported through os::active_processor_count().
>
> /Proposed change:
> http://cr.openjdk.java.net/~aharlap/8147906/webrev.00/
>
> Testing: JPRT

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20160128/602902b5/attachment.htm>


More information about the hotspot-gc-dev mailing list