RFR: 8336148: Test runtime/locking/TestRecursiveMonitorChurn.java failed: Unexpected Inflation

[Erik Österlund]

On Mon, 12 Aug 2024 14:41:55 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

> When writing the test I suspected that this might happen.
> 
> I wanted to see if we could get away without a fuzz factor on the post and pre inflation comparison.
> 
> I'll update the test with a fuzz factor that can handle that some spurious inflation from other sources can occur. Especially relevant now that class loading uses an ObjectLocker, which is why it occurred when instrumentation was added and not only with the -XX:+DeoptimizeALot flags.
> 
> Will run through the problematic tests and check that these help. All observed occurrences have been with a difference of one between pre and post inflation counts.

Looks good.

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

Marked as reviewed by eosterlund (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20551#pullrequestreview-2233403747