RFR: 8369631: Assess and remedy any unsafe usage of the sr_semaphore Semaphore in the Posix signal code [v3]

[Kim Barrett]

On Fri, 17 Oct 2025 02:37:52 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Replace the statically allocated "semaphore" with a pointer and lazily initialize it. I chose this approach as we already have a second Semaphore declared that way.
>> 
>> Testing:
>> - tiers 1-3 sanity
>> 
>> Thanks.
>
> David Holmes 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 four additional commits since the last revision:
> 
>  - Fix misleading indent.
>  - Merge branch 'master' into 8369631-sr_sem
>  - Switch to DeferredStatic per Kim's request
>  - 8369631: Assess and remedy any unsafe usage of the sr_semaphore Semaphore in the Posix signal code

Looks good.

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

Marked as reviewed by kbarrett (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27762#pullrequestreview-3348200889