diff options
author | Vladimír Dvo?ák <lada.dvorak7@gmail.com> | 2014-07-07 08:37:26 +0200 |
---|---|---|
committer | Vladimír Dvo?ák <lada.dvorak7@gmail.com> | 2014-07-07 08:37:26 +0200 |
commit | 3b4332bc31f592e140e994c6f56bdc052bbb7391 (patch) | |
tree | 02cb16184e0bb150105321459f191a9940bac8f4 | |
parent | 16351a553ce9ab9a426830a12396721592e15ef4 (diff) | |
download | dcevm-3b4332bc31f592e140e994c6f56bdc052bbb7391.tar.gz dcevm-3b4332bc31f592e140e994c6f56bdc052bbb7391.zip |
light-jdk7u60 deopt-cp patch
-rw-r--r-- | hotspot/.hg/patches/light-jdk7u60-deopt-cp.patch | 225 | ||||
-rw-r--r-- | hotspot/.hg/patches/series | 1 |
2 files changed, 226 insertions, 0 deletions
diff --git a/hotspot/.hg/patches/light-jdk7u60-deopt-cp.patch b/hotspot/.hg/patches/light-jdk7u60-deopt-cp.patch new file mode 100644 index 00000000..4c4b9dcd --- /dev/null +++ b/hotspot/.hg/patches/light-jdk7u60-deopt-cp.patch @@ -0,0 +1,225 @@ +diff --git a/src/share/vm/classfile/classFileParser.cpp b/src/share/vm/classfile/classFileParser.cpp +index 8c68dce..c2e3a43 100644 +--- a/src/share/vm/classfile/classFileParser.cpp ++++ b/src/share/vm/classfile/classFileParser.cpp +@@ -3798,6 +3798,11 @@ + } + } + ++ if (cfs->source() != NULL && HotswapDeoptClassPath != NULL) { ++ if (strstr(cfs->source(), HotswapDeoptClassPath) != NULL) ++ this_klass->set_deoptimization_incl(true); ++ } ++ + if (TraceClassResolution) { + // print out the superclass. + const char * from = Klass::cast(this_klass())->external_name(); +diff --git a/src/share/vm/classfile/systemDictionary.cpp b/src/share/vm/classfile/systemDictionary.cpp +index 6228964..c8abc4f 100644 +--- a/src/share/vm/classfile/systemDictionary.cpp ++++ b/src/share/vm/classfile/systemDictionary.cpp +@@ -1256,6 +1256,11 @@ + } + } + ++ if (HotswapDeoptClassPath != NULL) { ++ if (strstr(HotswapDeoptClassPath, ik->external_name()) != NULL) ++ ik->set_deoptimization_incl(true); ++ } ++ + if (TraceClassLoading) { + ResourceMark rm; + tty->print("[Loaded %s", ik->external_name()); +diff --git a/src/share/vm/code/codeCache.cpp b/src/share/vm/code/codeCache.cpp +index 70574bc..722ecfd 100644 +--- a/src/share/vm/code/codeCache.cpp ++++ b/src/share/vm/code/codeCache.cpp +@@ -713,6 +713,13 @@ + } + #endif // HOTSWAP + ++// Deoptimize all methods having deopt-include flag ++void CodeCache::mark_all_incl_nmethods_for_deoptimization() { ++ MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag); ++ FOR_ALL_ALIVE_NMETHODS(nm) { ++ nm->mark_for_deoptimization_incl(); ++ } ++} + + // Deoptimize all methods + void CodeCache::mark_all_nmethods_for_deoptimization() { +diff --git a/src/share/vm/code/codeCache.hpp b/src/share/vm/code/codeCache.hpp +index a670805..0206b48 100644 +--- a/src/share/vm/code/codeCache.hpp ++++ b/src/share/vm/code/codeCache.hpp +@@ -186,6 +186,7 @@ + #endif // HOTSWAP + + static void mark_all_nmethods_for_deoptimization(); ++ static void mark_all_incl_nmethods_for_deoptimization(); + static int mark_for_deoptimization(methodOop dependee); + static void make_marked_nmethods_zombies(); + static void make_marked_nmethods_not_entrant(); +diff --git a/src/share/vm/code/nmethod.cpp b/src/share/vm/code/nmethod.cpp +index d5918cd..ffef631 100644 +--- a/src/share/vm/code/nmethod.cpp ++++ b/src/share/vm/code/nmethod.cpp +@@ -465,6 +465,7 @@ + _lazy_critical_native = 0; + _has_wide_vectors = 0; + _marked_for_deoptimization = 0; ++ _deoptimization_incl = false; + _lock_count = 0; + _stack_traversal_mark = 0; + _unload_reported = false; // jvmti state +@@ -673,6 +674,9 @@ + _exception_cache = NULL; + _pc_desc_cache.reset_to(NULL); + ++ if (method != NULL) ++ _deoptimization_incl = Klass::cast(method->method_holder())->is_deoptimization_incl(); ++ + code_buffer->copy_oops_to(this); + if (ScavengeRootsInCode && detect_scavenge_root_oops()) { + CodeCache::add_scavenge_root_nmethod(this); +@@ -755,6 +759,9 @@ + _osr_entry_point = NULL; + _exception_cache = NULL; + _pc_desc_cache.reset_to(NULL); ++ ++ if (method != NULL) ++ _deoptimization_incl = Klass::cast(method->method_holder())->is_deoptimization_incl(); + + code_buffer->copy_oops_to(this); + debug_only(verify_scavenge_root_oops()); +@@ -864,6 +871,9 @@ + _exception_cache = NULL; + _pc_desc_cache.reset_to(scopes_pcs_begin()); + ++ if (method != NULL) ++ _deoptimization_incl = Klass::cast(method->method_holder())->is_deoptimization_incl(); ++ + // Copy contents of ScopeDescRecorder to nmethod + code_buffer->copy_oops_to(this); + debug_info->copy_to(this); +diff --git a/src/share/vm/code/nmethod.hpp b/src/share/vm/code/nmethod.hpp +index 1a4ad9e..70d220c 100644 +--- a/src/share/vm/code/nmethod.hpp ++++ b/src/share/vm/code/nmethod.hpp +@@ -169,6 +169,8 @@ + bool _marked_for_reclamation; // Used by NMethodSweeper (set only by sweeper) + bool _marked_for_deoptimization; // Used for stack deoptimization + ++ bool _deoptimization_incl; ++ + // used by jvmti to track if an unload event has been posted for this nmethod. + bool _unload_reported; + +@@ -412,6 +414,11 @@ + bool is_marked_for_deoptimization() const { return _marked_for_deoptimization; } + void mark_for_deoptimization() { _marked_for_deoptimization = true; } + ++ bool is_deoptimization_incl() const { return _deoptimization_incl; } ++ void set_deoptimization_incl(bool z) { _deoptimization_incl = z; } ++ ++ void mark_for_deoptimization_incl() { if (_deoptimization_incl) _marked_for_deoptimization = true; } ++ + void make_unloaded(BoolObjectClosure* is_alive, oop cause); + + bool has_dependencies() { return dependencies_size() != 0; } +diff --git a/src/share/vm/oops/klass.cpp b/src/share/vm/oops/klass.cpp +index a928777..d3714b7 100644 +--- a/src/share/vm/oops/klass.cpp ++++ b/src/share/vm/oops/klass.cpp +@@ -163,6 +163,7 @@ + + kl->set_redefinition_flags(Klass::NoRedefinition); + kl->set_redefining(false); ++ kl->set_deoptimization_incl(false); + kl->set_new_version(NULL); + kl->set_old_version(NULL); + kl->set_redefinition_index(-1); +@@ -233,6 +234,8 @@ + if (FastSuperclassLimit == 0) { + // None of the other machinery matters. + set_super(k); ++ if (k != NULL && k->klass_part()->is_deoptimization_incl()) ++ set_deoptimization_incl(true); + return; + } + if (k == NULL) { +@@ -244,6 +247,8 @@ + "initialize this only once to a non-trivial value"); + set_super(k); + Klass* sup = k->klass_part(); ++ if (sup->is_deoptimization_incl()) ++ set_deoptimization_incl(true); + int sup_depth = sup->super_depth(); + juint my_depth = MIN2(sup_depth + 1, (int)primary_super_limit()); + if (!can_be_primary_super_slow()) +diff --git a/src/share/vm/oops/klass.hpp b/src/share/vm/oops/klass.hpp +index bf242d9..884deae 100644 +--- a/src/share/vm/oops/klass.hpp ++++ b/src/share/vm/oops/klass.hpp +@@ -292,6 +292,7 @@ + bool _original_field_offsets_changed; // Did the original field offsets of this class change during class redefinition? + int * _update_information; // Update information + bool _is_redefining; ++ bool _deoptimization_incl; // True if class methods are included in deoptimization + + #ifndef PRODUCT + int _verify_count; // to avoid redundant verifies +@@ -376,6 +377,9 @@ + else { return _old_version->klass_part()->is_same_or_older_version(klass); } + } + ++ bool is_deoptimization_incl() const { return _deoptimization_incl; } ++ void set_deoptimization_incl(bool z) { _deoptimization_incl = z; } ++ + // Revision number for redefined classes, -1 for originally loaded classes + jint revision_number() const { + return _revision_number; +diff --git a/src/share/vm/prims/jvmtiRedefineClasses.cpp b/src/share/vm/prims/jvmtiRedefineClasses.cpp +index cb90823..6836fae 100644 +--- a/src/share/vm/prims/jvmtiRedefineClasses.cpp ++++ b/src/share/vm/prims/jvmtiRedefineClasses.cpp +@@ -482,6 +482,8 @@ + + new_class->set_redefinition_flags(redefinition_flags); + ++ new_class->set_deoptimization_incl(true); ++ + _max_redefinition_flags = _max_redefinition_flags | redefinition_flags; + + if ((redefinition_flags & Klass::ModifyInstances) != 0) { +@@ -1857,7 +1859,10 @@ + if (0 && JvmtiExport::all_dependencies_are_recorded()) { + Universe::flush_evol_dependents_on(k_h); + } else { +- CodeCache::mark_all_nmethods_for_deoptimization(); ++ if (HotswapDeoptClassPath == NULL) ++ CodeCache::mark_all_nmethods_for_deoptimization(); ++ else ++ CodeCache::mark_all_incl_nmethods_for_deoptimization(); + + ResourceMark rm(THREAD); + DeoptimizationMarker dm; +diff --git a/src/share/vm/runtime/globals.hpp b/src/share/vm/runtime/globals.hpp +index e779274..f61b82d 100644 +--- a/src/share/vm/runtime/globals.hpp ++++ b/src/share/vm/runtime/globals.hpp +@@ -3671,7 +3671,13 @@ + product(bool, EnableTracing, false, \ + "Enable event-based tracing") \ + product(bool, UseLockedTracing, false, \ +- "Use locked-tracing when doing event-based tracing") ++ "Use locked-tracing when doing event-based tracing") \ ++ product(ccstr, HotswapDeoptClassPath, NULL, \ ++ "Class path or fragment of the class path to a folder with " \ ++ "classes allowed to be deoptimized on hotswap. If is not " \ ++ "defined then all classes will be deoptimized on hotswap. " \ ++ "That's default behaviour. Using this option the performance " \ ++ "of hotswap can be considerably increased. ") + + /* + * Macros for factoring of globals
\ No newline at end of file diff --git a/hotspot/.hg/patches/series b/hotspot/.hg/patches/series index f6235eab..6851a7ce 100644 --- a/hotspot/.hg/patches/series +++ b/hotspot/.hg/patches/series @@ -21,3 +21,4 @@ light-jdk8u5-b13.patch #+light-jdk8u5-b13 full-jdk7u45-deopt-cp.patch #+full-jdk7u45-b08 full-jdk7u51-deopt-cp.patch #+full-jdk7u51-b13 light-jdk7u51-deopt-cp.patch #+light-jdk7u51-b13 +light-jdk7u60-deopt-cp.patch #+light-jdk7u60-b09 |