Integrated: 8322987: Remove gc/stress/gclocker/TestGCLocker* since they always fail with OOME

[Thomas Schatzl]

On Tue, 9 Jan 2024 09:56:03 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   please review this change that removes the obsolete and ever-problemlisted `gc/stress/TestGCLocker*` tests.
> 
> The test tries to verify JNI by filling the heap, causing GCs, but always OOME'ing because the parameters to tune this behavior are too hard to get right for every case and every collector.
> 
> GCLocker functionality is already being tested with other tests like gc/TestJNICriticalStressTest, gc/cslocker/TestCSLocker, vmTestbase/gc/lock/*, other tests that incidentally use JNI and collector specific JNI tests (e.g. gc/g1/pinnedObjs/*).
> 
> Fwiw, the test also fails when replacing the JNI code by equivalent Java code i.e. not causing gclocker gcs.
> 
> Given all this I think removing these tests is just fine.
> 
> Thanks,
>   Thomas

This pull request has now been integrated.

Changeset: 40861761
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/40861761c2b0bb5ae548afc4752dc7cee3bf506a
Stats:     448 lines in 7 files changed: 0 ins; 448 del; 0 mod

8322987: Remove gc/stress/gclocker/TestGCLocker* since they always fail with OOME

Reviewed-by: ayang, lmesnik

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

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