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