RFR: 8369631: Assess and remedy any unsafe usage of the sr_semaphore Semaphore in the Posix signal code [v3]
David Holmes
dholmes at openjdk.org
Fri Oct 17 02:37:52 UTC 2025
> 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
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/27762/files
- new: https://git.openjdk.org/jdk/pull/27762/files/a8982e70..d200692a
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=27762&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=27762&range=01-02
Stats: 9608 lines in 240 files changed: 5571 ins; 3533 del; 504 mod
Patch: https://git.openjdk.org/jdk/pull/27762.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/27762/head:pull/27762
PR: https://git.openjdk.org/jdk/pull/27762
More information about the hotspot-runtime-dev
mailing list