Skip to content
Permalink
Browse files
8257817: Shenandoah: Don't race with conc-weak-in-progress flag in we…
…ak-LRB

Reviewed-by: rkennke
  • Loading branch information
zhengyu123 committed Dec 7, 2020
1 parent a265c20 commit 395b6bdeee2d9c21234bc2ef37b290947b644d1d
Showing 1 changed file with 4 additions and 3 deletions.
@@ -2063,16 +2063,17 @@ void ShenandoahHeap::op_weak_roots() {
ShenandoahGCWorkerPhase worker_phase(ShenandoahPhaseTimings::conc_weak_roots_work);
ShenandoahConcurrentWeakRootsEvacUpdateTask task(ShenandoahPhaseTimings::conc_weak_roots_work);
workers()->run_task(&task);
if (!ShenandoahConcurrentRoots::should_do_concurrent_class_unloading()) {
set_concurrent_weak_root_in_progress(false);
}
}

// Perform handshake to flush out dead oops
{
ShenandoahTimingsTracker t(ShenandoahPhaseTimings::conc_weak_roots_rendezvous);
rendezvous_threads();
}

if (!ShenandoahConcurrentRoots::should_do_concurrent_class_unloading()) {
set_concurrent_weak_root_in_progress(false);
}
}
}

1 comment on commit 395b6bd

@openjdk-notifier
Copy link

@openjdk-notifier openjdk-notifier bot commented on 395b6bd Dec 7, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.