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

Severin Gehwolf sgehwolf at openjdk.java.net
Mon Feb 7 11:01:09 UTC 2022


On Fri, 4 Feb 2022 22:05:39 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Asserting the new output in the jtreg test would be good to have too.
>
> This format is fine me with me. You want to see both the container processor count and the actual count being used.

Sorry, I was wrong. I misread `int i = OSContainer::active_processor_count();` for `int i = os::active_processor_count();` which would then return the same value. As @dholmes-ora said, the original format was fine. Please add this in the test, though. Could be akin to `test/hotspot/jtreg/containers/docker/TestMisc.testPrintContainerInfo()`. I.e. set some CPU quota and also ActiveProcessorCount and assert the message in log output.

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

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


More information about the hotspot-runtime-dev mailing list