RFR: 8336148: Test runtime/locking/TestRecursiveMonitorChurn.java failed: Unexpected Inflation [v2]
Axel Boldt-Christmas
aboldtch at openjdk.org
Mon Aug 12 16:05:04 UTC 2024
> 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.
Axel Boldt-Christmas has updated the pull request incrementally with one additional commit since the last revision:
Remove superfluous whitespace
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/20551/files
- new: https://git.openjdk.org/jdk/pull/20551/files/a2d51e0e..db2eace2
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=20551&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=20551&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/20551.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/20551/head:pull/20551
PR: https://git.openjdk.org/jdk/pull/20551
More information about the hotspot-runtime-dev
mailing list