Integrated: 8351077: Shenandoah: Update comments in ShenandoahConcurrentGC::op_reset_after_collect

Xiaolong Peng xpeng at openjdk.org
Tue Mar 4 03:58:56 UTC 2025


On Mon, 3 Mar 2025 20:12:34 GMT, Xiaolong Peng <xpeng at openjdk.org> wrote:

> This is a trivial PR to update the code comments in ShenandoahConcurrentGC::op_reset_after_collect.
> 
> After doing more test and analysis, we have a better understanding why reset bitmap of young gen after concurrent cycle may cause crash if there is pending old GC cycle to execute: When there is soft reference in old gen, but the referent is in young, reseting bitmap of young will cause wrong state of the soft reference, which may lead to expected cashes.

This pull request has now been integrated.

Changeset: 7c173fde
Author:    Xiaolong Peng <xpeng at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/7c173fde4274a798f299876492a2cd833eee9fdd
Stats:     5 lines in 1 file changed: 0 ins; 2 del; 3 mod

8351077: Shenandoah: Update comments in ShenandoahConcurrentGC::op_reset_after_collect

Reviewed-by: wkemper, ysr

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

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


More information about the shenandoah-dev mailing list