author | xliu |
Wed, 09 Oct 2019 12:43:32 -0700 | |
changeset 58545 | 725244418646 |
parent 58398 | src/hotspot/share/runtime/compilationPolicy.cpp@ff559f979c4b |
child 58679 | 9c3209ff7550 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
53193
184c51e48260
8216262: Remove develop flag DelayCompilationDuringStartup
redestad
parents:
52934
diff
changeset
|
2 |
* Copyright (c) 2000, 2019, 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" |
51959
db0c3952de52
8209645: Split ClassLoaderData and ClassLoaderDataGraph into separate files
coleenp
parents:
51369
diff
changeset
|
26 |
#include "classfile/classLoaderDataGraph.inline.hpp" |
7397 | 27 |
#include "code/compiledIC.hpp" |
28 |
#include "code/nmethod.hpp" |
|
29 |
#include "code/scopeDesc.hpp" |
|
58545
725244418646
8165056: move JIT Compiler related files from runtime/ to compiler/ directory
xliu
parents:
58398
diff
changeset
|
30 |
#include "compiler/compilationPolicy.hpp" |
725244418646
8165056: move JIT Compiler related files from runtime/ to compiler/ directory
xliu
parents:
58398
diff
changeset
|
31 |
#include "compiler/tieredThresholdPolicy.hpp" |
7397 | 32 |
#include "interpreter/interpreter.hpp" |
37248 | 33 |
#include "memory/resourceArea.hpp" |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
34 |
#include "oops/methodData.hpp" |
49340
4e82736053ae
8191102: Incorrect include file use in classLoader.hpp
hseigel
parents:
48884
diff
changeset
|
35 |
#include "oops/method.inline.hpp" |
7397 | 36 |
#include "oops/oop.inline.hpp" |
37 |
#include "prims/nativeLookup.hpp" |
|
38 |
#include "runtime/frame.hpp" |
|
39 |
#include "runtime/handles.inline.hpp" |
|
40 |
#include "runtime/stubRoutines.hpp" |
|
41 |
#include "runtime/thread.hpp" |
|
42 |
#include "runtime/vframe.hpp" |
|
52877
9e041366c764
8214850: Rename vm_operations.?pp files to vmOperations.?pp files
tschatzl
parents:
52675
diff
changeset
|
43 |
#include "runtime/vmOperations.hpp" |
7397 | 44 |
#include "utilities/events.hpp" |
45 |
#include "utilities/globalDefinitions.hpp" |
|
1 | 46 |
|
52325
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
47 |
#ifdef COMPILER1 |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
48 |
#include "c1/c1_Compiler.hpp" |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
49 |
#endif |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
50 |
#ifdef COMPILER2 |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
51 |
#include "opto/c2compiler.hpp" |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
52 |
#endif |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
53 |
|
1 | 54 |
CompilationPolicy* CompilationPolicy::_policy; |
55 |
||
56 |
// Determine compilation policy based on command line argument |
|
57 |
void compilationPolicy_init() { |
|
58367 | 58 |
#ifdef TIERED |
59 |
if (TieredCompilation) { |
|
60 |
CompilationPolicy::set_policy(new TieredThresholdPolicy()); |
|
61 |
} else { |
|
1 | 62 |
CompilationPolicy::set_policy(new SimpleCompPolicy()); |
58367 | 63 |
} |
64 |
#else |
|
65 |
CompilationPolicy::set_policy(new SimpleCompPolicy()); |
|
66 |
#endif |
|
1 | 67 |
|
6453 | 68 |
CompilationPolicy::policy()->initialize(); |
1 | 69 |
} |
70 |
||
71 |
// Returns true if m must be compiled before executing it |
|
72 |
// This is intended to force compiles for methods (usually for |
|
73 |
// debugging) that would otherwise be interpreted for some reason. |
|
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
43455
diff
changeset
|
74 |
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
|
75 |
// Don't allow Xcomp to cause compiles in replay mode |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
76 |
if (ReplayCompiles) return false; |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
77 |
|
1 | 78 |
if (m->has_compiled_code()) return false; // already compiled |
6453 | 79 |
if (!can_be_compiled(m, comp_level)) return false; |
1 | 80 |
|
81 |
return !UseInterpreter || // must compile all methods |
|
4750 | 82 |
(UseCompiler && AlwaysCompileLoopMethods && m->has_loops() && CompileBroker::should_compile_new_jobs()); // eagerly compile loop methods |
1 | 83 |
} |
84 |
||
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
43455
diff
changeset
|
85 |
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
|
86 |
if (must_be_compiled(selected_method)) { |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
87 |
// 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
|
88 |
|
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
89 |
// 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
|
90 |
// 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
|
91 |
// 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
|
92 |
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
|
93 |
// 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
|
94 |
return; |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
95 |
} |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
96 |
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
|
97 |
// '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
|
98 |
// 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
|
99 |
// 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
|
100 |
// 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
|
101 |
// in CompileBroker::compile_method. |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
102 |
// 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
|
103 |
// even before classes are initialized. |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
104 |
return; |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
105 |
} |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
106 |
CompileBroker::compile_method(selected_method, InvocationEntryBci, |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
107 |
CompilationPolicy::policy()->initial_compile_level(), |
38218 | 108 |
methodHandle(), 0, CompileTask::Reason_MustBeCompiled, CHECK); |
38139
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
109 |
} |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
110 |
} |
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
38133
diff
changeset
|
111 |
|
1 | 112 |
// Returns true if m is allowed to be compiled |
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
43455
diff
changeset
|
113 |
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
|
114 |
// allow any levels for WhiteBox |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
115 |
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
|
116 |
|
1 | 117 |
if (m->is_abstract()) return false; |
118 |
if (DontCompileHugeMethods && m->code_size() > HugeMethodLimit) return false; |
|
119 |
||
4645
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
120 |
// 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
|
121 |
// 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
|
122 |
// 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
|
123 |
// 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
|
124 |
// 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
|
125 |
// modes. |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
126 |
if (!AbstractInterpreter::can_be_compiled(m)) { |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
127 |
return false; |
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
128 |
} |
6453 | 129 |
if (comp_level == CompLevel_all) { |
17126
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
130 |
if (TieredCompilation) { |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
131 |
// 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
|
132 |
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
|
133 |
} else { |
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
134 |
// 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
|
135 |
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
|
136 |
} |
16689
efce070b8d42
8007288: Additional WB API for compiler's testing
iignatyev
parents:
14477
diff
changeset
|
137 |
} else if (is_compile(comp_level)) { |
6453 | 138 |
return !m->is_not_compilable(comp_level); |
139 |
} |
|
16689
efce070b8d42
8007288: Additional WB API for compiler's testing
iignatyev
parents:
14477
diff
changeset
|
140 |
return false; |
6453 | 141 |
} |
4645
0c5f5b94e93a
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
1
diff
changeset
|
142 |
|
19332 | 143 |
// 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
|
144 |
bool CompilationPolicy::can_be_osr_compiled(const methodHandle& m, int comp_level) { |
19332 | 145 |
bool result = false; |
146 |
if (comp_level == CompLevel_all) { |
|
147 |
if (TieredCompilation) { |
|
148 |
// enough to be osr compilable at any level for tiered |
|
149 |
result = !m->is_not_osr_compilable(CompLevel_simple) || !m->is_not_osr_compilable(CompLevel_full_optimization); |
|
150 |
} else { |
|
151 |
// must be osr compilable at available level for non-tiered |
|
152 |
result = !m->is_not_osr_compilable(CompLevel_highest_tier); |
|
153 |
} |
|
154 |
} else if (is_compile(comp_level)) { |
|
155 |
result = !m->is_not_osr_compilable(comp_level); |
|
156 |
} |
|
157 |
return (result && can_be_compiled(m, comp_level)); |
|
158 |
} |
|
159 |
||
6453 | 160 |
bool CompilationPolicy::is_compilation_enabled() { |
161 |
// NOTE: CompileBroker::should_compile_new_jobs() checks for UseCompiler |
|
53193
184c51e48260
8216262: Remove develop flag DelayCompilationDuringStartup
redestad
parents:
52934
diff
changeset
|
162 |
return CompileBroker::should_compile_new_jobs(); |
1 | 163 |
} |
164 |
||
35547
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
165 |
CompileTask* CompilationPolicy::select_task_helper(CompileQueue* compile_queue) { |
54015
cd701366fcf8
8163511: Allocation of compile task fails with assert: "Leaking compilation tasks?"
thartmann
parents:
53194
diff
changeset
|
166 |
// Remove unloaded methods from the queue |
cd701366fcf8
8163511: Allocation of compile task fails with assert: "Leaking compilation tasks?"
thartmann
parents:
53194
diff
changeset
|
167 |
for (CompileTask* task = compile_queue->first(); task != NULL; ) { |
cd701366fcf8
8163511: Allocation of compile task fails with assert: "Leaking compilation tasks?"
thartmann
parents:
53194
diff
changeset
|
168 |
CompileTask* next = task->next(); |
cd701366fcf8
8163511: Allocation of compile task fails with assert: "Leaking compilation tasks?"
thartmann
parents:
53194
diff
changeset
|
169 |
if (task->is_unloaded()) { |
cd701366fcf8
8163511: Allocation of compile task fails with assert: "Leaking compilation tasks?"
thartmann
parents:
53194
diff
changeset
|
170 |
compile_queue->remove_and_mark_stale(task); |
cd701366fcf8
8163511: Allocation of compile task fails with assert: "Leaking compilation tasks?"
thartmann
parents:
53194
diff
changeset
|
171 |
} |
cd701366fcf8
8163511: Allocation of compile task fails with assert: "Leaking compilation tasks?"
thartmann
parents:
53194
diff
changeset
|
172 |
task = next; |
cd701366fcf8
8163511: Allocation of compile task fails with assert: "Leaking compilation tasks?"
thartmann
parents:
53194
diff
changeset
|
173 |
} |
35547
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
174 |
#if INCLUDE_JVMCI |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
175 |
if (UseJVMCICompiler && !BackgroundCompilation) { |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
176 |
/* |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
177 |
* In blocking compilation mode, the CompileBroker will make |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
178 |
* compilations submitted by a JVMCI compiler thread non-blocking. These |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
179 |
* compilations should be scheduled after all blocking compilations |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
180 |
* to service non-compiler related compilations sooner and reduce the |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
181 |
* chance of such compilations timing out. |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
182 |
*/ |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
183 |
for (CompileTask* task = compile_queue->first(); task != NULL; task = task->next()) { |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
184 |
if (task->is_blocking()) { |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
185 |
return task; |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
186 |
} |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
187 |
} |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
188 |
} |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
189 |
#endif |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
190 |
return compile_queue->first(); |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
191 |
} |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
192 |
|
1 | 193 |
#ifndef PRODUCT |
58398 | 194 |
void SimpleCompPolicy::trace_osr_completion(nmethod* osr_nm) { |
1 | 195 |
if (TraceOnStackReplacement) { |
196 |
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
|
197 |
else tty->print_cr("nmethod " INTPTR_FORMAT, p2i(osr_nm)); |
1 | 198 |
} |
199 |
} |
|
200 |
#endif // !PRODUCT |
|
201 |
||
58398 | 202 |
void SimpleCompPolicy::initialize() { |
6453 | 203 |
// Setup the compiler thread numbers |
204 |
if (CICompilerCountPerCPU) { |
|
205 |
// Example: if CICompilerCountPerCPU is true, then we get |
|
206 |
// max(log2(8)-1,1) = 2 compiler threads on an 8-way machine. |
|
207 |
// May help big-app startup time. |
|
52675 | 208 |
_compiler_count = MAX2(log2_int(os::active_processor_count())-1,1); |
52325
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
209 |
// Make sure there is enough space in the code cache to hold all the compiler buffers |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
210 |
size_t buffer_size = 1; |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
211 |
#ifdef COMPILER1 |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
212 |
buffer_size = is_client_compilation_mode_vm() ? Compiler::code_buffer_size() : buffer_size; |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
213 |
#endif |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
214 |
#ifdef COMPILER2 |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
215 |
buffer_size = is_server_compilation_mode_vm() ? C2Compiler::initial_code_buffer_size() : buffer_size; |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
216 |
#endif |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
217 |
int max_count = (ReservedCodeCacheSize - (CodeCacheMinimumUseSpace DEBUG_ONLY(* 3))) / (int)buffer_size; |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
218 |
if (_compiler_count > max_count) { |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
219 |
// Lower the compiler count such that all buffers fit into the code cache |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
220 |
_compiler_count = MAX2(max_count, 1); |
0451e0a2f1f5
8177899: Tests fail due to code cache exhaustion on machines with many cores
thartmann
parents:
51959
diff
changeset
|
221 |
} |
54983
81becad91321
8224203: Remove need to specify type when using FLAG_SET macros
stefank
parents:
54015
diff
changeset
|
222 |
FLAG_SET_ERGO(CICompilerCount, _compiler_count); |
6453 | 223 |
} else { |
224 |
_compiler_count = CICompilerCount; |
|
225 |
} |
|
226 |
} |
|
227 |
||
6747
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
228 |
// 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
|
229 |
// compiler_count() behaves the following way: |
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
230 |
// - 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
|
231 |
// 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
|
232 |
// 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
|
233 |
// - 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
|
234 |
// - 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
|
235 |
// - if neither is defined - always return zero. |
58398 | 236 |
int SimpleCompPolicy::compiler_count(CompLevel comp_level) { |
6747
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
237 |
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
|
238 |
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
|
239 |
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
|
240 |
return _compiler_count; |
ab166511728e
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
6453
diff
changeset
|
241 |
} |
6453 | 242 |
return 0; |
243 |
} |
|
244 |
||
58398 | 245 |
void SimpleCompPolicy::reset_counter_for_invocation_event(const methodHandle& m) { |
1 | 246 |
// Make sure invocation and backedge counter doesn't overflow again right away |
247 |
// as would be the case for native methods. |
|
248 |
||
249 |
// BUT also make sure the method doesn't look like it was never executed. |
|
250 |
// 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
|
251 |
MethodCounters* mcs = m->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
252 |
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
|
253 |
mcs->invocation_counter()->set_carry(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
254 |
mcs->backedge_counter()->set_carry(); |
1 | 255 |
|
256 |
assert(!m->was_never_executed(), "don't reset to 0 -- could be mistaken for never-executed"); |
|
257 |
} |
|
258 |
||
58398 | 259 |
void SimpleCompPolicy::reset_counter_for_back_branch_event(const methodHandle& m) { |
22551 | 260 |
// Delay next back-branch event but pump up invocation counter to trigger |
1 | 261 |
// whole method compilation. |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
262 |
MethodCounters* mcs = m->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
263 |
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
|
264 |
InvocationCounter* i = mcs->invocation_counter(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
265 |
InvocationCounter* b = mcs->backedge_counter(); |
1 | 266 |
|
267 |
// Don't set invocation_counter's value too low otherwise the method will |
|
268 |
// look like immature (ic < ~5300) which prevents the inlining based on |
|
269 |
// the type profiling. |
|
270 |
i->set(i->state(), CompileThreshold); |
|
271 |
// Don't reset counter too low - it is used to check if OSR method is ready. |
|
272 |
b->set(b->state(), CompileThreshold / 2); |
|
273 |
} |
|
274 |
||
6453 | 275 |
// |
276 |
// CounterDecay |
|
277 |
// |
|
22551 | 278 |
// Iterates through invocation counters and decrements them. This |
6453 | 279 |
// is done at each safepoint. |
280 |
// |
|
281 |
class CounterDecay : public AllStatic { |
|
282 |
static jlong _last_timestamp; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
283 |
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
|
284 |
MethodCounters* mcs = m->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
285 |
if (mcs != NULL) { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
286 |
mcs->invocation_counter()->decay(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
287 |
} |
6453 | 288 |
} |
289 |
public: |
|
290 |
static void decay(); |
|
291 |
static bool is_decay_needed() { |
|
292 |
return (os::javaTimeMillis() - _last_timestamp) > CounterDecayMinIntervalLength; |
|
293 |
} |
|
294 |
}; |
|
295 |
||
296 |
jlong CounterDecay::_last_timestamp = 0; |
|
297 |
||
298 |
void CounterDecay::decay() { |
|
299 |
_last_timestamp = os::javaTimeMillis(); |
|
300 |
||
301 |
// This operation is going to be performed only at the end of a safepoint |
|
302 |
// and hence GC's will not be going on, all Java mutators are suspended |
|
303 |
// at this point and hence SystemDictionary_lock is also not needed. |
|
304 |
assert(SafepointSynchronize::is_at_safepoint(), "can only be executed at a safepoint"); |
|
48884
7e17b00dc245
8196923: [REDO] NMT: Report array class count in NMT summary
zgu
parents:
48874
diff
changeset
|
305 |
size_t nclasses = ClassLoaderDataGraph::num_instance_classes(); |
7e17b00dc245
8196923: [REDO] NMT: Report array class count in NMT summary
zgu
parents:
48874
diff
changeset
|
306 |
size_t classes_per_tick = nclasses * (CounterDecayMinIntervalLength * 1e-3 / |
6453 | 307 |
CounterHalfLifeTime); |
48884
7e17b00dc245
8196923: [REDO] NMT: Report array class count in NMT summary
zgu
parents:
48874
diff
changeset
|
308 |
for (size_t i = 0; i < classes_per_tick; i++) { |
46729 | 309 |
InstanceKlass* k = ClassLoaderDataGraph::try_get_next_class(); |
310 |
if (k != NULL) { |
|
311 |
k->methods_do(do_method); |
|
6453 | 312 |
} |
313 |
} |
|
314 |
} |
|
315 |
||
316 |
// Called at the end of the safepoint |
|
58398 | 317 |
void SimpleCompPolicy::do_safepoint_work() { |
6453 | 318 |
if(UseCounterDecay && CounterDecay::is_decay_needed()) { |
319 |
CounterDecay::decay(); |
|
320 |
} |
|
321 |
} |
|
322 |
||
58398 | 323 |
void SimpleCompPolicy::reprofile(ScopeDesc* trap_scope, bool is_osr) { |
6453 | 324 |
ScopeDesc* sd = trap_scope; |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
325 |
MethodCounters* mcs; |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
326 |
InvocationCounter* c; |
6453 | 327 |
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
|
328 |
mcs = sd->method()->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
329 |
if (mcs != NULL) { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
330 |
// 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
|
331 |
mcs->invocation_counter()->reset(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
332 |
} |
6453 | 333 |
} |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
334 |
mcs = sd->method()->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
335 |
if (mcs != NULL) { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
336 |
c = mcs->invocation_counter(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
337 |
if (is_osr) { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
338 |
// 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
|
339 |
c->set(c->state(), CompileThreshold); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
340 |
} else { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
341 |
c->reset(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
342 |
} |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
343 |
mcs->backedge_counter()->reset(); |
6453 | 344 |
} |
345 |
} |
|
346 |
||
347 |
// This method can be called by any component of the runtime to notify the policy |
|
22551 | 348 |
// that it's recommended to delay the compilation of this method. |
58398 | 349 |
void SimpleCompPolicy::delay_compilation(Method* method) { |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
350 |
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
|
351 |
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
|
352 |
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
|
353 |
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
|
354 |
} |
6453 | 355 |
} |
356 |
||
58398 | 357 |
void SimpleCompPolicy::disable_compilation(Method* method) { |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
358 |
MethodCounters* mcs = method->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
359 |
if (mcs != NULL) { |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
360 |
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
|
361 |
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
|
362 |
} |
6453 | 363 |
} |
364 |
||
58398 | 365 |
CompileTask* SimpleCompPolicy::select_task(CompileQueue* compile_queue) { |
35547
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
33612
diff
changeset
|
366 |
return select_task_helper(compile_queue); |
6453 | 367 |
} |
368 |
||
58398 | 369 |
bool SimpleCompPolicy::is_mature(Method* method) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
370 |
MethodData* mdo = method->method_data(); |
6453 | 371 |
assert(mdo != NULL, "Should be"); |
372 |
uint current = mdo->mileage_of(method); |
|
373 |
uint initial = mdo->creation_mileage(); |
|
374 |
if (current < initial) |
|
375 |
return true; // some sort of overflow |
|
376 |
uint target; |
|
377 |
if (ProfileMaturityPercentage <= 0) |
|
378 |
target = (uint) -ProfileMaturityPercentage; // absolute value |
|
379 |
else |
|
380 |
target = (uint)( (ProfileMaturityPercentage * CompileThreshold) / 100 ); |
|
381 |
return (current >= initial + target); |
|
382 |
} |
|
383 |
||
58398 | 384 |
nmethod* SimpleCompPolicy::event(const methodHandle& method, const methodHandle& inlinee, int branch_bci, |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
37248
diff
changeset
|
385 |
int bci, CompLevel comp_level, CompiledMethod* nm, JavaThread* thread) { |
6453 | 386 |
assert(comp_level == CompLevel_none, "This should be only called from the interpreter"); |
387 |
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
|
388 |
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
|
389 |
// 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
|
390 |
// 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
|
391 |
// 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
|
392 |
// 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
|
393 |
// 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
|
394 |
// interpreted-only threads. |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
395 |
if (bci != InvocationEntryBci) { |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
396 |
reset_counter_for_back_branch_event(method); |
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
397 |
return NULL; |
6453 | 398 |
} |
399 |
} |
|
52934
8deeb7bba516
8214917: CTW testlibrary shouldn't ignore errors raised by the library itself
iignatyev
parents:
52877
diff
changeset
|
400 |
if (ReplayCompiles) { |
14477
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
401 |
// Don't trigger other compiles in testing mode |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
402 |
if (bci == InvocationEntryBci) { |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
403 |
reset_counter_for_invocation_event(method); |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
404 |
} else { |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
405 |
reset_counter_for_back_branch_event(method); |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
406 |
} |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
407 |
return NULL; |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
408 |
} |
95e66ea71f71
6830717: replay of compilations would help with debugging
minqi
parents:
14391
diff
changeset
|
409 |
|
6453 | 410 |
if (bci == InvocationEntryBci) { |
411 |
// when code cache is full, compilation gets switched off, UseCompiler |
|
412 |
// is set to false |
|
413 |
if (!method->has_compiled_code() && UseCompiler) { |
|
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
414 |
method_invocation_event(method, thread); |
6453 | 415 |
} else { |
416 |
// Force counter overflow on method entry, even if no compilation |
|
417 |
// happened. (The method_invocation_event call does this also.) |
|
418 |
reset_counter_for_invocation_event(method); |
|
419 |
} |
|
420 |
// compilation at an invocation overflow no longer goes and retries test for |
|
421 |
// compiled method. We always run the loser of the race as interpreted. |
|
422 |
// so return NULL |
|
423 |
return NULL; |
|
424 |
} else { |
|
425 |
// counter overflow in a loop => try to do on-stack-replacement |
|
426 |
nmethod* osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true); |
|
427 |
NOT_PRODUCT(trace_osr_request(method, osr_nm, bci)); |
|
428 |
// when code cache is full, we should not compile any more... |
|
429 |
if (osr_nm == NULL && UseCompiler) { |
|
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
10014
diff
changeset
|
430 |
method_back_branch_event(method, bci, thread); |
6453 | 431 |
osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true); |
432 |
} |
|
433 |
if (osr_nm == NULL) { |
|
434 |
reset_counter_for_back_branch_event(method); |
|
435 |
return NULL; |
|
436 |
} |
|
437 |
return osr_nm; |
|
438 |
} |
|
439 |
return NULL; |
|
440 |
} |
|
441 |
||
442 |
#ifndef PRODUCT |
|
58398 | 443 |
void SimpleCompPolicy::trace_frequency_counter_overflow(const methodHandle& m, int branch_bci, int bci) { |
6453 | 444 |
if (TraceInvocationCounterOverflow) { |
17000
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
445 |
MethodCounters* mcs = m->method_counters(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
446 |
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
|
447 |
InvocationCounter* ic = mcs->invocation_counter(); |
11bf92e571a2
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
14477
diff
changeset
|
448 |
InvocationCounter* bc = mcs->backedge_counter(); |
6453 | 449 |
ResourceMark rm; |
33604
ad1cd9269bd4
8139116: Fixes for warning "format not a string literal"
goetz
parents:
33593
diff
changeset
|
450 |
if (bci == InvocationEntryBci) { |
ad1cd9269bd4
8139116: Fixes for warning "format not a string literal"
goetz
parents:
33593
diff
changeset
|
451 |
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
|
452 |
} else { |
ad1cd9269bd4
8139116: Fixes for warning "format not a string literal"
goetz
parents:
33593
diff
changeset
|
453 |
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
|
454 |
} |
6453 | 455 |
m->print_value(); |
456 |
tty->cr(); |
|
457 |
ic->print(); |
|
458 |
bc->print(); |
|
459 |
if (ProfileInterpreter) { |
|
460 |
if (bci != InvocationEntryBci) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
461 |
MethodData* mdo = m->method_data(); |
6453 | 462 |
if (mdo != NULL) { |
48418
2207e2917a68
8191854: Null pointer dereference in methodData.hpp:462
dlong
parents:
47216
diff
changeset
|
463 |
ProfileData *pd = mdo->bci_to_data(branch_bci); |
2207e2917a68
8191854: Null pointer dereference in methodData.hpp:462
dlong
parents:
47216
diff
changeset
|
464 |
if (pd == NULL) { |
2207e2917a68
8191854: Null pointer dereference in methodData.hpp:462
dlong
parents:
47216
diff
changeset
|
465 |
tty->print_cr("back branch count = N/A (missing ProfileData)"); |
2207e2917a68
8191854: Null pointer dereference in methodData.hpp:462
dlong
parents:
47216
diff
changeset
|
466 |
} else { |
2207e2917a68
8191854: Null pointer dereference in methodData.hpp:462
dlong
parents:
47216
diff
changeset
|
467 |
tty->print_cr("back branch count = %d", pd->as_JumpData()->taken()); |
2207e2917a68
8191854: Null pointer dereference in methodData.hpp:462
dlong
parents:
47216
diff
changeset
|
468 |
} |
6453 | 469 |
} |
470 |
} |
|
471 |
} |
|
472 |
} |
|
473 |
} |
|
474 |
||
58398 | 475 |
void SimpleCompPolicy::trace_osr_request(const methodHandle& method, nmethod* osr, int bci) { |
6453 | 476 |
if (TraceOnStackReplacement) { |
477 |
ResourceMark rm; |
|
478 |
tty->print(osr != NULL ? "Reused OSR entry for " : "Requesting OSR entry for "); |
|
479 |
method->print_short_name(tty); |
|
480 |
tty->print_cr(" at bci %d", bci); |
|
481 |
} |
|
482 |
} |
|
483 |
#endif // !PRODUCT |
|
484 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
485 |
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
|
486 |
const int comp_level = CompLevel_highest_tier; |
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
487 |
const int hot_count = m->invocation_count(); |
1 | 488 |
reset_counter_for_invocation_event(m); |
489 |
||
17126
42a942feeea2
8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents:
17002
diff
changeset
|
490 |
if (is_compilation_enabled() && can_be_compiled(m, comp_level)) { |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
37248
diff
changeset
|
491 |
CompiledMethod* nm = m->code(); |
1 | 492 |
if (nm == NULL ) { |
38218 | 493 |
CompileBroker::compile_method(m, InvocationEntryBci, comp_level, m, hot_count, CompileTask::Reason_InvocationCount, thread); |
1 | 494 |
} |
495 |
} |
|
496 |
} |
|
497 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
32582
diff
changeset
|
498 |
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
|
499 |
const int comp_level = CompLevel_highest_tier; |
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
500 |
const int hot_count = m->backedge_count(); |
1 | 501 |
|
19332 | 502 |
if (is_compilation_enabled() && can_be_osr_compiled(m, comp_level)) { |
38218 | 503 |
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
|
504 |
NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, comp_level, true));) |
1 | 505 |
} |
506 |
} |