RFR: 8310228: Improve error reporting for uncaught native exceptions on Windows [v3]
Jorn Vernee
jvernee at openjdk.org
Wed Sep 20 15:23:29 UTC 2023
> Prevents a stack overflow (or other fatal error) from occurring when handling an unknown exception. See JBS issue for full problem outline.
>
> WRT implementation:
> - I've refactored `Handle_FLT_Exception` to check the exception code on 64 bit Windows as well. It now returns a boolean indicating whether the exception was recognized and handled.
> - For x86, I've created a new Uncaugh_Exception_Handler function which implements the uncaught exception handler. This new function replaces the previous use of `Handle_FLT_Exception` as uncaught exception handler. x86 also supports exception chaining through a previous exception handler stored in `prev_uef_handler`. But, on 64 bit windows this field is never set, so we don't need to try to execute the fallback handler on 64 bit. Hence, it seemed clearer to factor out that bit to a separate function. Both the x86 and 64 bit impls call the `handle_FLT_exception` function.
>
> I've tested this patch locally on Windows x64, I will need some help testing this on Windows x86 (it seems some of the test libraries are failing to build?)
>
> Testing: tier 1-4
Jorn Vernee 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 seven additional commits since the last revision:
- use executeTestJvm to forward arguments
- Merge branch 'master' into UncaughtExWin
- address review comments
- Merge branch 'master' into UncaughtExWin
- remove uncaught exception string
- Improve reporting of uncaught native exceptions
- improve uncaught exception handling on Windows
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/14523/files
- new: https://git.openjdk.org/jdk/pull/14523/files/03804dca..976176b3
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=14523&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=14523&range=01-02
Stats: 266543 lines in 5656 files changed: 108399 ins; 101993 del; 56151 mod
Patch: https://git.openjdk.org/jdk/pull/14523.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/14523/head:pull/14523
PR: https://git.openjdk.org/jdk/pull/14523
More information about the hotspot-runtime-dev
mailing list