-
Enhancement
-
Resolution: Fixed
-
P2
-
12
-
b16
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8231889 | 11.0.6-oracle | Erik Joelsson | P2 | Resolved | Fixed | b01 |
JDK-8232664 | 11.0.6 | Erik Joelsson | P2 | Resolved | Fixed | b01 |
To get this to work, there are number of issues that need to be fixed in run-test:
* Need JAVA_OPTIONS parameter in addition VM_OPTIONS on global level.
* The current TIMEOUT option is better named TIMEOUT_FACTOR as that is what it is used as.
* The default number of test jobs need to be consistently calculated, the same as the old test makefiles. The calculated number also needs to be modifiable by factors, from different sources.
* The default setting of test specific parameters from the TEST_OPTS parameters need to happen even if the TEST_OPTS parameters are set like TEST_OPTS_TIMEOUT_FACTOR=. Combining options into a single TEST_OPTS=OPTION1=,OPTION2= is just inconvenient for Mach5.
* Need support for extra problemlists for jtreg
* Langtools default memory size needs to be 768m
* The ignore keyword is set by default but really not needed. Lets not set default keywords.
* When setting keywords, since those strings contain shell sensitive characters (! &), try to handle quoting of them as flexible as possible.
For run-test-prebuilt target specifically
* Create the OUTPUTDIR if not present.
* Provide a fallback if NUM_CORES could not be determined.
* On Solaris use gnu tools
jib-profiles.js
* For greater flexibility, try to detect if running in a full source dir or in a src.conf dir and only require the full source to be downloaded in the latter case. This makes it much simpler to run tests using Jib locally.
Test timeouts
The current default timeoutFactor in the makefiles is 4. However, the old Mach5 executor overrides that to 10. I don't think it should dabble with such things and leave it to the makefiles, so with the new run-test executor, it no longer does. This means many tests now have a much shorter effective timeout. Because of this, we need to increase the timeout on some that are now prone to timing out. I have run tier1-5 a few times to try and find these and added /timeout=300 (which will result in the same effective timeout as before) when specific tests seemed problematic.
test/hotspot/jtreg/runtime/appcds/jvmti/InstrumentationTest.java
This test spawns a child process and tries to locate it using the attach api, by looking for a unique token in the command line string of the spawned JVM. The problem is that the command line string it gets from the attach api is truncated and the token is last on the command line. This normally works well, but the arguments before it are 3 files, with full absolute paths inside the jtreg work directory. With Mach5 we have pretty deep work directories, and with run-test, we make them even deeper. This unfortunately trips the limit and the test fails. I have fixed this by reordering the arguments to the child process.
Work is ongoing in the jdk-sandbox repo, branch ihse-runtestprebuilt-branch.
- backported by
-
JDK-8231889 Use run-test makefile framework for testing in Oracle's Mach5
-
- Resolved
-
-
JDK-8232664 Use run-test makefile framework for testing in Oracle's Mach5
-
- Resolved
-
- blocks
-
JDK-8210958 Rename "make run-test" to "make test"
-
- Resolved
-
- relates to
-
JDK-8213005 Missing symbols in hs_err files on Windows after JDK-8212028
-
- Resolved
-
-
JDK-8219251 Langtools tests default memory size needs to be 768m
-
- Resolved
-
-
JDK-8219260 Default number of test jobs needs to be consistently calculated
-
- Resolved
-
-
JDK-8240073 Fix 'test-make' build target in 11u
-
- Resolved
-
-
JDK-8233453 MLVM deoptimize stress test timed out
-
- Resolved
-
-
JDK-8291794 [11u] Corrections after backport of JDK-8212028
-
- Resolved
-
-
JDK-8213422 compilercontrol/DontInlineDirectiveTest.java needs a longer default timeout
-
- Closed
-
-
JDK-8152988 [AOT] Update test batch definitions to include aot-ed java.base module mode into hs-comp testing
-
- Resolved
-