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

[Leonid Mesnik]

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

Marked as reviewed by lmesnik (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/17322#pullrequestreview-1811602114