RFR: JDK-8259539: JDK-8255711 broke trap messages
Lois Foltan
lfoltan at openjdk.java.net
Mon Jan 11 20:26:59 UTC 2021
On Mon, 11 Jan 2021 09:20:56 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
> SIGTRAP/SIGILL are used on ppc/aarch64 to implement traps in compiled code (print a trap message and stop with a fatal error).
>
> JDK-8255711 reworked and unified that coding. Calls down into VMError::report_and_die() had been factored out of platform dependent code. Since that change, traps are handled the same as normal crashes, so we loose the detail message associated with a trap. This functionality should be restored.
LGTM with one suggestion.
Lois
src/hotspot/os_cpu/aix_ppc/os_aix_ppc.cpp line 325:
> 323: ShouldNotReachHere();
> 324:
> 325: return false; // Fatal error
In the two files following on you removed the return statement. For consistency can you do the same here?
-------------
Marked as reviewed by lfoltan (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/2022
More information about the hotspot-runtime-dev
mailing list