Skip to content

Commit 4a65a90

Browse files
author
William Kemper
committed
8351077: Shenandoah: Update comments in ShenandoahConcurrentGC::op_reset_after_collect
Backport-of: 7c173fd
1 parent ade23ea commit 4a65a90

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

src/hotspot/share/gc/shenandoah/shenandoahConcurrentGC.cpp

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1228,11 +1228,9 @@ void ShenandoahConcurrentGC::op_reset_after_collect() {
12281228

12291229
ShenandoahHeap* const heap = ShenandoahHeap::heap();
12301230
if (heap->mode()->is_generational()) {
1231-
// Resetting bitmaps of young gen when bootstrap old GC or there is preempted old GC
1232-
// causes crash due to remembered set violation, hence condition is added to fix the crash.
1233-
// Assuming bitmaps of young gen are not used at all after the cycle, the crash should not
1234-
// have happend, it seems to tickle a bug in remembered set scan. Root causing and fixing of the bug
1235-
// will be tracked via ticket https://bugs.openjdk.org/browse/JDK-8347371
1231+
// If we are in the midst of an old gc bootstrap or an old marking, we want to leave the mark bit map of
1232+
// the young generation intact. In particular, reference processing in the old generation may potentially
1233+
// need the reachability of a young generation referent of a Reference object in the old generation.
12361234
if (!_do_old_gc_bootstrap && !heap->is_concurrent_old_mark_in_progress()) {
12371235
heap->young_generation()->reset_mark_bitmap<false>();
12381236
}

0 commit comments

Comments
 (0)