[jdk17u-dev] RFR: 8321713: Harmonize executeTestJvm with create[Limited]TestJavaProcessBuilder [v2]
Goetz Lindenmaier
goetz at openjdk.org
Fri Apr 25 10:11:21 UTC 2025
> I backport this for parity with 17.0.16-oracle based on the push to 21.
> I verified that the tests omitted there have not been backported
> to 17 in the meantime etc.
> Anyways, already the build will detect if a change that needs
> to be adapted to this one is backported.
>
> I needed some adaptions on top of the jdk21 change:
>
> Omitted: test/hotspot/jtreg/compiler/arguments/TestCompileThresholdScaling.java
> Introduced by "8283807: Handle CompileThreshold the same as other thresholds when scaled with -XX:CompileThresholdScaling".
> This eventually is a candidate for backport. Omit for now I think.
>
> Omitted: test/hotspot/jtreg/compiler/c1/TestPrintC1Statistics.java
> Introduced by "8296969: C1: PrintC1Statistics is broken after JDK-8292878". JDK-8292878 is
> an assembler code cleanup that came in 20.
>
> test/hotspot/jtreg/compiler/loopstripmining/CheckLoopStripMining.java
> test/hotspot/jtreg/compiler/vectorapi/TestVectorErgonomics.java
> Trivial resolve.
>
> Omitted: test/hotspot/jtreg/gc/x/TestHighUsage.java
> Not in 17. Generational ZGC missing.
>
> test/hotspot/jtreg/gc/z/TestHighUsage.java
> Trivial resolve. Generational ZGC missing.
>
> Omitted: test/jdk/java/io/File/TempDirDoesNotExist.java
> Introduced by "8290313: Produce warning when user specified java.io.tmpdir directory doesn't exist". Not
> a candidate for backport, changes behavior.
>
> Omitted: test/jdk/jdk/jfr/jcmd/TestJcmdPreserveRepository.java
> Introduced by "8303229: JFR: Preserve disk repository after exit". Would be nice to have
> in 17, but larger change with CSR.
>
>
> **The following files need adidtional adaptions, see extra commit:**
>
> test/hotspot/jtreg/compiler/c2/Test8062950.java
> Was removed with BiasedLocking.
>
> test/hotspot/jtreg/runtime/jni/checked/TestCheckedEnsureLocalCapacity.java
> test/jdk/com/sun/management/DiagnosticCommandMBean/DcmdMBeanTestCheckJni.java
> Removed by "8297106: Remove the -Xcheck:jni local reference capacity checking" in 20.
>
> test/hotspot/jtreg/vmTestbase/nsk/jvmti/unit/extcallback/Test.java
> This test was only added in the backport of "8286490: JvmtiEventControllerPrivate::set_event_callbacks
> CLEARING_MASK computation is incorrect" to 17.
Goetz Lindenmaier has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
- Merge branch 'openjdk:master' into goetz_backport_8321713
- Adapt additional calls
- Backport e7ebae54a7e0250cb9f645bd2bdc04a60f6b0cba
-------------
Changes:
- all: https://git.openjdk.org/jdk17u-dev/pull/3495/files
- new: https://git.openjdk.org/jdk17u-dev/pull/3495/files/5ce77fa0..e19ded9b
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=3495&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=3495&range=00-01
Stats: 5506 lines in 90 files changed: 4121 ins; 1057 del; 328 mod
Patch: https://git.openjdk.org/jdk17u-dev/pull/3495.diff
Fetch: git fetch https://git.openjdk.org/jdk17u-dev.git pull/3495/head:pull/3495
PR: https://git.openjdk.org/jdk17u-dev/pull/3495
More information about the jdk-updates-dev
mailing list