Mercurial > hg > graal-jvmci-8
annotate src/share/vm/runtime/compilationPolicy.cpp @ 22764:9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
author | Doug Simon <doug.simon@oracle.com> |
---|---|
date | Fri, 08 Jan 2016 23:45:00 +0100 |
parents | 94b7354ef0e0 |
children | 7ae6a635fad0 |
rev | line source |
---|---|
0 | 1 /* |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17906
diff
changeset
|
2 * Copyright (c) 2000, 2014, 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: |
2348
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
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 |
12073 | 141 // Returns true if m is allowed to be osr compiled |
142 bool CompilationPolicy::can_be_osr_compiled(methodHandle m, int comp_level) { | |
143 bool result = false; | |
144 if (comp_level == CompLevel_all) { | |
145 if (TieredCompilation) { | |
146 // enough to be osr compilable at any level for tiered | |
147 result = !m->is_not_osr_compilable(CompLevel_simple) || !m->is_not_osr_compilable(CompLevel_full_optimization); | |
148 } else { | |
149 // must be osr compilable at available level for non-tiered | |
150 result = !m->is_not_osr_compilable(CompLevel_highest_tier); | |
151 } | |
152 } else if (is_compile(comp_level)) { | |
153 result = !m->is_not_osr_compilable(comp_level); | |
154 } | |
155 return (result && can_be_compiled(m, comp_level)); | |
156 } | |
157 | |
1783 | 158 bool CompilationPolicy::is_compilation_enabled() { |
159 // NOTE: CompileBroker::should_compile_new_jobs() checks for UseCompiler | |
160 return !delay_compilation_during_startup() && CompileBroker::should_compile_new_jobs(); | |
0 | 161 } |
162 | |
22764
9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
22763
diff
changeset
|
163 CompileTask* CompilationPolicy::select_task_helper(CompileQueue* compile_queue) { |
22762
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
164 #ifdef COMPILERJVMCI |
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
165 if (!BackgroundCompilation) { |
22764
9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
22763
diff
changeset
|
166 /* |
9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
22763
diff
changeset
|
167 * In blocking compilation mode, the CompileBroker will make |
9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
22763
diff
changeset
|
168 * compilations submitted by a JVMCI compiler thread non-blocking. These |
9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
22763
diff
changeset
|
169 * compilations should be scheduled after all blocking compilations |
9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
22763
diff
changeset
|
170 * to service non-compiler related compilations sooner and reduce the |
9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
22763
diff
changeset
|
171 * chance of such compilations timing out. |
9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
22763
diff
changeset
|
172 */ |
22762
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
173 for (CompileTask* task = compile_queue->first(); task != NULL; task = task->next()) { |
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
174 if (task->is_blocking()) { |
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
175 return task; |
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
176 } |
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
177 } |
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
178 } |
22764
9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
22763
diff
changeset
|
179 #endif |
22762
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
180 return compile_queue->first(); |
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
181 } |
9d78d44d3aac
improved JVMCI support for blocking compilation (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
21559
diff
changeset
|
182 |
0 | 183 #ifndef PRODUCT |
184 void CompilationPolicy::print_time() { | |
185 tty->print_cr ("Accumulated compilationPolicy times:"); | |
186 tty->print_cr ("---------------------------"); | |
187 tty->print_cr (" Total: %3.3f sec.", _accumulated_time.seconds()); | |
188 } | |
189 | |
1783 | 190 void NonTieredCompPolicy::trace_osr_completion(nmethod* osr_nm) { |
0 | 191 if (TraceOnStackReplacement) { |
192 if (osr_nm == NULL) tty->print_cr("compilation failed"); | |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17906
diff
changeset
|
193 else tty->print_cr("nmethod " INTPTR_FORMAT, p2i(osr_nm)); |
0 | 194 } |
195 } | |
196 #endif // !PRODUCT | |
197 | |
1783 | 198 void NonTieredCompPolicy::initialize() { |
199 // Setup the compiler thread numbers | |
200 if (CICompilerCountPerCPU) { | |
201 // Example: if CICompilerCountPerCPU is true, then we get | |
202 // max(log2(8)-1,1) = 2 compiler threads on an 8-way machine. | |
203 // May help big-app startup time. | |
204 _compiler_count = MAX2(log2_intptr(os::active_processor_count())-1,1); | |
17906
7150b16fda52
8029436: CICompilerCount is not updated when the number of compiler threads is adjusted to the number of CPUs
anoll
parents:
12073
diff
changeset
|
205 FLAG_SET_ERGO(intx, CICompilerCount, _compiler_count); |
1783 | 206 } else { |
207 _compiler_count = CICompilerCount; | |
208 } | |
209 } | |
210 | |
1821
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
211 // 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
|
212 // compiler_count() behaves the following way: |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
213 // - 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
|
214 // 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
|
215 // statements. |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
216 // - 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
|
217 // (server build without TIERED defined). |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
218 // - 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
|
219 // the c2 level. |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
220 // - if neither is defined - always return zero. |
1783 | 221 int NonTieredCompPolicy::compiler_count(CompLevel comp_level) { |
1821
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
222 assert(!TieredCompilation, "This policy should not be used with TieredCompilation"); |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
18929
diff
changeset
|
223 #if defined(COMPILER2) || defined(COMPILERJVMCI) |
1821
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
224 if (is_c2_compile(comp_level)) { |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
225 return _compiler_count; |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
226 } else { |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
227 return 0; |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
228 } |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
229 #endif |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
230 |
1783 | 231 #ifdef COMPILER1 |
232 if (is_c1_compile(comp_level)) { | |
233 return _compiler_count; | |
1821
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
234 } else { |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
235 return 0; |
1783 | 236 } |
237 #endif | |
238 | |
239 return 0; | |
240 } | |
241 | |
242 void NonTieredCompPolicy::reset_counter_for_invocation_event(methodHandle m) { | |
0 | 243 // Make sure invocation and backedge counter doesn't overflow again right away |
244 // as would be the case for native methods. | |
245 | |
246 // BUT also make sure the method doesn't look like it was never executed. | |
247 // 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
|
248 MethodCounters* mcs = m->method_counters(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
249 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
|
250 mcs->invocation_counter()->set_carry(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
251 mcs->backedge_counter()->set_carry(); |
0 | 252 |
253 assert(!m->was_never_executed(), "don't reset to 0 -- could be mistaken for never-executed"); | |
254 } | |
255 | |
1783 | 256 void NonTieredCompPolicy::reset_counter_for_back_branch_event(methodHandle m) { |
0 | 257 // Delay next back-branch event but pump up invocation counter to triger |
258 // whole method compilation. | |
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
259 MethodCounters* mcs = m->method_counters(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
260 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
|
261 InvocationCounter* i = mcs->invocation_counter(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
262 InvocationCounter* b = mcs->backedge_counter(); |
0 | 263 |
264 // Don't set invocation_counter's value too low otherwise the method will | |
265 // look like immature (ic < ~5300) which prevents the inlining based on | |
266 // the type profiling. | |
267 i->set(i->state(), CompileThreshold); | |
268 // Don't reset counter too low - it is used to check if OSR method is ready. | |
269 b->set(b->state(), CompileThreshold / 2); | |
270 } | |
271 | |
1783 | 272 // |
273 // CounterDecay | |
274 // | |
275 // Interates through invocation counters and decrements them. This | |
276 // is done at each safepoint. | |
277 // | |
278 class CounterDecay : public AllStatic { | |
279 static jlong _last_timestamp; | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
280 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
|
281 MethodCounters* mcs = m->method_counters(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
282 if (mcs != NULL) { |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
283 mcs->invocation_counter()->decay(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
284 } |
1783 | 285 } |
286 public: | |
287 static void decay(); | |
288 static bool is_decay_needed() { | |
289 return (os::javaTimeMillis() - _last_timestamp) > CounterDecayMinIntervalLength; | |
290 } | |
291 }; | |
292 | |
293 jlong CounterDecay::_last_timestamp = 0; | |
294 | |
295 void CounterDecay::decay() { | |
296 _last_timestamp = os::javaTimeMillis(); | |
297 | |
298 // This operation is going to be performed only at the end of a safepoint | |
299 // and hence GC's will not be going on, all Java mutators are suspended | |
300 // at this point and hence SystemDictionary_lock is also not needed. | |
301 assert(SafepointSynchronize::is_at_safepoint(), "can only be executed at a safepoint"); | |
302 int nclasses = SystemDictionary::number_of_classes(); | |
303 double classes_per_tick = nclasses * (CounterDecayMinIntervalLength * 1e-3 / | |
304 CounterHalfLifeTime); | |
305 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
|
306 Klass* k = SystemDictionary::try_get_next_class(); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
307 if (k != NULL && k->oop_is_instance()) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
308 InstanceKlass::cast(k)->methods_do(do_method); |
1783 | 309 } |
310 } | |
311 } | |
312 | |
313 // Called at the end of the safepoint | |
314 void NonTieredCompPolicy::do_safepoint_work() { | |
315 if(UseCounterDecay && CounterDecay::is_decay_needed()) { | |
316 CounterDecay::decay(); | |
317 } | |
318 } | |
319 | |
320 void NonTieredCompPolicy::reprofile(ScopeDesc* trap_scope, bool is_osr) { | |
321 ScopeDesc* sd = trap_scope; | |
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
322 MethodCounters* mcs; |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
323 InvocationCounter* c; |
1783 | 324 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
|
325 mcs = sd->method()->method_counters(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
326 if (mcs != NULL) { |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
327 // 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
|
328 mcs->invocation_counter()->reset(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
329 } |
1783 | 330 } |
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
331 mcs = sd->method()->method_counters(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
332 if (mcs != NULL) { |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
333 c = mcs->invocation_counter(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
334 if (is_osr) { |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
335 // 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
|
336 c->set(c->state(), CompileThreshold); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
337 } else { |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
338 c->reset(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
339 } |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
340 mcs->backedge_counter()->reset(); |
1783 | 341 } |
342 } | |
343 | |
344 // This method can be called by any component of the runtime to notify the policy | |
345 // 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
|
346 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
|
347 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
|
348 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
|
349 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
|
350 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
|
351 } |
1783 | 352 } |
353 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
354 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
|
355 MethodCounters* mcs = method->method_counters(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
356 if (mcs != NULL) { |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
357 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
|
358 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
|
359 } |
1783 | 360 } |
361 | |
362 CompileTask* NonTieredCompPolicy::select_task(CompileQueue* compile_queue) { | |
22764
9c0966b935a9
converted select_task_blocking_aware into select_task_helper (GRAAL-1387)
Doug Simon <doug.simon@oracle.com>
parents:
22763
diff
changeset
|
363 return select_task_helper(compile_queue); |
1783 | 364 } |
365 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
366 bool NonTieredCompPolicy::is_mature(Method* method) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
367 MethodData* mdo = method->method_data(); |
1783 | 368 assert(mdo != NULL, "Should be"); |
369 uint current = mdo->mileage_of(method); | |
370 uint initial = mdo->creation_mileage(); | |
371 if (current < initial) | |
372 return true; // some sort of overflow | |
373 uint target; | |
374 if (ProfileMaturityPercentage <= 0) | |
375 target = (uint) -ProfileMaturityPercentage; // absolute value | |
376 else | |
377 target = (uint)( (ProfileMaturityPercentage * CompileThreshold) / 100 ); | |
378 return (current >= initial + target); | |
379 } | |
380 | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
381 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
|
382 int bci, CompLevel comp_level, nmethod* nm, JavaThread* thread) { |
1783 | 383 assert(comp_level == CompLevel_none, "This should be only called from the interpreter"); |
384 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
|
385 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
|
386 // 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
|
387 // 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
|
388 // 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
|
389 // 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
|
390 // 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
|
391 // interpreted-only threads. |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
392 if (bci != InvocationEntryBci) { |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
393 reset_counter_for_back_branch_event(method); |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
394 return NULL; |
1783 | 395 } |
396 } | |
6972
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
397 if (CompileTheWorld || ReplayCompiles) { |
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
398 // Don't trigger other compiles in testing mode |
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
399 if (bci == InvocationEntryBci) { |
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
400 reset_counter_for_invocation_event(method); |
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
401 } else { |
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
402 reset_counter_for_back_branch_event(method); |
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
403 } |
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
404 return NULL; |
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
405 } |
bd7a7ce2e264
6830717: replay of compilations would help with debugging
minqi
parents:
6940
diff
changeset
|
406 |
1783 | 407 if (bci == InvocationEntryBci) { |
408 // when code cache is full, compilation gets switched off, UseCompiler | |
409 // is set to false | |
410 if (!method->has_compiled_code() && UseCompiler) { | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
411 method_invocation_event(method, thread); |
1783 | 412 } else { |
413 // Force counter overflow on method entry, even if no compilation | |
414 // happened. (The method_invocation_event call does this also.) | |
415 reset_counter_for_invocation_event(method); | |
416 } | |
417 // compilation at an invocation overflow no longer goes and retries test for | |
418 // compiled method. We always run the loser of the race as interpreted. | |
419 // so return NULL | |
420 return NULL; | |
421 } else { | |
422 // counter overflow in a loop => try to do on-stack-replacement | |
423 nmethod* osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true); | |
424 NOT_PRODUCT(trace_osr_request(method, osr_nm, bci)); | |
425 // when code cache is full, we should not compile any more... | |
426 if (osr_nm == NULL && UseCompiler) { | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
427 method_back_branch_event(method, bci, thread); |
1783 | 428 osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true); |
429 } | |
430 if (osr_nm == NULL) { | |
431 reset_counter_for_back_branch_event(method); | |
432 return NULL; | |
433 } | |
434 return osr_nm; | |
435 } | |
436 return NULL; | |
437 } | |
438 | |
439 #ifndef PRODUCT | |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17906
diff
changeset
|
440 PRAGMA_FORMAT_NONLITERAL_IGNORED_EXTERNAL |
1783 | 441 void NonTieredCompPolicy::trace_frequency_counter_overflow(methodHandle m, int branch_bci, int bci) { |
442 if (TraceInvocationCounterOverflow) { | |
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
443 MethodCounters* mcs = m->method_counters(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
444 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
|
445 InvocationCounter* ic = mcs->invocation_counter(); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6972
diff
changeset
|
446 InvocationCounter* bc = mcs->backedge_counter(); |
1783 | 447 ResourceMark rm; |
448 const char* msg = | |
449 bci == InvocationEntryBci | |
450 ? "comp-policy cntr ovfl @ %d in entry of " | |
451 : "comp-policy cntr ovfl @ %d in loop of "; | |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17906
diff
changeset
|
452 PRAGMA_DIAG_PUSH |
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17906
diff
changeset
|
453 PRAGMA_FORMAT_NONLITERAL_IGNORED_INTERNAL |
1783 | 454 tty->print(msg, bci); |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17906
diff
changeset
|
455 PRAGMA_DIAG_POP |
1783 | 456 m->print_value(); |
457 tty->cr(); | |
458 ic->print(); | |
459 bc->print(); | |
460 if (ProfileInterpreter) { | |
461 if (bci != InvocationEntryBci) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
462 MethodData* mdo = m->method_data(); |
1783 | 463 if (mdo != NULL) { |
464 int count = mdo->bci_to_data(branch_bci)->as_JumpData()->taken(); | |
465 tty->print_cr("back branch count = %d", count); | |
466 } | |
467 } | |
468 } | |
469 } | |
470 } | |
471 | |
472 void NonTieredCompPolicy::trace_osr_request(methodHandle method, nmethod* osr, int bci) { | |
473 if (TraceOnStackReplacement) { | |
474 ResourceMark rm; | |
475 tty->print(osr != NULL ? "Reused OSR entry for " : "Requesting OSR entry for "); | |
476 method->print_short_name(tty); | |
477 tty->print_cr(" at bci %d", bci); | |
478 } | |
479 } | |
480 #endif // !PRODUCT | |
481 | |
0 | 482 // SimpleCompPolicy - compile current method |
483 | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
484 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
|
485 const int comp_level = CompLevel_highest_tier; |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
486 const int hot_count = m->invocation_count(); |
0 | 487 reset_counter_for_invocation_event(m); |
488 const char* comment = "count"; | |
489 | |
10200
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
490 if (is_compilation_enabled() && can_be_compiled(m, comp_level)) { |
0 | 491 nmethod* nm = m->code(); |
492 if (nm == NULL ) { | |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
493 CompileBroker::compile_method(m, InvocationEntryBci, comp_level, m, hot_count, comment, thread); |
0 | 494 } |
495 } | |
496 } | |
497 | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
498 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
|
499 const int comp_level = CompLevel_highest_tier; |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
500 const int hot_count = m->backedge_count(); |
0 | 501 const char* comment = "backedge_count"; |
502 | |
12073 | 503 if (is_compilation_enabled() && can_be_osr_compiled(m, comp_level)) { |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
504 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
|
505 NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, comp_level, true));) |
0 | 506 } |
507 } | |
508 // StackWalkCompPolicy - walk up stack to find a suitable method to compile | |
509 | |
510 #ifdef COMPILER2 | |
511 const char* StackWalkCompPolicy::_msg = NULL; | |
512 | |
513 | |
514 // Consider m for compilation | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
515 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
|
516 const int comp_level = CompLevel_highest_tier; |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
517 const int hot_count = m->invocation_count(); |
0 | 518 reset_counter_for_invocation_event(m); |
519 const char* comment = "count"; | |
520 | |
10200
d1c9384eecb4
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
10107
diff
changeset
|
521 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
|
522 ResourceMark rm(thread); |
0 | 523 frame fr = thread->last_frame(); |
524 assert(fr.is_interpreted_frame(), "must be interpreted"); | |
525 assert(fr.interpreter_frame_method() == m(), "bad method"); | |
526 | |
527 if (TraceCompilationPolicy) { | |
528 tty->print("method invocation trigger: "); | |
529 m->print_short_name(tty); | |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17906
diff
changeset
|
530 tty->print(" ( interpreted " INTPTR_FORMAT ", size=%d ) ", p2i((address)m()), m->code_size()); |
0 | 531 } |
532 RegisterMap reg_map(thread, false); | |
533 javaVFrame* triggerVF = thread->last_java_vframe(®_map); | |
534 // triggerVF is the frame that triggered its counter | |
535 RFrame* first = new InterpretedRFrame(triggerVF->fr(), thread, m); | |
536 | |
537 if (first->top_method()->code() != NULL) { | |
538 // called obsolete method/nmethod -- no need to recompile | |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17906
diff
changeset
|
539 if (TraceCompilationPolicy) tty->print_cr(" --> " INTPTR_FORMAT, p2i(first->top_method()->code())); |
0 | 540 } else { |
541 if (TimeCompilationPolicy) accumulated_time()->start(); | |
542 GrowableArray<RFrame*>* stack = new GrowableArray<RFrame*>(50); | |
543 stack->push(first); | |
544 RFrame* top = findTopInlinableFrame(stack); | |
545 if (TimeCompilationPolicy) accumulated_time()->stop(); | |
546 assert(top != NULL, "findTopInlinableFrame returned null"); | |
547 if (TraceCompilationPolicy) top->print(); | |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
548 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
|
549 m, hot_count, comment, thread); |
0 | 550 } |
551 } | |
552 } | |
553 | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
554 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
|
555 const int comp_level = CompLevel_highest_tier; |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
556 const int hot_count = m->backedge_count(); |
0 | 557 const char* comment = "backedge_count"; |
558 | |
12073 | 559 if (is_compilation_enabled() && can_be_osr_compiled(m, comp_level)) { |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
560 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
|
561 NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, comp_level, true));) |
0 | 562 } |
563 } | |
564 | |
565 RFrame* StackWalkCompPolicy::findTopInlinableFrame(GrowableArray<RFrame*>* stack) { | |
566 // go up the stack until finding a frame that (probably) won't be inlined | |
567 // into its caller | |
568 RFrame* current = stack->at(0); // current choice for stopping | |
569 assert( current && !current->is_compiled(), "" ); | |
570 const char* msg = NULL; | |
571 | |
572 while (1) { | |
573 | |
574 // before going up the stack further, check if doing so would get us into | |
575 // compiled code | |
576 RFrame* next = senderOf(current, stack); | |
577 if( !next ) // No next frame up the stack? | |
578 break; // Then compile with current frame | |
579 | |
580 methodHandle m = current->top_method(); | |
581 methodHandle next_m = next->top_method(); | |
582 | |
583 if (TraceCompilationPolicy && Verbose) { | |
584 tty->print("[caller: "); | |
585 next_m->print_short_name(tty); | |
586 tty->print("] "); | |
587 } | |
588 | |
589 if( !Inline ) { // Inlining turned off | |
590 msg = "Inlining turned off"; | |
591 break; | |
592 } | |
593 if (next_m->is_not_compilable()) { // Did fail to compile this before/ | |
594 msg = "caller not compilable"; | |
595 break; | |
596 } | |
597 if (next->num() > MaxRecompilationSearchLength) { | |
598 // don't go up too high when searching for recompilees | |
599 msg = "don't go up any further: > MaxRecompilationSearchLength"; | |
600 break; | |
601 } | |
602 if (next->distance() > MaxInterpretedSearchLength) { | |
603 // don't go up too high when searching for recompilees | |
604 msg = "don't go up any further: next > MaxInterpretedSearchLength"; | |
605 break; | |
606 } | |
607 // Compiled frame above already decided not to inline; | |
608 // do not recompile him. | |
609 if (next->is_compiled()) { | |
610 msg = "not going up into optimized code"; | |
611 break; | |
612 } | |
613 | |
614 // Interpreted frame above us was already compiled. Do not force | |
615 // a recompile, although if the frame above us runs long enough an | |
616 // OSR might still happen. | |
617 if( current->is_interpreted() && next_m->has_compiled_code() ) { | |
618 msg = "not going up -- already compiled caller"; | |
619 break; | |
620 } | |
621 | |
622 // Compute how frequent this call site is. We have current method 'm'. | |
623 // We know next method 'next_m' is interpreted. Find the call site and | |
624 // check the various invocation counts. | |
625 int invcnt = 0; // Caller counts | |
626 if (ProfileInterpreter) { | |
627 invcnt = next_m->interpreter_invocation_count(); | |
628 } | |
629 int cnt = 0; // Call site counts | |
630 if (ProfileInterpreter && next_m->method_data() != NULL) { | |
631 ResourceMark rm; | |
632 int bci = next->top_vframe()->bci(); | |
633 ProfileData* data = next_m->method_data()->bci_to_data(bci); | |
634 if (data != NULL && data->is_CounterData()) | |
635 cnt = data->as_CounterData()->count(); | |
636 } | |
637 | |
638 // Caller counts / call-site counts; i.e. is this call site | |
639 // a hot call site for method next_m? | |
640 int freq = (invcnt) ? cnt/invcnt : cnt; | |
641 | |
642 // Check size and frequency limits | |
643 if ((msg = shouldInline(m, freq, cnt)) != NULL) { | |
644 break; | |
645 } | |
646 // Check inlining negative tests | |
647 if ((msg = shouldNotInline(m)) != NULL) { | |
648 break; | |
649 } | |
650 | |
651 | |
652 // If the caller method is too big or something then we do not want to | |
653 // 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
|
654 if (!can_be_compiled(next_m, CompLevel_any)) { |
0 | 655 msg = "caller cannot be compiled"; |
656 break; | |
657 } | |
658 | |
659 if( next_m->name() == vmSymbols::class_initializer_name() ) { | |
660 msg = "do not compile class initializer (OSR ok)"; | |
661 break; | |
662 } | |
663 | |
664 if (TraceCompilationPolicy && Verbose) { | |
665 tty->print("\n\t check caller: "); | |
666 next_m->print_short_name(tty); | |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17906
diff
changeset
|
667 tty->print(" ( interpreted " INTPTR_FORMAT ", size=%d ) ", p2i((address)next_m()), next_m->code_size()); |
0 | 668 } |
669 | |
670 current = next; | |
671 } | |
672 | |
673 assert( !current || !current->is_compiled(), "" ); | |
674 | |
675 if (TraceCompilationPolicy && msg) tty->print("(%s)\n", msg); | |
676 | |
677 return current; | |
678 } | |
679 | |
680 RFrame* StackWalkCompPolicy::senderOf(RFrame* rf, GrowableArray<RFrame*>* stack) { | |
681 RFrame* sender = rf->caller(); | |
682 if (sender && sender->num() == stack->length()) stack->push(sender); | |
683 return sender; | |
684 } | |
685 | |
686 | |
687 const char* StackWalkCompPolicy::shouldInline(methodHandle m, float freq, int cnt) { | |
688 // Allows targeted inlining | |
689 // positive filter: should send be inlined? returns NULL (--> yes) | |
690 // or rejection msg | |
691 int max_size = MaxInlineSize; | |
692 int cost = m->code_size(); | |
693 | |
694 // Check for too many throws (and not too huge) | |
695 if (m->interpreter_throwout_count() > InlineThrowCount && cost < InlineThrowMaxSize ) { | |
696 return NULL; | |
697 } | |
698 | |
699 // bump the max size if the call is frequent | |
700 if ((freq >= InlineFrequencyRatio) || (cnt >= InlineFrequencyCount)) { | |
701 if (TraceFrequencyInlining) { | |
702 tty->print("(Inlined frequent method)\n"); | |
703 m->print(); | |
704 } | |
705 max_size = FreqInlineSize; | |
706 } | |
707 if (cost > max_size) { | |
708 return (_msg = "too big"); | |
709 } | |
710 return NULL; | |
711 } | |
712 | |
713 | |
714 const char* StackWalkCompPolicy::shouldNotInline(methodHandle m) { | |
715 // negative filter: should send NOT be inlined? returns NULL (--> inline) or rejection msg | |
716 if (m->is_abstract()) return (_msg = "abstract method"); | |
717 // 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
|
718 if (!m->method_holder()->is_initialized()) return (_msg = "method holder not initialized"); |
0 | 719 if (m->is_native()) return (_msg = "native method"); |
720 nmethod* m_code = m->code(); | |
1748 | 721 if (m_code != NULL && m_code->code_size() > InlineSmallCode) |
0 | 722 return (_msg = "already compiled into a big method"); |
723 | |
724 // use frequency-based objections only for non-trivial methods | |
725 if (m->code_size() <= MaxTrivialSize) return NULL; | |
726 if (UseInterpreter) { // don't use counts with -Xcomp | |
727 if ((m->code() == NULL) && m->was_never_executed()) return (_msg = "never executed"); | |
728 if (!m->was_executed_more_than(MIN2(MinInliningThreshold, CompileThreshold >> 1))) return (_msg = "executed < MinInliningThreshold times"); | |
729 } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
730 if (Method::has_unloaded_classes_in_signature(m, JavaThread::current())) return (_msg = "unloaded signature classes"); |
0 | 731 |
732 return NULL; | |
733 } | |
734 | |
735 | |
736 | |
737 #endif // COMPILER2 |