RFR: JDK-8281274: deal with ActiveProcessorCount in os::Linux::print_container_info [v3]
David Holmes
dholmes at openjdk.java.net
Tue Feb 8 02:43:07 UTC 2022
On Mon, 7 Feb 2022 14:34:34 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:
>
> Adjust output again, adjust test
test/lib/jdk/test/lib/containers/docker/Common.java line 74:
> 72: }
> 73:
> 74: public static DockerRunOptions newOpts(String imageNameAndTag, String testClass, String addJavaOpts) {
It would be good to avoid code duplication and just e.g. have the no-arg `newOpts` pass `null` for `addJavaOpts`.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7354
More information about the hotspot-runtime-dev
mailing list