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