On Sat, 10 Oct 2026 03:41:35 GMT, Leonid Mesnik <[email protected]> wrote:

>> Yunbo Zhang has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8384612: Test serviceability/dcmd/gc/HeapDumpParallelTest.java failed on a 
>> single core machine
>
> test/hotspot/jtreg/serviceability/dcmd/gc/HeapDumpParallelTest.java line 120:
> 
>> 118: 
>> 119:             // Expect parallel dump
>> 120:             test(heapDumpFile, "-gz=9 -overwrite -parallel=2", false);
> 
> Wouldn't be better to left max(Runtime.getRuntime().availableProcessors(), 2) 
> to use all cores?

Thanks, changed it to `Math.max(Runtime.getRuntime().availableProcessors(), 2)`.

Retested with `ActiveProcessorCount=1`, `2`, and `4`; all passed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33212#discussion_r4236870269

Reply via email to