RFR: 8321066: Multiple JFR tests have started failing

[Daniel D. Daugherty]

On Thu, 30 Nov 2023 15:28:51 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:

>> Before integrating https://bugs.openjdk.org/browse/JDK-8310644 we added a seemingly innocent NoSafepointVerifier in some code that really shouldn't safepoint and reran tier1 only.
>> 
>> What nobody anticipated is that the JFR dumping during crash reporting code probably introduced by https://bugs.openjdk.org/browse/JDK-8233706 performs safepoint polls from inside the crash reporter. These JFR tests try to provoke a crash and check that the JFR recording gets dumped. But we crash during crash reporting in debug builds, because the NSV doesn't like the safepoint polls inside the crash reporter.
>> 
>> Now while this crash reporting code can seemingly make any NSV in the JVM fail if you get a crash there, and even worse, accept safepoints and do GC while crash reporting from totally safepoint unsafe code and what not, this change merely removes the new NSV from the fix that introduced the test failures. But maybe going forward we shouldn't poll for safepoints in the crash reporter.
>> 
>> I tested that the reported test failures fail deterministically without this patch and do not fail with this patch. I also re-ran tier1 just to be safe.
>
> Since this is causing a bit of a christmas tree in the CI and christmas shouldn't come quite yet, I'm going to go ahead and /integrate

@fisk - Has a follow-up bug been filed for the fact that we're polling
for safepoints in the crash reporting code? I don't see a link in
[JDK-8321066](https://bugs.openjdk.org/browse/JDK-8321066) Multiple JFR tests have started failing

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

PR Comment: https://git.openjdk.org/jdk/pull/16900#issuecomment-1836717912