Integrated: 8336148: Test runtime/locking/TestRecursiveMonitorChurn.java failed: Unexpected Inflation
Axel Boldt-Christmas
aboldtch at openjdk.org
Tue Aug 13 13:15:42 UTC 2024
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.
This pull request has now been integrated.
Changeset: 76e33b6c
Author: Axel Boldt-Christmas <aboldtch at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/76e33b6c1517599e14ee34371c945aafcd752e4e
Stats: 12 lines in 1 file changed: 12 ins; 0 del; 0 mod
8336148: Test runtime/locking/TestRecursiveMonitorChurn.java failed: Unexpected Inflation
Reviewed-by: dholmes, eosterlund
-------------
PR: https://git.openjdk.org/jdk/pull/20551
More information about the hotspot-runtime-dev
mailing list