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

David Holmes david.holmes at oracle.com
Thu Jan 28 05:22:59 UTC 2016


Hi Alex,

Thanks for doing this so promptly. Changes look good to me. Just update 
copyright year before pushing.

David

On 28/01/2016 6:11 AM, 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



More information about the hotspot-gc-dev mailing list