RFR: 8321066: Multiple JFR tests have started failing

[Erik Österlund]

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.

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

Commit messages:
 - 8321066: Multiple JFR tests have started failing

Changes: https://git.openjdk.org/jdk/pull/16900/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=16900&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8321066
  Stats: 3 lines in 1 file changed: 0 ins; 1 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/16900.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/16900/head:pull/16900

PR: https://git.openjdk.org/jdk/pull/16900