On Wed, 13 Sep 2023 12:27:39 GMT, Soumadipta Roy <d...@openjdk.org> wrote:

>> This test is running in tier1, and takes about 400 seconds to complete. 
>> Thus, it drags the execution time of tier1 on large machines. The commit 
>> includes changing the sequential run of test cases in 
>> java/util/concurrent/tck/JSR166TestCase.java to the best possible 
>> combination of parallelization to reduce the total runtime of the overall 
>> test and causing least possible change to user and system times. The below 
>> comparison shows existing and newer combinations of parallelizations:
>> 
>> * before                           : **200.64s user 20.94s system 204% cpu 
>> 1:48.47 total**
>> * fully parallel                  : **308.84s user 23.75s system 479% cpu 
>> 1:09.42 total**
>> * default-details-tck       : **244.69s user 22.03s system 314% cpu 1:24.94 
>> total**
>> * default-fjp_details-others : **260.12s user 24.47s system 404% cpu 1:10.31 
>> total**
>> 
>> Based on the comparison above, the fourth combination has a result very 
>> close to full parallelization and at the same time having the least 
>> deviation of system and user times among the newer combinations. Hence the 
>> commit includes the changes corresponding to the fourth combination.
>
> Soumadipta Roy has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Adding summary for each parallelized section.

Marked as reviewed by martin (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/15619#pullrequestreview-1625807482

Reply via email to