RFR: 8359820: Improve handshake/safepoint timeout diagnostic messages [v3]

Anton Artemov duke at openjdk.org
Fri Jul 18 09:04:50 UTC 2025


On Fri, 18 Jul 2025 08:02:32 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Anton Artemov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8359820: Addressed reviewer's comments
>
> src/hotspot/share/utilities/vmError.cpp line 827:
> 
>> 825:         } else {
>> 826:           st->print(" (sent by kill)");
>> 827:         }
> 
> I don't understand.
> 
> Should not the receiving thread be the timeouted thread? So the one that is executing right now?

I think it can be that the receiving thread does not manage to report the error, but `safepointTimedOutThread` already got a value, which could confuse the reporting method of `VMError` called on another thread. I think this check is needed, as pointed out by @dholmes-ora .

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26309#discussion_r2215489017


More information about the hotspot-runtime-dev mailing list