Integrated: 8258239: Shenandoah: Used wrong closure to mark concurrent roots

Zhengyu Gu zgu at openjdk.java.net
Mon Dec 14 17:58:56 UTC 2020


On Mon, 14 Dec 2020 15:20:42 GMT, Zhengyu Gu <zgu at openjdk.org> wrote:

> During concurrent mark phase, there should not have forwarded objects in roots. Therefore, it should use ShenandoahMarkRefsClosure to mark concurrent roots, instead of ShenandoahMarkResolveRefsClosure.
> 
> Note: this is *not* a correctness bug, but performance one, as ShenandoahMarkResolveRefsClosure closure unnecessarily resolves forwarding pointers, where always resolved to themselves.
> 
> - [x] hotspot_gc_shenandoah

This pull request has now been integrated.

Changeset: 2c3ae19a
Author:    Zhengyu Gu <zgu at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/2c3ae19a
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

8258239: Shenandoah: Used wrong closure to mark concurrent roots

Reviewed-by: rkennke

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

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


More information about the shenandoah-dev mailing list