RFR: 8359820: Improve handshake/safepoint timeout diagnostic messages [v8]
David Holmes
dholmes at openjdk.org
Tue Aug 5 02:59:12 UTC 2025
On Mon, 4 Aug 2025 08:37:23 GMT, Anton Artemov <duke at openjdk.org> wrote:
>> Hi, please consider the following changes:
>>
>> The problem in the issue description is not a problem by itself, the behavior is not unexpected, but it is somewhat difficult to find out what caused SIGILL to be fired.
>>
>> We propagate this information from `handshake::handle_timeout()` to `VMError::report()` with a help of a global variable. The same mechanism is used to address a similar issue in the safepoint timeout handler.
>>
>> Tested in tiers 1-3.
>
> Anton Artemov 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 17 additional commits since the last revision:
>
> - 8359820: Addressed reviewer's comments
> - Merge remote-tracking branch 'origin/master' into JDK-8359820-SIGILL-with-low-handshake-timeout-on-intel-sde
> - 8359820: Addressed reviewer's comments
> - 8359820: Addressed reviewer's comments
> - Merge remote-tracking branch 'origin/master' into JDK-8359820-SIGILL-with-low-handshake-timeout-on-intel-sde
> - 8359820: Fixed spaces
> - 8359820: Addressed reviewer's comments
> - 8359820: Addressed reviewer's comments
> - 8359820: Addressed reviewer's comments
> - Merge remote-tracking branch 'origin/master' into JDK-8359820-SIGILL-with-low-handshake-timeout-on-intel-sde
> - ... and 7 more: https://git.openjdk.org/jdk/compare/cd8cf2b8...d85769af
LGTM!
Thanks
-------------
Marked as reviewed by dholmes (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26309#pullrequestreview-3086347740
More information about the hotspot-runtime-dev
mailing list