RFR: 8269135: TestDifferentProtectionDomains runs into timeout in client VM [v2]

Christoph Göttschkes cgo at openjdk.java.net
Wed Jun 23 10:44:53 UTC 2021


> The test tries to compile a method using the WhiteBox with optimization level 4, which is not available in client VMs. The WhiteBox only prints out a warning, and the call to enqueueMethodForCompilation succeeds. After that, the test case goes into an endless loop to wait for the compilation to finish, which never happens, because the method is not enqueued for compilation.
> 
> I fixed this by choosing a different compilation level, if C2 is not included in the JVM. I think this test should be enough, since there is already an `@requires` which checks, that we are in mixed mode and that C2 should be used.
> 
> In addition, I bound the loop in order to fail with a timeout more early, if the compilation does not happen (or takes too long because of platform restrictions).

Christoph Göttschkes has updated the pull request incrementally with one additional commit since the last revision:

  Adds requires compiler2 & revert compilation level detection.

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/4558/files
  - new: https://git.openjdk.java.net/jdk/pull/4558/files/b1a40f61..2aabea99

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4558&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4558&range=00-01

  Stats: 10 lines in 1 file changed: 1 ins; 5 del; 4 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4558.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4558/head:pull/4558

PR: https://git.openjdk.java.net/jdk/pull/4558


More information about the hotspot-runtime-dev mailing list