RFR: 8371014: Dump JFR recording on CrashOnOutOfMemoryError is incorrectly implemented

[Matthias Baesken]

On Sat, 29 Nov 2025 06:06:16 GMT, Yasumasa Suenaga <ysuenaga at openjdk.org> wrote:

> The jtreg test TestEmergencyDumpAtOOM.java runs into the following error on ppc64 platforms.
> 
> JFR emergency dump would be kicked at `VMError::report_and_die()`, then Java thread for JFR would not work due to secondary signal handler for error reporting.
> 
> Passed all of jdk_jfr tests on Linux AMD64.

With your PR added, we do not observe the error in test TestEmergencyDumpAtOOM any more.

src/hotspot/share/jfr/jfr.cpp line 159:

> 157: 
> 158: void Jfr::on_vm_error_report(outputStream* st) {
> 159:   assert(!JfrRecorder::is_recording(), "JFR should be stopped at erorr reporting");

'erorr' - please fix the little typo !

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

PR Comment: https://git.openjdk.org/jdk/pull/28563#issuecomment-3605855107
PR Review Comment: https://git.openjdk.org/jdk/pull/28563#discussion_r2584283966