stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8295839: G1: Single threaded phases (within parallel phases) report as using multiple threads in logs

Browse Source 
Reviewed-by: ayang, iwalulya
This commit is contained in:
Thomas Schatzl 2022-10-25 18:49:06 +00:00
parent 427f50624f
commit 3e49f8fd28
1 changed files with 9 additions and 13 deletions

22
src/hotspot/share/gc/g1/g1RootProcessor.cpp

@ -62,16 +62,14 @@ void G1RootProcessor::evacuate_roots(G1ParScanThreadState* pss, uint worker_id)
process_vm_roots(closures, phase_times, worker_id);
{
// Now the CM ref_processor roots.
// Now the CM ref_processor roots.
if (_process_strong_tasks.try_claim_task(G1RP_PS_refProcessor_oops_do)) {
G1GCParPhaseTimesTracker x(phase_times, G1GCPhaseTimes::CMRefRoots, worker_id);
if (_process_strong_tasks.try_claim_task(G1RP_PS_refProcessor_oops_do)) {
// We need to treat the discovered reference lists of the
// concurrent mark ref processor as roots and keep entries
// (which are added by the marking threads) on them live
// until they can be processed at the end of marking.
_g1h->ref_processor_cm()->weak_oops_do(closures->strong_oops());
}
// We need to treat the discovered reference lists of the
// concurrent mark ref processor as roots and keep entries
// (which are added by the marking threads) on them live
// until they can be processed at the end of marking.
_g1h->ref_processor_cm()->weak_oops_do(closures->strong_oops());
}
// CodeCache is already processed in java roots
@ -186,11 +184,9 @@ void G1RootProcessor::process_java_roots(G1RootClosures* closures,
closures->strong_codeblobs());
}
{
if (_process_strong_tasks.try_claim_task(G1RP_PS_ClassLoaderDataGraph_oops_do)) {
G1GCParPhaseTimesTracker x(phase_times, G1GCPhaseTimes::CLDGRoots, worker_id);
if (_process_strong_tasks.try_claim_task(G1RP_PS_ClassLoaderDataGraph_oops_do)) {
ClassLoaderDataGraph::roots_cld_do(closures->strong_clds(), closures->weak_clds());
}
ClassLoaderDataGraph::roots_cld_do(closures->strong_clds(), closures->weak_clds());
}
}