Skip to content

Commit ee0dec8

Browse files
author
Thomas Schatzl
committedOct 25, 2022
8295838: Document why we do not print Code Cache Roots phase in G1 logging
Reviewed-by: ayang, iwalulya
1 parent ef60608 commit ee0dec8

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed
 

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

+3
Original file line numberDiff line numberDiff line change
@@ -205,6 +205,9 @@ void G1RootProcessor::process_vm_roots(G1RootClosures* closures,
205205
void G1RootProcessor::process_code_cache_roots(CodeBlobClosure* code_closure,
206206
G1GCPhaseTimes* phase_times,
207207
uint worker_id) {
208+
// We do not track timing of this phase. It is only required with class unloading
209+
// disabled, which is an extremely uncommon use case and would otherwise only ever
210+
// show up as "skipped" in the logs.
208211
if (_process_strong_tasks.try_claim_task(G1RP_PS_CodeCache_oops_do)) {
209212
CodeCache::blobs_do(code_closure);
210213
}

0 commit comments

Comments
 (0)
Please sign in to comment.