[jdk21u-dev] RFR: 8321812: Update GC tests to use execute[Limited]TestJava

[Matthias Baesken]

On Tue, 21 May 2024 16:35:16 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

> Mostly clean backport of [JDK-8321812](https://bugs.openjdk.org/browse/JDK-8321812).
> Manual integration was needed in the following files, but trivial to resolve:
> 	test/hotspot/jtreg/gc/arguments/TestSmallInitialHeapWithLargePageAndNUMA.java
> 	test/hotspot/jtreg/gc/g1/TestEvacuationFailure.java
> 	test/hotspot/jtreg/gc/g1/TestGCLogMessages.java
> 	test/hotspot/jtreg/gc/g1/TestLargePageUseForAuxMemory.java
> 	test/hotspot/jtreg/gc/logging/TestMetaSpaceLog.java
> TestAllocationFailure.java was renamed from TestEvacuationFailure.java by [JDK-8319313](https://bugs.openjdk.org/browse/JDK-8319313).
> Skipped hunks which belong to https://github.com/openjdk/jdk/commit/38cfb220ddadbb401cc15f313aadb8234f626210 (not in 21).

Marked as reviewed by mbaesken (Reviewer).

test/hotspot/jtreg/gc/testlibrary/Helpers.java

first hunk throws Exception  now (not IOException) see javadoc ; but this is wrong already in the original change not the backport so a separate JBS issue should be filed .

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

PR Review: https://git.openjdk.org/jdk21u-dev/pull/594#pullrequestreview-2070435251
PR Comment: https://git.openjdk.org/jdk21u-dev/pull/594#issuecomment-2124130545