RFR: 8286829: Shenandoah: fix Shenandoah Loom support [v2]
Zhengyu Gu
zgu at openjdk.java.net
Tue May 31 13:18:29 UTC 2022
> Please review the patch that fixes Loom support in Shenandoah GC.
>
> - stackChunkOop is a special oop that contains stack metadata, we need to utilize nmethod entry barrier to mark, evacuate and update stack metadata.
> - During evacuation and reference updating phase, we can not guarantee all metadata in stackChunkOop is deeply good, so I force it to take slow path for the correctness, will try to optimize it later in separate CR.
> - Shenandoah uses similar way to arm and disarm nmethod as the new mechanism introduced in JDK-8284161. We may want to migrate to it in followup CR.
>
> Test:
> - [x] tier1 with ShenandoahGC (Linux x86_64 and Windows x64)
> - [x] tier2 with Shenandoah GC (Linux x86_64 and Windows x64)
> - [x] hotspot_gc_shenandoah on Linux x86_32
> - [x] tier1 with ShenandoahGC + Loom (Linux x86_64 and Windows x64)
> - [x] tier2 with ShenandoahGC + Loom (Linux x86_64 and Windows x64)
Zhengyu Gu has updated the pull request incrementally with one additional commit since the last revision:
Aleksey's comment
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/8924/files
- new: https://git.openjdk.java.net/jdk/pull/8924/files/7db9c0b8..b86b51b1
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=8924&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=8924&range=00-01
Stats: 7 lines in 2 files changed: 6 ins; 0 del; 1 mod
Patch: https://git.openjdk.java.net/jdk/pull/8924.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/8924/head:pull/8924
PR: https://git.openjdk.java.net/jdk/pull/8924
More information about the shenandoah-dev
mailing list