RFR: 8255019: Shenandoah: Split STW and concurrent mark into separate classes [v24]

[Zhengyu Gu]

On Tue, 5 Jan 2021 14:00:53 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Zhengyu Gu has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 31 commits:
>> 
>>  - Merge
>>  - Update copyright years
>>  - Merge
>>  - Merge branch 'master' into JDK-8255019-sh-mark
>>  - Concurrent mark does not expect forwarded objects
>>  - Merge branch 'master' into JDK-8255019-sh-mark
>>  - Merge branch 'master' into JDK-8255019-sh-mark
>>  - Silent valgrind on potential memory leak
>>  - Merge branch 'master' into JDK-8255019-sh-mark
>>  - Removed ShenandoahConcurrentMark parameter from concurrent GC entry/op, etc.
>>  - ... and 21 more: https://git.openjdk.java.net/jdk/compare/a6c08813...b7390c08
>
> src/hotspot/share/gc/shenandoah/shenandoahMark.inline.hpp line 306:
> 
>> 304: ShenandoahHeap* ShenandoahMark::heap() const {
>> 305:   return _heap;
>> 306: }
> 
> Do we really need this method? `ShenandoahHeap::heap()` is supposed to be as fast.

No. Quite messy on how to access heap in mark code ... removed _heap member and all access via ShenandoahHeap::heap()

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

PR: https://git.openjdk.java.net/jdk/pull/1009