RFR: 8328235: GenShen: Robustify ShenandoahGCSession and fix missing use [v4]
Y. Srinivas Ramakrishna
ysr at openjdk.org
Tue May 28 13:58:18 UTC 2024
On Thu, 23 May 2024 20:52:02 GMT, William Kemper <wkemper at openjdk.org> wrote:
>> Y. Srinivas Ramakrishna has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Small clean-ups.
>
> src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp line 1578:
>
>> 1576: void ShenandoahHeap::set_gc_generation(ShenandoahGeneration* generation) {
>> 1577: _gc_generation = generation;
>> 1578: assert(Thread::current() == ShenandoahController::thread() ||
>
> Could use `shenandoah_assert_control_or_vm_thread` from `shenandoahAsserts.hpp` here?
Done (defined and used a stronger version of the assertion that checks that we are at a safepoint if vm thread).
-------------
PR Review Comment: https://git.openjdk.org/shenandoah/pull/407#discussion_r1617292646
More information about the shenandoah-dev
mailing list