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

Ashutosh Mehra duke at openjdk.org
Thu Sep 22 01:45:42 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 two additional commits since the last revision:
>> 
>>  - Use HeapAccess instead of RawAccess when iterating the heap objects
>>    
>>    Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>
>>  - Make oop_store_common() as private method and remove extraneous condition
>>    
>>    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?

I have made `oop_store_common` a private method. It needs to be part of `ShenandoahBarrierSet::AccessBarrier` to be able to access `RawAccessBarrier::oop_store()`.

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

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



More information about the hotspot-gc-dev mailing list