author | erikj |
Tue, 12 Sep 2017 19:03:39 +0200 | |
changeset 47216 | 71c04702a3d5 |
parent 46739 | hotspot/src/share/vm/runtime/compilationPolicy.cpp@89569d6a77c0 |
child 48418 | 2207e2917a68 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
43455
diff
changeset
|
2 |
* Copyright (c) 2000, 2017, Oracle and/or its affiliates. All rights reserved. |
1 | 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 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4750
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4750
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4750
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "code/compiledIC.hpp" |
|
27 |
#include "code/nmethod.hpp" |
|
28 |
#include "code/scopeDesc.hpp" |
|
29 |
#include "interpreter/interpreter.hpp" |
|
37248 | 30 |
#include "memory/resourceArea.hpp" |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
31 |
#include "oops/methodData.hpp" |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
32 |
#include "oops/method.hpp" |
7397 | 33 |
#include "oops/oop.inline.hpp" |
34 |
#include "prims/nativeLookup.hpp" |
|
8667
b32929355d27
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
7397
diff
changeset
|
35 |
#include "runtime/advancedThresholdPolicy.hpp" |
7397 | 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" |
|
1 | 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; |
|
6453 | 69 |
case 2: |
70 |
#ifdef TIERED |
|
71 |
CompilationPolicy::set_policy(new SimpleThresholdPolicy()); |
|
72 |
#else |
|
73 |
Unimplemented(); |
|
74 |
#endif |
|
75 |
break; |
|
8667
b32929355d27
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
7397
diff
changeset
|
76 |
case 3: |
b32929355d27
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
7397
diff
changeset
|
77 |
#ifdef TIERED |
b32929355d27
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
7397
diff
changeset
|
78 |
CompilationPolicy::set_policy(new AdvancedThresholdPolicy()); |
b32929355d27
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
7397
diff
changeset
|
79 |
#else |
b32929355d27
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
7397
diff
changeset
|
80 |
Unimplemented(); |
b32929355d27
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
7397
diff
changeset
|
81 |
#endif |
b32929355d27
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
7397
diff
changeset
|
82 |
break; |
1 | 83 |
default: |
8667
b32929355d27
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
7397
diff
changeset
|
84 |
fatal("CompilationPolicyChoice must be in the range: [0-3]"); |
1 | 85 |
} |
6453 | 86 |
CompilationPolicy::policy()->initialize(); |
1 | 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. |
|
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
43455
diff
changeset
|
99 |
bool CompilationPolicy::must_be_compiled(const methodHandle& m, int comp_level) { |
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
100 |
// Don't allow Xcomp to cause compiles in replay mode |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
101 |
if (ReplayCompiles) return false; |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
102 |
|
1 | 103 |
if (m->has_compiled_code()) return false; // already compiled |
6453 | 104 |
if (!can_be_compiled(m, comp_level)) return false; |
1 | 105 |
|
106 |
return !UseInterpreter || // must compile all methods |
|
4750 | 107 |
(UseCompiler && AlwaysCompileLoopMethods && m->has_loops() && CompileBroker::should_compile_new_jobs()); // eagerly compile loop methods |
1 | 108 |
} |
109 |
||
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
43455
diff
changeset
|
110 |
void CompilationPolicy::compile_if_required(const methodHandle& selected_method, TRAPS) { |
38139
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
111 |
if (must_be_compiled(selected_method)) { |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
112 |
// This path is unusual, mostly used by the '-Xcomp' stress test mode. |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
113 |
|
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
114 |
// Note: with several active threads, the must_be_compiled may be true |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
115 |
// while can_be_compiled is false; remove assert |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
116 |
// assert(CompilationPolicy::can_be_compiled(selected_method), "cannot compile"); |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
117 |
if (!THREAD->can_call_java() || THREAD->is_Compiler_thread()) { |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
118 |
// don't force compilation, resolve was on behalf of compiler |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
119 |
return; |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
120 |
} |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
121 |
if (selected_method->method_holder()->is_not_initialized()) { |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
122 |
// 'is_not_initialized' means not only '!is_initialized', but also that |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
123 |
// initialization has not been started yet ('!being_initialized') |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
124 |
// Do not force compilation of methods in uninitialized classes. |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
125 |
// Note that doing this would throw an assert later, |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
126 |
// in CompileBroker::compile_method. |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
127 |
// We sometimes use the link resolver to do reflective lookups |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
128 |
// even before classes are initialized. |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
129 |
return; |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
130 |
} |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
131 |
CompileBroker::compile_method(selected_method, InvocationEntryBci, |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
132 |
CompilationPolicy::policy()->initial_compile_level(), |
38218 | 133 |
methodHandle(), 0, CompileTask::Reason_MustBeCompiled, CHECK); |
38139
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
134 |
} |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
135 |
} |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
136 |
|
1 | 137 |
// Returns true if m is allowed to be compiled |
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
43455
diff
changeset
|
138 |
bool CompilationPolicy::can_be_compiled(const methodHandle& m, int comp_level) { |
17126
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
139 |
// allow any levels for WhiteBox |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
140 |
assert(WhiteBoxAPI || comp_level == CompLevel_all || is_compile(comp_level), "illegal compilation level"); |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
141 |
|
1 | 142 |
if (m->is_abstract()) return false; |
143 |
if (DontCompileHugeMethods && m->code_size() > HugeMethodLimit) return false; |
|
144 |
||
4645
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
145 |
// Math intrinsics should never be compiled as this can lead to |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
146 |
// monotonicity problems because the interpreter will prefer the |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
147 |
// compiled code to the intrinsic version. This can't happen in |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
148 |
// production because the invocation counter can't be incremented |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
149 |
// but we shouldn't expose the system to this problem in testing |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
150 |
// modes. |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
151 |
if (!AbstractInterpreter::can_be_compiled(m)) { |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
152 |
return false; |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
153 |
} |
6453 | 154 |
if (comp_level == CompLevel_all) { |
17126
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
155 |
if (TieredCompilation) { |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
156 |
// enough to be compilable at any level for tiered |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
157 |
return !m->is_not_compilable(CompLevel_simple) || !m->is_not_compilable(CompLevel_full_optimization); |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
158 |
} else { |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
159 |
// must be compilable at available level for non-tiered |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
160 |
return !m->is_not_compilable(CompLevel_highest_tier); |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
161 |
} |
16689
efce070b8d42
8007288: Additional WB API for compiler's testing
iignatyev
parents:
14477
diff
changeset
|
162 |
} else if (is_compile(comp_level)) { |
6453 | 163 |
return !m->is_not_compilable(comp_level); |
164 |
} |
|
16689
efce070b8d42
8007288: Additional WB API for compiler's testing
iignatyev
parents:
14477
diff
changeset
|
165 |
return false; |
6453 | 166 |
} |
4645
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
167 |
|
19332 | 168 |
// Returns true if m is allowed to be osr compiled |
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
43455
diff
changeset
|
169 |
bool CompilationPolicy::can_be_osr_compiled(const methodHandle& m, int comp_level) { |
19332 | 170 |
bool result = false; |
171 |
if (comp_level == CompLevel_all) { |
|
172 |
if (TieredCompilation) { |
|
173 |
// enough to be osr compilable at any level for tiered |
|
174 |
result = !m->is_not_osr_compilable(CompLevel_simple) || !m->is_not_osr_compilable(CompLevel_full_optimization); |
|
175 |
} else { |
|
176 |
// must be osr compilable at available level for non-tiered |
|
177 |
result = !m->is_not_osr_compilable(CompLevel_highest_tier); |
|
178 |
} |
|
179 |
} else if (is_compile(comp_level)) { |
|
180 |
result = !m->is_not_osr_compilable(comp_level); |
|
181 |
} |
|
182 |
return (result && can_be_compiled(m, comp_level)); |
|
183 |
} |
|
184 |
||
6453 | 185 |
bool CompilationPolicy::is_compilation_enabled() { |
186 |
// NOTE: CompileBroker::should_compile_new_jobs() checks for UseCompiler |
|
187 |
return !delay_compilation_during_startup() && CompileBroker::should_compile_new_jobs(); |
|
1 | 188 |
} |
189 |
||
35547
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
190 |
CompileTask* CompilationPolicy::select_task_helper(CompileQueue* compile_queue) { |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
191 |
#if INCLUDE_JVMCI |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
192 |
if (UseJVMCICompiler && !BackgroundCompilation) { |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
193 |
/* |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
194 |
* In blocking compilation mode, the CompileBroker will make |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
195 |
* compilations submitted by a JVMCI compiler thread non-blocking. These |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
196 |
* compilations should be scheduled after all blocking compilations |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
197 |
* to service non-compiler related compilations sooner and reduce the |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
198 |
* chance of such compilations timing out. |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
199 |
*/ |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
200 |
for (CompileTask* task = compile_queue->first(); task != NULL; task = task->next()) { |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
201 |
if (task->is_blocking()) { |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
202 |
return task; |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
203 |
} |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
204 |
} |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
205 |
} |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
206 |
#endif |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
207 |
return compile_queue->first(); |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
208 |
} |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
209 |
|
1 | 210 |
#ifndef PRODUCT |
211 |
void CompilationPolicy::print_time() { |
|
212 |
tty->print_cr ("Accumulated compilationPolicy times:"); |
|
213 |
tty->print_cr ("---------------------------"); |
|
214 |
tty->print_cr (" Total: %3.3f sec.", _accumulated_time.seconds()); |
|
215 |
} |
|
216 |
||
6453 | 217 |
void NonTieredCompPolicy::trace_osr_completion(nmethod* osr_nm) { |
1 | 218 |
if (TraceOnStackReplacement) { |
219 |
if (osr_nm == NULL) tty->print_cr("compilation failed"); |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
24013
diff
changeset
|
220 |
else tty->print_cr("nmethod " INTPTR_FORMAT, p2i(osr_nm)); |
1 | 221 |
} |
222 |
} |
|
223 |
#endif // !PRODUCT |
|
224 |
||
6453 | 225 |
void NonTieredCompPolicy::initialize() { |
226 |
// Setup the compiler thread numbers |
|
227 |
if (CICompilerCountPerCPU) { |
|
228 |
// Example: if CICompilerCountPerCPU is true, then we get |
|
229 |
// max(log2(8)-1,1) = 2 compiler threads on an 8-way machine. |
|
230 |
// May help big-app startup time. |
|
231 |
_compiler_count = MAX2(log2_intptr(os::active_processor_count())-1,1); |
|
24013
1d16b0f1060d
8029436: CICompilerCount is not updated when the number of compiler threads is adjusted to the number of CPUs
anoll
parents:
22551
diff
changeset
|
232 |
FLAG_SET_ERGO(intx, CICompilerCount, _compiler_count); |
6453 | 233 |
} else { |
234 |
_compiler_count = CICompilerCount; |
|
235 |
} |
|
236 |
} |
|
237 |
||
6747
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
238 |
// Note: this policy is used ONLY if TieredCompilation is off. |
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
239 |
// compiler_count() behaves the following way: |
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
240 |
// - with TIERED build (with both COMPILER1 and COMPILER2 defined) it should return |
43455
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
38218
diff
changeset
|
241 |
// zero for the c1 compilation levels in server compilation mode runs |
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
38218
diff
changeset
|
242 |
// and c2 compilation levels in client compilation mode runs. |
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
38218
diff
changeset
|
243 |
// - with COMPILER2 not defined it should return zero for c2 compilation levels. |
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
38218
diff
changeset
|
244 |
// - with COMPILER1 not defined it should return zero for c1 compilation levels. |
6747
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
245 |
// - if neither is defined - always return zero. |
6453 | 246 |
int NonTieredCompPolicy::compiler_count(CompLevel comp_level) { |
6747
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
247 |
assert(!TieredCompilation, "This policy should not be used with TieredCompilation"); |
43455
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
38218
diff
changeset
|
248 |
if (COMPILER2_PRESENT(is_server_compilation_mode_vm() && is_c2_compile(comp_level) ||) |
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
38218
diff
changeset
|
249 |
is_client_compilation_mode_vm() && is_c1_compile(comp_level)) { |
6747
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
250 |
return _compiler_count; |
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
251 |
} |
6453 | 252 |
return 0; |
253 |
} |
|
254 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
255 |
void NonTieredCompPolicy::reset_counter_for_invocation_event(const methodHandle& m) { |
1 | 256 |
// Make sure invocation and backedge counter doesn't overflow again right away |
257 |
// as would be the case for native methods. |
|
258 |
||
259 |
// BUT also make sure the method doesn't look like it was never executed. |
|
260 |
// Set carry bit and reduce counter's value to min(count, CompileThreshold/2). |
|
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
261 |
MethodCounters* mcs = m->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
262 |
assert(mcs != NULL, "MethodCounters cannot be NULL for profiling"); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
263 |
mcs->invocation_counter()->set_carry(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
264 |
mcs->backedge_counter()->set_carry(); |
1 | 265 |
|
266 |
assert(!m->was_never_executed(), "don't reset to 0 -- could be mistaken for never-executed"); |
|
267 |
} |
|
268 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
269 |
void NonTieredCompPolicy::reset_counter_for_back_branch_event(const methodHandle& m) { |
22551 | 270 |
// Delay next back-branch event but pump up invocation counter to trigger |
1 | 271 |
// whole method compilation. |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
272 |
MethodCounters* mcs = m->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
273 |
assert(mcs != NULL, "MethodCounters cannot be NULL for profiling"); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
274 |
InvocationCounter* i = mcs->invocation_counter(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
275 |
InvocationCounter* b = mcs->backedge_counter(); |
1 | 276 |
|
277 |
// Don't set invocation_counter's value too low otherwise the method will |
|
278 |
// look like immature (ic < ~5300) which prevents the inlining based on |
|
279 |
// the type profiling. |
|
280 |
i->set(i->state(), CompileThreshold); |
|
281 |
// Don't reset counter too low - it is used to check if OSR method is ready. |
|
282 |
b->set(b->state(), CompileThreshold / 2); |
|
283 |
} |
|
284 |
||
6453 | 285 |
// |
286 |
// CounterDecay |
|
287 |
// |
|
22551 | 288 |
// Iterates through invocation counters and decrements them. This |
6453 | 289 |
// is done at each safepoint. |
290 |
// |
|
291 |
class CounterDecay : public AllStatic { |
|
292 |
static jlong _last_timestamp; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
293 |
static void do_method(Method* m) { |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
294 |
MethodCounters* mcs = m->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
295 |
if (mcs != NULL) { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
296 |
mcs->invocation_counter()->decay(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
297 |
} |
6453 | 298 |
} |
299 |
public: |
|
300 |
static void decay(); |
|
301 |
static bool is_decay_needed() { |
|
302 |
return (os::javaTimeMillis() - _last_timestamp) > CounterDecayMinIntervalLength; |
|
303 |
} |
|
304 |
}; |
|
305 |
||
306 |
jlong CounterDecay::_last_timestamp = 0; |
|
307 |
||
308 |
void CounterDecay::decay() { |
|
309 |
_last_timestamp = os::javaTimeMillis(); |
|
310 |
||
311 |
// This operation is going to be performed only at the end of a safepoint |
|
312 |
// and hence GC's will not be going on, all Java mutators are suspended |
|
313 |
// at this point and hence SystemDictionary_lock is also not needed. |
|
314 |
assert(SafepointSynchronize::is_at_safepoint(), "can only be executed at a safepoint"); |
|
46729 | 315 |
int nclasses = InstanceKlass::number_of_instance_classes(); |
46739
89569d6a77c0
8185590: ShouldNotReachHere from ClassLoaderData::try_get_next_class()
coleenp
parents:
46729
diff
changeset
|
316 |
int classes_per_tick = nclasses * (CounterDecayMinIntervalLength * 1e-3 / |
6453 | 317 |
CounterHalfLifeTime); |
318 |
for (int i = 0; i < classes_per_tick; i++) { |
|
46729 | 319 |
InstanceKlass* k = ClassLoaderDataGraph::try_get_next_class(); |
320 |
if (k != NULL) { |
|
321 |
k->methods_do(do_method); |
|
6453 | 322 |
} |
323 |
} |
|
324 |
} |
|
325 |
||
326 |
// Called at the end of the safepoint |
|
327 |
void NonTieredCompPolicy::do_safepoint_work() { |
|
328 |
if(UseCounterDecay && CounterDecay::is_decay_needed()) { |
|
329 |
CounterDecay::decay(); |
|
330 |
} |
|
331 |
} |
|
332 |
||
333 |
void NonTieredCompPolicy::reprofile(ScopeDesc* trap_scope, bool is_osr) { |
|
334 |
ScopeDesc* sd = trap_scope; |
|
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
335 |
MethodCounters* mcs; |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
336 |
InvocationCounter* c; |
6453 | 337 |
for (; !sd->is_top(); sd = sd->sender()) { |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
338 |
mcs = sd->method()->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
339 |
if (mcs != NULL) { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
340 |
// Reset ICs of inlined methods, since they can trigger compilations also. |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
341 |
mcs->invocation_counter()->reset(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
342 |
} |
6453 | 343 |
} |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
344 |
mcs = sd->method()->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
345 |
if (mcs != NULL) { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
346 |
c = mcs->invocation_counter(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
347 |
if (is_osr) { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
348 |
// It was an OSR method, so bump the count higher. |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
349 |
c->set(c->state(), CompileThreshold); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
350 |
} else { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
351 |
c->reset(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
352 |
} |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
353 |
mcs->backedge_counter()->reset(); |
6453 | 354 |
} |
355 |
} |
|
356 |
||
357 |
// This method can be called by any component of the runtime to notify the policy |
|
22551 | 358 |
// that it's recommended to delay the compilation of this method. |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
359 |
void NonTieredCompPolicy::delay_compilation(Method* method) { |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
360 |
MethodCounters* mcs = method->method_counters(); |
17001
1ce8485b1907
8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents:
17000
diff
changeset
|
361 |
if (mcs != NULL) { |
1ce8485b1907
8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents:
17000
diff
changeset
|
362 |
mcs->invocation_counter()->decay(); |
1ce8485b1907
8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents:
17000
diff
changeset
|
363 |
mcs->backedge_counter()->decay(); |
1ce8485b1907
8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents:
17000
diff
changeset
|
364 |
} |
6453 | 365 |
} |
366 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
367 |
void NonTieredCompPolicy::disable_compilation(Method* method) { |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
368 |
MethodCounters* mcs = method->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
369 |
if (mcs != NULL) { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
370 |
mcs->invocation_counter()->set_state(InvocationCounter::wait_for_nothing); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
371 |
mcs->backedge_counter()->set_state(InvocationCounter::wait_for_nothing); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
372 |
} |
6453 | 373 |
} |
374 |
||
375 |
CompileTask* NonTieredCompPolicy::select_task(CompileQueue* compile_queue) { |
|
35547
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
376 |
return select_task_helper(compile_queue); |
6453 | 377 |
} |
378 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
379 |
bool NonTieredCompPolicy::is_mature(Method* method) { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
380 |
MethodData* mdo = method->method_data(); |
6453 | 381 |
assert(mdo != NULL, "Should be"); |
382 |
uint current = mdo->mileage_of(method); |
|
383 |
uint initial = mdo->creation_mileage(); |
|
384 |
if (current < initial) |
|
385 |
return true; // some sort of overflow |
|
386 |
uint target; |
|
387 |
if (ProfileMaturityPercentage <= 0) |
|
388 |
target = (uint) -ProfileMaturityPercentage; // absolute value |
|
389 |
else |
|
390 |
target = (uint)( (ProfileMaturityPercentage * CompileThreshold) / 100 ); |
|
391 |
return (current >= initial + target); |
|
392 |
} |
|
393 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
394 |
nmethod* NonTieredCompPolicy::event(const methodHandle& method, const methodHandle& inlinee, int branch_bci, |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
37248
diff
changeset
|
395 |
int bci, CompLevel comp_level, CompiledMethod* nm, JavaThread* thread) { |
6453 | 396 |
assert(comp_level == CompLevel_none, "This should be only called from the interpreter"); |
397 |
NOT_PRODUCT(trace_frequency_counter_overflow(method, branch_bci, bci)); |
|
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
398 |
if (JvmtiExport::can_post_interpreter_events() && thread->is_interp_only_mode()) { |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
399 |
// If certain JVMTI events (e.g. frame pop event) are requested then the |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
400 |
// thread is forced to remain in interpreted code. This is |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
401 |
// implemented partly by a check in the run_compiled_code |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
402 |
// section of the interpreter whether we should skip running |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
403 |
// compiled code, and partly by skipping OSR compiles for |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
404 |
// interpreted-only threads. |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
405 |
if (bci != InvocationEntryBci) { |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
406 |
reset_counter_for_back_branch_event(method); |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
407 |
return NULL; |
6453 | 408 |
} |
409 |
} |
|
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
410 |
if (CompileTheWorld || ReplayCompiles) { |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
411 |
// Don't trigger other compiles in testing mode |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
412 |
if (bci == InvocationEntryBci) { |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
413 |
reset_counter_for_invocation_event(method); |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
414 |
} else { |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
415 |
reset_counter_for_back_branch_event(method); |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
416 |
} |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
417 |
return NULL; |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
418 |
} |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
419 |
|
6453 | 420 |
if (bci == InvocationEntryBci) { |
421 |
// when code cache is full, compilation gets switched off, UseCompiler |
|
422 |
// is set to false |
|
423 |
if (!method->has_compiled_code() && UseCompiler) { |
|
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
424 |
method_invocation_event(method, thread); |
6453 | 425 |
} else { |
426 |
// Force counter overflow on method entry, even if no compilation |
|
427 |
// happened. (The method_invocation_event call does this also.) |
|
428 |
reset_counter_for_invocation_event(method); |
|
429 |
} |
|
430 |
// compilation at an invocation overflow no longer goes and retries test for |
|
431 |
// compiled method. We always run the loser of the race as interpreted. |
|
432 |
// so return NULL |
|
433 |
return NULL; |
|
434 |
} else { |
|
435 |
// counter overflow in a loop => try to do on-stack-replacement |
|
436 |
nmethod* osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true); |
|
437 |
NOT_PRODUCT(trace_osr_request(method, osr_nm, bci)); |
|
438 |
// when code cache is full, we should not compile any more... |
|
439 |
if (osr_nm == NULL && UseCompiler) { |
|
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
440 |
method_back_branch_event(method, bci, thread); |
6453 | 441 |
osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true); |
442 |
} |
|
443 |
if (osr_nm == NULL) { |
|
444 |
reset_counter_for_back_branch_event(method); |
|
445 |
return NULL; |
|
446 |
} |
|
447 |
return osr_nm; |
|
448 |
} |
|
449 |
return NULL; |
|
450 |
} |
|
451 |
||
452 |
#ifndef PRODUCT |
|
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
453 |
void NonTieredCompPolicy::trace_frequency_counter_overflow(const methodHandle& m, int branch_bci, int bci) { |
6453 | 454 |
if (TraceInvocationCounterOverflow) { |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
455 |
MethodCounters* mcs = m->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
456 |
assert(mcs != NULL, "MethodCounters cannot be NULL for profiling"); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
457 |
InvocationCounter* ic = mcs->invocation_counter(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
458 |
InvocationCounter* bc = mcs->backedge_counter(); |
6453 | 459 |
ResourceMark rm; |
33604
ad1cd9269bd4
8139116: Fixes for warning "format not a string literal"
goetz
parents:
33593
diff
changeset
|
460 |
if (bci == InvocationEntryBci) { |
ad1cd9269bd4
8139116: Fixes for warning "format not a string literal"
goetz
parents:
33593
diff
changeset
|
461 |
tty->print("comp-policy cntr ovfl @ %d in entry of ", bci); |
ad1cd9269bd4
8139116: Fixes for warning "format not a string literal"
goetz
parents:
33593
diff
changeset
|
462 |
} else { |
ad1cd9269bd4
8139116: Fixes for warning "format not a string literal"
goetz
parents:
33593
diff
changeset
|
463 |
tty->print("comp-policy cntr ovfl @ %d in loop of ", bci); |
ad1cd9269bd4
8139116: Fixes for warning "format not a string literal"
goetz
parents:
33593
diff
changeset
|
464 |
} |
6453 | 465 |
m->print_value(); |
466 |
tty->cr(); |
|
467 |
ic->print(); |
|
468 |
bc->print(); |
|
469 |
if (ProfileInterpreter) { |
|
470 |
if (bci != InvocationEntryBci) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
471 |
MethodData* mdo = m->method_data(); |
6453 | 472 |
if (mdo != NULL) { |
473 |
int count = mdo->bci_to_data(branch_bci)->as_JumpData()->taken(); |
|
474 |
tty->print_cr("back branch count = %d", count); |
|
475 |
} |
|
476 |
} |
|
477 |
} |
|
478 |
} |
|
479 |
} |
|
480 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
481 |
void NonTieredCompPolicy::trace_osr_request(const methodHandle& method, nmethod* osr, int bci) { |
6453 | 482 |
if (TraceOnStackReplacement) { |
483 |
ResourceMark rm; |
|
484 |
tty->print(osr != NULL ? "Reused OSR entry for " : "Requesting OSR entry for "); |
|
485 |
method->print_short_name(tty); |
|
486 |
tty->print_cr(" at bci %d", bci); |
|
487 |
} |
|
488 |
} |
|
489 |
#endif // !PRODUCT |
|
490 |
||
1 | 491 |
// SimpleCompPolicy - compile current method |
492 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
493 |
void SimpleCompPolicy::method_invocation_event(const methodHandle& m, JavaThread* thread) { |
13891
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
494 |
const int comp_level = CompLevel_highest_tier; |
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
495 |
const int hot_count = m->invocation_count(); |
1 | 496 |
reset_counter_for_invocation_event(m); |
497 |
||
17126
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
498 |
if (is_compilation_enabled() && can_be_compiled(m, comp_level)) { |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
37248
diff
changeset
|
499 |
CompiledMethod* nm = m->code(); |
1 | 500 |
if (nm == NULL ) { |
38218 | 501 |
CompileBroker::compile_method(m, InvocationEntryBci, comp_level, m, hot_count, CompileTask::Reason_InvocationCount, thread); |
1 | 502 |
} |
503 |
} |
|
504 |
} |
|
505 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
506 |
void SimpleCompPolicy::method_back_branch_event(const methodHandle& m, int bci, JavaThread* thread) { |
13891
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
507 |
const int comp_level = CompLevel_highest_tier; |
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
508 |
const int hot_count = m->backedge_count(); |
1 | 509 |
|
19332 | 510 |
if (is_compilation_enabled() && can_be_osr_compiled(m, comp_level)) { |
38218 | 511 |
CompileBroker::compile_method(m, bci, comp_level, m, hot_count, CompileTask::Reason_BackedgeCount, thread); |
13891
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
512 |
NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, comp_level, true));) |
1 | 513 |
} |
514 |
} |
|
515 |
// StackWalkCompPolicy - walk up stack to find a suitable method to compile |
|
516 |
||
517 |
#ifdef COMPILER2 |
|
518 |
const char* StackWalkCompPolicy::_msg = NULL; |
|
519 |
||
520 |
||
521 |
// Consider m for compilation |
|
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
522 |
void StackWalkCompPolicy::method_invocation_event(const methodHandle& m, JavaThread* thread) { |
13891
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
523 |
const int comp_level = CompLevel_highest_tier; |
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
524 |
const int hot_count = m->invocation_count(); |
1 | 525 |
reset_counter_for_invocation_event(m); |
526 |
||
17126
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
527 |
if (is_compilation_enabled() && m->code() == NULL && can_be_compiled(m, comp_level)) { |
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
528 |
ResourceMark rm(thread); |
1 | 529 |
frame fr = thread->last_frame(); |
530 |
assert(fr.is_interpreted_frame(), "must be interpreted"); |
|
531 |
assert(fr.interpreter_frame_method() == m(), "bad method"); |
|
532 |
||
533 |
if (TraceCompilationPolicy) { |
|
534 |
tty->print("method invocation trigger: "); |
|
535 |
m->print_short_name(tty); |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
24013
diff
changeset
|
536 |
tty->print(" ( interpreted " INTPTR_FORMAT ", size=%d ) ", p2i((address)m()), m->code_size()); |
1 | 537 |
} |
538 |
RegisterMap reg_map(thread, false); |
|
539 |
javaVFrame* triggerVF = thread->last_java_vframe(®_map); |
|
540 |
// triggerVF is the frame that triggered its counter |
|
31521
f57b2ce43484
8079775: Java 9-fastdebug ia32 Error: Unimplemented with "-XX:CompilationPolicyChoice=1 -XX:-TieredCompilation" options
iveresov
parents:
24424
diff
changeset
|
541 |
RFrame* first = new InterpretedRFrame(triggerVF->fr(), thread, m()); |
1 | 542 |
|
543 |
if (first->top_method()->code() != NULL) { |
|
544 |
// called obsolete method/nmethod -- no need to recompile |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
24013
diff
changeset
|
545 |
if (TraceCompilationPolicy) tty->print_cr(" --> " INTPTR_FORMAT, p2i(first->top_method()->code())); |
1 | 546 |
} else { |
547 |
if (TimeCompilationPolicy) accumulated_time()->start(); |
|
548 |
GrowableArray<RFrame*>* stack = new GrowableArray<RFrame*>(50); |
|
549 |
stack->push(first); |
|
550 |
RFrame* top = findTopInlinableFrame(stack); |
|
551 |
if (TimeCompilationPolicy) accumulated_time()->stop(); |
|
552 |
assert(top != NULL, "findTopInlinableFrame returned null"); |
|
553 |
if (TraceCompilationPolicy) top->print(); |
|
13891
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
554 |
CompileBroker::compile_method(top->top_method(), InvocationEntryBci, comp_level, |
38218 | 555 |
m, hot_count, CompileTask::Reason_InvocationCount, thread); |
1 | 556 |
} |
557 |
} |
|
558 |
} |
|
559 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
560 |
void StackWalkCompPolicy::method_back_branch_event(const methodHandle& m, int bci, JavaThread* thread) { |
13891
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
561 |
const int comp_level = CompLevel_highest_tier; |
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
562 |
const int hot_count = m->backedge_count(); |
1 | 563 |
|
19332 | 564 |
if (is_compilation_enabled() && can_be_osr_compiled(m, comp_level)) { |
38218 | 565 |
CompileBroker::compile_method(m, bci, comp_level, m, hot_count, CompileTask::Reason_BackedgeCount, thread); |
13891
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
566 |
NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, comp_level, true));) |
1 | 567 |
} |
568 |
} |
|
569 |
||
570 |
RFrame* StackWalkCompPolicy::findTopInlinableFrame(GrowableArray<RFrame*>* stack) { |
|
571 |
// go up the stack until finding a frame that (probably) won't be inlined |
|
572 |
// into its caller |
|
573 |
RFrame* current = stack->at(0); // current choice for stopping |
|
574 |
assert( current && !current->is_compiled(), "" ); |
|
575 |
const char* msg = NULL; |
|
576 |
||
577 |
while (1) { |
|
578 |
||
579 |
// before going up the stack further, check if doing so would get us into |
|
580 |
// compiled code |
|
581 |
RFrame* next = senderOf(current, stack); |
|
582 |
if( !next ) // No next frame up the stack? |
|
583 |
break; // Then compile with current frame |
|
584 |
||
31521
f57b2ce43484
8079775: Java 9-fastdebug ia32 Error: Unimplemented with "-XX:CompilationPolicyChoice=1 -XX:-TieredCompilation" options
iveresov
parents:
24424
diff
changeset
|
585 |
Method* m = current->top_method(); |
f57b2ce43484
8079775: Java 9-fastdebug ia32 Error: Unimplemented with "-XX:CompilationPolicyChoice=1 -XX:-TieredCompilation" options
iveresov
parents:
24424
diff
changeset
|
586 |
Method* next_m = next->top_method(); |
1 | 587 |
|
588 |
if (TraceCompilationPolicy && Verbose) { |
|
589 |
tty->print("[caller: "); |
|
590 |
next_m->print_short_name(tty); |
|
591 |
tty->print("] "); |
|
592 |
} |
|
593 |
||
594 |
if( !Inline ) { // Inlining turned off |
|
595 |
msg = "Inlining turned off"; |
|
596 |
break; |
|
597 |
} |
|
598 |
if (next_m->is_not_compilable()) { // Did fail to compile this before/ |
|
599 |
msg = "caller not compilable"; |
|
600 |
break; |
|
601 |
} |
|
602 |
if (next->num() > MaxRecompilationSearchLength) { |
|
603 |
// don't go up too high when searching for recompilees |
|
604 |
msg = "don't go up any further: > MaxRecompilationSearchLength"; |
|
605 |
break; |
|
606 |
} |
|
607 |
if (next->distance() > MaxInterpretedSearchLength) { |
|
608 |
// don't go up too high when searching for recompilees |
|
609 |
msg = "don't go up any further: next > MaxInterpretedSearchLength"; |
|
610 |
break; |
|
611 |
} |
|
612 |
// Compiled frame above already decided not to inline; |
|
613 |
// do not recompile him. |
|
614 |
if (next->is_compiled()) { |
|
615 |
msg = "not going up into optimized code"; |
|
616 |
break; |
|
617 |
} |
|
618 |
||
619 |
// Interpreted frame above us was already compiled. Do not force |
|
620 |
// a recompile, although if the frame above us runs long enough an |
|
621 |
// OSR might still happen. |
|
622 |
if( current->is_interpreted() && next_m->has_compiled_code() ) { |
|
623 |
msg = "not going up -- already compiled caller"; |
|
624 |
break; |
|
625 |
} |
|
626 |
||
627 |
// Compute how frequent this call site is. We have current method 'm'. |
|
628 |
// We know next method 'next_m' is interpreted. Find the call site and |
|
629 |
// check the various invocation counts. |
|
630 |
int invcnt = 0; // Caller counts |
|
631 |
if (ProfileInterpreter) { |
|
632 |
invcnt = next_m->interpreter_invocation_count(); |
|
633 |
} |
|
634 |
int cnt = 0; // Call site counts |
|
635 |
if (ProfileInterpreter && next_m->method_data() != NULL) { |
|
636 |
ResourceMark rm; |
|
637 |
int bci = next->top_vframe()->bci(); |
|
638 |
ProfileData* data = next_m->method_data()->bci_to_data(bci); |
|
639 |
if (data != NULL && data->is_CounterData()) |
|
640 |
cnt = data->as_CounterData()->count(); |
|
641 |
} |
|
642 |
||
643 |
// Caller counts / call-site counts; i.e. is this call site |
|
644 |
// a hot call site for method next_m? |
|
645 |
int freq = (invcnt) ? cnt/invcnt : cnt; |
|
646 |
||
647 |
// Check size and frequency limits |
|
648 |
if ((msg = shouldInline(m, freq, cnt)) != NULL) { |
|
649 |
break; |
|
650 |
} |
|
651 |
// Check inlining negative tests |
|
652 |
if ((msg = shouldNotInline(m)) != NULL) { |
|
653 |
break; |
|
654 |
} |
|
655 |
||
656 |
||
657 |
// If the caller method is too big or something then we do not want to |
|
658 |
// compile it just to inline a method |
|
17126
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
659 |
if (!can_be_compiled(next_m, CompLevel_any)) { |
1 | 660 |
msg = "caller cannot be compiled"; |
661 |
break; |
|
662 |
} |
|
663 |
||
664 |
if( next_m->name() == vmSymbols::class_initializer_name() ) { |
|
665 |
msg = "do not compile class initializer (OSR ok)"; |
|
666 |
break; |
|
667 |
} |
|
668 |
||
669 |
if (TraceCompilationPolicy && Verbose) { |
|
670 |
tty->print("\n\t check caller: "); |
|
671 |
next_m->print_short_name(tty); |
|
31521
f57b2ce43484
8079775: Java 9-fastdebug ia32 Error: Unimplemented with "-XX:CompilationPolicyChoice=1 -XX:-TieredCompilation" options
iveresov
parents:
24424
diff
changeset
|
672 |
tty->print(" ( interpreted " INTPTR_FORMAT ", size=%d ) ", p2i((address)next_m), next_m->code_size()); |
1 | 673 |
} |
674 |
||
675 |
current = next; |
|
676 |
} |
|
677 |
||
678 |
assert( !current || !current->is_compiled(), "" ); |
|
679 |
||
680 |
if (TraceCompilationPolicy && msg) tty->print("(%s)\n", msg); |
|
681 |
||
682 |
return current; |
|
683 |
} |
|
684 |
||
685 |
RFrame* StackWalkCompPolicy::senderOf(RFrame* rf, GrowableArray<RFrame*>* stack) { |
|
686 |
RFrame* sender = rf->caller(); |
|
687 |
if (sender && sender->num() == stack->length()) stack->push(sender); |
|
688 |
return sender; |
|
689 |
} |
|
690 |
||
691 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
692 |
const char* StackWalkCompPolicy::shouldInline(const methodHandle& m, float freq, int cnt) { |
1 | 693 |
// Allows targeted inlining |
694 |
// positive filter: should send be inlined? returns NULL (--> yes) |
|
695 |
// or rejection msg |
|
696 |
int max_size = MaxInlineSize; |
|
697 |
int cost = m->code_size(); |
|
698 |
||
699 |
// Check for too many throws (and not too huge) |
|
700 |
if (m->interpreter_throwout_count() > InlineThrowCount && cost < InlineThrowMaxSize ) { |
|
701 |
return NULL; |
|
702 |
} |
|
703 |
||
704 |
// bump the max size if the call is frequent |
|
705 |
if ((freq >= InlineFrequencyRatio) || (cnt >= InlineFrequencyCount)) { |
|
706 |
if (TraceFrequencyInlining) { |
|
707 |
tty->print("(Inlined frequent method)\n"); |
|
708 |
m->print(); |
|
709 |
} |
|
710 |
max_size = FreqInlineSize; |
|
711 |
} |
|
712 |
if (cost > max_size) { |
|
713 |
return (_msg = "too big"); |
|
714 |
} |
|
715 |
return NULL; |
|
716 |
} |
|
717 |
||
718 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
719 |
const char* StackWalkCompPolicy::shouldNotInline(const methodHandle& m) { |
1 | 720 |
// negative filter: should send NOT be inlined? returns NULL (--> inline) or rejection msg |
721 |
if (m->is_abstract()) return (_msg = "abstract method"); |
|
722 |
// note: we allow ik->is_abstract() |
|
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
13891
diff
changeset
|
723 |
if (!m->method_holder()->is_initialized()) return (_msg = "method holder not initialized"); |
1 | 724 |
if (m->is_native()) return (_msg = "native method"); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
37248
diff
changeset
|
725 |
CompiledMethod* m_code = m->code(); |
6418 | 726 |
if (m_code != NULL && m_code->code_size() > InlineSmallCode) |
1 | 727 |
return (_msg = "already compiled into a big method"); |
728 |
||
729 |
// use frequency-based objections only for non-trivial methods |
|
730 |
if (m->code_size() <= MaxTrivialSize) return NULL; |
|
731 |
if (UseInterpreter) { // don't use counts with -Xcomp |
|
732 |
if ((m->code() == NULL) && m->was_never_executed()) return (_msg = "never executed"); |
|
733 |
if (!m->was_executed_more_than(MIN2(MinInliningThreshold, CompileThreshold >> 1))) return (_msg = "executed < MinInliningThreshold times"); |
|
734 |
} |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
735 |
if (Method::has_unloaded_classes_in_signature(m, JavaThread::current())) return (_msg = "unloaded signature classes"); |
1 | 736 |
|
737 |
return NULL; |
|
738 |
} |
|
739 |
||
740 |
||
741 |
||
742 |
#endif // COMPILER2 |