> 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 two additional 
commits since the last revision:

 - Changing test id name from security-manager-allow to security-manager
 - Creating a separate parallelized test for java.security.manager=allow

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15619/files
  - new: https://git.openjdk.org/jdk/pull/15619/files/12e1a02a..74b97be9

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15619&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15619&range=02-03

  Stats: 8 lines in 1 file changed: 8 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/15619.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15619/head:pull/15619

PR: https://git.openjdk.org/jdk/pull/15619

Reply via email to