infernap12/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2026-02-03 23:18:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

8361404: Parallel: Group all class unloading logc at the end of marking phase

Browse Source 
Reviewed-by: tschatzl, iwalulya
This commit is contained in:
Albert Mingkun Yang 2025-07-17 13:55:47 +00:00
parent dc08cf016e
commit 5ed72775a7
1 changed files with 14 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
src/hotspot/share/gc/parallel/psParallelCompact.cpp
View File

@ -732,16 +732,6 @@ void PSParallelCompact::post_compact()
ct->dirty_MemRegion(old_mr);
}
{
// Delete metaspaces for unloaded class loaders and clean up loader_data graph
GCTraceTime(Debug, gc, phases) t("Purge Class Loader Data", gc_timer());
ClassLoaderDataGraph::purge(true /* at_safepoint */);
DEBUG_ONLY(MetaspaceUtils::verify();)
}
// Need to clear claim bits for the next mark.
ClassLoaderDataGraph::clear_claimed_marks();
heap->prune_scavengable_nmethods();
#if COMPILER2_OR_JVMCI
@ -1044,10 +1034,6 @@ bool PSParallelCompact::invoke_no_policy(bool clear_all_soft_refs) {
ref_processor()->start_discovery(clear_all_soft_refs);
ClassUnloadingContext ctx(1 /* num_nmethod_unlink_workers */,
false /* unregister_nmethods_during_purge */,
false /* lock_nmethod_free_separately */);
marking_phase(&_gc_tracer);
summary_phase();
@ -1337,7 +1323,9 @@ void PSParallelCompact::marking_phase(ParallelOldTracer *gc_tracer) {
{
GCTraceTime(Debug, gc, phases) tm_m("Class Unloading", &_gc_timer);
ClassUnloadingContext* ctx = ClassUnloadingContext::context();
ClassUnloadingContext ctx(1 /* num_nmethod_unlink_workers */,
false /* unregister_nmethods_during_purge */,
false /* lock_nmethod_free_separately */);
bool unloading_occurred;
{
@ -1353,7 +1341,7 @@ void PSParallelCompact::marking_phase(ParallelOldTracer *gc_tracer) {
{
GCTraceTime(Debug, gc, phases) t("Purge Unlinked NMethods", gc_timer());
// Release unloaded nmethod's memory.
ctx->purge_nmethods();
ctx.purge_nmethods();
}
{
GCTraceTime(Debug, gc, phases) ur("Unregister NMethods", &_gc_timer);
@ -1361,7 +1349,7 @@ void PSParallelCompact::marking_phase(ParallelOldTracer *gc_tracer) {
}
{
GCTraceTime(Debug, gc, phases) t("Free Code Blobs", gc_timer());
ctx->free_nmethods();
ctx.free_nmethods();
}
// Prune dead klasses from subklass/sibling/implementor lists.
@ -1369,6 +1357,15 @@ void PSParallelCompact::marking_phase(ParallelOldTracer *gc_tracer) {
// Clean JVMCI metadata handles.
JVMCI_ONLY(JVMCI::do_unloading(unloading_occurred));
{
// Delete metaspaces for unloaded class loaders and clean up loader_data graph
GCTraceTime(Debug, gc, phases) t("Purge Class Loader Data", gc_timer());
ClassLoaderDataGraph::purge(true /* at_safepoint */);
DEBUG_ONLY(MetaspaceUtils::verify();)
}
// Need to clear claim bits for the next mark.
ClassLoaderDataGraph::clear_claimed_marks();
}
{