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