Mercurial > hg > truffle
annotate src/share/vm/runtime/compilationPolicy.cpp @ 11893:ec267141f753
more javadoc and output tweaks for dynamic counters
| author | Lukas Stadler <lukas.stadler@jku.at> |
|---|---|
| date | Fri, 04 Oct 2013 11:56:18 +0200 |
| parents | e7d07c9bb779 |
| children | cefad50507d8 |
| rev | line source |
|---|---|
| 0 | 1 /* |
|
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
2 * Copyright (c) 2000, 2013, Oracle and/or its affiliates. All rights reserved. |
| 0 | 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
| 4 * | |
| 5 * This code is free software; you can redistribute it and/or modify it | |
| 6 * under the terms of the GNU General Public License version 2 only, as | |
| 7 * published by the Free Software Foundation. | |
| 8 * | |
| 9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
| 10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
| 11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
| 12 * version 2 for more details (a copy is included in the LICENSE file that | |
| 13 * accompanied this code). | |
| 14 * | |
| 15 * You should have received a copy of the GNU General Public License version | |
| 16 * 2 along with this work; if not, write to the Free Software Foundation, | |
| 17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
| 18 * | |
|
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1202
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1202
diff
changeset
|
20 * or visit www.oracle.com if you need additional information or have any |
|
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1202
diff
changeset
|
21 * questions. |
| 0 | 22 * |
| 23 */ | |
| 24 | |
| 1972 | 25 #include "precompiled.hpp" |
| 26 #include "code/compiledIC.hpp" | |
| 27 #include "code/nmethod.hpp" | |
| 28 #include "code/scopeDesc.hpp" | |
| 29 #include "compiler/compilerOracle.hpp" | |
| 30 #include "interpreter/interpreter.hpp" | |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
31 #include "oops/methodData.hpp" |
|
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
32 #include "oops/method.hpp" |
| 1972 | 33 #include "oops/oop.inline.hpp" |
| 34 #include "prims/nativeLookup.hpp" | |
|
2348
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
35 #include "runtime/advancedThresholdPolicy.hpp" |
| 1972 | 36 #include "runtime/compilationPolicy.hpp" |
| 37 #include "runtime/frame.hpp" | |
| 38 #include "runtime/handles.inline.hpp" | |
| 39 #include "runtime/rframe.hpp" | |
| 40 #include "runtime/simpleThresholdPolicy.hpp" | |
| 41 #include "runtime/stubRoutines.hpp" | |
| 42 #include "runtime/thread.hpp" | |
| 43 #include "runtime/timer.hpp" | |
| 44 #include "runtime/vframe.hpp" | |
| 45 #include "runtime/vm_operations.hpp" | |
| 46 #include "utilities/events.hpp" | |
| 47 #include "utilities/globalDefinitions.hpp" | |
| 0 | 48 |
| 49 CompilationPolicy* CompilationPolicy::_policy; | |
| 50 elapsedTimer CompilationPolicy::_accumulated_time; | |
| 51 bool CompilationPolicy::_in_vm_startup; | |
| 52 | |
| 53 // Determine compilation policy based on command line argument | |
| 54 void compilationPolicy_init() { | |
| 55 CompilationPolicy::set_in_vm_startup(DelayCompilationDuringStartup); | |
| 56 | |
| 57 switch(CompilationPolicyChoice) { | |
| 58 case 0: | |
| 59 CompilationPolicy::set_policy(new SimpleCompPolicy()); | |
| 60 break; | |
| 61 | |
| 62 case 1: | |
| 63 #ifdef COMPILER2 | |
| 64 CompilationPolicy::set_policy(new StackWalkCompPolicy()); | |
| 65 #else | |
| 66 Unimplemented(); | |
| 67 #endif | |
| 68 break; | |
| 1783 | 69 case 2: |
| 70 #ifdef TIERED | |
| 71 CompilationPolicy::set_policy(new SimpleThresholdPolicy()); | |
| 72 #else | |
| 73 Unimplemented(); | |
| 74 #endif | |
| 75 break; | |
|
2348
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
76 case 3: |
|
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
77 #ifdef TIERED |
|
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
78 CompilationPolicy::set_policy(new AdvancedThresholdPolicy()); |
|
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
79 #else |
|
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
80 Unimplemented(); |
|
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
81 #endif |
|
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
82 break; |
| 0 | 83 default: |
|
10672
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10664
diff
changeset
|
84 fatal("CompilationPolicyChoice must be in the range: [0-3]"); |
| 0 | 85 } |
| 1783 | 86 CompilationPolicy::policy()->initialize(); |
| 0 | 87 } |
| 88 | |
| 89 void CompilationPolicy::completed_vm_startup() { | |
| 90 if (TraceCompilationPolicy) { | |
| 91 tty->print("CompilationPolicy: completed vm startup.\n"); | |
| 92 } | |
| 93 _in_vm_startup = false; | |
| 94 } | |
| 95 | |
| 96 // Returns true if m must be compiled before executing it | |
| 97 // This is intended to force compiles for methods (usually for | |
| 98 // debugging) that would otherwise be interpreted for some reason. | |
| 1783 | 99 bool CompilationPolicy::must_be_compiled(methodHandle m, int comp_level) { |
|
6972
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
100 // Don't allow Xcomp to cause compiles in replay mode |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
101 if (ReplayCompiles) return false; |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
102 |
| 0 | 103 if (m->has_compiled_code()) return false; // already compiled |
| 1783 | 104 if (!can_be_compiled(m, comp_level)) return false; |
| 0 | 105 |
| 106 return !UseInterpreter || // must compile all methods | |
| 1202 | 107 (UseCompiler && AlwaysCompileLoopMethods && m->has_loops() && CompileBroker::should_compile_new_jobs()); // eagerly compile loop methods |
| 0 | 108 } |
| 109 | |
| 110 // Returns true if m is allowed to be compiled | |
| 1783 | 111 bool CompilationPolicy::can_be_compiled(methodHandle m, int comp_level) { |
|
10200
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
112 // allow any levels for WhiteBox |
|
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
113 assert(WhiteBoxAPI || comp_level == CompLevel_all || is_compile(comp_level), "illegal compilation level"); |
|
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
114 |
| 0 | 115 if (m->is_abstract()) return false; |
| 116 if (DontCompileHugeMethods && m->code_size() > HugeMethodLimit) return false; | |
| 117 | |
|
1174
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
118 // Math intrinsics should never be compiled as this can lead to |
|
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
119 // monotonicity problems because the interpreter will prefer the |
|
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
120 // compiled code to the intrinsic version. This can't happen in |
|
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
121 // production because the invocation counter can't be incremented |
|
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
122 // but we shouldn't expose the system to this problem in testing |
|
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
123 // modes. |
|
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
124 if (!AbstractInterpreter::can_be_compiled(m)) { |
|
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
125 return false; |
|
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
126 } |
| 1783 | 127 if (comp_level == CompLevel_all) { |
|
10200
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
128 if (TieredCompilation) { |
|
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
129 // enough to be compilable at any level for tiered |
|
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
130 return !m->is_not_compilable(CompLevel_simple) || !m->is_not_compilable(CompLevel_full_optimization); |
|
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
131 } else { |
|
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
132 // must be compilable at available level for non-tiered |
|
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
133 return !m->is_not_compilable(CompLevel_highest_tier); |
|
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
134 } |
|
9080
b84fd7d73702
8007288: Additional WB API for compiler's testing
iignatyev
parents:
6972
diff
changeset
|
135 } else if (is_compile(comp_level)) { |
| 1783 | 136 return !m->is_not_compilable(comp_level); |
| 137 } | |
|
9080
b84fd7d73702
8007288: Additional WB API for compiler's testing
iignatyev
parents:
6972
diff
changeset
|
138 return false; |
| 1783 | 139 } |
|
1174
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
140 |
| 1783 | 141 bool CompilationPolicy::is_compilation_enabled() { |
| 142 // NOTE: CompileBroker::should_compile_new_jobs() checks for UseCompiler | |
| 143 return !delay_compilation_during_startup() && CompileBroker::should_compile_new_jobs(); | |
| 0 | 144 } |
| 145 | |
| 146 #ifndef PRODUCT | |
| 147 void CompilationPolicy::print_time() { | |
| 148 tty->print_cr ("Accumulated compilationPolicy times:"); | |
| 149 tty->print_cr ("---------------------------"); | |
| 150 tty->print_cr (" Total: %3.3f sec.", _accumulated_time.seconds()); | |
| 151 } | |
| 152 | |
| 1783 | 153 void NonTieredCompPolicy::trace_osr_completion(nmethod* osr_nm) { |
| 0 | 154 if (TraceOnStackReplacement) { |
| 155 if (osr_nm == NULL) tty->print_cr("compilation failed"); | |
| 156 else tty->print_cr("nmethod " INTPTR_FORMAT, osr_nm); | |
| 157 } | |
| 158 } | |
| 159 #endif // !PRODUCT | |
| 160 | |
| 1783 | 161 void NonTieredCompPolicy::initialize() { |
| 162 // Setup the compiler thread numbers | |
| 163 if (CICompilerCountPerCPU) { | |
| 164 // Example: if CICompilerCountPerCPU is true, then we get | |
| 165 // max(log2(8)-1,1) = 2 compiler threads on an 8-way machine. | |
| 166 // May help big-app startup time. | |
| 167 _compiler_count = MAX2(log2_intptr(os::active_processor_count())-1,1); | |
| 168 } else { | |
| 169 _compiler_count = CICompilerCount; | |
| 170 } | |
| 171 } | |
| 172 | |
|
1821
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
173 // Note: this policy is used ONLY if TieredCompilation is off. |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
174 // compiler_count() behaves the following way: |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
175 // - with TIERED build (with both COMPILER1 and COMPILER2 defined) it should return |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
176 // zero for the c1 compilation levels, hence the particular ordering of the |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
177 // statements. |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
178 // - the same should happen when COMPILER2 is defined and COMPILER1 is not |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
179 // (server build without TIERED defined). |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
180 // - if only COMPILER1 is defined (client build), zero should be returned for |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
181 // the c2 level. |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
182 // - if neither is defined - always return zero. |
| 1783 | 183 int NonTieredCompPolicy::compiler_count(CompLevel comp_level) { |
|
1821
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
184 assert(!TieredCompilation, "This policy should not be used with TieredCompilation"); |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
185 #ifdef COMPILER2 |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
186 if (is_c2_compile(comp_level)) { |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
187 return _compiler_count; |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
188 } else { |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
189 return 0; |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
190 } |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
191 #endif |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
192 |
| 1783 | 193 #ifdef COMPILER1 |
| 194 if (is_c1_compile(comp_level)) { | |
| 195 return _compiler_count; | |
|
1821
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
196 } else { |
|
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
197 return 0; |
| 1783 | 198 } |
| 199 #endif | |
| 200 | |
| 201 return 0; | |
| 202 } | |
| 203 | |
| 204 void NonTieredCompPolicy::reset_counter_for_invocation_event(methodHandle m) { | |
| 0 | 205 // Make sure invocation and backedge counter doesn't overflow again right away |
| 206 // as would be the case for native methods. | |
| 207 | |
| 208 // BUT also make sure the method doesn't look like it was never executed. | |
| 209 // Set carry bit and reduce counter's value to min(count, CompileThreshold/2). | |
|
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
210 MethodCounters* mcs = m->method_counters(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
211 assert(mcs != NULL, "MethodCounters cannot be NULL for profiling"); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
212 mcs->invocation_counter()->set_carry(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
213 mcs->backedge_counter()->set_carry(); |
| 0 | 214 |
| 215 assert(!m->was_never_executed(), "don't reset to 0 -- could be mistaken for never-executed"); | |
| 216 } | |
| 217 | |
| 1783 | 218 void NonTieredCompPolicy::reset_counter_for_back_branch_event(methodHandle m) { |
| 0 | 219 // Delay next back-branch event but pump up invocation counter to triger |
| 220 // whole method compilation. | |
|
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
221 MethodCounters* mcs = m->method_counters(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
222 assert(mcs != NULL, "MethodCounters cannot be NULL for profiling"); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
223 InvocationCounter* i = mcs->invocation_counter(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
224 InvocationCounter* b = mcs->backedge_counter(); |
| 0 | 225 |
| 226 // Don't set invocation_counter's value too low otherwise the method will | |
| 227 // look like immature (ic < ~5300) which prevents the inlining based on | |
| 228 // the type profiling. | |
| 229 i->set(i->state(), CompileThreshold); | |
| 230 // Don't reset counter too low - it is used to check if OSR method is ready. | |
| 231 b->set(b->state(), CompileThreshold / 2); | |
| 232 } | |
| 233 | |
| 1783 | 234 // |
| 235 // CounterDecay | |
| 236 // | |
| 237 // Interates through invocation counters and decrements them. This | |
| 238 // is done at each safepoint. | |
| 239 // | |
| 240 class CounterDecay : public AllStatic { | |
| 241 static jlong _last_timestamp; | |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
242 static void do_method(Method* m) { |
|
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
243 MethodCounters* mcs = m->method_counters(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
244 if (mcs != NULL) { |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
245 mcs->invocation_counter()->decay(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
246 } |
| 1783 | 247 } |
| 248 public: | |
| 249 static void decay(); | |
| 250 static bool is_decay_needed() { | |
| 251 return (os::javaTimeMillis() - _last_timestamp) > CounterDecayMinIntervalLength; | |
| 252 } | |
| 253 }; | |
| 254 | |
| 255 jlong CounterDecay::_last_timestamp = 0; | |
| 256 | |
| 257 void CounterDecay::decay() { | |
| 258 _last_timestamp = os::javaTimeMillis(); | |
| 259 | |
| 260 // This operation is going to be performed only at the end of a safepoint | |
| 261 // and hence GC's will not be going on, all Java mutators are suspended | |
| 262 // at this point and hence SystemDictionary_lock is also not needed. | |
| 263 assert(SafepointSynchronize::is_at_safepoint(), "can only be executed at a safepoint"); | |
| 264 int nclasses = SystemDictionary::number_of_classes(); | |
| 265 double classes_per_tick = nclasses * (CounterDecayMinIntervalLength * 1e-3 / | |
| 266 CounterHalfLifeTime); | |
| 267 for (int i = 0; i < classes_per_tick; i++) { | |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
268 Klass* k = SystemDictionary::try_get_next_class(); |
|
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
269 if (k != NULL && k->oop_is_instance()) { |
|
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
270 InstanceKlass::cast(k)->methods_do(do_method); |
| 1783 | 271 } |
| 272 } | |
| 273 } | |
| 274 | |
| 275 // Called at the end of the safepoint | |
| 276 void NonTieredCompPolicy::do_safepoint_work() { | |
| 277 if(UseCounterDecay && CounterDecay::is_decay_needed()) { | |
| 278 CounterDecay::decay(); | |
| 279 } | |
| 280 } | |
| 281 | |
| 282 void NonTieredCompPolicy::reprofile(ScopeDesc* trap_scope, bool is_osr) { | |
| 283 ScopeDesc* sd = trap_scope; | |
|
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
284 MethodCounters* mcs; |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
285 InvocationCounter* c; |
| 1783 | 286 for (; !sd->is_top(); sd = sd->sender()) { |
|
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
287 mcs = sd->method()->method_counters(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
288 if (mcs != NULL) { |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
289 // Reset ICs of inlined methods, since they can trigger compilations also. |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
290 mcs->invocation_counter()->reset(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
291 } |
| 1783 | 292 } |
|
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
293 mcs = sd->method()->method_counters(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
294 if (mcs != NULL) { |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
295 c = mcs->invocation_counter(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
296 if (is_osr) { |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
297 // It was an OSR method, so bump the count higher. |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
298 c->set(c->state(), CompileThreshold); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
299 } else { |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
300 c->reset(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
301 } |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
302 mcs->backedge_counter()->reset(); |
| 1783 | 303 } |
| 304 } | |
| 305 | |
| 306 // This method can be called by any component of the runtime to notify the policy | |
| 307 // that it's recommended to delay the complation of this method. | |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
308 void NonTieredCompPolicy::delay_compilation(Method* method) { |
|
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
309 MethodCounters* mcs = method->method_counters(); |
|
10106
42a42da29fd7
8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents:
10105
diff
changeset
|
310 if (mcs != NULL) { |
|
42a42da29fd7
8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents:
10105
diff
changeset
|
311 mcs->invocation_counter()->decay(); |
|
42a42da29fd7
8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents:
10105
diff
changeset
|
312 mcs->backedge_counter()->decay(); |
|
42a42da29fd7
8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents:
10105
diff
changeset
|
313 } |
| 1783 | 314 } |
| 315 | |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
316 void NonTieredCompPolicy::disable_compilation(Method* method) { |
|
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
317 MethodCounters* mcs = method->method_counters(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
318 if (mcs != NULL) { |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
319 mcs->invocation_counter()->set_state(InvocationCounter::wait_for_nothing); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
320 mcs->backedge_counter()->set_state(InvocationCounter::wait_for_nothing); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
321 } |
| 1783 | 322 } |
| 323 | |
| 324 CompileTask* NonTieredCompPolicy::select_task(CompileQueue* compile_queue) { | |
| 325 return compile_queue->first(); | |
| 326 } | |
| 327 | |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
328 bool NonTieredCompPolicy::is_mature(Method* method) { |
|
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
329 MethodData* mdo = method->method_data(); |
| 1783 | 330 assert(mdo != NULL, "Should be"); |
| 331 uint current = mdo->mileage_of(method); | |
| 332 uint initial = mdo->creation_mileage(); | |
| 333 if (current < initial) | |
| 334 return true; // some sort of overflow | |
| 335 uint target; | |
| 336 if (ProfileMaturityPercentage <= 0) | |
| 337 target = (uint) -ProfileMaturityPercentage; // absolute value | |
| 338 else | |
| 339 target = (uint)( (ProfileMaturityPercentage * CompileThreshold) / 100 ); | |
| 340 return (current >= initial + target); | |
| 341 } | |
| 342 | |
|
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
343 nmethod* NonTieredCompPolicy::event(methodHandle method, methodHandle inlinee, int branch_bci, |
|
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
344 int bci, CompLevel comp_level, nmethod* nm, JavaThread* thread) { |
| 1783 | 345 assert(comp_level == CompLevel_none, "This should be only called from the interpreter"); |
| 346 NOT_PRODUCT(trace_frequency_counter_overflow(method, branch_bci, bci)); | |
|
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
347 if (JvmtiExport::can_post_interpreter_events() && thread->is_interp_only_mode()) { |
|
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
348 // If certain JVMTI events (e.g. frame pop event) are requested then the |
|
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
349 // thread is forced to remain in interpreted code. This is |
|
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
350 // implemented partly by a check in the run_compiled_code |
|
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
351 // section of the interpreter whether we should skip running |
|
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
352 // compiled code, and partly by skipping OSR compiles for |
|
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
353 // interpreted-only threads. |
|
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
354 if (bci != InvocationEntryBci) { |
|
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
355 reset_counter_for_back_branch_event(method); |
|
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
356 return NULL; |
| 1783 | 357 } |
| 358 } | |
|
6972
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
359 if (CompileTheWorld || ReplayCompiles) { |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
360 // Don't trigger other compiles in testing mode |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
361 if (bci == InvocationEntryBci) { |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
362 reset_counter_for_invocation_event(method); |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
363 } else { |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
364 reset_counter_for_back_branch_event(method); |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
365 } |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
366 return NULL; |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
367 } |
|
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
368 |
| 1783 | 369 if (bci == InvocationEntryBci) { |
| 370 // when code cache is full, compilation gets switched off, UseCompiler | |
| 371 // is set to false | |
| 372 if (!method->has_compiled_code() && UseCompiler) { | |
|
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
373 method_invocation_event(method, thread); |
| 1783 | 374 } else { |
| 375 // Force counter overflow on method entry, even if no compilation | |
| 376 // happened. (The method_invocation_event call does this also.) | |
| 377 reset_counter_for_invocation_event(method); | |
| 378 } | |
| 379 // compilation at an invocation overflow no longer goes and retries test for | |
| 380 // compiled method. We always run the loser of the race as interpreted. | |
| 381 // so return NULL | |
| 382 return NULL; | |
| 383 } else { | |
| 384 // counter overflow in a loop => try to do on-stack-replacement | |
| 385 nmethod* osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true); | |
| 386 NOT_PRODUCT(trace_osr_request(method, osr_nm, bci)); | |
| 387 // when code cache is full, we should not compile any more... | |
| 388 if (osr_nm == NULL && UseCompiler) { | |
|
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
389 method_back_branch_event(method, bci, thread); |
| 1783 | 390 osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true); |
| 391 } | |
| 392 if (osr_nm == NULL) { | |
| 393 reset_counter_for_back_branch_event(method); | |
| 394 return NULL; | |
| 395 } | |
| 396 return osr_nm; | |
| 397 } | |
| 398 return NULL; | |
| 399 } | |
| 400 | |
| 401 #ifndef PRODUCT | |
| 402 void NonTieredCompPolicy::trace_frequency_counter_overflow(methodHandle m, int branch_bci, int bci) { | |
| 403 if (TraceInvocationCounterOverflow) { | |
|
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
404 MethodCounters* mcs = m->method_counters(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
405 assert(mcs != NULL, "MethodCounters cannot be NULL for profiling"); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
406 InvocationCounter* ic = mcs->invocation_counter(); |
|
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
407 InvocationCounter* bc = mcs->backedge_counter(); |
| 1783 | 408 ResourceMark rm; |
| 409 const char* msg = | |
| 410 bci == InvocationEntryBci | |
| 411 ? "comp-policy cntr ovfl @ %d in entry of " | |
| 412 : "comp-policy cntr ovfl @ %d in loop of "; | |
| 413 tty->print(msg, bci); | |
| 414 m->print_value(); | |
| 415 tty->cr(); | |
| 416 ic->print(); | |
| 417 bc->print(); | |
| 418 if (ProfileInterpreter) { | |
| 419 if (bci != InvocationEntryBci) { | |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
420 MethodData* mdo = m->method_data(); |
| 1783 | 421 if (mdo != NULL) { |
| 422 int count = mdo->bci_to_data(branch_bci)->as_JumpData()->taken(); | |
| 423 tty->print_cr("back branch count = %d", count); | |
| 424 } | |
| 425 } | |
| 426 } | |
| 427 } | |
| 428 } | |
| 429 | |
| 430 void NonTieredCompPolicy::trace_osr_request(methodHandle method, nmethod* osr, int bci) { | |
| 431 if (TraceOnStackReplacement) { | |
| 432 ResourceMark rm; | |
| 433 tty->print(osr != NULL ? "Reused OSR entry for " : "Requesting OSR entry for "); | |
| 434 method->print_short_name(tty); | |
| 435 tty->print_cr(" at bci %d", bci); | |
| 436 } | |
| 437 } | |
| 438 #endif // !PRODUCT | |
| 439 | |
| 0 | 440 // SimpleCompPolicy - compile current method |
| 441 | |
|
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
442 void SimpleCompPolicy::method_invocation_event(methodHandle m, JavaThread* thread) { |
|
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
443 const int comp_level = CompLevel_highest_tier; |
|
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
444 const int hot_count = m->invocation_count(); |
| 0 | 445 reset_counter_for_invocation_event(m); |
| 446 const char* comment = "count"; | |
| 447 | |
|
10200
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
448 if (is_compilation_enabled() && can_be_compiled(m, comp_level)) { |
| 0 | 449 nmethod* nm = m->code(); |
| 450 if (nm == NULL ) { | |
|
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
451 CompileBroker::compile_method(m, InvocationEntryBci, comp_level, m, hot_count, comment, thread); |
| 0 | 452 } |
| 453 } | |
| 454 } | |
| 455 | |
|
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
456 void SimpleCompPolicy::method_back_branch_event(methodHandle m, int bci, JavaThread* thread) { |
|
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
457 const int comp_level = CompLevel_highest_tier; |
|
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
458 const int hot_count = m->backedge_count(); |
| 0 | 459 const char* comment = "backedge_count"; |
| 460 | |
|
10200
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
461 if (is_compilation_enabled() && !m->is_not_osr_compilable(comp_level) && can_be_compiled(m, comp_level)) { |
|
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
462 CompileBroker::compile_method(m, bci, comp_level, m, hot_count, comment, thread); |
|
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
463 NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, comp_level, true));) |
| 0 | 464 } |
| 465 } | |
|
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
466 |
| 0 | 467 // StackWalkCompPolicy - walk up stack to find a suitable method to compile |
| 468 | |
| 469 #ifdef COMPILER2 | |
| 470 const char* StackWalkCompPolicy::_msg = NULL; | |
| 471 | |
| 472 | |
| 473 // Consider m for compilation | |
|
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
474 void StackWalkCompPolicy::method_invocation_event(methodHandle m, JavaThread* thread) { |
|
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
475 const int comp_level = CompLevel_highest_tier; |
|
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
476 const int hot_count = m->invocation_count(); |
| 0 | 477 reset_counter_for_invocation_event(m); |
| 478 const char* comment = "count"; | |
| 479 | |
|
10200
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
480 if (is_compilation_enabled() && m->code() == NULL && can_be_compiled(m, comp_level)) { |
|
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
481 ResourceMark rm(thread); |
| 0 | 482 frame fr = thread->last_frame(); |
| 483 assert(fr.is_interpreted_frame(), "must be interpreted"); | |
| 484 assert(fr.interpreter_frame_method() == m(), "bad method"); | |
| 485 | |
| 486 if (TraceCompilationPolicy) { | |
| 487 tty->print("method invocation trigger: "); | |
| 488 m->print_short_name(tty); | |
| 489 tty->print(" ( interpreted " INTPTR_FORMAT ", size=%d ) ", (address)m(), m->code_size()); | |
| 490 } | |
| 491 RegisterMap reg_map(thread, false); | |
| 492 javaVFrame* triggerVF = thread->last_java_vframe(®_map); | |
| 493 // triggerVF is the frame that triggered its counter | |
| 494 RFrame* first = new InterpretedRFrame(triggerVF->fr(), thread, m); | |
| 495 | |
| 496 if (first->top_method()->code() != NULL) { | |
| 497 // called obsolete method/nmethod -- no need to recompile | |
| 498 if (TraceCompilationPolicy) tty->print_cr(" --> " INTPTR_FORMAT, first->top_method()->code()); | |
| 499 } else { | |
| 500 if (TimeCompilationPolicy) accumulated_time()->start(); | |
| 501 GrowableArray<RFrame*>* stack = new GrowableArray<RFrame*>(50); | |
| 502 stack->push(first); | |
| 503 RFrame* top = findTopInlinableFrame(stack); | |
| 504 if (TimeCompilationPolicy) accumulated_time()->stop(); | |
| 505 assert(top != NULL, "findTopInlinableFrame returned null"); | |
| 506 if (TraceCompilationPolicy) top->print(); | |
|
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
507 CompileBroker::compile_method(top->top_method(), InvocationEntryBci, comp_level, |
|
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
508 m, hot_count, comment, thread); |
| 0 | 509 } |
| 510 } | |
| 511 } | |
| 512 | |
|
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
513 void StackWalkCompPolicy::method_back_branch_event(methodHandle m, int bci, JavaThread* thread) { |
|
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
514 const int comp_level = CompLevel_highest_tier; |
|
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
515 const int hot_count = m->backedge_count(); |
| 0 | 516 const char* comment = "backedge_count"; |
| 517 | |
|
10200
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
518 if (is_compilation_enabled() && !m->is_not_osr_compilable(comp_level) && can_be_compiled(m, comp_level)) { |
|
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
519 CompileBroker::compile_method(m, bci, comp_level, m, hot_count, comment, thread); |
|
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
520 NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, comp_level, true));) |
| 0 | 521 } |
| 522 } | |
| 523 | |
| 524 RFrame* StackWalkCompPolicy::findTopInlinableFrame(GrowableArray<RFrame*>* stack) { | |
| 525 // go up the stack until finding a frame that (probably) won't be inlined | |
| 526 // into its caller | |
| 527 RFrame* current = stack->at(0); // current choice for stopping | |
| 528 assert( current && !current->is_compiled(), "" ); | |
| 529 const char* msg = NULL; | |
| 530 | |
| 531 while (1) { | |
| 532 | |
| 533 // before going up the stack further, check if doing so would get us into | |
| 534 // compiled code | |
| 535 RFrame* next = senderOf(current, stack); | |
| 536 if( !next ) // No next frame up the stack? | |
| 537 break; // Then compile with current frame | |
| 538 | |
| 539 methodHandle m = current->top_method(); | |
| 540 methodHandle next_m = next->top_method(); | |
| 541 | |
| 542 if (TraceCompilationPolicy && Verbose) { | |
| 543 tty->print("[caller: "); | |
| 544 next_m->print_short_name(tty); | |
| 545 tty->print("] "); | |
| 546 } | |
| 547 | |
| 548 if( !Inline ) { // Inlining turned off | |
| 549 msg = "Inlining turned off"; | |
| 550 break; | |
| 551 } | |
| 552 if (next_m->is_not_compilable()) { // Did fail to compile this before/ | |
| 553 msg = "caller not compilable"; | |
| 554 break; | |
| 555 } | |
| 556 if (next->num() > MaxRecompilationSearchLength) { | |
| 557 // don't go up too high when searching for recompilees | |
| 558 msg = "don't go up any further: > MaxRecompilationSearchLength"; | |
| 559 break; | |
| 560 } | |
| 561 if (next->distance() > MaxInterpretedSearchLength) { | |
| 562 // don't go up too high when searching for recompilees | |
| 563 msg = "don't go up any further: next > MaxInterpretedSearchLength"; | |
| 564 break; | |
| 565 } | |
| 566 // Compiled frame above already decided not to inline; | |
| 567 // do not recompile him. | |
| 568 if (next->is_compiled()) { | |
| 569 msg = "not going up into optimized code"; | |
| 570 break; | |
| 571 } | |
| 572 | |
| 573 // Interpreted frame above us was already compiled. Do not force | |
| 574 // a recompile, although if the frame above us runs long enough an | |
| 575 // OSR might still happen. | |
| 576 if( current->is_interpreted() && next_m->has_compiled_code() ) { | |
| 577 msg = "not going up -- already compiled caller"; | |
| 578 break; | |
| 579 } | |
| 580 | |
| 581 // Compute how frequent this call site is. We have current method 'm'. | |
| 582 // We know next method 'next_m' is interpreted. Find the call site and | |
| 583 // check the various invocation counts. | |
| 584 int invcnt = 0; // Caller counts | |
| 585 if (ProfileInterpreter) { | |
| 586 invcnt = next_m->interpreter_invocation_count(); | |
| 587 } | |
| 588 int cnt = 0; // Call site counts | |
| 589 if (ProfileInterpreter && next_m->method_data() != NULL) { | |
| 590 ResourceMark rm; | |
| 591 int bci = next->top_vframe()->bci(); | |
| 592 ProfileData* data = next_m->method_data()->bci_to_data(bci); | |
| 593 if (data != NULL && data->is_CounterData()) | |
| 594 cnt = data->as_CounterData()->count(); | |
| 595 } | |
| 596 | |
| 597 // Caller counts / call-site counts; i.e. is this call site | |
| 598 // a hot call site for method next_m? | |
| 599 int freq = (invcnt) ? cnt/invcnt : cnt; | |
| 600 | |
| 601 // Check size and frequency limits | |
| 602 if ((msg = shouldInline(m, freq, cnt)) != NULL) { | |
| 603 break; | |
| 604 } | |
| 605 // Check inlining negative tests | |
| 606 if ((msg = shouldNotInline(m)) != NULL) { | |
| 607 break; | |
| 608 } | |
| 609 | |
| 610 | |
| 611 // If the caller method is too big or something then we do not want to | |
| 612 // compile it just to inline a method | |
|
10200
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
613 if (!can_be_compiled(next_m, CompLevel_any)) { |
| 0 | 614 msg = "caller cannot be compiled"; |
| 615 break; | |
| 616 } | |
| 617 | |
| 618 if( next_m->name() == vmSymbols::class_initializer_name() ) { | |
| 619 msg = "do not compile class initializer (OSR ok)"; | |
| 620 break; | |
| 621 } | |
| 622 | |
| 623 if (TraceCompilationPolicy && Verbose) { | |
| 624 tty->print("\n\t check caller: "); | |
| 625 next_m->print_short_name(tty); | |
| 626 tty->print(" ( interpreted " INTPTR_FORMAT ", size=%d ) ", (address)next_m(), next_m->code_size()); | |
| 627 } | |
| 628 | |
| 629 current = next; | |
| 630 } | |
| 631 | |
| 632 assert( !current || !current->is_compiled(), "" ); | |
| 633 | |
| 634 if (TraceCompilationPolicy && msg) tty->print("(%s)\n", msg); | |
| 635 | |
| 636 return current; | |
| 637 } | |
| 638 | |
| 639 RFrame* StackWalkCompPolicy::senderOf(RFrame* rf, GrowableArray<RFrame*>* stack) { | |
| 640 RFrame* sender = rf->caller(); | |
| 641 if (sender && sender->num() == stack->length()) stack->push(sender); | |
| 642 return sender; | |
| 643 } | |
| 644 | |
| 645 | |
| 646 const char* StackWalkCompPolicy::shouldInline(methodHandle m, float freq, int cnt) { | |
| 647 // Allows targeted inlining | |
| 648 // positive filter: should send be inlined? returns NULL (--> yes) | |
| 649 // or rejection msg | |
| 650 int max_size = MaxInlineSize; | |
| 651 int cost = m->code_size(); | |
| 652 | |
| 653 // Check for too many throws (and not too huge) | |
| 654 if (m->interpreter_throwout_count() > InlineThrowCount && cost < InlineThrowMaxSize ) { | |
| 655 return NULL; | |
| 656 } | |
| 657 | |
| 658 // bump the max size if the call is frequent | |
| 659 if ((freq >= InlineFrequencyRatio) || (cnt >= InlineFrequencyCount)) { | |
| 660 if (TraceFrequencyInlining) { | |
| 661 tty->print("(Inlined frequent method)\n"); | |
| 662 m->print(); | |
| 663 } | |
| 664 max_size = FreqInlineSize; | |
| 665 } | |
| 666 if (cost > max_size) { | |
| 667 return (_msg = "too big"); | |
| 668 } | |
| 669 return NULL; | |
| 670 } | |
| 671 | |
| 672 | |
| 673 const char* StackWalkCompPolicy::shouldNotInline(methodHandle m) { | |
| 674 // negative filter: should send NOT be inlined? returns NULL (--> inline) or rejection msg | |
| 675 if (m->is_abstract()) return (_msg = "abstract method"); | |
| 676 // note: we allow ik->is_abstract() | |
|
6940
18fb7da42534
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
6800
diff
changeset
|
677 if (!m->method_holder()->is_initialized()) return (_msg = "method holder not initialized"); |
| 0 | 678 if (m->is_native()) return (_msg = "native method"); |
| 679 nmethod* m_code = m->code(); | |
| 1748 | 680 if (m_code != NULL && m_code->code_size() > InlineSmallCode) |
| 0 | 681 return (_msg = "already compiled into a big method"); |
| 682 | |
| 683 // use frequency-based objections only for non-trivial methods | |
| 684 if (m->code_size() <= MaxTrivialSize) return NULL; | |
| 685 if (UseInterpreter) { // don't use counts with -Xcomp | |
| 686 if ((m->code() == NULL) && m->was_never_executed()) return (_msg = "never executed"); | |
| 687 if (!m->was_executed_more_than(MIN2(MinInliningThreshold, CompileThreshold >> 1))) return (_msg = "executed < MinInliningThreshold times"); | |
| 688 } | |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
689 if (Method::has_unloaded_classes_in_signature(m, JavaThread::current())) return (_msg = "unloaded signature classes"); |
| 0 | 690 |
| 691 return NULL; | |
| 692 } | |
| 693 | |
| 694 | |
| 695 | |
| 696 #endif // COMPILER2 |