aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVladimír Dvo?ák <lada.dvorak7@gmail.com>2014-07-07 08:37:26 +0200
committerVladimír Dvo?ák <lada.dvorak7@gmail.com>2014-07-07 08:37:26 +0200
commit3b4332bc31f592e140e994c6f56bdc052bbb7391 (patch)
tree02cb16184e0bb150105321459f191a9940bac8f4
parent16351a553ce9ab9a426830a12396721592e15ef4 (diff)
downloaddcevm-3b4332bc31f592e140e994c6f56bdc052bbb7391.tar.gz
dcevm-3b4332bc31f592e140e994c6f56bdc052bbb7391.zip
light-jdk7u60 deopt-cp patch
-rw-r--r--hotspot/.hg/patches/light-jdk7u60-deopt-cp.patch225
-rw-r--r--hotspot/.hg/patches/series1
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