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