RFR: 8302798: Refactor -XX:+UseOSErrorReporting for noreturn crash reporting [v3]

[Kim Barrett]

> Please review this change to the implementation of the Windows-specific option
> UseOSErrorReporting, toward allowing crash reporting functions to be declared
> noreturn.  VMError::report_and_die no longer conditionally returns if the
> Windows-only option UseOSErrorReporting is true.
> 
> The Windows-only sections of report_and_die now call RaiseFailFastException
> (https://learn.microsoft.com/en-us/windows/win32/api/errhandlingapi/nf-errhandlingapi-raisefailfastexception),
> which immediately invokes WER (Windows Error Reporting) if it is enabled,
> without executing structured exception handler.  If WER is not enabled, it
> just immediately terminates the program.  Thus, we no longer return to walk up
> thestructured exception handler chain to pop out at the top as unhandled in
> order to invoke WER.
> 
> This permits declaring report_and_die as [[noreturn]], once some functions
> from the os class are also so declared.  Also adding that attribute as
> appropriate to other functions in the os class.  This of course assumes
> the use of [[noreturn]] in HotSpot code is approved (JDK-8302124).
> 
> There is a pre-existing bug that I'll be reporting separately.  If
> UseOSErrorReporting and CreateCoredumpOnCrash are both true, we create an
> empty .mdmp file.  We shouldn't create that file when UseOSErrorReporting.
> 
> Testing:
> mach5 tier1-3
> 
> Manual testing with the following, to verify desired behavior.
> 
> -XX:ErrorHandlerTest=N
> 1: assertion failure
> 2: guarantee failure
> 14: SIGSEGV
> 15: divide by zero
> path/to/bin/java \
>   -XX:+UnlockDiagnosticVMOptions \
>   -XX:+ErrorLogSecondaryErrorDetails \
>   -XX:+UseOSErrorReporting \
>   -XX:ErrorHandlerTest=1 \
>   TestDebug.java
> 
> --- TestDebug.java ---
> import java.lang.String;
> public class TestDebug {
>     static private volatile String dummy;
>     public static void main(String[] args) throws Exception {
>         while (true) {
>             dummy = new String("foo bar");
>         }
>     }
> }
> --- end TestDebug.java ---
> 
> The state of WER can be examined and modified using Power Shell commands
> {Get,Enable,Disable}-WindowsErrorReporting.
> 
> The state of reporting WER captured errors can be examined and modified using
> Control Panel > Security and Maintenance > Maintenance : Report Problems [on,off]
> 
> With Report Problems off, reports are placed in
> c:\ProgramData\Microsoft\Windows\WER\ReportArchive
> 
> I verified that executing the above test with WER enabled adds an entry in
> that directory, but not when it's disabled.  Also nothing is added there when
> the test is run with -XX:-UseOSErrorReporting.

Kim Barrett has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:

 - Merge branch 'master' into failfast
 - remove failfast cuttoff of secondary errors
 - failfast

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/12759/files
  - new: https://git.openjdk.org/jdk/pull/12759/files/e3717037..8a7c86ee

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=12759&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12759&range=01-02

  Stats: 3684 lines in 195 files changed: 2505 ins; 453 del; 726 mod
  Patch: https://git.openjdk.org/jdk/pull/12759.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12759/head:pull/12759

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