RFR: 8373819: GenShen: Requested generation may be null [v3]
Kelvin Nilsen
kdnilsen at openjdk.org
Thu Jan 8 00:14:17 UTC 2026
On Mon, 5 Jan 2026 17:13:08 GMT, William Kemper <wkemper at openjdk.org> wrote:
>> This PR attempts to simplify the generational control thread by decoupling it somewhat from the heap/gc cancellation mechanism. This is meant to prevent the control thread from seeing inconsistencies between `shHeap::_cancelled_gc` and `shGenControlThread::_requested_gc_cause`.
>
> William Kemper has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 13 commits:
>
> - Merge remote-tracking branch 'jdk/master' into fix-null-generation-crash
> - Fix typo in assertion message
> - Take regulator thread out of STS before requesting GC
>
> The request may block while it waits for control thread to stop old marking. If workers are already in the STS, and the regulator thread is still in the STS, but cannot yield, the safepoint will not run. Control, worker and regulator threads deadlock each other.
> - Add comments
> - Revert back to what should be on this branch
> - Merge remote-tracking branch 'jdk/master' into fix-null-generation-crash
> - Don't know how this file got deleted
> - Carry over gc cancellation to gc request
> - Do not let allocation failure requests be overwritten by other requests
> - Fix degen point handling
> - ... and 3 more: https://git.openjdk.org/jdk/compare/4458cab4...8f4f55db
Thanks for talking us through this PR. Lots of subtle issues here. Looks good to me.
-------------
Marked as reviewed by kdnilsen (Committer).
PR Review: https://git.openjdk.org/jdk/pull/28932#pullrequestreview-3637243572
More information about the shenandoah-dev
mailing list