RFR: 8364819: Post-integration cleanups for JDK-8359820 [v9]

duke duke at openjdk.org
Wed Aug 20 07:23:41 UTC 2025


On Tue, 19 Aug 2025 07:34:30 GMT, Anton Artemov <duke at openjdk.org> wrote:

>> Hi, please consider the following changes:
>> 
>> This is a cleanup after https://github.com/openjdk/jdk/pull/26309
>> 
>> 1) `_handshake_timed_out_thread `and `_safepoint_timed_out_thread` are now `Thread*` and not `intptr_t`, no conversions `p2i <-> i2p` needed.
>> 
>> 2) Added a missed brace in the error message. 
>> 
>> 3) Updates are done with `Atomic::replace_if_null()` to address possible multiple updates and visibility among all threads. 
>> 
>> Trivial change.
>
> Anton Artemov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8364819:  Addressed reviewer's comments

@toxaart 
Your change (at version ee4ceadb1d3c3a028246dcc8ba9cf611be1d9f6d) is now ready to be sponsored by a Committer.

@toxaart 
Your change (at version ee4ceadb1d3c3a028246dcc8ba9cf611be1d9f6d) is now ready to be sponsored by a Committer.

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

PR Comment: https://git.openjdk.org/jdk/pull/26656#issuecomment-3204531242
PR Comment: https://git.openjdk.org/jdk/pull/26656#issuecomment-3204537276


More information about the hotspot-dev mailing list