On Fri, 16 Oct 2026 21:11:50 GMT, Patricio Chilano Mateo
<[email protected]> wrote:
>> Fredrik Bredberg has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update for Patricio
>
> src/hotspot/share/runtime/threadSMR.cpp line 1007:
>
>> 1005: }
>> 1006: } else {
>> 1007: if (thread_is_safe_to_delete &&
>> current->includes(thread_to_delete)) {
>
> If `current == threads` we know the list will include the thread being
> removed so we can avoid the list search.
Yes, but of course. Well spotted!
I've added a check for `current == threads` to avoid the list search.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/33166#discussion_r4315836197
