Integrated: 8294197: Zero: JVM_handle_linux_signal should not assume deopt NOPs
Aleksey Shipilev
shade at openjdk.org
Thu Sep 22 14:54:08 UTC 2022
On Thu, 22 Sep 2022 09:45:22 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> This is a simple regression from Loom added code. In `JVM_handle_linux_signal` there is a new block that handles deopt NOPs. Zero does not have them, and so it currently encounters `ShouldNotReachHere()` when entering the block and doing either `os::Posix::ucontext_get_pc(uc)` or `NativeDeoptInstruction::is_deopt_at(pc)`.
>
> This block should not be entered with Zero at all.
>
> Additional testing:
> - [x] Artificially crashing Linux x86_64 Zero fastdebug now produces a sensible hs_err
This pull request has now been integrated.
Changeset: f751e608
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/f751e6087d24012ecc0e6bc74d5572794a7c85d6
Stats: 2 lines in 1 file changed: 2 ins; 0 del; 0 mod
8294197: Zero: JVM_handle_linux_signal should not assume deopt NOPs
Reviewed-by: coleenp
-------------
PR: https://git.openjdk.org/jdk/pull/10389
More information about the hotspot-runtime-dev
mailing list