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

David Holmes david.holmes at
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.


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:
> Testing: JPRT

More information about the hotspot-gc-dev mailing list