RFR: 8314139: TEST_BUG: runtime/os/THPsInThreadStackPreventionTest.java could fail on machine with large number of cores [v2]

Aleksey Shipilev shade at openjdk.org
Fri Aug 11 17:30:28 UTC 2023


On Fri, 11 Aug 2023 17:03:38 GMT, Man Cao <manc at openjdk.org> wrote:

>> Hi all, @tstuefe 
>> 
>> Could anyone review this fix for a test failure that could happen on machines with many cores? This fix caps `-XX:ActiveProcessorCount` to 64, which limits the number of internal threads the JVM creates.
>> 
>> An alternative is to bump up the `acceptableRSSLimitMB` such as increasing `basicRSSOverheadMB`. However, this approach could be fragile for machines with even more number of cores (e.g. 256 or more).
>> 
>> -Man
>
> Man Cao has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Always set ActiveProcessorCount=2 instead

This looks good to me, but @tstuefe needs to look at it too.

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

Marked as reviewed by shade (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/15241#pullrequestreview-1574082503


More information about the hotspot-runtime-dev mailing list