RFR: JDK-8281274: deal with ActiveProcessorCount in os::Linux::print_container_info [v5]

David Holmes dholmes at openjdk.java.net
Wed Feb 9 12:24:07 UTC 2022


On Wed, 9 Feb 2022 08:14:39 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

>> The function os::Linux::print_container_info outputs memory- and cpu-related container information (when running in a containerized environment).
>> However in the case of active processor count , it currently just outputs the info from OSContainer::active_processor_count without  looking at  ActiveProcessorCount that can be set to overwrite the OSContainer::active_processor_count() - information. This should be improved.
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Avoid changing Common.java, simplify test

Only a single reviewer approved the final version of these changes so it may have been prudent to seek a least one additional re-review. I had it on my to-do list. :)

The updated test looks much cleaner and simpler.

Thanks,
David

-------------

PR: https://git.openjdk.java.net/jdk/pull/7354


More information about the hotspot-runtime-dev mailing list