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

[David Holmes]

On Thu, 16 Oct 2025 17:50:04 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> src/hotspot/os/posix/signals_posix.cpp line 173:
>> 
>>> 171:   static DeferredStatic<OSXSemaphore> sr_semaphore;
>>> 172: #else
>>> 173:   static DeferredStatic<PosixSemaphore> sr_semaphore;
>> 
>> [pre-existing] Oh, ick. Please remove the indentation for the definitions of sr_semaphore.  In the context of this diff I thought these were declarations at class scope, and was quite confused because I couldn't find a definition. Nope, these are at file scope, and *are* definitions. (This might be what is confusing @stefank about how they are/were being constructed.)
>
> Ahh. Thanks for solving that conundrum.

Fixed

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27762#discussion_r2438052402