author | thartmann |
Thu, 04 Aug 2016 08:58:22 +0200 | |
changeset 40364 | 5bf8fa4fddfd |
parent 38666 | 5ff19807abd5 |
child 42650 | 1f304d0c888b |
permissions | -rw-r--r-- |
8667 | 1 |
/* |
36597
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
2 |
* Copyright (c) 2010, 2016, Oracle and/or its affiliates. All rights reserved. |
9625 | 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 |
* |
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
20 |
* or visit www.oracle.com if you need additional information or have any |
|
21 |
* questions. |
|
22 |
* |
|
23 |
*/ |
|
8667 | 24 |
|
25 |
#include "precompiled.hpp" |
|
25715
d5a8dbdc5150
8049325: Introduce and clean up umbrella headers for the files in the cpu subdirectories.
goetz
parents:
24443
diff
changeset
|
26 |
#include "code/codeCache.hpp" |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
27 |
#include "compiler/compileTask.hpp" |
8667 | 28 |
#include "runtime/advancedThresholdPolicy.hpp" |
29 |
#include "runtime/simpleThresholdPolicy.inline.hpp" |
|
38666
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
30 |
#if INCLUDE_JVMCI |
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
31 |
#include "jvmci/jvmciRuntime.hpp" |
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
32 |
#endif |
8667 | 33 |
|
34 |
#ifdef TIERED |
|
35 |
// Print an event. |
|
36 |
void AdvancedThresholdPolicy::print_specific(EventType type, methodHandle mh, methodHandle imh, |
|
37 |
int bci, CompLevel level) { |
|
13891
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
38 |
tty->print(" rate="); |
8667 | 39 |
if (mh->prev_time() == 0) tty->print("n/a"); |
40 |
else tty->print("%f", mh->rate()); |
|
41 |
||
13891
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
42 |
tty->print(" k=%.2lf,%.2lf", threshold_scale(CompLevel_full_profile, Tier3LoadFeedback), |
35dabd293e56
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
13728
diff
changeset
|
43 |
threshold_scale(CompLevel_full_optimization, Tier4LoadFeedback)); |
8667 | 44 |
|
45 |
} |
|
46 |
||
47 |
void AdvancedThresholdPolicy::initialize() { |
|
36556
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
48 |
int count = CICompilerCount; |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
49 |
#ifdef _LP64 |
8667 | 50 |
// Turn on ergonomic compiler count selection |
51 |
if (FLAG_IS_DEFAULT(CICompilerCountPerCPU) && FLAG_IS_DEFAULT(CICompilerCount)) { |
|
52 |
FLAG_SET_DEFAULT(CICompilerCountPerCPU, true); |
|
53 |
} |
|
54 |
if (CICompilerCountPerCPU) { |
|
55 |
// Simple log n seems to grow too slowly for tiered, try something faster: log n * log log n |
|
56 |
int log_cpu = log2_intptr(os::active_processor_count()); |
|
57 |
int loglog_cpu = log2_intptr(MAX2(log_cpu, 1)); |
|
40364
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
58 |
count = MAX2(log_cpu * loglog_cpu * 3 / 2, 2); |
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
59 |
FLAG_SET_ERGO(intx, CICompilerCount, count); |
8667 | 60 |
} |
36556
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
61 |
#else |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
62 |
// On 32-bit systems, the number of compiler threads is limited to 3. |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
63 |
// On these systems, the virtual address space available to the JVM |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
64 |
// is usually limited to 2-4 GB (the exact value depends on the platform). |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
65 |
// As the compilers (especially C2) can consume a large amount of |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
66 |
// memory, scaling the number of compiler threads with the number of |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
67 |
// available cores can result in the exhaustion of the address space |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
68 |
/// available to the VM and thus cause the VM to crash. |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
69 |
if (FLAG_IS_DEFAULT(CICompilerCount)) { |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
70 |
count = 3; |
40364
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
71 |
FLAG_SET_ERGO(intx, CICompilerCount, count); |
36556
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
72 |
} |
7f092a7ce938
8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents:
35547
diff
changeset
|
73 |
#endif |
8667 | 74 |
|
40364
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
75 |
if (TieredStopAtLevel < CompLevel_full_optimization) { |
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
76 |
// No C2 compiler thread required |
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
77 |
set_c1_count(count); |
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
78 |
} else { |
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
79 |
set_c1_count(MAX2(count / 3, 1)); |
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
80 |
set_c2_count(MAX2(count - c1_count(), 1)); |
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
81 |
} |
5bf8fa4fddfd
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents:
38666
diff
changeset
|
82 |
assert(count == c1_count() + c2_count(), "inconsistent compiler thread count"); |
8667 | 83 |
|
84 |
// Some inlining tuning |
|
85 |
#ifdef X86 |
|
86 |
if (FLAG_IS_DEFAULT(InlineSmallCode)) { |
|
87 |
FLAG_SET_DEFAULT(InlineSmallCode, 2000); |
|
88 |
} |
|
89 |
#endif |
|
90 |
||
31053
91b652361bce
8085805: aarch64: AdvancedThresholdPolicy lacks tuning of InlineSmallCode size
enevill
parents:
29335
diff
changeset
|
91 |
#if defined SPARC || defined AARCH64 |
8667 | 92 |
if (FLAG_IS_DEFAULT(InlineSmallCode)) { |
93 |
FLAG_SET_DEFAULT(InlineSmallCode, 2500); |
|
94 |
} |
|
95 |
#endif |
|
96 |
||
17617
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
97 |
set_increase_threshold_at_ratio(); |
8667 | 98 |
set_start_time(os::javaTimeMillis()); |
99 |
} |
|
100 |
||
101 |
// update_rate() is called from select_task() while holding a compile queue lock. |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
102 |
void AdvancedThresholdPolicy::update_rate(jlong t, Method* m) { |
24443
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
103 |
// Skip update if counters are absent. |
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
104 |
// Can't allocate them since we are holding compile queue lock. |
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
105 |
if (m->method_counters() == NULL) return; |
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
106 |
|
8667 | 107 |
if (is_old(m)) { |
108 |
// We don't remove old methods from the queue, |
|
109 |
// so we can just zero the rate. |
|
24443
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
110 |
m->set_rate(0); |
8667 | 111 |
return; |
112 |
} |
|
113 |
||
114 |
// We don't update the rate if we've just came out of a safepoint. |
|
115 |
// delta_s is the time since last safepoint in milliseconds. |
|
116 |
jlong delta_s = t - SafepointSynchronize::end_of_last_safepoint(); |
|
117 |
jlong delta_t = t - (m->prev_time() != 0 ? m->prev_time() : start_time()); // milliseconds since the last measurement |
|
118 |
// How many events were there since the last time? |
|
119 |
int event_count = m->invocation_count() + m->backedge_count(); |
|
120 |
int delta_e = event_count - m->prev_event_count(); |
|
121 |
||
122 |
// We should be running for at least 1ms. |
|
123 |
if (delta_s >= TieredRateUpdateMinTime) { |
|
124 |
// And we must've taken the previous point at least 1ms before. |
|
125 |
if (delta_t >= TieredRateUpdateMinTime && delta_e > 0) { |
|
24443
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
126 |
m->set_prev_time(t); |
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
127 |
m->set_prev_event_count(event_count); |
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
128 |
m->set_rate((float)delta_e / (float)delta_t); // Rate is events per millisecond |
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
129 |
} else { |
8667 | 130 |
if (delta_t > TieredRateUpdateMaxTime && delta_e == 0) { |
131 |
// If nothing happened for 25ms, zero the rate. Don't modify prev values. |
|
24443
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
132 |
m->set_rate(0); |
8667 | 133 |
} |
24443
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
134 |
} |
8667 | 135 |
} |
136 |
} |
|
137 |
||
138 |
// Check if this method has been stale from a given number of milliseconds. |
|
139 |
// See select_task(). |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
140 |
bool AdvancedThresholdPolicy::is_stale(jlong t, jlong timeout, Method* m) { |
8667 | 141 |
jlong delta_s = t - SafepointSynchronize::end_of_last_safepoint(); |
142 |
jlong delta_t = t - m->prev_time(); |
|
143 |
if (delta_t > timeout && delta_s > timeout) { |
|
144 |
int event_count = m->invocation_count() + m->backedge_count(); |
|
145 |
int delta_e = event_count - m->prev_event_count(); |
|
146 |
// Return true if there were no events. |
|
147 |
return delta_e == 0; |
|
148 |
} |
|
149 |
return false; |
|
150 |
} |
|
151 |
||
152 |
// We don't remove old methods from the compile queue even if they have |
|
153 |
// very low activity. See select_task(). |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
154 |
bool AdvancedThresholdPolicy::is_old(Method* method) { |
8667 | 155 |
return method->invocation_count() > 50000 || method->backedge_count() > 500000; |
156 |
} |
|
157 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
158 |
double AdvancedThresholdPolicy::weight(Method* method) { |
35155 | 159 |
return (double)(method->rate() + 1) * |
160 |
(method->invocation_count() + 1) * (method->backedge_count() + 1); |
|
8667 | 161 |
} |
162 |
||
163 |
// Apply heuristics and return true if x should be compiled before y |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
164 |
bool AdvancedThresholdPolicy::compare_methods(Method* x, Method* y) { |
8667 | 165 |
if (x->highest_comp_level() > y->highest_comp_level()) { |
166 |
// recompilation after deopt |
|
167 |
return true; |
|
168 |
} else |
|
169 |
if (x->highest_comp_level() == y->highest_comp_level()) { |
|
170 |
if (weight(x) > weight(y)) { |
|
171 |
return true; |
|
172 |
} |
|
173 |
} |
|
174 |
return false; |
|
175 |
} |
|
176 |
||
177 |
// Is method profiled enough? |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
178 |
bool AdvancedThresholdPolicy::is_method_profiled(Method* method) { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
179 |
MethodData* mdo = method->method_data(); |
8667 | 180 |
if (mdo != NULL) { |
181 |
int i = mdo->invocation_count_delta(); |
|
182 |
int b = mdo->backedge_count_delta(); |
|
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
183 |
return call_predicate_helper<CompLevel_full_profile>(i, b, 1, method); |
8667 | 184 |
} |
185 |
return false; |
|
186 |
} |
|
187 |
||
188 |
// Called with the queue locked and with at least one element |
|
189 |
CompileTask* AdvancedThresholdPolicy::select_task(CompileQueue* compile_queue) { |
|
35547
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
35155
diff
changeset
|
190 |
CompileTask *max_blocking_task = NULL; |
8667 | 191 |
CompileTask *max_task = NULL; |
13952
e3cf184080bc
8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents:
13891
diff
changeset
|
192 |
Method* max_method = NULL; |
8667 | 193 |
jlong t = os::javaTimeMillis(); |
194 |
// Iterate through the queue and find a method with a maximum rate. |
|
195 |
for (CompileTask* task = compile_queue->first(); task != NULL;) { |
|
196 |
CompileTask* next_task = task->next(); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
197 |
Method* method = task->method(); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
198 |
update_rate(t, method); |
8667 | 199 |
if (max_task == NULL) { |
200 |
max_task = task; |
|
201 |
max_method = method; |
|
202 |
} else { |
|
203 |
// If a method has been stale for some time, remove it from the queue. |
|
38218 | 204 |
// Blocking tasks and tasks submitted from whitebox API don't become stale |
205 |
if (task->can_become_stale() && is_stale(t, TieredCompileTaskTimeout, method) && !is_old(method)) { |
|
8667 | 206 |
if (PrintTieredEvents) { |
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
207 |
print_event(REMOVE_FROM_QUEUE, method, method, task->osr_bci(), (CompLevel)task->comp_level()); |
8667 | 208 |
} |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
209 |
task->log_task_dequeued("stale"); |
24443
7aaf1b306b55
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
24321
diff
changeset
|
210 |
compile_queue->remove_and_mark_stale(task); |
8667 | 211 |
method->clear_queued_for_compilation(); |
212 |
task = next_task; |
|
213 |
continue; |
|
214 |
} |
|
215 |
||
216 |
// Select a method with a higher rate |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
217 |
if (compare_methods(method, max_method)) { |
8667 | 218 |
max_task = task; |
219 |
max_method = method; |
|
220 |
} |
|
221 |
} |
|
36597
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
222 |
|
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
223 |
if (task->is_blocking()) { |
35547
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
35155
diff
changeset
|
224 |
if (max_blocking_task == NULL || compare_methods(method, max_blocking_task->method())) { |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
35155
diff
changeset
|
225 |
max_blocking_task = task; |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
35155
diff
changeset
|
226 |
} |
0ee84aa8e705
8146705: Improve JVMCI support for blocking compilation
dnsimon
parents:
35155
diff
changeset
|
227 |
} |
36597
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
228 |
|
8667 | 229 |
task = next_task; |
230 |
} |
|
231 |
||
36597
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
232 |
if (max_blocking_task != NULL) { |
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
233 |
// In blocking compilation mode, the CompileBroker will make |
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
234 |
// compilations submitted by a JVMCI compiler thread non-blocking. These |
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
235 |
// compilations should be scheduled after all blocking compilations |
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
236 |
// to service non-compiler related compilations sooner and reduce the |
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
237 |
// chance of such compilations timing out. |
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
238 |
max_task = max_blocking_task; |
ee256e343585
8150646: Add support for blocking compiles though whitebox API
simonis
parents:
36556
diff
changeset
|
239 |
max_method = max_task->method(); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
240 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
241 |
|
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
242 |
if (max_task->comp_level() == CompLevel_full_profile && TieredStopAtLevel > CompLevel_full_profile |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
243 |
&& is_method_profiled(max_method)) { |
8667 | 244 |
max_task->set_comp_level(CompLevel_limited_profile); |
245 |
if (PrintTieredEvents) { |
|
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
246 |
print_event(UPDATE_IN_QUEUE, max_method, max_method, max_task->osr_bci(), (CompLevel)max_task->comp_level()); |
8667 | 247 |
} |
248 |
} |
|
249 |
||
250 |
return max_task; |
|
251 |
} |
|
252 |
||
253 |
double AdvancedThresholdPolicy::threshold_scale(CompLevel level, int feedback_k) { |
|
254 |
double queue_size = CompileBroker::queue_size(level); |
|
255 |
int comp_count = compiler_count(level); |
|
256 |
double k = queue_size / (feedback_k * comp_count) + 1; |
|
17617
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
257 |
|
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
258 |
// Increase C1 compile threshold when the code cache is filled more |
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
259 |
// than specified by IncreaseFirstTierCompileThresholdAt percentage. |
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
260 |
// The main intention is to keep enough free space for C2 compiled code |
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
261 |
// to achieve peak performance if the code cache is under stress. |
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
262 |
if ((TieredStopAtLevel == CompLevel_full_optimization) && (level != CompLevel_full_optimization)) { |
26796 | 263 |
double current_reverse_free_ratio = CodeCache::reverse_free_ratio(CodeCache::get_code_blob_type(level)); |
17617
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
264 |
if (current_reverse_free_ratio > _increase_threshold_at_ratio) { |
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
265 |
k *= exp(current_reverse_free_ratio - _increase_threshold_at_ratio); |
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
266 |
} |
4e330bce1812
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
17000
diff
changeset
|
267 |
} |
8667 | 268 |
return k; |
269 |
} |
|
270 |
||
271 |
// Call and loop predicates determine whether a transition to a higher |
|
272 |
// compilation level should be performed (pointers to predicate functions |
|
273 |
// are passed to common()). |
|
274 |
// Tier?LoadFeedback is basically a coefficient that determines of |
|
275 |
// how many methods per compiler thread can be in the queue before |
|
276 |
// the threshold values double. |
|
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
277 |
bool AdvancedThresholdPolicy::loop_predicate(int i, int b, CompLevel cur_level, Method* method) { |
8667 | 278 |
switch(cur_level) { |
279 |
case CompLevel_none: |
|
280 |
case CompLevel_limited_profile: { |
|
281 |
double k = threshold_scale(CompLevel_full_profile, Tier3LoadFeedback); |
|
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
282 |
return loop_predicate_helper<CompLevel_none>(i, b, k, method); |
8667 | 283 |
} |
284 |
case CompLevel_full_profile: { |
|
285 |
double k = threshold_scale(CompLevel_full_optimization, Tier4LoadFeedback); |
|
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
286 |
return loop_predicate_helper<CompLevel_full_profile>(i, b, k, method); |
8667 | 287 |
} |
288 |
default: |
|
289 |
return true; |
|
290 |
} |
|
291 |
} |
|
292 |
||
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
293 |
bool AdvancedThresholdPolicy::call_predicate(int i, int b, CompLevel cur_level, Method* method) { |
8667 | 294 |
switch(cur_level) { |
295 |
case CompLevel_none: |
|
296 |
case CompLevel_limited_profile: { |
|
297 |
double k = threshold_scale(CompLevel_full_profile, Tier3LoadFeedback); |
|
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
298 |
return call_predicate_helper<CompLevel_none>(i, b, k, method); |
8667 | 299 |
} |
300 |
case CompLevel_full_profile: { |
|
301 |
double k = threshold_scale(CompLevel_full_optimization, Tier4LoadFeedback); |
|
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
302 |
return call_predicate_helper<CompLevel_full_profile>(i, b, k, method); |
8667 | 303 |
} |
304 |
default: |
|
305 |
return true; |
|
306 |
} |
|
307 |
} |
|
308 |
||
309 |
// If a method is old enough and is still in the interpreter we would want to |
|
310 |
// start profiling without waiting for the compiled method to arrive. |
|
311 |
// We also take the load on compilers into the account. |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
312 |
bool AdvancedThresholdPolicy::should_create_mdo(Method* method, CompLevel cur_level) { |
8667 | 313 |
if (cur_level == CompLevel_none && |
314 |
CompileBroker::queue_size(CompLevel_full_optimization) <= |
|
315 |
Tier3DelayOn * compiler_count(CompLevel_full_optimization)) { |
|
316 |
int i = method->invocation_count(); |
|
317 |
int b = method->backedge_count(); |
|
318 |
double k = Tier0ProfilingStartPercentage / 100.0; |
|
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
319 |
return call_predicate_helper<CompLevel_none>(i, b, k, method) || loop_predicate_helper<CompLevel_none>(i, b, k, method); |
8667 | 320 |
} |
321 |
return false; |
|
322 |
} |
|
323 |
||
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
324 |
// Inlining control: if we're compiling a profiled method with C1 and the callee |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
325 |
// is known to have OSRed in a C2 version, don't inline it. |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
326 |
bool AdvancedThresholdPolicy::should_not_inline(ciEnv* env, ciMethod* callee) { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
327 |
CompLevel comp_level = (CompLevel)env->comp_level(); |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
328 |
if (comp_level == CompLevel_full_profile || |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
329 |
comp_level == CompLevel_limited_profile) { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
330 |
return callee->highest_osr_comp_level() == CompLevel_full_optimization; |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
331 |
} |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
332 |
return false; |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
333 |
} |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
334 |
|
8667 | 335 |
// Create MDO if necessary. |
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
11424
diff
changeset
|
336 |
void AdvancedThresholdPolicy::create_mdo(methodHandle mh, JavaThread* THREAD) { |
29335 | 337 |
if (mh->is_native() || |
338 |
mh->is_abstract() || |
|
339 |
mh->is_accessor() || |
|
340 |
mh->is_constant_getter()) { |
|
341 |
return; |
|
342 |
} |
|
8667 | 343 |
if (mh->method_data() == NULL) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
344 |
Method::build_interpreter_method_data(mh, CHECK_AND_CLEAR); |
8667 | 345 |
} |
346 |
} |
|
347 |
||
348 |
||
349 |
/* |
|
350 |
* Method states: |
|
351 |
* 0 - interpreter (CompLevel_none) |
|
352 |
* 1 - pure C1 (CompLevel_simple) |
|
353 |
* 2 - C1 with invocation and backedge counting (CompLevel_limited_profile) |
|
354 |
* 3 - C1 with full profiling (CompLevel_full_profile) |
|
355 |
* 4 - C2 (CompLevel_full_optimization) |
|
356 |
* |
|
357 |
* Common state transition patterns: |
|
358 |
* a. 0 -> 3 -> 4. |
|
359 |
* The most common path. But note that even in this straightforward case |
|
360 |
* profiling can start at level 0 and finish at level 3. |
|
361 |
* |
|
362 |
* b. 0 -> 2 -> 3 -> 4. |
|
22551 | 363 |
* This case occurs when the load on C2 is deemed too high. So, instead of transitioning |
8667 | 364 |
* into state 3 directly and over-profiling while a method is in the C2 queue we transition to |
365 |
* level 2 and wait until the load on C2 decreases. This path is disabled for OSRs. |
|
366 |
* |
|
367 |
* c. 0 -> (3->2) -> 4. |
|
368 |
* In this case we enqueue a method for compilation at level 3, but the C1 queue is long enough |
|
369 |
* to enable the profiling to fully occur at level 0. In this case we change the compilation level |
|
27643
fe8f95a2d9bc
8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
thartmann
parents:
26796
diff
changeset
|
370 |
* of the method to 2 while the request is still in-queue, because it'll allow it to run much faster |
fe8f95a2d9bc
8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
thartmann
parents:
26796
diff
changeset
|
371 |
* without full profiling while c2 is compiling. |
8667 | 372 |
* |
373 |
* d. 0 -> 3 -> 1 or 0 -> 2 -> 1. |
|
374 |
* After a method was once compiled with C1 it can be identified as trivial and be compiled to |
|
375 |
* level 1. These transition can also occur if a method can't be compiled with C2 but can with C1. |
|
376 |
* |
|
377 |
* e. 0 -> 4. |
|
378 |
* This can happen if a method fails C1 compilation (it will still be profiled in the interpreter) |
|
379 |
* or because of a deopt that didn't require reprofiling (compilation won't happen in this case because |
|
380 |
* the compiled version already exists). |
|
381 |
* |
|
382 |
* Note that since state 0 can be reached from any other state via deoptimization different loops |
|
383 |
* are possible. |
|
384 |
* |
|
385 |
*/ |
|
386 |
||
387 |
// Common transition function. Given a predicate determines if a method should transition to another level. |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
388 |
CompLevel AdvancedThresholdPolicy::common(Predicate p, Method* method, CompLevel cur_level, bool disable_feedback) { |
8667 | 389 |
CompLevel next_level = cur_level; |
390 |
int i = method->invocation_count(); |
|
391 |
int b = method->backedge_count(); |
|
392 |
||
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
393 |
if (is_trivial(method)) { |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
394 |
next_level = CompLevel_simple; |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
395 |
} else { |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
396 |
switch(cur_level) { |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
397 |
case CompLevel_none: |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
398 |
// If we were at full profile level, would we switch to full opt? |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
399 |
if (common(p, method, CompLevel_full_profile, disable_feedback) == CompLevel_full_optimization) { |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
400 |
next_level = CompLevel_full_optimization; |
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
401 |
} else if ((this->*p)(i, b, cur_level, method)) { |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
402 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
403 |
if (UseJVMCICompiler) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
404 |
// Since JVMCI takes a while to warm up, its queue inevitably backs up during |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
405 |
// early VM execution. |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
406 |
next_level = CompLevel_full_profile; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
407 |
break; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
408 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
409 |
#endif |
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
410 |
// C1-generated fully profiled code is about 30% slower than the limited profile |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
411 |
// code that has only invocation and backedge counters. The observation is that |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
412 |
// if C2 queue is large enough we can spend too much time in the fully profiled code |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
413 |
// while waiting for C2 to pick the method from the queue. To alleviate this problem |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
414 |
// we introduce a feedback on the C2 queue size. If the C2 queue is sufficiently long |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
415 |
// we choose to compile a limited profiled version and then recompile with full profiling |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
416 |
// when the load on C2 goes down. |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
417 |
if (!disable_feedback && CompileBroker::queue_size(CompLevel_full_optimization) > |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31053
diff
changeset
|
418 |
Tier3DelayOn * compiler_count(CompLevel_full_optimization)) { |
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
419 |
next_level = CompLevel_limited_profile; |
8667 | 420 |
} else { |
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
421 |
next_level = CompLevel_full_profile; |
8667 | 422 |
} |
423 |
} |
|
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
424 |
break; |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
425 |
case CompLevel_limited_profile: |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
426 |
if (is_method_profiled(method)) { |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
427 |
// Special case: we got here because this method was fully profiled in the interpreter. |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
428 |
next_level = CompLevel_full_optimization; |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
429 |
} else { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
430 |
MethodData* mdo = method->method_data(); |
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
431 |
if (mdo != NULL) { |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
432 |
if (mdo->would_profile()) { |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
433 |
if (disable_feedback || (CompileBroker::queue_size(CompLevel_full_optimization) <= |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
434 |
Tier3DelayOff * compiler_count(CompLevel_full_optimization) && |
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
435 |
(this->*p)(i, b, cur_level, method))) { |
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
436 |
next_level = CompLevel_full_profile; |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
437 |
} |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
438 |
} else { |
8667 | 439 |
next_level = CompLevel_full_optimization; |
440 |
} |
|
441 |
} |
|
442 |
} |
|
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
443 |
break; |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
444 |
case CompLevel_full_profile: |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
445 |
{ |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
446 |
MethodData* mdo = method->method_data(); |
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
447 |
if (mdo != NULL) { |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
448 |
if (mdo->would_profile()) { |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
449 |
int mdo_i = mdo->invocation_count_delta(); |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
450 |
int mdo_b = mdo->backedge_count_delta(); |
28650
772aaab2582f
8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents:
27643
diff
changeset
|
451 |
if ((this->*p)(mdo_i, mdo_b, cur_level, method)) { |
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
452 |
next_level = CompLevel_full_optimization; |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
453 |
} |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
454 |
} else { |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
455 |
next_level = CompLevel_full_optimization; |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
456 |
} |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
457 |
} |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
458 |
} |
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
459 |
break; |
8667 | 460 |
} |
461 |
} |
|
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
462 |
return MIN2(next_level, (CompLevel)TieredStopAtLevel); |
8667 | 463 |
} |
464 |
||
465 |
// Determine if a method should be compiled with a normal entry point at a different level. |
|
38666
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
466 |
CompLevel AdvancedThresholdPolicy::call_event(Method* method, CompLevel cur_level, JavaThread * thread) { |
10013
714ad59b56cb
7058689: Tiered: Reprofiling doesn't happen in presence of level 4 OSR methods
iveresov
parents:
9625
diff
changeset
|
467 |
CompLevel osr_level = MIN2((CompLevel) method->highest_osr_comp_level(), |
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
468 |
common(&AdvancedThresholdPolicy::loop_predicate, method, cur_level, true)); |
8667 | 469 |
CompLevel next_level = common(&AdvancedThresholdPolicy::call_predicate, method, cur_level); |
470 |
||
471 |
// If OSR method level is greater than the regular method level, the levels should be |
|
472 |
// equalized by raising the regular method level in order to avoid OSRs during each |
|
473 |
// invocation of the method. |
|
474 |
if (osr_level == CompLevel_full_optimization && cur_level == CompLevel_full_profile) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11572
diff
changeset
|
475 |
MethodData* mdo = method->method_data(); |
8667 | 476 |
guarantee(mdo != NULL, "MDO should not be NULL"); |
477 |
if (mdo->invocation_count() >= 1) { |
|
478 |
next_level = CompLevel_full_optimization; |
|
479 |
} |
|
480 |
} else { |
|
481 |
next_level = MAX2(osr_level, next_level); |
|
482 |
} |
|
38666
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
483 |
#if INCLUDE_JVMCI |
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
484 |
if (UseJVMCICompiler) { |
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
485 |
next_level = JVMCIRuntime::adjust_comp_level(method, false, next_level, thread); |
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
486 |
} |
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
487 |
#endif |
8667 | 488 |
return next_level; |
489 |
} |
|
490 |
||
491 |
// Determine if we should do an OSR compilation of a given method. |
|
38666
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
492 |
CompLevel AdvancedThresholdPolicy::loop_event(Method* method, CompLevel cur_level, JavaThread * thread) { |
10250
0794cd144834
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
10014
diff
changeset
|
493 |
CompLevel next_level = common(&AdvancedThresholdPolicy::loop_predicate, method, cur_level, true); |
8667 | 494 |
if (cur_level == CompLevel_none) { |
495 |
// If there is a live OSR method that means that we deopted to the interpreter |
|
496 |
// for the transition. |
|
10013
714ad59b56cb
7058689: Tiered: Reprofiling doesn't happen in presence of level 4 OSR methods
iveresov
parents:
9625
diff
changeset
|
497 |
CompLevel osr_level = MIN2((CompLevel)method->highest_osr_comp_level(), next_level); |
8667 | 498 |
if (osr_level > CompLevel_none) { |
499 |
return osr_level; |
|
500 |
} |
|
501 |
} |
|
38666
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
502 |
#if INCLUDE_JVMCI |
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
503 |
if (UseJVMCICompiler) { |
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
504 |
next_level = JVMCIRuntime::adjust_comp_level(method, true, next_level, thread); |
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
505 |
} |
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
506 |
#endif |
10013
714ad59b56cb
7058689: Tiered: Reprofiling doesn't happen in presence of level 4 OSR methods
iveresov
parents:
9625
diff
changeset
|
507 |
return next_level; |
8667 | 508 |
} |
509 |
||
510 |
// Update the rate and submit compile |
|
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
511 |
void AdvancedThresholdPolicy::submit_compile(const methodHandle& mh, int bci, CompLevel level, JavaThread* thread) { |
8667 | 512 |
int hot_count = (bci == InvocationEntryBci) ? mh->invocation_count() : mh->backedge_count(); |
513 |
update_rate(os::javaTimeMillis(), mh()); |
|
38218 | 514 |
CompileBroker::compile_method(mh, bci, level, mh, hot_count, CompileTask::Reason_Tiered, thread); |
8667 | 515 |
} |
516 |
||
517 |
// Handle the invocation event. |
|
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
518 |
void AdvancedThresholdPolicy::method_invocation_event(const methodHandle& mh, const methodHandle& imh, |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36597
diff
changeset
|
519 |
CompLevel level, CompiledMethod* nm, JavaThread* thread) { |
8667 | 520 |
if (should_create_mdo(mh(), level)) { |
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
11424
diff
changeset
|
521 |
create_mdo(mh, thread); |
8667 | 522 |
} |
24321
621f7e09fc0a
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
24013
diff
changeset
|
523 |
if (is_compilation_enabled() && !CompileBroker::compilation_is_in_queue(mh)) { |
38666
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
524 |
CompLevel next_level = call_event(mh(), level, thread); |
8667 | 525 |
if (next_level != level) { |
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
11424
diff
changeset
|
526 |
compile(mh, InvocationEntryBci, next_level, thread); |
8667 | 527 |
} |
528 |
} |
|
529 |
} |
|
530 |
||
531 |
// Handle the back branch event. Notice that we can compile the method |
|
532 |
// with a regular entry from here. |
|
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
533 |
void AdvancedThresholdPolicy::method_back_branch_event(const methodHandle& mh, const methodHandle& imh, |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
36597
diff
changeset
|
534 |
int bci, CompLevel level, CompiledMethod* nm, JavaThread* thread) { |
8667 | 535 |
if (should_create_mdo(mh(), level)) { |
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
11424
diff
changeset
|
536 |
create_mdo(mh, thread); |
8667 | 537 |
} |
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
538 |
// Check if MDO should be created for the inlined method |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
539 |
if (should_create_mdo(imh(), level)) { |
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
11424
diff
changeset
|
540 |
create_mdo(imh, thread); |
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
541 |
} |
8667 | 542 |
|
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
543 |
if (is_compilation_enabled()) { |
38666
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
544 |
CompLevel next_osr_level = loop_event(imh(), level, thread); |
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
545 |
CompLevel max_osr_level = (CompLevel)imh->highest_osr_comp_level(); |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
546 |
// At the very least compile the OSR version |
24321
621f7e09fc0a
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
24013
diff
changeset
|
547 |
if (!CompileBroker::compilation_is_in_queue(imh) && (next_osr_level != level)) { |
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
11424
diff
changeset
|
548 |
compile(imh, bci, next_osr_level, thread); |
8667 | 549 |
} |
550 |
||
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
551 |
// Use loop event as an opportunity to also check if there's been |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
552 |
// enough calls. |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
553 |
CompLevel cur_level, next_level; |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
554 |
if (mh() != imh()) { // If there is an enclosing method |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
555 |
guarantee(nm != NULL, "Should have nmethod here"); |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
556 |
cur_level = comp_level(mh()); |
38666
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
557 |
next_level = call_event(mh(), cur_level, thread); |
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
558 |
|
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
559 |
if (max_osr_level == CompLevel_full_optimization) { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
560 |
// The inlinee OSRed to full opt, we need to modify the enclosing method to avoid deopts |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
561 |
bool make_not_entrant = false; |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
562 |
if (nm->is_osr_method()) { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
563 |
// This is an osr method, just make it not entrant and recompile later if needed |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
564 |
make_not_entrant = true; |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
565 |
} else { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
566 |
if (next_level != CompLevel_full_optimization) { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
567 |
// next_level is not full opt, so we need to recompile the |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
568 |
// enclosing method without the inlinee |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
569 |
cur_level = CompLevel_none; |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
570 |
make_not_entrant = true; |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
571 |
} |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
572 |
} |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
573 |
if (make_not_entrant) { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
574 |
if (PrintTieredEvents) { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
575 |
int osr_bci = nm->is_osr_method() ? nm->osr_entry_bci() : InvocationEntryBci; |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
576 |
print_event(MAKE_NOT_ENTRANT, mh(), mh(), osr_bci, level); |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
577 |
} |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
578 |
nm->make_not_entrant(); |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
579 |
} |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
580 |
} |
24321
621f7e09fc0a
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
24013
diff
changeset
|
581 |
if (!CompileBroker::compilation_is_in_queue(mh)) { |
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
582 |
// Fix up next_level if necessary to avoid deopts |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
583 |
if (next_level == CompLevel_limited_profile && max_osr_level == CompLevel_full_profile) { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
584 |
next_level = CompLevel_full_profile; |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
585 |
} |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
586 |
if (cur_level != next_level) { |
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
11424
diff
changeset
|
587 |
compile(mh, InvocationEntryBci, next_level, thread); |
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
588 |
} |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
589 |
} |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
590 |
} else { |
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
591 |
cur_level = comp_level(imh()); |
38666
5ff19807abd5
8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents:
38218
diff
changeset
|
592 |
next_level = call_event(imh(), cur_level, thread); |
24321
621f7e09fc0a
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
24013
diff
changeset
|
593 |
if (!CompileBroker::compilation_is_in_queue(imh) && (next_level != cur_level)) { |
11572
84afef481892
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
11424
diff
changeset
|
594 |
compile(imh, InvocationEntryBci, next_level, thread); |
10014
a5c2141ee857
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
10013
diff
changeset
|
595 |
} |
8667 | 596 |
} |
597 |
} |
|
598 |
} |
|
599 |
||
600 |
#endif // TIERED |