diff -r be588be89f68 -r 436097b038a1 src/hotspot/share/prims/methodHandles.cpp --- a/src/hotspot/share/prims/methodHandles.cpp Fri Nov 30 20:15:25 2018 +0900 +++ b/src/hotspot/share/prims/methodHandles.cpp Fri Nov 30 11:40:48 2018 +0100 @@ -1064,14 +1064,6 @@ return rfill + overflow; } -// Is it safe to remove stale entries from a dependency list? -static bool safe_to_expunge() { - // Since parallel GC threads can concurrently iterate over a dependency - // list during safepoint, it is safe to remove entries only when - // CodeCache lock is held. - return CodeCache_lock->owned_by_self(); -} - void MethodHandles::add_dependent_nmethod(oop call_site, nmethod* nm) { assert_locked_or_safepoint(CodeCache_lock); @@ -1082,7 +1074,7 @@ // in order to avoid memory leak, stale entries are purged whenever a dependency list // is changed (both on addition and removal). Though memory reclamation is delayed, // it avoids indefinite memory usage growth. - deps.add_dependent_nmethod(nm, /*expunge_stale_entries=*/safe_to_expunge()); + deps.add_dependent_nmethod(nm); } void MethodHandles::remove_dependent_nmethod(oop call_site, nmethod* nm) { @@ -1090,7 +1082,15 @@ oop context = java_lang_invoke_CallSite::context_no_keepalive(call_site); DependencyContext deps = java_lang_invoke_MethodHandleNatives_CallSiteContext::vmdependencies(context); - deps.remove_dependent_nmethod(nm, /*expunge_stale_entries=*/safe_to_expunge()); + deps.remove_dependent_nmethod(nm); +} + +void MethodHandles::clean_dependency_context(oop call_site) { + assert_locked_or_safepoint(CodeCache_lock); + + oop context = java_lang_invoke_CallSite::context_no_keepalive(call_site); + DependencyContext deps = java_lang_invoke_MethodHandleNatives_CallSiteContext::vmdependencies(context); + deps.clean_unloading_dependents(); } void MethodHandles::flush_dependent_nmethods(Handle call_site, Handle target) { @@ -1500,7 +1500,6 @@ { NoSafepointVerifier nsv; MutexLockerEx mu2(CodeCache_lock, Mutex::_no_safepoint_check_flag); - assert(safe_to_expunge(), "removal is not safe"); DependencyContext deps = java_lang_invoke_MethodHandleNatives_CallSiteContext::vmdependencies(context()); marked = deps.remove_all_dependents(); }