RFR: 8364090: Dump JFR recording on CrashOnOutOfMemoryError [v3]

Erik Gahlin egahlin at openjdk.org
Mon Jul 28 08:57:54 UTC 2025


On Mon, 28 Jul 2025 07:06:48 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> Looks reasonable to me. But could you please add a simple regression test? Easiest would be to modify TestCrashOnOutOfMemoryError.java - add a second `@test ` section, start the test with a different parameter that signals that you run the testee with JFR enabled, and then test for existence of the JFR dump.

It might be better to have a test in test/jdk/jdk/jfr/jvm/, where we have other crash tests.

It's probably best to create a separate test for this, as checking for OOM can be flaky, and it may need to be added to the ProblemList separately.

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

PR Comment: https://git.openjdk.org/jdk/pull/26468#issuecomment-3126226149


More information about the hotspot-jfr-dev mailing list