RFR: 8293252: Shenandoah: ThreadMXBean synchronizer tests crash with aggressive heuristics [v3]

Ashutosh Mehra duke at openjdk.org
Fri Sep 23 15:49:18 UTC 2022


On Wed, 14 Sep 2022 18:55:55 GMT, Dan Heidinga <heidinga at openjdk.org> wrote:

>> Ashutosh Mehra has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use lrb directly instead of using HeapAccess to avoid evacuating doomed
>>   objects referred by weak roots.
>>   
>>   Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>
>
> src/hotspot/share/gc/shenandoah/shenandoahBarrierSet.hpp line 179:
> 
>> 177: 
>> 178:     template <typename T>
>> 179:     static void oop_store_common(T* addr, oop value);
> 
> This is an implementation detail and a public part of the barrier set. We wouldn't expect users - other than the two existing callers - to ever use this api.
> 
> Can it be refactored out of this header and kept entirely within shenandoahBarrierSet.inline.hpp?

@DanHeidinga are you okay with the updates?

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

PR: https://git.openjdk.org/jdk/pull/10268



More information about the hotspot-gc-dev mailing list