RFR: 8285773: Replace Algorithms.eatMemory(...) with WB.fullGC() in vmTestbase/gc/gctests/ReferencesGC/ReferencesGC.java

[Thomas Schatzl]

On Thu, 28 Apr 2022 02:23:47 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

> Please review this fix. 
> OOME might happen in any thread, causing test failure.
> 
> The reference tests provoke OOME only to force reference clearing.
> 
> The possible fix is to force it directly with WB.fullGC(). Another possible fix would be to throw SkippedException in DefaultUncaughtExceptionHandler. However, it might also cause OOME and still fail.
> 
> We might also want to verify that a last-ditch GC or something similar is called for each GC.

Seems good.

test/hotspot/jtreg/vmTestbase/gc/gctests/ReferencesGC/ReferencesGC.java line 134:

> 132:                 }
> 133: 
> 134:                 // WB.fullGC() is a lst ditch GC which should provoke

Suggestion:

                // WB.fullGC() is a last ditch GC which should provoke

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

Marked as reviewed by tschatzl (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/8440