Mercurial > hg > graal-jvmci-8
annotate src/share/vm/compiler/compileBroker.cpp @ 22714:9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
author | Doug Simon <doug.simon@oracle.com> |
---|---|
date | Sun, 01 Nov 2015 21:43:49 +0100 |
parents | cfd7ebda543b |
children | 44bc739eae23 |
rev | line source |
---|---|
0 | 1 /* |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
14456
diff
changeset
|
2 * Copyright (c) 1999, 2014, Oracle and/or its affiliates. All rights reserved. |
0 | 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 * | |
5 * This code is free software; you can redistribute it and/or modify it | |
6 * under the terms of the GNU General Public License version 2 only, as | |
7 * published by the Free Software Foundation. | |
8 * | |
9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
12 * version 2 for more details (a copy is included in the LICENSE file that | |
13 * accompanied this code). | |
14 * | |
15 * You should have received a copy of the GNU General Public License version | |
16 * 2 along with this work; if not, write to the Free Software Foundation, | |
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
18 * | |
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1538
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1538
diff
changeset
|
20 * or visit www.oracle.com if you need additional information or have any |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1538
diff
changeset
|
21 * questions. |
0 | 22 * |
23 */ | |
24 | |
1972 | 25 #include "precompiled.hpp" |
26 #include "classfile/systemDictionary.hpp" | |
27 #include "classfile/vmSymbols.hpp" | |
28 #include "code/codeCache.hpp" | |
29 #include "compiler/compileBroker.hpp" | |
30 #include "compiler/compileLog.hpp" | |
31 #include "compiler/compilerOracle.hpp" | |
32 #include "interpreter/linkResolver.hpp" | |
33 #include "memory/allocation.inline.hpp" | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
34 #include "oops/methodData.hpp" |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
35 #include "oops/method.hpp" |
1972 | 36 #include "oops/oop.inline.hpp" |
37 #include "prims/nativeLookup.hpp" | |
38 #include "runtime/arguments.hpp" | |
39 #include "runtime/compilationPolicy.hpp" | |
40 #include "runtime/init.hpp" | |
41 #include "runtime/interfaceSupport.hpp" | |
42 #include "runtime/javaCalls.hpp" | |
43 #include "runtime/os.hpp" | |
44 #include "runtime/sharedRuntime.hpp" | |
45 #include "runtime/sweeper.hpp" | |
10405 | 46 #include "trace/tracing.hpp" |
1972 | 47 #include "utilities/dtrace.hpp" |
4872
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
48 #include "utilities/events.hpp" |
1972 | 49 #ifdef COMPILER1 |
50 #include "c1/c1_Compiler.hpp" | |
51 #endif | |
22298 | 52 #if INCLUDE_JVMCI |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
53 #include "jvmci/jvmciCompiler.hpp" |
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
54 #ifdef COMPILERJVMCI |
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
55 #include "jvmci/jvmciRuntime.hpp" |
16447
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
56 #include "runtime/vframe.hpp" |
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
57 #endif |
7125
1baf7f1e3f23
decoupled C++ Graal runtime from C1
Doug Simon <doug.simon@oracle.com>
parents:
6948
diff
changeset
|
58 #endif |
1972 | 59 #ifdef COMPILER2 |
60 #include "opto/c2compiler.hpp" | |
61 #endif | |
62 #ifdef SHARK | |
63 #include "shark/sharkCompiler.hpp" | |
64 #endif | |
0 | 65 |
66 #ifdef DTRACE_ENABLED | |
67 | |
68 // Only bother with this argument setup if dtrace is available | |
69 | |
4006 | 70 #ifndef USDT2 |
0 | 71 HS_DTRACE_PROBE_DECL8(hotspot, method__compile__begin, |
72 char*, intptr_t, char*, intptr_t, char*, intptr_t, char*, intptr_t); | |
73 HS_DTRACE_PROBE_DECL9(hotspot, method__compile__end, | |
74 char*, intptr_t, char*, intptr_t, char*, intptr_t, char*, intptr_t, bool); | |
75 | |
10207
e4e131b15d5c
8013532: Remove unused parameter "compiler" from DTRACE_METHOD_COMPILE* macros
roland
parents:
10206
diff
changeset
|
76 #define DTRACE_METHOD_COMPILE_BEGIN_PROBE(method, comp_name) \ |
0 | 77 { \ |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
78 Symbol* klass_name = (method)->klass_name(); \ |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
79 Symbol* name = (method)->name(); \ |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
80 Symbol* signature = (method)->signature(); \ |
0 | 81 HS_DTRACE_PROBE8(hotspot, method__compile__begin, \ |
82 comp_name, strlen(comp_name), \ | |
83 klass_name->bytes(), klass_name->utf8_length(), \ | |
84 name->bytes(), name->utf8_length(), \ | |
85 signature->bytes(), signature->utf8_length()); \ | |
86 } | |
87 | |
10207
e4e131b15d5c
8013532: Remove unused parameter "compiler" from DTRACE_METHOD_COMPILE* macros
roland
parents:
10206
diff
changeset
|
88 #define DTRACE_METHOD_COMPILE_END_PROBE(method, comp_name, success) \ |
0 | 89 { \ |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
90 Symbol* klass_name = (method)->klass_name(); \ |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
91 Symbol* name = (method)->name(); \ |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
92 Symbol* signature = (method)->signature(); \ |
0 | 93 HS_DTRACE_PROBE9(hotspot, method__compile__end, \ |
94 comp_name, strlen(comp_name), \ | |
95 klass_name->bytes(), klass_name->utf8_length(), \ | |
96 name->bytes(), name->utf8_length(), \ | |
97 signature->bytes(), signature->utf8_length(), (success)); \ | |
98 } | |
99 | |
4006 | 100 #else /* USDT2 */ |
101 | |
10207
e4e131b15d5c
8013532: Remove unused parameter "compiler" from DTRACE_METHOD_COMPILE* macros
roland
parents:
10206
diff
changeset
|
102 #define DTRACE_METHOD_COMPILE_BEGIN_PROBE(method, comp_name) \ |
4006 | 103 { \ |
104 Symbol* klass_name = (method)->klass_name(); \ | |
105 Symbol* name = (method)->name(); \ | |
106 Symbol* signature = (method)->signature(); \ | |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
107 HOTSPOT_METHOD_COMPILE_BEGIN( \ |
4006 | 108 comp_name, strlen(comp_name), \ |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
109 (char *) klass_name->bytes(), klass_name->utf8_length(), \ |
4006 | 110 (char *) name->bytes(), name->utf8_length(), \ |
111 (char *) signature->bytes(), signature->utf8_length()); \ | |
112 } | |
113 | |
10207
e4e131b15d5c
8013532: Remove unused parameter "compiler" from DTRACE_METHOD_COMPILE* macros
roland
parents:
10206
diff
changeset
|
114 #define DTRACE_METHOD_COMPILE_END_PROBE(method, comp_name, success) \ |
4006 | 115 { \ |
116 Symbol* klass_name = (method)->klass_name(); \ | |
117 Symbol* name = (method)->name(); \ | |
118 Symbol* signature = (method)->signature(); \ | |
119 HOTSPOT_METHOD_COMPILE_END( \ | |
120 comp_name, strlen(comp_name), \ | |
121 (char *) klass_name->bytes(), klass_name->utf8_length(), \ | |
122 (char *) name->bytes(), name->utf8_length(), \ | |
123 (char *) signature->bytes(), signature->utf8_length(), (success)); \ | |
124 } | |
125 #endif /* USDT2 */ | |
126 | |
0 | 127 #else // ndef DTRACE_ENABLED |
128 | |
10207
e4e131b15d5c
8013532: Remove unused parameter "compiler" from DTRACE_METHOD_COMPILE* macros
roland
parents:
10206
diff
changeset
|
129 #define DTRACE_METHOD_COMPILE_BEGIN_PROBE(method, comp_name) |
e4e131b15d5c
8013532: Remove unused parameter "compiler" from DTRACE_METHOD_COMPILE* macros
roland
parents:
10206
diff
changeset
|
130 #define DTRACE_METHOD_COMPILE_END_PROBE(method, comp_name, success) |
0 | 131 |
132 #endif // ndef DTRACE_ENABLED | |
133 | |
134 bool CompileBroker::_initialized = false; | |
135 volatile bool CompileBroker::_should_block = false; | |
13074
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
136 volatile jint CompileBroker::_print_compilation_warning = 0; |
1202 | 137 volatile jint CompileBroker::_should_compile_new_jobs = run_compilation; |
0 | 138 |
139 // The installed compiler(s) | |
140 AbstractCompiler* CompileBroker::_compilers[2]; | |
141 | |
14236 | 142 // These counters are used to assign an unique ID to each compilation. |
143 volatile jint CompileBroker::_compilation_id = 0; | |
144 volatile jint CompileBroker::_osr_compilation_id = 0; | |
0 | 145 |
146 // Debugging information | |
147 int CompileBroker::_last_compile_type = no_compile; | |
148 int CompileBroker::_last_compile_level = CompLevel_none; | |
149 char CompileBroker::_last_method_compiled[CompileBroker::name_buffer_length]; | |
150 | |
151 // Performance counters | |
152 PerfCounter* CompileBroker::_perf_total_compilation = NULL; | |
153 PerfCounter* CompileBroker::_perf_osr_compilation = NULL; | |
154 PerfCounter* CompileBroker::_perf_standard_compilation = NULL; | |
155 | |
156 PerfCounter* CompileBroker::_perf_total_bailout_count = NULL; | |
157 PerfCounter* CompileBroker::_perf_total_invalidated_count = NULL; | |
158 PerfCounter* CompileBroker::_perf_total_compile_count = NULL; | |
159 PerfCounter* CompileBroker::_perf_total_osr_compile_count = NULL; | |
160 PerfCounter* CompileBroker::_perf_total_standard_compile_count = NULL; | |
161 | |
162 PerfCounter* CompileBroker::_perf_sum_osr_bytes_compiled = NULL; | |
163 PerfCounter* CompileBroker::_perf_sum_standard_bytes_compiled = NULL; | |
164 PerfCounter* CompileBroker::_perf_sum_nmethod_size = NULL; | |
165 PerfCounter* CompileBroker::_perf_sum_nmethod_code_size = NULL; | |
166 | |
167 PerfStringVariable* CompileBroker::_perf_last_method = NULL; | |
168 PerfStringVariable* CompileBroker::_perf_last_failed_method = NULL; | |
169 PerfStringVariable* CompileBroker::_perf_last_invalidated_method = NULL; | |
170 PerfVariable* CompileBroker::_perf_last_compile_type = NULL; | |
171 PerfVariable* CompileBroker::_perf_last_compile_size = NULL; | |
172 PerfVariable* CompileBroker::_perf_last_failed_type = NULL; | |
173 PerfVariable* CompileBroker::_perf_last_invalidated_type = NULL; | |
174 | |
175 // Timers and counters for generating statistics | |
176 elapsedTimer CompileBroker::_t_total_compilation; | |
177 elapsedTimer CompileBroker::_t_osr_compilation; | |
178 elapsedTimer CompileBroker::_t_standard_compilation; | |
179 | |
180 int CompileBroker::_total_bailout_count = 0; | |
181 int CompileBroker::_total_invalidated_count = 0; | |
182 int CompileBroker::_total_compile_count = 0; | |
183 int CompileBroker::_total_osr_compile_count = 0; | |
184 int CompileBroker::_total_standard_compile_count = 0; | |
185 | |
186 int CompileBroker::_sum_osr_bytes_compiled = 0; | |
187 int CompileBroker::_sum_standard_bytes_compiled = 0; | |
188 int CompileBroker::_sum_nmethod_size = 0; | |
189 int CompileBroker::_sum_nmethod_code_size = 0; | |
190 | |
10405 | 191 long CompileBroker::_peak_compilation_time = 0; |
192 | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
193 CompileQueue* CompileBroker::_c2_compile_queue = NULL; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
194 CompileQueue* CompileBroker::_c1_compile_queue = NULL; |
0 | 195 |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
196 GrowableArray<CompilerThread*>* CompileBroker::_compiler_threads = NULL; |
0 | 197 |
198 | |
4872
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
199 class CompilationLog : public StringEventLog { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
200 public: |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
201 CompilationLog() : StringEventLog("Compilation events") { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
202 } |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
203 |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
204 void log_compile(JavaThread* thread, CompileTask* task) { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
205 StringLogMessage lm; |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
206 stringStream sstr = lm.stream(); |
4872
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
207 // msg.time_stamp().update_to(tty->time_stamp().ticks()); |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
208 task->print_compilation(&sstr, NULL, true); |
4872
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
209 log(thread, "%s", (const char*)lm); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
210 } |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
211 |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
212 void log_nmethod(JavaThread* thread, nmethod* nm) { |
4944 | 213 log(thread, "nmethod %d%s " INTPTR_FORMAT " code ["INTPTR_FORMAT ", " INTPTR_FORMAT "]", |
214 nm->compile_id(), nm->is_osr_method() ? "%" : "", | |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
14456
diff
changeset
|
215 p2i(nm), p2i(nm->code_begin()), p2i(nm->code_end())); |
4872
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
216 } |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
217 |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
218 void log_failure(JavaThread* thread, CompileTask* task, const char* reason, const char* retry_message) { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
219 StringLogMessage lm; |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
220 lm.print("%4d COMPILE SKIPPED: %s", task->compile_id(), reason); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
221 if (retry_message != NULL) { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
222 lm.append(" (%s)", retry_message); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
223 } |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
224 lm.print("\n"); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
225 log(thread, "%s", (const char*)lm); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
226 } |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
227 }; |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
228 |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
229 static CompilationLog* _compilation_log = NULL; |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
230 |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
231 void compileBroker_init() { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
232 if (LogEvents) { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
233 _compilation_log = new CompilationLog(); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
234 } |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
235 } |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
236 |
0 | 237 CompileTaskWrapper::CompileTaskWrapper(CompileTask* task) { |
238 CompilerThread* thread = CompilerThread::current(); | |
239 thread->set_task(task); | |
240 CompileLog* log = thread->log(); | |
241 if (log != NULL) task->log_task_start(log); | |
242 } | |
243 | |
244 CompileTaskWrapper::~CompileTaskWrapper() { | |
245 CompilerThread* thread = CompilerThread::current(); | |
246 CompileTask* task = thread->task(); | |
247 CompileLog* log = thread->log(); | |
248 if (log != NULL) task->log_task_done(log); | |
249 thread->set_task(NULL); | |
250 task->set_code_handle(NULL); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
251 thread->set_env(NULL); |
0 | 252 if (task->is_blocking()) { |
253 MutexLocker notifier(task->lock(), thread); | |
254 task->mark_complete(); | |
255 // Notify the waiting thread that the compilation has completed. | |
256 task->lock()->notify_all(); | |
22714
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
257 #ifdef COMPILERJVMCI |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
258 if (CompileBroker::compiler(task->comp_level())->is_jvmci()) { |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
259 // Blocking JVMCI compilations are performed with a timeout so as |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
260 // to avoid deadlock between an application thread and a JVMCI |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
261 // compiler thread (both of which execute Java code). In this case, |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
262 // the compiling thread recycles the CompileTask. |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
263 CompileTask::free(task); |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
264 } |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
265 #endif |
0 | 266 } else { |
267 task->mark_complete(); | |
268 | |
269 // By convention, the compiling thread is responsible for | |
270 // recycling a non-blocking CompileTask. | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
271 CompileTask::free(task); |
0 | 272 } |
273 } | |
274 | |
275 | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
276 CompileTask* CompileTask::_task_free_list = NULL; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
277 #ifdef ASSERT |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
278 int CompileTask::_num_allocated_tasks = 0; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
279 #endif |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
280 /** |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
281 * Allocate a CompileTask, from the free list if possible. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
282 */ |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
283 CompileTask* CompileTask::allocate() { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
284 MutexLocker locker(CompileTaskAlloc_lock); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
285 CompileTask* task = NULL; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
286 |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
287 if (_task_free_list != NULL) { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
288 task = _task_free_list; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
289 _task_free_list = task->next(); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
290 task->set_next(NULL); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
291 } else { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
292 task = new CompileTask(); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
293 DEBUG_ONLY(_num_allocated_tasks++;) |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
294 assert (_num_allocated_tasks < 10000, "Leaking compilation tasks?"); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
295 task->set_next(NULL); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
296 task->set_is_free(true); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
297 } |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
298 assert(task->is_free(), "Task must be free."); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
299 task->set_is_free(false); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
300 return task; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
301 } |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
302 |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
303 |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
304 /** |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
305 * Add a task to the free list. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
306 */ |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
307 void CompileTask::free(CompileTask* task) { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
308 MutexLocker locker(CompileTaskAlloc_lock); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
309 if (!task->is_free()) { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
310 task->set_code(NULL); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
311 assert(!task->lock()->is_locked(), "Should not be locked when freed"); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
312 JNIHandles::destroy_global(task->_method_holder); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
313 JNIHandles::destroy_global(task->_hot_method_holder); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
314 |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
315 task->set_is_free(true); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
316 task->set_next(_task_free_list); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
317 _task_free_list = task; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
318 } |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
319 } |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
320 |
0 | 321 void CompileTask::initialize(int compile_id, |
322 methodHandle method, | |
323 int osr_bci, | |
324 int comp_level, | |
325 methodHandle hot_method, | |
326 int hot_count, | |
327 const char* comment, | |
328 bool is_blocking) { | |
329 assert(!_lock->is_locked(), "bad locking"); | |
330 | |
331 _compile_id = compile_id; | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
332 _method = method(); |
7409 | 333 _method_holder = JNIHandles::make_global(method->method_holder()->klass_holder()); |
0 | 334 _osr_bci = osr_bci; |
335 _is_blocking = is_blocking; | |
336 _comp_level = comp_level; | |
337 _num_inlined_bytecodes = 0; | |
338 | |
339 _is_complete = false; | |
340 _is_success = false; | |
341 _code_handle = NULL; | |
342 | |
343 _hot_method = NULL; | |
7185
90273fc0a981
8000662: NPG: nashorn ant clean test262 out-of-memory with Java heap
coleenp
parents:
6940
diff
changeset
|
344 _hot_method_holder = NULL; |
0 | 345 _hot_count = hot_count; |
346 _time_queued = 0; // tidy | |
347 _comment = comment; | |
20469
dd89808e49ba
8049530: Provide descriptive failure reason for compilation tasks removed for the queue
vlivanov
parents:
20465
diff
changeset
|
348 _failure_reason = NULL; |
0 | 349 |
350 if (LogCompilation) { | |
351 _time_queued = os::elapsed_counter(); | |
352 if (hot_method.not_null()) { | |
353 if (hot_method == method) { | |
354 _hot_method = _method; | |
355 } else { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
356 _hot_method = hot_method(); |
7185
90273fc0a981
8000662: NPG: nashorn ant clean test262 out-of-memory with Java heap
coleenp
parents:
6940
diff
changeset
|
357 // only add loader or mirror if different from _method_holder |
7409 | 358 _hot_method_holder = JNIHandles::make_global(hot_method->method_holder()->klass_holder()); |
0 | 359 } |
360 } | |
361 } | |
362 | |
363 _next = NULL; | |
364 } | |
365 | |
366 // ------------------------------------------------------------------ | |
367 // CompileTask::code/set_code | |
368 nmethod* CompileTask::code() const { | |
369 if (_code_handle == NULL) return NULL; | |
370 return _code_handle->code(); | |
371 } | |
372 void CompileTask::set_code(nmethod* nm) { | |
373 if (_code_handle == NULL && nm == NULL) return; | |
374 guarantee(_code_handle != NULL, ""); | |
375 _code_handle->set_code(nm); | |
376 if (nm == NULL) _code_handle = NULL; // drop the handle also | |
377 } | |
378 | |
379 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
380 void CompileTask::mark_on_stack() { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
381 // Mark these methods as something redefine classes cannot remove. |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
382 _method->set_on_stack(true); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
383 if (_hot_method != NULL) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
384 _hot_method->set_on_stack(true); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
385 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
386 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
387 |
0 | 388 // ------------------------------------------------------------------ |
389 // CompileTask::print | |
390 void CompileTask::print() { | |
391 tty->print("<CompileTask compile_id=%d ", _compile_id); | |
392 tty->print("method="); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
393 _method->print_name(tty); |
0 | 394 tty->print_cr(" osr_bci=%d is_blocking=%s is_complete=%s is_success=%s>", |
395 _osr_bci, bool_to_str(_is_blocking), | |
396 bool_to_str(_is_complete), bool_to_str(_is_success)); | |
397 } | |
398 | |
1783 | 399 |
0 | 400 // ------------------------------------------------------------------ |
401 // CompileTask::print_line_on_error | |
402 // | |
403 // This function is called by fatal error handler when the thread | |
404 // causing troubles is a compiler thread. | |
405 // | |
406 // Do not grab any lock, do not allocate memory. | |
407 // | |
408 // Otherwise it's the same as CompileTask::print_line() | |
409 // | |
410 void CompileTask::print_line_on_error(outputStream* st, char* buf, int buflen) { | |
411 // print compiler name | |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
412 st->print("%s:", CompileBroker::compiler_name(comp_level())); |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
413 print_compilation(st); |
0 | 414 } |
415 | |
416 // ------------------------------------------------------------------ | |
417 // CompileTask::print_line | |
418 void CompileTask::print_line() { | |
419 ttyLocker ttyl; // keep the following output all in one block | |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
420 print_compilation(); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
421 } |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
422 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
423 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
424 // ------------------------------------------------------------------ |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
425 // CompileTask::print_compilation_impl |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
426 void CompileTask::print_compilation_impl(outputStream* st, Method* method, int compile_id, int comp_level, |
4872
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
427 bool is_osr_method, int osr_bci, bool is_blocking, |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
428 const char* msg, bool short_form) { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
429 if (!short_form) { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
430 st->print("%7d ", (int) st->time_stamp().milliseconds()); // print timestamp |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
431 } |
15494
e20a45d17181
Move CIPrintCompilerName handling into CompileTask::print_compilation_impl
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
15487
diff
changeset
|
432 // print compiler name if requested |
16180
404d2e86488a
CIPrintCompilerName: compiler name should be printed on supplied stream
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
16145
diff
changeset
|
433 if (CIPrintCompilerName) st->print("%s:", CompileBroker::compiler_name(comp_level)); |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
434 st->print("%4d ", compile_id); // print compilation number |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
435 |
3441
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
436 // For unloaded methods the transition to zombie occurs after the |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
437 // method is cleared so it's impossible to report accurate |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
438 // information for that case. |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
439 bool is_synchronized = false; |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
440 bool has_exception_handler = false; |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
441 bool is_native = false; |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
442 if (method != NULL) { |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
443 is_synchronized = method->is_synchronized(); |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
444 has_exception_handler = method->has_exception_handler(); |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
445 is_native = method->is_native(); |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
446 } |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
447 // method attributes |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
448 const char compile_type = is_osr_method ? '%' : ' '; |
3441
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
449 const char sync_char = is_synchronized ? 's' : ' '; |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
450 const char exception_char = has_exception_handler ? '!' : ' '; |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
451 const char blocking_char = is_blocking ? 'b' : ' '; |
3441
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
452 const char native_char = is_native ? 'n' : ' '; |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
453 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
454 // print method attributes |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
455 st->print("%c%c%c%c%c ", compile_type, sync_char, exception_char, blocking_char, native_char); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
456 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
457 if (TieredCompilation) { |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
458 if (comp_level != -1) st->print("%d ", comp_level); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
459 else st->print("- "); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
460 } |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
461 st->print(" "); // more indent |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
462 |
3441
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
463 if (method == NULL) { |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
464 st->print("(method)"); |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
465 } else { |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
466 method->print_short_name(st); |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
467 if (is_osr_method) { |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
468 st->print(" @ %d", osr_bci); |
f56542cb325a
7050554: JSR 292 - need optimization for selectAlternative
never
parents:
3294
diff
changeset
|
469 } |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
470 if (method->is_native()) |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
471 st->print(" (native)"); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
472 else |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
473 st->print(" (%d bytes)", method->code_size()); |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
474 } |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
475 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
476 if (msg != NULL) { |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
477 st->print(" %s", msg); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
478 } |
4872
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
479 if (!short_form) { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
480 st->cr(); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
481 } |
0 | 482 } |
483 | |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
484 // ------------------------------------------------------------------ |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
485 // CompileTask::print_inlining |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
486 void CompileTask::print_inlining(outputStream* st, ciMethod* method, int inline_level, int bci, const char* msg) { |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
487 // 1234567 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
488 st->print(" "); // print timestamp |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
489 // 1234 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
490 st->print(" "); // print compilation number |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
491 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
492 // method attributes |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
493 if (method->is_loaded()) { |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
494 const char sync_char = method->is_synchronized() ? 's' : ' '; |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
495 const char exception_char = method->has_exception_handlers() ? '!' : ' '; |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
496 const char monitors_char = method->has_monitor_bytecodes() ? 'm' : ' '; |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
497 |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
498 // print method attributes |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
499 st->print(" %c%c%c ", sync_char, exception_char, monitors_char); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
500 } else { |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
501 // %s!bn |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
502 st->print(" "); // print method attributes |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
503 } |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
504 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
505 if (TieredCompilation) { |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
506 st->print(" "); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
507 } |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
508 st->print(" "); // more indent |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
509 st->print(" "); // initial inlining indent |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
510 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
511 for (int i = 0; i < inline_level; i++) st->print(" "); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
512 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
513 st->print("@ %d ", bci); // print bci |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
514 method->print_short_name(st); |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
515 if (method->is_loaded()) |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
516 st->print(" (%d bytes)", method->code_size()); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
517 else |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
518 st->print(" (not loaded)"); |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
519 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
520 if (msg != NULL) { |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
521 st->print(" %s", msg); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
522 } |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
523 st->cr(); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
524 } |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
525 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
526 // ------------------------------------------------------------------ |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
527 // CompileTask::print_inline_indent |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
528 void CompileTask::print_inline_indent(int inline_level, outputStream* st) { |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
529 // 1234567 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
530 st->print(" "); // print timestamp |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
531 // 1234 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
532 st->print(" "); // print compilation number |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
533 // %s!bn |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
534 st->print(" "); // print method attributes |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
535 if (TieredCompilation) { |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
536 st->print(" "); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
537 } |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
538 st->print(" "); // more indent |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
539 st->print(" "); // initial inlining indent |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
540 for (int i = 0; i < inline_level; i++) st->print(" "); |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
541 } |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
542 |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
543 // ------------------------------------------------------------------ |
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
544 // CompileTask::print_compilation |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
545 void CompileTask::print_compilation(outputStream* st, const char* msg, bool short_form) { |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
546 bool is_osr_method = osr_bci() != InvocationEntryBci; |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
547 print_compilation_impl(st, method(), compile_id(), comp_level(), is_osr_method, osr_bci(), is_blocking(), msg, short_form); |
2405
3d58a4983660
7022998: JSR 292 recursive method handle calls inline themselves infinitely
twisti
parents:
2403
diff
changeset
|
548 } |
0 | 549 |
550 // ------------------------------------------------------------------ | |
551 // CompileTask::log_task | |
552 void CompileTask::log_task(xmlStream* log) { | |
553 Thread* thread = Thread::current(); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
554 methodHandle method(thread, this->method()); |
0 | 555 ResourceMark rm(thread); |
556 | |
557 // <task id='9' method='M' osr_bci='X' level='1' blocking='1' stamp='1.234'> | |
8730
b7c2c5b2572c
8005772: Stubs report compile id -1 in phase events
neliasso
parents:
7998
diff
changeset
|
558 log->print(" compile_id='%d'", _compile_id); |
0 | 559 if (_osr_bci != CompileBroker::standard_entry_bci) { |
560 log->print(" compile_kind='osr'"); // same as nmethod::compile_kind | |
561 } // else compile_kind='c2c' | |
562 if (!method.is_null()) log->method(method); | |
563 if (_osr_bci != CompileBroker::standard_entry_bci) { | |
564 log->print(" osr_bci='%d'", _osr_bci); | |
565 } | |
15487
07fac8558d7b
update state flag after initialization to allow other compiler threads to execute
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
15463
diff
changeset
|
566 // Always print the level in tiered. |
07fac8558d7b
update state flag after initialization to allow other compiler threads to execute
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
15463
diff
changeset
|
567 if (_comp_level != CompLevel_highest_tier || TieredCompilation) { |
0 | 568 log->print(" level='%d'", _comp_level); |
569 } | |
570 if (_is_blocking) { | |
571 log->print(" blocking='1'"); | |
572 } | |
573 log->stamp(); | |
574 } | |
575 | |
576 | |
577 // ------------------------------------------------------------------ | |
578 // CompileTask::log_task_queued | |
579 void CompileTask::log_task_queued() { | |
580 Thread* thread = Thread::current(); | |
581 ttyLocker ttyl; | |
582 ResourceMark rm(thread); | |
583 | |
584 xtty->begin_elem("task_queued"); | |
585 log_task(xtty); | |
586 if (_comment != NULL) { | |
587 xtty->print(" comment='%s'", _comment); | |
588 } | |
589 if (_hot_method != NULL) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
590 methodHandle hot(thread, _hot_method); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
591 methodHandle method(thread, _method); |
0 | 592 if (hot() != method()) { |
593 xtty->method(hot); | |
594 } | |
595 } | |
596 if (_hot_count != 0) { | |
597 xtty->print(" hot_count='%d'", _hot_count); | |
598 } | |
599 xtty->end_elem(); | |
600 } | |
601 | |
602 | |
603 // ------------------------------------------------------------------ | |
19943
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
604 // CompileTask::log_task_dequeued |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
605 void CompileTask::log_task_dequeued(const char* comment) { |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
606 if (LogCompilation && xtty != NULL) { |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
607 Thread* thread = Thread::current(); |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
608 ttyLocker ttyl; |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
609 ResourceMark rm(thread); |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
610 |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
611 xtty->begin_elem("task_dequeued"); |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
612 log_task(xtty); |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
613 if (comment != NULL) { |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
614 xtty->print(" comment='%s'", comment); |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
615 } |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
616 xtty->end_elem(); |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
617 } |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
618 } |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
619 |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
620 |
5119e7f07d93
Log dequeue of methods from CompileQueue
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19895
diff
changeset
|
621 // ------------------------------------------------------------------ |
0 | 622 // CompileTask::log_task_start |
623 void CompileTask::log_task_start(CompileLog* log) { | |
624 log->begin_head("task"); | |
625 log_task(log); | |
626 log->end_head(); | |
627 } | |
628 | |
629 | |
630 // ------------------------------------------------------------------ | |
631 // CompileTask::log_task_done | |
632 void CompileTask::log_task_done(CompileLog* log) { | |
633 Thread* thread = Thread::current(); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
634 methodHandle method(thread, this->method()); |
0 | 635 ResourceMark rm(thread); |
636 | |
20469
dd89808e49ba
8049530: Provide descriptive failure reason for compilation tasks removed for the queue
vlivanov
parents:
20465
diff
changeset
|
637 if (!_is_success) { |
dd89808e49ba
8049530: Provide descriptive failure reason for compilation tasks removed for the queue
vlivanov
parents:
20465
diff
changeset
|
638 const char* reason = _failure_reason != NULL ? _failure_reason : "unknown"; |
dd89808e49ba
8049530: Provide descriptive failure reason for compilation tasks removed for the queue
vlivanov
parents:
20465
diff
changeset
|
639 log->elem("failure reason='%s'", reason); |
dd89808e49ba
8049530: Provide descriptive failure reason for compilation tasks removed for the queue
vlivanov
parents:
20465
diff
changeset
|
640 } |
dd89808e49ba
8049530: Provide descriptive failure reason for compilation tasks removed for the queue
vlivanov
parents:
20465
diff
changeset
|
641 |
0 | 642 // <task_done ... stamp='1.234'> </task> |
643 nmethod* nm = code(); | |
644 log->begin_elem("task_done success='%d' nmsize='%d' count='%d'", | |
1748 | 645 _is_success, nm == NULL ? 0 : nm->content_size(), |
0 | 646 method->invocation_count()); |
647 int bec = method->backedge_count(); | |
648 if (bec != 0) log->print(" backedge_count='%d'", bec); | |
649 // Note: "_is_complete" is about to be set, but is not. | |
650 if (_num_inlined_bytecodes != 0) { | |
651 log->print(" inlined_bytes='%d'", _num_inlined_bytecodes); | |
652 } | |
653 log->stamp(); | |
654 log->end_elem(); | |
655 log->tail("task"); | |
656 log->clear_identities(); // next task will have different CI | |
657 if (log->unflushed_count() > 2000) { | |
658 log->flush(); | |
659 } | |
660 log->mark_file_end(); | |
661 } | |
662 | |
663 | |
664 | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
665 /** |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
666 * Add a CompileTask to a CompileQueue |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
667 */ |
0 | 668 void CompileQueue::add(CompileTask* task) { |
669 assert(lock()->owned_by_self(), "must own lock"); | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
670 assert(!CompileBroker::is_compilation_disabled_forever(), "Do not add task if compilation is turned off forever"); |
0 | 671 |
672 task->set_next(NULL); | |
1783 | 673 task->set_prev(NULL); |
0 | 674 |
675 if (_last == NULL) { | |
676 // The compile queue is empty. | |
677 assert(_first == NULL, "queue is empty"); | |
678 _first = task; | |
679 _last = task; | |
680 } else { | |
681 // Append the task to the queue. | |
682 assert(_last->next() == NULL, "not last"); | |
683 _last->set_next(task); | |
1783 | 684 task->set_prev(_last); |
0 | 685 _last = task; |
686 } | |
1783 | 687 ++_size; |
0 | 688 |
689 // Mark the method as being in the compile queue. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
690 task->method()->set_queued_for_compilation(); |
0 | 691 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
692 NOT_PRODUCT(print();) |
0 | 693 |
694 if (LogCompilation && xtty != NULL) { | |
695 task->log_task_queued(); | |
696 } | |
697 | |
698 // Notify CompilerThreads that a task is available. | |
1783 | 699 lock()->notify_all(); |
0 | 700 } |
701 | |
20590
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
702 /** |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
703 * Empties compilation queue by putting all compilation tasks onto |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
704 * a freelist. Furthermore, the method wakes up all threads that are |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
705 * waiting on a compilation task to finish. This can happen if background |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
706 * compilation is disabled. |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
707 */ |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
708 void CompileQueue::free_all() { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
709 MutexLocker mu(lock()); |
20590
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
710 CompileTask* next = _first; |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
711 |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
712 // Iterate over all tasks in the compile queue |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
713 while (next != NULL) { |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
714 CompileTask* current = next; |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
715 next = current->next(); |
20591
28051f14c328
8043158: Crash in CodeSweeperSweepNoFlushTest in CompileQueue::free_all()
anoll
parents:
20590
diff
changeset
|
716 { |
28051f14c328
8043158: Crash in CodeSweeperSweepNoFlushTest in CompileQueue::free_all()
anoll
parents:
20590
diff
changeset
|
717 // Wake up thread that blocks on the compile task. |
28051f14c328
8043158: Crash in CodeSweeperSweepNoFlushTest in CompileQueue::free_all()
anoll
parents:
20590
diff
changeset
|
718 MutexLocker ct_lock(current->lock()); |
28051f14c328
8043158: Crash in CodeSweeperSweepNoFlushTest in CompileQueue::free_all()
anoll
parents:
20590
diff
changeset
|
719 current->lock()->notify(); |
28051f14c328
8043158: Crash in CodeSweeperSweepNoFlushTest in CompileQueue::free_all()
anoll
parents:
20590
diff
changeset
|
720 } |
20590
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
721 // Put the task back on the freelist. |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
722 CompileTask::free(current); |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
723 } |
20590
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
724 _first = NULL; |
c47fcf523fff
8042428: CompileQueue::free_all() code is incorrect
anoll
parents:
20589
diff
changeset
|
725 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
726 // Wake up all threads that block on the queue. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
727 lock()->notify_all(); |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
728 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
729 |
0 | 730 // ------------------------------------------------------------------ |
731 // CompileQueue::get | |
732 // | |
733 // Get the next CompileTask from a CompileQueue | |
734 CompileTask* CompileQueue::get() { | |
1538
bfe29ec02863
6950075: nmethod sweeper should operate concurrently
never
parents:
1353
diff
changeset
|
735 NMethodSweeper::possibly_sweep(); |
bfe29ec02863
6950075: nmethod sweeper should operate concurrently
never
parents:
1353
diff
changeset
|
736 |
0 | 737 MutexLocker locker(lock()); |
12324
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
738 // If _first is NULL we have no more compile jobs. There are two reasons for |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
739 // having no compile jobs: First, we compiled everything we wanted. Second, |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
740 // we ran out of code cache so compilation has been disabled. In the latter |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
741 // case we perform code cache sweeps to free memory such that we can re-enable |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
742 // compilation. |
0 | 743 while (_first == NULL) { |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
744 // Exit loop if compilation is disabled forever |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
745 if (CompileBroker::is_compilation_disabled_forever()) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
746 return NULL; |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
747 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
748 |
12324
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
749 if (UseCodeCacheFlushing && !CompileBroker::should_compile_new_jobs()) { |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
750 // Wait a certain amount of time to possibly do another sweep. |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
751 // We must wait until stack scanning has happened so that we can |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
752 // transition a method's state from 'not_entrant' to 'zombie'. |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
753 long wait_time = NmethodSweepCheckInterval * 1000; |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
754 if (FLAG_IS_DEFAULT(NmethodSweepCheckInterval)) { |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
755 // Only one thread at a time can do sweeping. Scale the |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
756 // wait time according to the number of compiler threads. |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
757 // As a result, the next sweep is likely to happen every 100ms |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
758 // with an arbitrary number of threads that do sweeping. |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
759 wait_time = 100 * CICompilerCount; |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
760 } |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
761 bool timeout = lock()->wait(!Mutex::_no_safepoint_check_flag, wait_time); |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
762 if (timeout) { |
1538
bfe29ec02863
6950075: nmethod sweeper should operate concurrently
never
parents:
1353
diff
changeset
|
763 MutexUnlocker ul(lock()); |
bfe29ec02863
6950075: nmethod sweeper should operate concurrently
never
parents:
1353
diff
changeset
|
764 NMethodSweeper::possibly_sweep(); |
bfe29ec02863
6950075: nmethod sweeper should operate concurrently
never
parents:
1353
diff
changeset
|
765 } |
bfe29ec02863
6950075: nmethod sweeper should operate concurrently
never
parents:
1353
diff
changeset
|
766 } else { |
12324
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
767 // If there are no compilation tasks and we can compile new jobs |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
768 // (i.e., there is enough free space in the code cache) there is |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
769 // no need to invoke the sweeper. As a result, the hotness of methods |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
770 // remains unchanged. This behavior is desired, since we want to keep |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
771 // the stable state, i.e., we do not want to evict methods from the |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
772 // code cache if it is unnecessary. |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
773 // We need a timed wait here, since compiler threads can exit if compilation |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
774 // is disabled forever. We use 5 seconds wait time; the exiting of compiler threads |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
775 // is not critical and we do not want idle compiler threads to wake up too often. |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
776 lock()->wait(!Mutex::_no_safepoint_check_flag, 5*1000); |
1538
bfe29ec02863
6950075: nmethod sweeper should operate concurrently
never
parents:
1353
diff
changeset
|
777 } |
0 | 778 } |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
779 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
780 if (CompileBroker::is_compilation_disabled_forever()) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
781 return NULL; |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
782 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
783 |
20465
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
784 CompileTask* task; |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
785 { |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
786 No_Safepoint_Verifier nsv; |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
787 task = CompilationPolicy::policy()->select_task(this); |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
788 } |
1783 | 789 remove(task); |
20465
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
790 purge_stale_tasks(); // may temporarily release MCQ lock |
1783 | 791 return task; |
792 } | |
0 | 793 |
20465
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
794 // Clean & deallocate stale compile tasks. |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
795 // Temporarily releases MethodCompileQueue lock. |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
796 void CompileQueue::purge_stale_tasks() { |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
797 assert(lock()->owned_by_self(), "must own lock"); |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
798 if (_first_stale != NULL) { |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
799 // Stale tasks are purged when MCQ lock is released, |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
800 // but _first_stale updates are protected by MCQ lock. |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
801 // Once task processing starts and MCQ lock is released, |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
802 // other compiler threads can reuse _first_stale. |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
803 CompileTask* head = _first_stale; |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
804 _first_stale = NULL; |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
805 { |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
806 MutexUnlocker ul(lock()); |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
807 for (CompileTask* task = head; task != NULL; ) { |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
808 CompileTask* next_task = task->next(); |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
809 CompileTaskWrapper ctw(task); // Frees the task |
20469
dd89808e49ba
8049530: Provide descriptive failure reason for compilation tasks removed for the queue
vlivanov
parents:
20465
diff
changeset
|
810 task->set_failure_reason("stale task"); |
20465
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
811 task = next_task; |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
812 } |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
813 } |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
814 } |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
815 } |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
816 |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
817 void CompileQueue::remove(CompileTask* task) { |
1783 | 818 assert(lock()->owned_by_self(), "must own lock"); |
819 if (task->prev() != NULL) { | |
820 task->prev()->set_next(task->next()); | |
821 } else { | |
822 // max is the first element | |
823 assert(task == _first, "Sanity"); | |
824 _first = task->next(); | |
0 | 825 } |
826 | |
1783 | 827 if (task->next() != NULL) { |
828 task->next()->set_prev(task->prev()); | |
829 } else { | |
830 // max is the last element | |
831 assert(task == _last, "Sanity"); | |
832 _last = task->prev(); | |
833 } | |
834 --_size; | |
0 | 835 } |
836 | |
20465
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
837 void CompileQueue::remove_and_mark_stale(CompileTask* task) { |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
838 assert(lock()->owned_by_self(), "must own lock"); |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
839 remove(task); |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
840 |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
841 // Enqueue the task for reclamation (should be done outside MCQ lock) |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
842 task->set_next(_first_stale); |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
843 task->set_prev(NULL); |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
844 _first_stale = task; |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
845 } |
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
846 |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
847 // methods in the compile queue need to be marked as used on the stack |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
848 // so that they don't get reclaimed by Redefine Classes |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
849 void CompileQueue::mark_on_stack() { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
850 CompileTask* task = _first; |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
851 while (task != NULL) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
852 task->mark_on_stack(); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
853 task = task->next(); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
854 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
855 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
856 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
857 #ifndef PRODUCT |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
858 /** |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
859 * Print entire compilation queue. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
860 */ |
0 | 861 void CompileQueue::print() { |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
862 if (CIPrintCompileQueue) { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
863 ttyLocker ttyl; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
864 tty->print_cr("Contents of %s", name()); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
865 tty->print_cr("----------------------"); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
866 CompileTask* task = _first; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
867 while (task != NULL) { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
868 task->print_line(); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
869 task = task->next(); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
870 } |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
871 tty->print_cr("----------------------"); |
0 | 872 } |
873 } | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
874 #endif // PRODUCT |
0 | 875 |
876 CompilerCounters::CompilerCounters(const char* thread_name, int instance, TRAPS) { | |
877 | |
878 _current_method[0] = '\0'; | |
879 _compile_type = CompileBroker::no_compile; | |
880 | |
881 if (UsePerfData) { | |
882 ResourceMark rm; | |
883 | |
884 // create the thread instance name space string - don't create an | |
885 // instance subspace if instance is -1 - keeps the adapterThread | |
886 // counters from having a ".0" namespace. | |
887 const char* thread_i = (instance == -1) ? thread_name : | |
888 PerfDataManager::name_space(thread_name, instance); | |
889 | |
890 | |
891 char* name = PerfDataManager::counter_name(thread_i, "method"); | |
892 _perf_current_method = | |
893 PerfDataManager::create_string_variable(SUN_CI, name, | |
894 cmname_buffer_length, | |
895 _current_method, CHECK); | |
896 | |
897 name = PerfDataManager::counter_name(thread_i, "type"); | |
898 _perf_compile_type = PerfDataManager::create_variable(SUN_CI, name, | |
899 PerfData::U_None, | |
900 (jlong)_compile_type, | |
901 CHECK); | |
902 | |
903 name = PerfDataManager::counter_name(thread_i, "time"); | |
904 _perf_time = PerfDataManager::create_counter(SUN_CI, name, | |
905 PerfData::U_Ticks, CHECK); | |
906 | |
907 name = PerfDataManager::counter_name(thread_i, "compiles"); | |
908 _perf_compiles = PerfDataManager::create_counter(SUN_CI, name, | |
909 PerfData::U_Events, CHECK); | |
910 } | |
911 } | |
912 | |
913 // ------------------------------------------------------------------ | |
914 // CompileBroker::compilation_init | |
915 // | |
916 // Initialize the Compilation object | |
917 void CompileBroker::compilation_init() { | |
918 _last_method_compiled[0] = '\0'; | |
919 | |
13016
a196f1aaec86
8026949: -Xint flag prints wrong warning: Initialization of C1 thread failed (no space to run compilers)
anoll
parents:
12965
diff
changeset
|
920 // No need to initialize compilation system if we do not use it. |
a196f1aaec86
8026949: -Xint flag prints wrong warning: Initialization of C1 thread failed (no space to run compilers)
anoll
parents:
12965
diff
changeset
|
921 if (!UseCompiler) { |
a196f1aaec86
8026949: -Xint flag prints wrong warning: Initialization of C1 thread failed (no space to run compilers)
anoll
parents:
12965
diff
changeset
|
922 return; |
a196f1aaec86
8026949: -Xint flag prints wrong warning: Initialization of C1 thread failed (no space to run compilers)
anoll
parents:
12965
diff
changeset
|
923 } |
1970 | 924 #ifndef SHARK |
0 | 925 // Set the interface to the current compiler(s). |
1783 | 926 int c1_count = CompilationPolicy::policy()->compiler_count(CompLevel_simple); |
927 int c2_count = CompilationPolicy::policy()->compiler_count(CompLevel_full_optimization); | |
22298 | 928 #if INCLUDE_JVMCI |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
929 JVMCICompiler* jvmci = new JVMCICompiler(); |
7154
5d0bb7d52783
changes to support Graal co-existing with the other HotSpot compiler(s) and being used for explicit compilation requests and code installation via the Graal API
Doug Simon <doug.simon@oracle.com>
parents:
7125
diff
changeset
|
930 #endif |
14107
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13969
diff
changeset
|
931 |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
932 #if defined(COMPILERJVMCI) |
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
933 _compilers[1] = jvmci; |
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
934 if (FLAG_IS_DEFAULT(JVMCIThreads)) { |
22449
cfd7ebda543b
run JVMCI bootstrap iff BootstrapJVMCI is true
Doug Simon <doug.simon@oracle.com>
parents:
22298
diff
changeset
|
935 if (BootstrapJVMCI) { |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
936 // JVMCI will bootstrap so give it more threads |
17166
9df38e5fbed6
Limit the maximum Graal thread amount to 32 if nothing else is specified (On SPARC with 256 threads we exceed code cache size)
Stefan Anzinger <stefan.anzinger@oracle.com>
parents:
16625
diff
changeset
|
937 c2_count = MIN2(32, os::active_processor_count()); |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16185
diff
changeset
|
938 } |
16145
237508bd73b9
when Graal bootstrap is enabled (explicitly or implicitly), allocate same number of Graal compiler threads in -UseGraalCompilationQueue as in +UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16142
diff
changeset
|
939 } else { |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
940 c2_count = JVMCIThreads; |
16145
237508bd73b9
when Graal bootstrap is enabled (explicitly or implicitly), allocate same number of Graal compiler threads in -UseGraalCompilationQueue as in +UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16142
diff
changeset
|
941 } |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
942 if (FLAG_IS_DEFAULT(JVMCIHostThreads)) { |
19895
c3d9637c98ed
Add GraalHostThreads flag to overwrite the number of threads spawned for the baseline compiler.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
943 } else { |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
944 c1_count = JVMCIHostThreads; |
19895
c3d9637c98ed
Add GraalHostThreads flag to overwrite the number of threads spawned for the baseline compiler.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
945 } |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
946 #endif // COMPILERJVMCI |
14107
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13969
diff
changeset
|
947 |
0 | 948 #ifdef COMPILER1 |
1783 | 949 if (c1_count > 0) { |
950 _compilers[0] = new Compiler(); | |
951 } | |
0 | 952 #endif // COMPILER1 |
953 | |
954 #ifdef COMPILER2 | |
1783 | 955 if (c2_count > 0) { |
956 _compilers[1] = new C2Compiler(); | |
957 } | |
0 | 958 #endif // COMPILER2 |
959 | |
1970 | 960 #else // SHARK |
961 int c1_count = 0; | |
962 int c2_count = 1; | |
963 | |
964 _compilers[1] = new SharkCompiler(); | |
965 #endif // SHARK | |
1692 | 966 |
0 | 967 // Start the CompilerThreads |
1783 | 968 init_compiler_threads(c1_count, c2_count); |
0 | 969 // totalTime performance counter is always created as it is required |
970 // by the implementation of java.lang.management.CompilationMBean. | |
971 { | |
972 EXCEPTION_MARK; | |
973 _perf_total_compilation = | |
974 PerfDataManager::create_counter(JAVA_CI, "totalTime", | |
975 PerfData::U_Ticks, CHECK); | |
976 } | |
977 | |
978 | |
979 if (UsePerfData) { | |
980 | |
981 EXCEPTION_MARK; | |
982 | |
983 // create the jvmstat performance counters | |
984 _perf_osr_compilation = | |
985 PerfDataManager::create_counter(SUN_CI, "osrTime", | |
986 PerfData::U_Ticks, CHECK); | |
987 | |
988 _perf_standard_compilation = | |
989 PerfDataManager::create_counter(SUN_CI, "standardTime", | |
990 PerfData::U_Ticks, CHECK); | |
991 | |
992 _perf_total_bailout_count = | |
993 PerfDataManager::create_counter(SUN_CI, "totalBailouts", | |
994 PerfData::U_Events, CHECK); | |
995 | |
996 _perf_total_invalidated_count = | |
997 PerfDataManager::create_counter(SUN_CI, "totalInvalidates", | |
998 PerfData::U_Events, CHECK); | |
999 | |
1000 _perf_total_compile_count = | |
1001 PerfDataManager::create_counter(SUN_CI, "totalCompiles", | |
1002 PerfData::U_Events, CHECK); | |
1003 _perf_total_osr_compile_count = | |
1004 PerfDataManager::create_counter(SUN_CI, "osrCompiles", | |
1005 PerfData::U_Events, CHECK); | |
1006 | |
1007 _perf_total_standard_compile_count = | |
1008 PerfDataManager::create_counter(SUN_CI, "standardCompiles", | |
1009 PerfData::U_Events, CHECK); | |
1010 | |
1011 _perf_sum_osr_bytes_compiled = | |
1012 PerfDataManager::create_counter(SUN_CI, "osrBytes", | |
1013 PerfData::U_Bytes, CHECK); | |
1014 | |
1015 _perf_sum_standard_bytes_compiled = | |
1016 PerfDataManager::create_counter(SUN_CI, "standardBytes", | |
1017 PerfData::U_Bytes, CHECK); | |
1018 | |
1019 _perf_sum_nmethod_size = | |
1020 PerfDataManager::create_counter(SUN_CI, "nmethodSize", | |
1021 PerfData::U_Bytes, CHECK); | |
1022 | |
1023 _perf_sum_nmethod_code_size = | |
1024 PerfDataManager::create_counter(SUN_CI, "nmethodCodeSize", | |
1025 PerfData::U_Bytes, CHECK); | |
1026 | |
1027 _perf_last_method = | |
1028 PerfDataManager::create_string_variable(SUN_CI, "lastMethod", | |
1029 CompilerCounters::cmname_buffer_length, | |
1030 "", CHECK); | |
1031 | |
1032 _perf_last_failed_method = | |
1033 PerfDataManager::create_string_variable(SUN_CI, "lastFailedMethod", | |
1034 CompilerCounters::cmname_buffer_length, | |
1035 "", CHECK); | |
1036 | |
1037 _perf_last_invalidated_method = | |
1038 PerfDataManager::create_string_variable(SUN_CI, "lastInvalidatedMethod", | |
1039 CompilerCounters::cmname_buffer_length, | |
1040 "", CHECK); | |
1041 | |
1042 _perf_last_compile_type = | |
1043 PerfDataManager::create_variable(SUN_CI, "lastType", | |
1044 PerfData::U_None, | |
1045 (jlong)CompileBroker::no_compile, | |
1046 CHECK); | |
1047 | |
1048 _perf_last_compile_size = | |
1049 PerfDataManager::create_variable(SUN_CI, "lastSize", | |
1050 PerfData::U_Bytes, | |
1051 (jlong)CompileBroker::no_compile, | |
1052 CHECK); | |
1053 | |
1054 | |
1055 _perf_last_failed_type = | |
1056 PerfDataManager::create_variable(SUN_CI, "lastFailedType", | |
1057 PerfData::U_None, | |
1058 (jlong)CompileBroker::no_compile, | |
1059 CHECK); | |
1060 | |
1061 _perf_last_invalidated_type = | |
1062 PerfDataManager::create_variable(SUN_CI, "lastInvalidatedType", | |
1063 PerfData::U_None, | |
1064 (jlong)CompileBroker::no_compile, | |
1065 CHECK); | |
1066 } | |
1067 | |
1068 _initialized = true; | |
1069 } | |
1070 | |
1071 | |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1072 CompilerThread* CompileBroker::make_compiler_thread(const char* name, CompileQueue* queue, CompilerCounters* counters, |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1073 AbstractCompiler* comp, TRAPS) { |
0 | 1074 CompilerThread* compiler_thread = NULL; |
1075 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1076 Klass* k = |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
1077 SystemDictionary::resolve_or_fail(vmSymbols::java_lang_Thread(), |
0 | 1078 true, CHECK_0); |
1079 instanceKlassHandle klass (THREAD, k); | |
1080 instanceHandle thread_oop = klass->allocate_instance_handle(CHECK_0); | |
1081 Handle string = java_lang_String::create_from_str(name, CHECK_0); | |
1082 | |
1083 // Initialize thread_oop to put it into the system threadGroup | |
1084 Handle thread_group (THREAD, Universe::system_thread_group()); | |
1085 JavaValue result(T_VOID); | |
1086 JavaCalls::call_special(&result, thread_oop, | |
1087 klass, | |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
1088 vmSymbols::object_initializer_name(), |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
1089 vmSymbols::threadgroup_string_void_signature(), |
0 | 1090 thread_group, |
1091 string, | |
1092 CHECK_0); | |
1093 | |
1094 { | |
1095 MutexLocker mu(Threads_lock, THREAD); | |
1096 compiler_thread = new CompilerThread(queue, counters); | |
1097 // At this point the new CompilerThread data-races with this startup | |
1098 // thread (which I believe is the primoridal thread and NOT the VM | |
1099 // thread). This means Java bytecodes being executed at startup can | |
1100 // queue compile jobs which will run at whatever default priority the | |
1101 // newly created CompilerThread runs at. | |
1102 | |
1103 | |
1104 // At this point it may be possible that no osthread was created for the | |
1105 // JavaThread due to lack of memory. We would have to throw an exception | |
1106 // in that case. However, since this must work and we do not allow | |
1107 // exceptions anyway, check and abort if this fails. | |
1108 | |
1109 if (compiler_thread == NULL || compiler_thread->osthread() == NULL){ | |
1110 vm_exit_during_initialization("java.lang.OutOfMemoryError", | |
1111 "unable to create new native thread"); | |
1112 } | |
1113 | |
1114 java_lang_Thread::set_thread(thread_oop(), compiler_thread); | |
1115 | |
1116 // Note that this only sets the JavaThread _priority field, which by | |
1117 // definition is limited to Java priorities and not OS priorities. | |
1118 // The os-priority is set in the CompilerThread startup code itself | |
4854
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1119 |
0 | 1120 java_lang_Thread::set_priority(thread_oop(), NearMaxPriority); |
4854
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1121 |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1122 // Note that we cannot call os::set_priority because it expects Java |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1123 // priorities and we are *explicitly* using OS priorities so that it's |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1124 // possible to set the compiler thread priority higher than any Java |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1125 // thread. |
0 | 1126 |
4854
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1127 int native_prio = CompilerThreadPriority; |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1128 if (native_prio == -1) { |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1129 if (UseCriticalCompilerThreadPriority) { |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1130 native_prio = os::java_to_os_priority[CriticalPriority]; |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1131 } else { |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1132 native_prio = os::java_to_os_priority[NearMaxPriority]; |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1133 } |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1134 } |
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1135 os::set_native_priority(compiler_thread, native_prio); |
0 | 1136 |
1137 java_lang_Thread::set_daemon(thread_oop()); | |
1138 | |
1139 compiler_thread->set_threadObj(thread_oop()); | |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1140 compiler_thread->set_compiler(comp); |
0 | 1141 Threads::add(compiler_thread); |
1142 Thread::start(compiler_thread); | |
1143 } | |
4854
de268c8a8075
7082553: Interpret Thread.setPriority(Thread.MAX_PRIORITY) to mean FX60 on Solaris 10 and 11
phh
parents:
4122
diff
changeset
|
1144 |
0 | 1145 // Let go of Threads_lock before yielding |
1146 os::yield(); // make sure that the compiler thread is started early (especially helpful on SOLARIS) | |
1147 | |
1148 return compiler_thread; | |
1149 } | |
1150 | |
1151 | |
1783 | 1152 void CompileBroker::init_compiler_threads(int c1_compiler_count, int c2_compiler_count) { |
0 | 1153 EXCEPTION_MARK; |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
1154 #if !defined(ZERO) && !defined(SHARK) && !defined(COMPILERJVMCI) |
1783 | 1155 assert(c2_compiler_count > 0 || c1_compiler_count > 0, "No compilers?"); |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
1156 #endif // !ZERO && !SHARK && !COMPILERJVMCI |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1157 // Initialize the compilation queue |
1783 | 1158 if (c2_compiler_count > 0) { |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1159 _c2_compile_queue = new CompileQueue("C2 CompileQueue", MethodCompileQueue_lock); |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1160 _compilers[1]->set_num_compiler_threads(c2_compiler_count); |
1783 | 1161 } |
1162 if (c1_compiler_count > 0) { | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1163 _c1_compile_queue = new CompileQueue("C1 CompileQueue", MethodCompileQueue_lock); |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1164 _compilers[0]->set_num_compiler_threads(c1_compiler_count); |
1783 | 1165 } |
0 | 1166 |
1783 | 1167 int compiler_count = c1_compiler_count + c2_compiler_count; |
1168 | |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1169 _compiler_threads = |
6197 | 1170 new (ResourceObj::C_HEAP, mtCompiler) GrowableArray<CompilerThread*>(compiler_count, true); |
0 | 1171 |
1172 char name_buffer[256]; | |
1783 | 1173 for (int i = 0; i < c2_compiler_count; i++) { |
0 | 1174 // Create a name for our thread. |
16102
64d8c6c8a00c
derive native compiler thread name from Compiler::name
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
16006
diff
changeset
|
1175 sprintf(name_buffer, "%s CompilerThread%d", _compilers[1]->name(), i); |
0 | 1176 CompilerCounters* counters = new CompilerCounters("compilerThread", i, CHECK); |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1177 // Shark and C2 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1178 CompilerThread* new_thread = make_compiler_thread(name_buffer, _c2_compile_queue, counters, _compilers[1], CHECK); |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1179 _compiler_threads->append(new_thread); |
0 | 1180 } |
1783 | 1181 |
1182 for (int i = c2_compiler_count; i < compiler_count; i++) { | |
1183 // Create a name for our thread. | |
1184 sprintf(name_buffer, "C1 CompilerThread%d", i); | |
1185 CompilerCounters* counters = new CompilerCounters("compilerThread", i, CHECK); | |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1186 // C1 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1187 CompilerThread* new_thread = make_compiler_thread(name_buffer, _c1_compile_queue, counters, _compilers[0], CHECK); |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1188 _compiler_threads->append(new_thread); |
1783 | 1189 } |
1190 | |
0 | 1191 if (UsePerfData) { |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1192 PerfDataManager::create_constant(SUN_CI, "threads", PerfData::U_Bytes, compiler_count, CHECK); |
0 | 1193 } |
1194 } | |
1195 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1196 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1197 /** |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1198 * Set the methods on the stack as on_stack so that redefine classes doesn't |
20589
0c0e68524c17
8042570: Excessive number of tests timing out on nightly testing due to fix for 8040798
anoll
parents:
20588
diff
changeset
|
1199 * reclaim them. This method is executed at a safepoint. |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1200 */ |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1201 void CompileBroker::mark_on_stack() { |
20589
0c0e68524c17
8042570: Excessive number of tests timing out on nightly testing due to fix for 8040798
anoll
parents:
20588
diff
changeset
|
1202 assert(SafepointSynchronize::is_at_safepoint(), "sanity check"); |
0c0e68524c17
8042570: Excessive number of tests timing out on nightly testing due to fix for 8040798
anoll
parents:
20588
diff
changeset
|
1203 // Since we are at a safepoint, we do not need a lock to access |
0c0e68524c17
8042570: Excessive number of tests timing out on nightly testing due to fix for 8040798
anoll
parents:
20588
diff
changeset
|
1204 // the compile queues. |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1205 if (_c2_compile_queue != NULL) { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1206 _c2_compile_queue->mark_on_stack(); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1207 } |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1208 if (_c1_compile_queue != NULL) { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1209 _c1_compile_queue->mark_on_stack(); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1210 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1211 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1212 |
0 | 1213 // ------------------------------------------------------------------ |
1214 // CompileBroker::compile_method | |
1215 // | |
1216 // Request compilation of a method. | |
1217 void CompileBroker::compile_method_base(methodHandle method, | |
1218 int osr_bci, | |
1219 int comp_level, | |
1220 methodHandle hot_method, | |
1221 int hot_count, | |
1222 const char* comment, | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4122
diff
changeset
|
1223 Thread* thread) { |
0 | 1224 // do nothing if compiler thread(s) is not available |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1225 if (!_initialized) { |
0 | 1226 return; |
1227 } | |
1228 | |
1229 guarantee(!method->is_abstract(), "cannot compile abstract methods"); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1230 assert(method->method_holder()->oop_is_instance(), |
0 | 1231 "sanity check"); |
6940
18fb7da42534
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
6843
diff
changeset
|
1232 assert(!method->method_holder()->is_not_initialized(), |
0 | 1233 "method holder must be initialized"); |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
1234 assert(!method->is_method_handle_intrinsic(), "do not enqueue these guys"); |
0 | 1235 |
1236 if (CIPrintRequests) { | |
1237 tty->print("request: "); | |
1238 method->print_short_name(tty); | |
1239 if (osr_bci != InvocationEntryBci) { | |
1240 tty->print(" osr_bci: %d", osr_bci); | |
1241 } | |
14107
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13969
diff
changeset
|
1242 tty->print(" level: %d comment: %s count: %d", comp_level, comment, hot_count); |
0 | 1243 if (!hot_method.is_null()) { |
1244 tty->print(" hot: "); | |
1245 if (hot_method() != method()) { | |
1246 hot_method->print_short_name(tty); | |
1247 } else { | |
1248 tty->print("yes"); | |
1249 } | |
1250 } | |
1251 tty->cr(); | |
1252 } | |
1253 | |
1254 // A request has been made for compilation. Before we do any | |
1255 // real work, check to see if the method has been compiled | |
1256 // in the meantime with a definitive result. | |
1257 if (compilation_is_complete(method, osr_bci, comp_level)) { | |
1258 return; | |
1259 } | |
1260 | |
2403 | 1261 #ifndef PRODUCT |
1262 if (osr_bci != -1 && !FLAG_IS_DEFAULT(OSROnlyBCI)) { | |
1263 if ((OSROnlyBCI > 0) ? (OSROnlyBCI != osr_bci) : (-OSROnlyBCI == osr_bci)) { | |
1264 // Positive OSROnlyBCI means only compile that bci. Negative means don't compile that BCI. | |
1265 return; | |
1266 } | |
1267 } | |
1268 #endif | |
1783 | 1269 |
0 | 1270 // If this method is already in the compile queue, then |
1271 // we do not block the current thread. | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1272 if (compilation_is_in_queue(method)) { |
0 | 1273 // We may want to decay our counter a bit here to prevent |
1274 // multiple denied requests for compilation. This is an | |
1275 // open compilation policy issue. Note: The other possibility, | |
1276 // in the case that this is a blocking compile request, is to have | |
1277 // all subsequent blocking requesters wait for completion of | |
1278 // ongoing compiles. Note that in this case we'll need a protocol | |
1279 // for freeing the associated compile tasks. [Or we could have | |
1280 // a single static monitor on which all these waiters sleep.] | |
1281 return; | |
1282 } | |
1283 | |
3294
86ebb26bcdeb
7037756: Deadlock in compiler thread similiar to 6789220
johnc
parents:
2482
diff
changeset
|
1284 // If the requesting thread is holding the pending list lock |
86ebb26bcdeb
7037756: Deadlock in compiler thread similiar to 6789220
johnc
parents:
2482
diff
changeset
|
1285 // then we just return. We can't risk blocking while holding |
86ebb26bcdeb
7037756: Deadlock in compiler thread similiar to 6789220
johnc
parents:
2482
diff
changeset
|
1286 // the pending list lock or a 3-way deadlock may occur |
86ebb26bcdeb
7037756: Deadlock in compiler thread similiar to 6789220
johnc
parents:
2482
diff
changeset
|
1287 // between the reference handler thread, a GC (instigated |
86ebb26bcdeb
7037756: Deadlock in compiler thread similiar to 6789220
johnc
parents:
2482
diff
changeset
|
1288 // by a compiler thread), and compiled method registration. |
6735
aed758eda82a
7195833: NPG: Rename instanceClassLoaderKlass, instanceRefKlass and instanceMirrorKlass
coleenp
parents:
6725
diff
changeset
|
1289 if (InstanceRefKlass::owns_pending_list_lock(JavaThread::current())) { |
3294
86ebb26bcdeb
7037756: Deadlock in compiler thread similiar to 6789220
johnc
parents:
2482
diff
changeset
|
1290 return; |
86ebb26bcdeb
7037756: Deadlock in compiler thread similiar to 6789220
johnc
parents:
2482
diff
changeset
|
1291 } |
86ebb26bcdeb
7037756: Deadlock in compiler thread similiar to 6789220
johnc
parents:
2482
diff
changeset
|
1292 |
20489
966205f0e717
8058564: Tiered compilation performance drop in PIT
iveresov
parents:
20469
diff
changeset
|
1293 if (TieredCompilation) { |
966205f0e717
8058564: Tiered compilation performance drop in PIT
iveresov
parents:
20469
diff
changeset
|
1294 // Tiered policy requires MethodCounters to exist before adding a method to |
966205f0e717
8058564: Tiered compilation performance drop in PIT
iveresov
parents:
20469
diff
changeset
|
1295 // the queue. Create if we don't have them yet. |
966205f0e717
8058564: Tiered compilation performance drop in PIT
iveresov
parents:
20469
diff
changeset
|
1296 method->get_method_counters(thread); |
966205f0e717
8058564: Tiered compilation performance drop in PIT
iveresov
parents:
20469
diff
changeset
|
1297 } |
966205f0e717
8058564: Tiered compilation performance drop in PIT
iveresov
parents:
20469
diff
changeset
|
1298 |
0 | 1299 // Outputs from the following MutexLocker block: |
1300 CompileTask* task = NULL; | |
1301 bool blocking = false; | |
1783 | 1302 CompileQueue* queue = compile_queue(comp_level); |
0 | 1303 |
1304 // Acquire our lock. | |
1305 { | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4122
diff
changeset
|
1306 MutexLocker locker(queue->lock(), thread); |
0 | 1307 |
1308 // Make sure the method has not slipped into the queues since | |
1309 // last we checked; note that those checks were "fast bail-outs". | |
1310 // Here we need to be more careful, see 14012000 below. | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1311 if (compilation_is_in_queue(method)) { |
0 | 1312 return; |
1313 } | |
1314 | |
1315 // We need to check again to see if the compilation has | |
1316 // completed. A previous compilation may have registered | |
1317 // some result. | |
1318 if (compilation_is_complete(method, osr_bci, comp_level)) { | |
1319 return; | |
1320 } | |
1321 | |
1322 // We now know that this compilation is not pending, complete, | |
1323 // or prohibited. Assign a compile_id to this compilation | |
1324 // and check to see if it is in our [Start..Stop) range. | |
14236 | 1325 int compile_id = assign_compile_id(method, osr_bci); |
0 | 1326 if (compile_id == 0) { |
1327 // The compilation falls outside the allowed range. | |
1328 return; | |
1329 } | |
1330 | |
1331 // Should this thread wait for completion of the compile? | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1332 blocking = is_compile_blocking(); |
0 | 1333 |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
1334 #ifdef COMPILERJVMCI |
16447
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1335 if (blocking) { |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
1336 // Don't allow blocking compiles for requests triggered by JVMCI. |
16447
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1337 if (thread->is_Compiler_thread()) { |
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1338 blocking = false; |
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1339 } |
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1340 |
19947
6a0692faf9fd
Ignore -Xbatch when the thread is in the middle of class loading
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19943
diff
changeset
|
1341 // Don't allow blocking compiles if inside a class initializer or while performing class loading |
16447
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1342 vframeStream vfst((JavaThread*) thread); |
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1343 for (; !vfst.at_end(); vfst.next()) { |
19947
6a0692faf9fd
Ignore -Xbatch when the thread is in the middle of class loading
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19943
diff
changeset
|
1344 if (vfst.method()->is_static_initializer() || |
6a0692faf9fd
Ignore -Xbatch when the thread is in the middle of class loading
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19943
diff
changeset
|
1345 (vfst.method()->method_holder()->is_subclass_of(SystemDictionary::ClassLoader_klass()) && |
6a0692faf9fd
Ignore -Xbatch when the thread is in the middle of class loading
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19943
diff
changeset
|
1346 vfst.method()->name() == vmSymbols::loadClass_name())) { |
16447
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1347 blocking = false; |
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1348 break; |
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1349 } |
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1350 } |
16625
d2aa48d54db5
don't allow blocking compilation requests to Graal if Graal itself is not yet initialized
Doug Simon <doug.simon@oracle.com>
parents:
16447
diff
changeset
|
1351 |
21527
07b088d61d5d
added HotSpotJVMCIRuntime* classes, replaced references to HotSpotGraalRuntime in VM with HotSpotJVMCIRuntime (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
20804
diff
changeset
|
1352 // Don't allow blocking compilation requests to JVMCI |
07b088d61d5d
added HotSpotJVMCIRuntime* classes, replaced references to HotSpotGraalRuntime in VM with HotSpotJVMCIRuntime (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
20804
diff
changeset
|
1353 // if JVMCI itself is not yet initialized |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
1354 if (!JVMCIRuntime::is_HotSpotJVMCIRuntime_initialized() && compiler(comp_level)->is_jvmci()) { |
16625
d2aa48d54db5
don't allow blocking compilation requests to Graal if Graal itself is not yet initialized
Doug Simon <doug.simon@oracle.com>
parents:
16447
diff
changeset
|
1355 blocking = false; |
d2aa48d54db5
don't allow blocking compilation requests to Graal if Graal itself is not yet initialized
Doug Simon <doug.simon@oracle.com>
parents:
16447
diff
changeset
|
1356 } |
18303
ab47ef2f2207
disable blocking compilation requests once GraalRuntime::shutdown has been called
Doug Simon <doug.simon@oracle.com>
parents:
18171
diff
changeset
|
1357 |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
1358 // Don't allow blocking compilation requests if we are in JVMCIRuntime::shutdown |
18303
ab47ef2f2207
disable blocking compilation requests once GraalRuntime::shutdown has been called
Doug Simon <doug.simon@oracle.com>
parents:
18171
diff
changeset
|
1359 // to avoid deadlock between compiler thread(s) and threads run at shutdown |
ab47ef2f2207
disable blocking compilation requests once GraalRuntime::shutdown has been called
Doug Simon <doug.simon@oracle.com>
parents:
18171
diff
changeset
|
1360 // such as the DestroyJavaVM thread. |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
1361 if (JVMCIRuntime::shutdown_called()) { |
18303
ab47ef2f2207
disable blocking compilation requests once GraalRuntime::shutdown has been called
Doug Simon <doug.simon@oracle.com>
parents:
18171
diff
changeset
|
1362 blocking = false; |
ab47ef2f2207
disable blocking compilation requests once GraalRuntime::shutdown has been called
Doug Simon <doug.simon@oracle.com>
parents:
18171
diff
changeset
|
1363 } |
16185
518a221dbbde
don't allow blocking compile requests from the compiler thread
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
16180
diff
changeset
|
1364 } |
16447
ce6696559683
better fix for deadlock in GraalVM under -Xcomp (JBS:GRAAL-48)
Doug Simon <doug.simon@oracle.com>
parents:
16399
diff
changeset
|
1365 // Don't allow blocking compiles |
16185
518a221dbbde
don't allow blocking compile requests from the compiler thread
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
16180
diff
changeset
|
1366 #endif |
518a221dbbde
don't allow blocking compile requests from the compiler thread
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
16180
diff
changeset
|
1367 |
0 | 1368 // We will enter the compilation in the queue. |
1369 // 14012000: Note that this sets the queued_for_compile bits in | |
1370 // the target method. We can now reason that a method cannot be | |
1371 // queued for compilation more than once, as follows: | |
1372 // Before a thread queues a task for compilation, it first acquires | |
1373 // the compile queue lock, then checks if the method's queued bits | |
1374 // are set or it has already been compiled. Thus there can not be two | |
1375 // instances of a compilation task for the same method on the | |
1376 // compilation queue. Consider now the case where the compilation | |
1377 // thread has already removed a task for that method from the queue | |
1378 // and is in the midst of compiling it. In this case, the | |
1379 // queued_for_compile bits must be set in the method (and these | |
1380 // will be visible to the current thread, since the bits were set | |
1381 // under protection of the compile queue lock, which we hold now. | |
1382 // When the compilation completes, the compiler thread first sets | |
1383 // the compilation result and then clears the queued_for_compile | |
1384 // bits. Neither of these actions are protected by a barrier (or done | |
1385 // under the protection of a lock), so the only guarantee we have | |
1386 // (on machines with TSO (Total Store Order)) is that these values | |
1387 // will update in that order. As a result, the only combinations of | |
1388 // these bits that the current thread will see are, in temporal order: | |
1389 // <RESULT, QUEUE> : | |
1390 // <0, 1> : in compile queue, but not yet compiled | |
1391 // <1, 1> : compiled but queue bit not cleared | |
1392 // <1, 0> : compiled and queue bit cleared | |
1393 // Because we first check the queue bits then check the result bits, | |
1394 // we are assured that we cannot introduce a duplicate task. | |
1395 // Note that if we did the tests in the reverse order (i.e. check | |
1396 // result then check queued bit), we could get the result bit before | |
1397 // the compilation completed, and the queue bit after the compilation | |
1398 // completed, and end up introducing a "duplicate" (redundant) task. | |
1399 // In that case, the compiler thread should first check if a method | |
1400 // has already been compiled before trying to compile it. | |
1401 // NOTE: in the event that there are multiple compiler threads and | |
1402 // there is de-optimization/recompilation, things will get hairy, | |
1403 // and in that case it's best to protect both the testing (here) of | |
1404 // these bits, and their updating (here and elsewhere) under a | |
1405 // common lock. | |
1783 | 1406 task = create_compile_task(queue, |
0 | 1407 compile_id, method, |
1408 osr_bci, comp_level, | |
1409 hot_method, hot_count, comment, | |
1410 blocking); | |
1411 } | |
1412 | |
1413 if (blocking) { | |
1414 wait_for_completion(task); | |
1415 } | |
1416 } | |
1417 | |
1418 | |
1419 nmethod* CompileBroker::compile_method(methodHandle method, int osr_bci, | |
1783 | 1420 int comp_level, |
0 | 1421 methodHandle hot_method, int hot_count, |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4122
diff
changeset
|
1422 const char* comment, Thread* THREAD) { |
0 | 1423 // make sure arguments make sense |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1424 assert(method->method_holder()->oop_is_instance(), "not an instance method"); |
0 | 1425 assert(osr_bci == InvocationEntryBci || (0 <= osr_bci && osr_bci < method->code_size()), "bci out of range"); |
1426 assert(!method->is_abstract() && (osr_bci == InvocationEntryBci || !method->is_native()), "cannot compile abstract/native methods"); | |
6940
18fb7da42534
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
6843
diff
changeset
|
1427 assert(!method->method_holder()->is_not_initialized(), "method holder must be initialized"); |
9080
b84fd7d73702
8007288: Additional WB API for compiler's testing
iignatyev
parents:
8860
diff
changeset
|
1428 // allow any levels for WhiteBox |
b84fd7d73702
8007288: Additional WB API for compiler's testing
iignatyev
parents:
8860
diff
changeset
|
1429 assert(WhiteBoxAPI || TieredCompilation || comp_level == CompLevel_highest_tier, "only CompLevel_highest_tier must be used in non-tiered"); |
0 | 1430 // return quickly if possible |
1431 | |
1432 // lock, make sure that the compilation | |
1433 // isn't prohibited in a straightforward way. | |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
1434 AbstractCompiler *comp = CompileBroker::compiler(comp_level); |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
1435 if (comp == NULL || !comp->can_compile_method(method) || |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
1436 compilation_is_prohibited(method, osr_bci, comp_level)) { |
0 | 1437 return NULL; |
1438 } | |
1439 | |
1440 if (osr_bci == InvocationEntryBci) { | |
1441 // standard compilation | |
1442 nmethod* method_code = method->code(); | |
1783 | 1443 if (method_code != NULL) { |
1444 if (compilation_is_complete(method, osr_bci, comp_level)) { | |
1445 return method_code; | |
1446 } | |
0 | 1447 } |
12324
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
1448 if (method->is_not_compilable(comp_level)) { |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
1449 return NULL; |
1202 | 1450 } |
0 | 1451 } else { |
1452 // osr compilation | |
1453 #ifndef TIERED | |
1454 // seems like an assert of dubious value | |
1783 | 1455 assert(comp_level == CompLevel_highest_tier, |
0 | 1456 "all OSR compiles are assumed to be at a single compilation lavel"); |
1457 #endif // TIERED | |
1783 | 1458 // We accept a higher level osr method |
1459 nmethod* nm = method->lookup_osr_nmethod_for(osr_bci, comp_level, false); | |
0 | 1460 if (nm != NULL) return nm; |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
1461 if (method->is_not_osr_compilable(comp_level)) return NULL; |
0 | 1462 } |
1463 | |
1464 assert(!HAS_PENDING_EXCEPTION, "No exception should be present"); | |
1465 // some prerequisites that are compiler specific | |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
1466 if (comp->is_c2() || comp->is_shark()) { |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4122
diff
changeset
|
1467 method->constants()->resolve_string_constants(CHECK_AND_CLEAR_NULL); |
0 | 1468 // Resolve all classes seen in the signature of the method |
1469 // we are compiling. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1470 Method::load_signature_classes(method, CHECK_AND_CLEAR_NULL); |
0 | 1471 } |
1472 | |
1473 // If the method is native, do the lookup in the thread requesting | |
1474 // the compilation. Native lookups can load code, which is not | |
1475 // permitted during compilation. | |
1476 // | |
1477 // Note: A native method implies non-osr compilation which is | |
1478 // checked with an assertion at the entry of this method. | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
1479 if (method->is_native() && !method->is_method_handle_intrinsic()) { |
0 | 1480 bool in_base_library; |
1481 address adr = NativeLookup::lookup(method, in_base_library, THREAD); | |
1482 if (HAS_PENDING_EXCEPTION) { | |
1483 // In case of an exception looking up the method, we just forget | |
1484 // about it. The interpreter will kick-in and throw the exception. | |
1485 method->set_not_compilable(); // implies is_not_osr_compilable() | |
1486 CLEAR_PENDING_EXCEPTION; | |
1487 return NULL; | |
1488 } | |
1489 assert(method->has_native_function(), "must have native code by now"); | |
1490 } | |
1491 | |
1492 // RedefineClasses() has replaced this method; just return | |
1493 if (method->is_old()) { | |
1494 return NULL; | |
1495 } | |
1496 | |
1497 // JVMTI -- post_compile_event requires jmethod_id() that may require | |
1498 // a lock the compiling thread can not acquire. Prefetch it here. | |
1499 if (JvmtiExport::should_post_compiled_method_load()) { | |
1500 method->jmethod_id(); | |
1501 } | |
1502 | |
1503 // do the compilation | |
1504 if (method->is_native()) { | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
6197
diff
changeset
|
1505 if (!PreferInterpreterNativeStubs || method->is_method_handle_intrinsic()) { |
12965
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1506 // To properly handle the appendix argument for out-of-line calls we are using a small trampoline that |
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1507 // pops off the appendix argument and jumps to the target (see gen_special_dispatch in SharedRuntime). |
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1508 // |
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1509 // Since normal compiled-to-compiled calls are not able to handle such a thing we MUST generate an adapter |
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1510 // in this case. If we can't generate one and use it we can not execute the out-of-line method handle calls. |
14236 | 1511 AdapterHandlerLibrary::create_native_wrapper(method); |
0 | 1512 } else { |
1513 return NULL; | |
1514 } | |
1515 } else { | |
12965
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1516 // If the compiler is shut off due to code cache getting full |
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1517 // fail out now so blocking compiles dont hang the java thread |
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1518 if (!should_compile_new_jobs()) { |
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1519 CompilationPolicy::policy()->delay_compilation(method()); |
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1520 return NULL; |
8b4bbba322d3
8026407: VM crashes on linux-ppc and linux-i586 when there is not enough ReservedCodeCacheSize specified
anoll
parents:
12880
diff
changeset
|
1521 } |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4122
diff
changeset
|
1522 compile_method_base(method, osr_bci, comp_level, hot_method, hot_count, comment, THREAD); |
0 | 1523 } |
1524 | |
1525 // return requested nmethod | |
1783 | 1526 // We accept a higher level osr method |
1527 return osr_bci == InvocationEntryBci ? method->code() : method->lookup_osr_nmethod_for(osr_bci, comp_level, false); | |
0 | 1528 } |
1529 | |
1530 | |
1531 // ------------------------------------------------------------------ | |
1532 // CompileBroker::compilation_is_complete | |
1533 // | |
1534 // See if compilation of this method is already complete. | |
1535 bool CompileBroker::compilation_is_complete(methodHandle method, | |
1536 int osr_bci, | |
1537 int comp_level) { | |
1538 bool is_osr = (osr_bci != standard_entry_bci); | |
1539 if (is_osr) { | |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
1540 if (method->is_not_osr_compilable(comp_level)) { |
0 | 1541 return true; |
1542 } else { | |
1783 | 1543 nmethod* result = method->lookup_osr_nmethod_for(osr_bci, comp_level, true); |
0 | 1544 return (result != NULL); |
1545 } | |
1546 } else { | |
1547 if (method->is_not_compilable(comp_level)) { | |
1548 return true; | |
1549 } else { | |
1550 nmethod* result = method->code(); | |
1551 if (result == NULL) return false; | |
1783 | 1552 return comp_level == result->comp_level(); |
0 | 1553 } |
1554 } | |
1555 } | |
1556 | |
1557 | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1558 /** |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1559 * See if this compilation is already requested. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1560 * |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1561 * Implementation note: there is only a single "is in queue" bit |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1562 * for each method. This means that the check below is overly |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1563 * conservative in the sense that an osr compilation in the queue |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1564 * will block a normal compilation from entering the queue (and vice |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1565 * versa). This can be remedied by a full queue search to disambiguate |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1566 * cases. If it is deemed profitable, this may be done. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1567 */ |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1568 bool CompileBroker::compilation_is_in_queue(methodHandle method) { |
0 | 1569 return method->queued_for_compilation(); |
1570 } | |
1571 | |
1572 // ------------------------------------------------------------------ | |
1573 // CompileBroker::compilation_is_prohibited | |
1574 // | |
1575 // See if this compilation is not allowed. | |
1576 bool CompileBroker::compilation_is_prohibited(methodHandle method, int osr_bci, int comp_level) { | |
1577 bool is_native = method->is_native(); | |
1578 // Some compilers may not support the compilation of natives. | |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
1579 AbstractCompiler *comp = compiler(comp_level); |
0 | 1580 if (is_native && |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
1581 (!CICompileNatives || comp == NULL || !comp->supports_native())) { |
1783 | 1582 method->set_not_compilable_quietly(comp_level); |
0 | 1583 return true; |
1584 } | |
1585 | |
1586 bool is_osr = (osr_bci != standard_entry_bci); | |
1587 // Some compilers may not support on stack replacement. | |
1588 if (is_osr && | |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
1589 (!CICompileOSR || comp == NULL || !comp->supports_osr())) { |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
1590 method->set_not_osr_compilable(comp_level); |
0 | 1591 return true; |
1592 } | |
1593 | |
1594 // The method may be explicitly excluded by the user. | |
1595 bool quietly; | |
1596 if (CompilerOracle::should_exclude(method, quietly)) { | |
1597 if (!quietly) { | |
1598 // This does not happen quietly... | |
1599 ResourceMark rm; | |
1600 tty->print("### Excluding %s:%s", | |
1601 method->is_native() ? "generation of native wrapper" : "compile", | |
1602 (method->is_static() ? " static" : "")); | |
1603 method->print_short_name(tty); | |
1604 tty->cr(); | |
1605 } | |
7998 | 1606 method->set_not_compilable(CompLevel_all, !quietly, "excluded by CompilerOracle"); |
0 | 1607 } |
1608 | |
1609 return false; | |
1610 } | |
1611 | |
14236 | 1612 /** |
1613 * Generate serialized IDs for compilation requests. If certain debugging flags are used | |
1614 * and the ID is not within the specified range, the method is not compiled and 0 is returned. | |
1615 * The function also allows to generate separate compilation IDs for OSR compilations. | |
1616 */ | |
1617 int CompileBroker::assign_compile_id(methodHandle method, int osr_bci) { | |
1618 #ifdef ASSERT | |
0 | 1619 bool is_osr = (osr_bci != standard_entry_bci); |
14236 | 1620 int id; |
1621 if (method->is_native()) { | |
1622 assert(!is_osr, "can't be osr"); | |
1623 // Adapters, native wrappers and method handle intrinsics | |
1624 // should be generated always. | |
1625 return Atomic::add(1, &_compilation_id); | |
1626 } else if (CICountOSR && is_osr) { | |
1627 id = Atomic::add(1, &_osr_compilation_id); | |
1628 if (CIStartOSR <= id && id < CIStopOSR) { | |
0 | 1629 return id; |
1630 } | |
1631 } else { | |
14236 | 1632 id = Atomic::add(1, &_compilation_id); |
1633 if (CIStart <= id && id < CIStop) { | |
0 | 1634 return id; |
1635 } | |
1636 } | |
1637 | |
1638 // Method was not in the appropriate compilation range. | |
1208
cef333a48af6
6923043: failed nightly tests which use -XX:+PrintCompilation -Xcomp -XX:CompileOnly
kvn
parents:
1202
diff
changeset
|
1639 method->set_not_compilable_quietly(); |
0 | 1640 return 0; |
14236 | 1641 #else |
1642 // CICountOSR is a develop flag and set to 'false' by default. In a product built, | |
1643 // only _compilation_id is incremented. | |
1644 return Atomic::add(1, &_compilation_id); | |
1645 #endif | |
0 | 1646 } |
1647 | |
1648 | |
1649 // ------------------------------------------------------------------ | |
13969
fe034af88233
Acquire proper locks before calling assign_compile_id
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13731
diff
changeset
|
1650 // CompileBroker::assign_compile_id_unlocked |
fe034af88233
Acquire proper locks before calling assign_compile_id
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13731
diff
changeset
|
1651 // |
fe034af88233
Acquire proper locks before calling assign_compile_id
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13731
diff
changeset
|
1652 // Public wrapper for assign_compile_id that acquires the needed locks |
fe034af88233
Acquire proper locks before calling assign_compile_id
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13731
diff
changeset
|
1653 uint CompileBroker::assign_compile_id_unlocked(Thread* thread, methodHandle method, int osr_bci) { |
fe034af88233
Acquire proper locks before calling assign_compile_id
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13731
diff
changeset
|
1654 MutexLocker locker(MethodCompileQueue_lock, thread); |
fe034af88233
Acquire proper locks before calling assign_compile_id
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13731
diff
changeset
|
1655 return assign_compile_id(method, osr_bci); |
fe034af88233
Acquire proper locks before calling assign_compile_id
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13731
diff
changeset
|
1656 } |
fe034af88233
Acquire proper locks before calling assign_compile_id
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13731
diff
changeset
|
1657 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1658 /** |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1659 * Should the current thread block until this compilation request |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1660 * has been fulfilled? |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1661 */ |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1662 bool CompileBroker::is_compile_blocking() { |
6735
aed758eda82a
7195833: NPG: Rename instanceClassLoaderKlass, instanceRefKlass and instanceMirrorKlass
coleenp
parents:
6725
diff
changeset
|
1663 assert(!InstanceRefKlass::owns_pending_list_lock(JavaThread::current()), "possible deadlock"); |
3294
86ebb26bcdeb
7037756: Deadlock in compiler thread similiar to 6789220
johnc
parents:
2482
diff
changeset
|
1664 return !BackgroundCompilation; |
0 | 1665 } |
1666 | |
1667 | |
1668 // ------------------------------------------------------------------ | |
1669 // CompileBroker::preload_classes | |
1670 void CompileBroker::preload_classes(methodHandle method, TRAPS) { | |
1671 // Move this code over from c1_Compiler.cpp | |
1672 ShouldNotReachHere(); | |
1673 } | |
1674 | |
1675 | |
1676 // ------------------------------------------------------------------ | |
1677 // CompileBroker::create_compile_task | |
1678 // | |
1679 // Create a CompileTask object representing the current request for | |
1680 // compilation. Add this task to the queue. | |
1681 CompileTask* CompileBroker::create_compile_task(CompileQueue* queue, | |
1682 int compile_id, | |
1683 methodHandle method, | |
1684 int osr_bci, | |
1685 int comp_level, | |
1686 methodHandle hot_method, | |
1687 int hot_count, | |
1688 const char* comment, | |
1689 bool blocking) { | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1690 CompileTask* new_task = CompileTask::allocate(); |
0 | 1691 new_task->initialize(compile_id, method, osr_bci, comp_level, |
1692 hot_method, hot_count, comment, | |
1693 blocking); | |
1694 queue->add(new_task); | |
1695 return new_task; | |
1696 } | |
1697 | |
22714
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1698 // 1 second should be long enough to complete most JVMCI compilations |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1699 // and not too long to stall a blocking JVMCI compilation that |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1700 // is trying to acquire a lock held by the app thread that submitted the |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1701 // compilation. |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1702 static const long BLOCKING_JVMCI_COMPILATION_TIMEOUT = 1000; |
0 | 1703 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1704 /** |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1705 * Wait for the compilation task to complete. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1706 */ |
0 | 1707 void CompileBroker::wait_for_completion(CompileTask* task) { |
1708 if (CIPrintCompileQueue) { | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1709 ttyLocker ttyl; |
0 | 1710 tty->print_cr("BLOCKING FOR COMPILE"); |
1711 } | |
1712 | |
1713 assert(task->is_blocking(), "can only wait on blocking task"); | |
1714 | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1715 JavaThread* thread = JavaThread::current(); |
0 | 1716 thread->set_blocked_on_compilation(true); |
1717 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
1718 methodHandle method(thread, task->method()); |
0 | 1719 { |
1720 MutexLocker waiter(task->lock(), thread); | |
1721 | |
22714
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1722 #ifdef COMPILERJVMCI |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1723 if (compiler(task->comp_level())->is_jvmci()) { |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1724 // No need to check if compilation has completed - just |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1725 // rely on the time out. The JVMCI compiler thread will |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1726 // recycle the CompileTask. |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1727 task->lock()->wait(!Mutex::_no_safepoint_check_flag, BLOCKING_JVMCI_COMPILATION_TIMEOUT); |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1728 thread->set_blocked_on_compilation(false); |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1729 return; |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1730 } |
9b3b66634d17
use timeout for JVMCI blocking compilation to avoid deadlock between compiler and app thread that submitted the blocking compilation
Doug Simon <doug.simon@oracle.com>
parents:
22449
diff
changeset
|
1731 #endif |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1732 while (!task->is_complete() && !is_compilation_disabled_forever()) { |
0 | 1733 task->lock()->wait(); |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1734 } |
0 | 1735 } |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1736 |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1737 thread->set_blocked_on_compilation(false); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1738 if (is_compilation_disabled_forever()) { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1739 CompileTask::free(task); |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1740 return; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1741 } |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1742 |
0 | 1743 // It is harmless to check this status without the lock, because |
1744 // completion is a stable property (until the task object is recycled). | |
1745 assert(task->is_complete(), "Compilation should have completed"); | |
1746 assert(task->code_handle() == NULL, "must be reset"); | |
1747 | |
1748 // By convention, the waiter is responsible for recycling a | |
1749 // blocking CompileTask. Since there is only one waiter ever | |
1750 // waiting on a CompileTask, we know that no one else will | |
1751 // be using this CompileTask; we can free it. | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1752 CompileTask::free(task); |
0 | 1753 } |
1754 | |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1755 /** |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1756 * Initialize compiler thread(s) + compiler object(s). The postcondition |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1757 * of this function is that the compiler runtimes are initialized and that |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1758 * compiler threads can start compiling. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1759 */ |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1760 bool CompileBroker::init_compiler_runtime() { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1761 CompilerThread* thread = CompilerThread::current(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1762 AbstractCompiler* comp = thread->compiler(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1763 // Final sanity check - the compiler object must exist |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1764 guarantee(comp != NULL, "Compiler object must exist"); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1765 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1766 int system_dictionary_modification_counter; |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1767 { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1768 MutexLocker locker(Compile_lock, thread); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1769 system_dictionary_modification_counter = SystemDictionary::number_of_modifications(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1770 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1771 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1772 { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1773 // Must switch to native to allocate ci_env |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1774 ThreadToNativeFromVM ttn(thread); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1775 ciEnv ci_env(NULL, system_dictionary_modification_counter); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1776 // Cache Jvmti state |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1777 ci_env.cache_jvmti_state(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1778 // Cache DTrace flags |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1779 ci_env.cache_dtrace_flags(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1780 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1781 // Switch back to VM state to do compiler initialization |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1782 ThreadInVMfromNative tv(thread); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1783 ResetNoHandleMark rnhm; |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1784 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1785 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1786 if (!comp->is_shark()) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1787 // Perform per-thread and global initializations |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1788 comp->initialize(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1789 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1790 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1791 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1792 if (comp->is_failed()) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1793 disable_compilation_forever(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1794 // If compiler initialization failed, no compiler thread that is specific to a |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1795 // particular compiler runtime will ever start to compile methods. |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1796 shutdown_compiler_runtime(comp, thread); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1797 return false; |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1798 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1799 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1800 // C1 specific check |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1801 if (comp->is_c1() && (thread->get_buffer_blob() == NULL)) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1802 warning("Initialization of %s thread failed (no space to run compilers)", thread->name()); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1803 return false; |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1804 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1805 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1806 return true; |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1807 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1808 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1809 /** |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1810 * If C1 and/or C2 initialization failed, we shut down all compilation. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1811 * We do this to keep things simple. This can be changed if it ever turns |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1812 * out to be a problem. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1813 */ |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1814 void CompileBroker::shutdown_compiler_runtime(AbstractCompiler* comp, CompilerThread* thread) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1815 // Free buffer blob, if allocated |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1816 if (thread->get_buffer_blob() != NULL) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1817 MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1818 CodeCache::free(thread->get_buffer_blob()); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1819 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1820 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1821 if (comp->should_perform_shutdown()) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1822 // There are two reasons for shutting down the compiler |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1823 // 1) compiler runtime initialization failed |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1824 // 2) The code cache is full and the following flag is set: -XX:-UseCodeCacheFlushing |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1825 warning("%s initialization failed. Shutting down all compilers", comp->name()); |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1826 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1827 // Only one thread per compiler runtime object enters here |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1828 // Set state to shut down |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1829 comp->set_shut_down(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1830 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1831 // Delete all queued compilation tasks to make compiler threads exit faster. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1832 if (_c1_compile_queue != NULL) { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1833 _c1_compile_queue->free_all(); |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1834 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1835 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1836 if (_c2_compile_queue != NULL) { |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1837 _c2_compile_queue->free_all(); |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1838 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1839 |
20588
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1840 // Set flags so that we continue execution with using interpreter only. |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1841 UseCompiler = false; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1842 UseInterpreter = true; |
41dcdd636080
8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents:
20575
diff
changeset
|
1843 |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1844 // We could delete compiler runtimes also. However, there are references to |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1845 // the compiler runtime(s) (e.g., nmethod::is_compiled_by_c1()) which then |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1846 // fail. This can be done later if necessary. |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1847 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1848 } |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1849 |
0 | 1850 // ------------------------------------------------------------------ |
1851 // CompileBroker::compiler_thread_loop | |
1852 // | |
1853 // The main loop run by a CompilerThread. | |
1854 void CompileBroker::compiler_thread_loop() { | |
1855 CompilerThread* thread = CompilerThread::current(); | |
1856 CompileQueue* queue = thread->queue(); | |
1857 // For the thread that initializes the ciObjectFactory | |
1858 // this resource mark holds all the shared objects | |
1859 ResourceMark rm; | |
1860 | |
1861 // First thread to get here will initialize the compiler interface | |
1862 | |
1863 if (!ciObjectFactory::is_initialized()) { | |
1864 ASSERT_IN_VM; | |
1865 MutexLocker only_one (CompileThread_lock, thread); | |
1866 if (!ciObjectFactory::is_initialized()) { | |
1867 ciObjectFactory::initialize(); | |
1868 } | |
1869 } | |
1870 | |
1871 // Open a log. | |
1872 if (LogCompilation) { | |
1873 init_compiler_thread_log(); | |
1874 } | |
1875 CompileLog* log = thread->log(); | |
1876 if (log != NULL) { | |
6843 | 1877 log->begin_elem("start_compile_thread name='%s' thread='" UINTX_FORMAT "' process='%d'", |
1878 thread->name(), | |
0 | 1879 os::current_thread_id(), |
1880 os::current_process_id()); | |
1881 log->stamp(); | |
1882 log->end_elem(); | |
1883 } | |
1884 | |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1885 // If compiler thread/runtime initialization fails, exit the compiler thread |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1886 if (!init_compiler_runtime()) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1887 return; |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1888 } |
1202 | 1889 |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1890 // Poll for new compilation tasks as long as the JVM runs. Compilation |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1891 // should only be disabled if something went wrong while initializing the |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1892 // compiler runtimes. This, in turn, should not happen. The only known case |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1893 // when compiler runtime initialization fails is if there is not enough free |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1894 // space in the code cache to generate the necessary stubs, etc. |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1895 while (!is_compilation_disabled_forever()) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1896 // We need this HandleMark to avoid leaking VM handles. |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1897 HandleMark hm(thread); |
0 | 1898 |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1899 if (CodeCache::unallocated_capacity() < CodeCacheMinimumFreeSpace) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1900 // the code cache is really full |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1901 handle_full_code_cache(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1902 } |
0 | 1903 |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1904 CompileTask* task = queue->get(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1905 if (task == NULL) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1906 continue; |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1907 } |
0 | 1908 |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1909 // Give compiler threads an extra quanta. They tend to be bursty and |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1910 // this helps the compiler to finish up the job. |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1911 if( CompilerThreadHintNoPreempt ) |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1912 os::hint_no_preempt(); |
0 | 1913 |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1914 // trace per thread time and compile statistics |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1915 CompilerCounters* counters = ((CompilerThread*)thread)->counters(); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1916 PerfTraceTimedEvent(counters->time_counter(), counters->compile_counter()); |
0 | 1917 |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1918 // Assign the task to the current thread. Mark this compilation |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1919 // thread as active for the profiler. |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1920 CompileTaskWrapper ctw(task); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1921 nmethodLocker result_handle; // (handle for the nmethod produced by this task) |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1922 task->set_code_handle(&result_handle); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1923 methodHandle method(thread, task->method()); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1924 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1925 // Never compile a method if breakpoints are present in it |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1926 if (method()->number_of_breakpoints() == 0) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1927 // Compile the method. |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1928 if ((UseCompiler || AlwaysCompileLoopMethods) && CompileBroker::should_compile_new_jobs()) { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1929 invoke_compiler_on_method(task); |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1930 } else { |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1931 // After compilation is disabled, remove remaining methods from queue |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1932 method->clear_queued_for_compilation(); |
20469
dd89808e49ba
8049530: Provide descriptive failure reason for compilation tasks removed for the queue
vlivanov
parents:
20465
diff
changeset
|
1933 task->set_failure_reason("compilation is disabled"); |
0 | 1934 } |
1935 } | |
1936 } | |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1937 |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1938 // Shut down compiler runtime |
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
1939 shutdown_compiler_runtime(thread->compiler(), thread); |
0 | 1940 } |
1941 | |
1942 // ------------------------------------------------------------------ | |
1943 // CompileBroker::init_compiler_thread_log | |
1944 // | |
1945 // Set up state required by +LogCompilation. | |
1946 void CompileBroker::init_compiler_thread_log() { | |
1947 CompilerThread* thread = CompilerThread::current(); | |
10394
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1948 char file_name[4*K]; |
0 | 1949 FILE* fp = NULL; |
1950 intx thread_id = os::current_thread_id(); | |
1951 for (int try_temp_dir = 1; try_temp_dir >= 0; try_temp_dir--) { | |
1952 const char* dir = (try_temp_dir ? os::get_temp_directory() : NULL); | |
1353
a2ea687fdc7c
6938627: Make temporary directory use property java.io.tmpdir when specified
coleenp
parents:
1300
diff
changeset
|
1953 if (dir == NULL) { |
10394
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1954 jio_snprintf(file_name, sizeof(file_name), "hs_c" UINTX_FORMAT "_pid%u.log", |
1353
a2ea687fdc7c
6938627: Make temporary directory use property java.io.tmpdir when specified
coleenp
parents:
1300
diff
changeset
|
1955 thread_id, os::current_process_id()); |
a2ea687fdc7c
6938627: Make temporary directory use property java.io.tmpdir when specified
coleenp
parents:
1300
diff
changeset
|
1956 } else { |
10394
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1957 jio_snprintf(file_name, sizeof(file_name), |
1353
a2ea687fdc7c
6938627: Make temporary directory use property java.io.tmpdir when specified
coleenp
parents:
1300
diff
changeset
|
1958 "%s%shs_c" UINTX_FORMAT "_pid%u.log", dir, |
a2ea687fdc7c
6938627: Make temporary directory use property java.io.tmpdir when specified
coleenp
parents:
1300
diff
changeset
|
1959 os::file_separator(), thread_id, os::current_process_id()); |
a2ea687fdc7c
6938627: Make temporary directory use property java.io.tmpdir when specified
coleenp
parents:
1300
diff
changeset
|
1960 } |
10394
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1961 |
20679
ecccc23346fe
8007993: hotspot.log w/ enabled LogCompilation can be an invalid XML
thartmann
parents:
20591
diff
changeset
|
1962 fp = fopen(file_name, "wt"); |
0 | 1963 if (fp != NULL) { |
10394
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1964 if (LogCompilation && Verbose) { |
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1965 tty->print_cr("Opening compilation log %s", file_name); |
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1966 } |
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1967 CompileLog* log = new(ResourceObj::C_HEAP, mtCompiler) CompileLog(file_name, fp, thread_id); |
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1968 thread->init_log(log); |
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1969 |
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1970 if (xtty != NULL) { |
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1971 ttyLocker ttyl; |
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1972 // Record any per thread log files |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
14456
diff
changeset
|
1973 xtty->elem("thread_logfile thread='" INTX_FORMAT "' filename='%s'", thread_id, file_name); |
10394
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1974 } |
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1975 return; |
0 | 1976 } |
1977 } | |
10394
813f26e34135
8013329: File leak in hotspot/src/share/vm/compiler/compileBroker.cpp
anoll
parents:
10278
diff
changeset
|
1978 warning("Cannot open log file: %s", file_name); |
0 | 1979 } |
1980 | |
1981 // ------------------------------------------------------------------ | |
1982 // CompileBroker::set_should_block | |
1983 // | |
1984 // Set _should_block. | |
1985 // Call this from the VM, with Threads_lock held and a safepoint requested. | |
1986 void CompileBroker::set_should_block() { | |
1987 assert(Threads_lock->owner() == Thread::current(), "must have threads lock"); | |
1988 assert(SafepointSynchronize::is_at_safepoint(), "must be at a safepoint already"); | |
1989 #ifndef PRODUCT | |
1990 if (PrintCompilation && (Verbose || WizardMode)) | |
1991 tty->print_cr("notifying compiler thread pool to block"); | |
1992 #endif | |
1993 _should_block = true; | |
1994 } | |
1995 | |
1996 // ------------------------------------------------------------------ | |
1997 // CompileBroker::maybe_block | |
1998 // | |
1999 // Call this from the compiler at convenient points, to poll for _should_block. | |
2000 void CompileBroker::maybe_block() { | |
2001 if (_should_block) { | |
2002 #ifndef PRODUCT | |
2003 if (PrintCompilation && (Verbose || WizardMode)) | |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
14456
diff
changeset
|
2004 tty->print_cr("compiler thread " INTPTR_FORMAT " poll detects block request", p2i(Thread::current())); |
0 | 2005 #endif |
2006 ThreadInVMfromNative tivfn(JavaThread::current()); | |
2007 } | |
2008 } | |
2009 | |
7595
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2010 // wrapper for CodeCache::print_summary() |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2011 static void codecache_print(bool detailed) |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2012 { |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2013 ResourceMark rm; |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2014 stringStream s; |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2015 // Dump code cache into a buffer before locking the tty, |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2016 { |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2017 MutexLockerEx mu(CodeCache_lock, Mutex::_no_safepoint_check_flag); |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2018 CodeCache::print_summary(&s, detailed); |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2019 } |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2020 ttyLocker ttyl; |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
14456
diff
changeset
|
2021 tty->print("%s", s.as_string()); |
7595
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2022 } |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2023 |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2024 void CompileBroker::post_compile(CompilerThread* thread, CompileTask* task, EventCompilation& event, bool success, ciEnv* ci_env) { |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2025 |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2026 if (success) { |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2027 task->mark_success(); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2028 if (ci_env != NULL) { |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2029 task->set_num_inlined_bytecodes(ci_env->num_inlined_bytecodes()); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2030 } |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2031 if (_compilation_log != NULL) { |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2032 nmethod* code = task->code(); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2033 if (code != NULL) { |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2034 _compilation_log->log_nmethod(thread, code); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2035 } |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2036 } |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2037 } |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2038 |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2039 // simulate crash during compilation |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2040 assert(task->compile_id() != CICrashAt, "just as planned"); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2041 if (event.should_commit()) { |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2042 event.set_method(task->method()); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2043 event.set_compileID(task->compile_id()); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2044 event.set_compileLevel(task->comp_level()); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2045 event.set_succeded(task->is_success()); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2046 event.set_isOsr(task->osr_bci() != CompileBroker::standard_entry_bci); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2047 event.set_codeSize((task->code() == NULL) ? 0 : task->code()->total_size()); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2048 event.set_inlinedBytes(task->num_inlined_bytecodes()); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2049 event.commit(); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2050 } |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2051 } |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2052 |
0 | 2053 // ------------------------------------------------------------------ |
2054 // CompileBroker::invoke_compiler_on_method | |
2055 // | |
2056 // Compile a method. | |
2057 // | |
2058 void CompileBroker::invoke_compiler_on_method(CompileTask* task) { | |
2059 if (PrintCompilation) { | |
2060 ResourceMark rm; | |
2061 task->print_line(); | |
2062 } | |
2063 elapsedTimer time; | |
2064 | |
2065 CompilerThread* thread = CompilerThread::current(); | |
2066 ResourceMark rm(thread); | |
2067 | |
4872
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2068 if (LogEvents) { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2069 _compilation_log->log_compile(thread, task); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2070 } |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2071 |
0 | 2072 // Common flags. |
2073 uint compile_id = task->compile_id(); | |
2074 int osr_bci = task->osr_bci(); | |
2075 bool is_osr = (osr_bci != standard_entry_bci); | |
2076 bool should_log = (thread->log() != NULL); | |
2077 bool should_break = false; | |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2078 int task_level = task->comp_level(); |
0 | 2079 { |
2080 // create the handle inside it's own block so it can't | |
2081 // accidentally be referenced once the thread transitions to | |
2082 // native. The NoHandleMark before the transition should catch | |
2083 // any cases where this occurs in the future. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
2084 methodHandle method(thread, task->method()); |
0 | 2085 should_break = check_break_at(method, compile_id, is_osr); |
2086 if (should_log && !CompilerOracle::should_log(method)) { | |
2087 should_log = false; | |
2088 } | |
2089 assert(!method->is_native(), "no longer compile natives"); | |
2090 | |
2091 // Save information about this method in case of failure. | |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2092 set_last_compile(thread, method, is_osr, task_level); |
0 | 2093 |
10207
e4e131b15d5c
8013532: Remove unused parameter "compiler" from DTRACE_METHOD_COMPILE* macros
roland
parents:
10206
diff
changeset
|
2094 DTRACE_METHOD_COMPILE_BEGIN_PROBE(method, compiler_name(task_level)); |
0 | 2095 } |
2096 | |
2097 // Allocate a new set of JNI handles. | |
2098 push_jni_handle_block(); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
2099 Method* target_handle = task->method(); |
0 | 2100 int compilable = ciEnv::MethodCompilable; |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2101 AbstractCompiler *comp = compiler(task_level); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2102 |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
18303
diff
changeset
|
2103 int system_dictionary_modification_counter; |
0 | 2104 { |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
18303
diff
changeset
|
2105 MutexLocker locker(Compile_lock, thread); |
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
18303
diff
changeset
|
2106 system_dictionary_modification_counter = SystemDictionary::number_of_modifications(); |
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
18303
diff
changeset
|
2107 } |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2108 #ifdef COMPILERJVMCI |
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2109 if (comp != NULL && comp->is_jvmci()) { |
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2110 JVMCICompiler* jvmci = (JVMCICompiler*) comp; |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2111 |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2112 TraceTime t1("compilation", &time); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2113 EventCompilation event; |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2114 |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2115 JVMCIEnv env(task, system_dictionary_modification_counter); |
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2116 jvmci->compile_method(target_handle, osr_bci, &env); |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2117 |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2118 post_compile(thread, task, event, task->code() != NULL, NULL); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2119 } else |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2120 #endif // COMPILERJVMCI |
0 | 2121 { |
2122 | |
2123 NoHandleMark nhm; | |
2124 ThreadToNativeFromVM ttn(thread); | |
2125 | |
2126 ciEnv ci_env(task, system_dictionary_modification_counter); | |
2127 if (should_break) { | |
2128 ci_env.set_break_at_compile(true); | |
2129 } | |
2130 if (should_log) { | |
2131 ci_env.set_log(thread->log()); | |
2132 } | |
2133 assert(thread->env() == &ci_env, "set by ci_env"); | |
2134 // The thread-env() field is cleared in ~CompileTaskWrapper. | |
2135 | |
780
c96bf21b756f
6788527: Server vm intermittently fails with assertion "live value must not be garbage" with fastdebug bits
kvn
parents:
0
diff
changeset
|
2136 // Cache Jvmti state |
c96bf21b756f
6788527: Server vm intermittently fails with assertion "live value must not be garbage" with fastdebug bits
kvn
parents:
0
diff
changeset
|
2137 ci_env.cache_jvmti_state(); |
c96bf21b756f
6788527: Server vm intermittently fails with assertion "live value must not be garbage" with fastdebug bits
kvn
parents:
0
diff
changeset
|
2138 |
c96bf21b756f
6788527: Server vm intermittently fails with assertion "live value must not be garbage" with fastdebug bits
kvn
parents:
0
diff
changeset
|
2139 // Cache DTrace flags |
c96bf21b756f
6788527: Server vm intermittently fails with assertion "live value must not be garbage" with fastdebug bits
kvn
parents:
0
diff
changeset
|
2140 ci_env.cache_dtrace_flags(); |
c96bf21b756f
6788527: Server vm intermittently fails with assertion "live value must not be garbage" with fastdebug bits
kvn
parents:
0
diff
changeset
|
2141 |
0 | 2142 ciMethod* target = ci_env.get_method_from_handle(target_handle); |
2143 | |
2144 TraceTime t1("compilation", &time); | |
10405 | 2145 EventCompilation event; |
0 | 2146 |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2147 if (comp == NULL) { |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2148 ci_env.record_method_not_compilable("no compiler", !TieredCompilation); |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2149 } else { |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2150 comp->compile_method(&ci_env, target, osr_bci); |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2151 } |
0 | 2152 |
2153 if (!ci_env.failing() && task->code() == NULL) { | |
2154 //assert(false, "compiler should always document failure"); | |
2155 // The compiler elected, without comment, not to register a result. | |
2156 // Do not attempt further compilations of this method. | |
1964
22ef3370343b
7000349: Tiered reacts incorrectly to C1 compilation failures
iveresov
parents:
1783
diff
changeset
|
2157 ci_env.record_method_not_compilable("compile failed", !TieredCompilation); |
0 | 2158 } |
2159 | |
4872
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2160 // Copy this bit to the enclosing block: |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2161 compilable = ci_env.compilable(); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2162 |
0 | 2163 if (ci_env.failing()) { |
20469
dd89808e49ba
8049530: Provide descriptive failure reason for compilation tasks removed for the queue
vlivanov
parents:
20465
diff
changeset
|
2164 task->set_failure_reason(ci_env.failure_reason()); |
4872
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2165 const char* retry_message = ci_env.retry_message(); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2166 if (_compilation_log != NULL) { |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2167 _compilation_log->log_failure(thread, task, ci_env.failure_reason(), retry_message); |
aa3d708d67c4
7141200: log some interesting information in ring buffers for crashes
never
parents:
4825
diff
changeset
|
2168 } |
0 | 2169 if (PrintCompilation) { |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2170 FormatBufferResource msg = retry_message != NULL ? |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2171 err_msg_res("COMPILE SKIPPED: %s (%s)", ci_env.failure_reason(), retry_message) : |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2172 err_msg_res("COMPILE SKIPPED: %s", ci_env.failure_reason()); |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2173 task->print_compilation(tty, msg); |
0 | 2174 } |
2175 } | |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2176 |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
14957
diff
changeset
|
2177 post_compile(thread, task, event, !ci_env.failing(), &ci_env); |
0 | 2178 } |
2179 pop_jni_handle_block(); | |
2180 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
2181 methodHandle method(thread, task->method()); |
0 | 2182 |
10207
e4e131b15d5c
8013532: Remove unused parameter "compiler" from DTRACE_METHOD_COMPILE* macros
roland
parents:
10206
diff
changeset
|
2183 DTRACE_METHOD_COMPILE_END_PROBE(method, compiler_name(task_level), task->is_success()); |
0 | 2184 |
2185 collect_statistics(thread, time, task); | |
2186 | |
4064
670a74b863fc
7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents:
4047
diff
changeset
|
2187 if (PrintCompilation && PrintCompilation2) { |
4047 | 2188 tty->print("%7d ", (int) tty->time_stamp().milliseconds()); // print timestamp |
2189 tty->print("%4d ", compile_id); // print compilation number | |
2190 tty->print("%s ", (is_osr ? "%" : " ")); | |
10278 | 2191 if (task->code() != NULL) { |
2192 tty->print("size: %d(%d) ", task->code()->total_size(), task->code()->insts_size()); | |
2193 } | |
2194 tty->print_cr("time: %d inlined: %d bytes", (int)time.milliseconds(), task->num_inlined_bytecodes()); | |
4047 | 2195 } |
2196 | |
7595
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2197 if (PrintCodeCacheOnCompilation) |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2198 codecache_print(/* detailed= */ false); |
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2199 |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2200 // Disable compilation, if required. |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2201 switch (compilable) { |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2202 case ciEnv::MethodCompilable_never: |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2203 if (is_osr) |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2204 method->set_not_osr_compilable_quietly(); |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2205 else |
1208
cef333a48af6
6923043: failed nightly tests which use -XX:+PrintCompilation -Xcomp -XX:CompileOnly
kvn
parents:
1202
diff
changeset
|
2206 method->set_not_compilable_quietly(); |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2207 break; |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2208 case ciEnv::MethodCompilable_not_at_tier: |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2209 if (is_osr) |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2210 method->set_not_osr_compilable_quietly(task_level); |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2211 else |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2212 method->set_not_compilable_quietly(task_level); |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6735
diff
changeset
|
2213 break; |
0 | 2214 } |
2215 | |
2216 // Note that the queued_for_compilation bits are cleared without | |
2217 // protection of a mutex. [They were set by the requester thread, | |
20465
7301840ea20e
8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents:
18009
diff
changeset
|
2218 // when adding the task to the compile queue -- at which time the |
0 | 2219 // compile queue lock was held. Subsequently, we acquired the compile |
2220 // queue lock to get this task off the compile queue; thus (to belabour | |
2221 // the point somewhat) our clearing of the bits must be occurring | |
2222 // only after the setting of the bits. See also 14012000 above. | |
2223 method->clear_queued_for_compilation(); | |
2224 | |
2225 #ifdef ASSERT | |
2226 if (CollectedHeap::fired_fake_oom()) { | |
2227 // The current compile received a fake OOM during compilation so | |
2228 // go ahead and exit the VM since the test apparently succeeded | |
2229 tty->print_cr("*** Shutting down VM after successful fake OOM"); | |
2230 vm_exit(0); | |
2231 } | |
2232 #endif | |
2233 } | |
2234 | |
13074
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2235 /** |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2236 * The CodeCache is full. Print out warning and disable compilation |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2237 * or try code cache cleaning so compilation can continue later. |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2238 */ |
1202 | 2239 void CompileBroker::handle_full_code_cache() { |
2240 UseInterpreter = true; | |
2241 if (UseCompiler || AlwaysCompileLoopMethods ) { | |
1728
a62d332029cf
6976372: # assert(_owner == Thread::current()) failed: invariant
never
parents:
1692
diff
changeset
|
2242 if (xtty != NULL) { |
7595
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2243 ResourceMark rm; |
2482
dbccacb79c63
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
2447
diff
changeset
|
2244 stringStream s; |
dbccacb79c63
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
2447
diff
changeset
|
2245 // Dump code cache state into a buffer before locking the tty, |
dbccacb79c63
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
2447
diff
changeset
|
2246 // because log_state() will use locks causing lock conflicts. |
dbccacb79c63
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
2447
diff
changeset
|
2247 CodeCache::log_state(&s); |
dbccacb79c63
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
2447
diff
changeset
|
2248 // Lock to prevent tearing |
dbccacb79c63
7036236: VM crashes assert((!inside_attrs()) || is_error_reported()) failed ...
iveresov
parents:
2447
diff
changeset
|
2249 ttyLocker ttyl; |
1728
a62d332029cf
6976372: # assert(_owner == Thread::current()) failed: invariant
never
parents:
1692
diff
changeset
|
2250 xtty->begin_elem("code_cache_full"); |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
14456
diff
changeset
|
2251 xtty->print("%s", s.as_string()); |
1728
a62d332029cf
6976372: # assert(_owner == Thread::current()) failed: invariant
never
parents:
1692
diff
changeset
|
2252 xtty->stamp(); |
a62d332029cf
6976372: # assert(_owner == Thread::current()) failed: invariant
never
parents:
1692
diff
changeset
|
2253 xtty->end_elem(); |
1202 | 2254 } |
10405 | 2255 |
2256 CodeCache::report_codemem_full(); | |
2257 | |
1578
ca3dceda776c
6930994: Code cache is full warning should be visible in product
never
parents:
1538
diff
changeset
|
2258 #ifndef PRODUCT |
1202 | 2259 if (CompileTheWorld || ExitOnFullCodeCache) { |
7595
9deda4d8e126
8005204: Code Cache Reduction: command line options implementation
vladidan
parents:
7409
diff
changeset
|
2260 codecache_print(/* detailed= */ true); |
1202 | 2261 before_exit(JavaThread::current()); |
2262 exit_globals(); // will delete tty | |
2263 vm_direct_exit(CompileTheWorld ? 0 : 1); | |
2264 } | |
1578
ca3dceda776c
6930994: Code cache is full warning should be visible in product
never
parents:
1538
diff
changeset
|
2265 #endif |
1202 | 2266 if (UseCodeCacheFlushing) { |
12324
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
2267 // Since code cache is full, immediately stop new compiles |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
2268 if (CompileBroker::set_should_compile_new_jobs(CompileBroker::stop_compilation)) { |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
2269 NMethodSweeper::log_sweep("disable_compiler"); |
510fbd28919c
8020151: PSR:PERF Large performance regressions when code cache is filled
anoll
parents:
12158
diff
changeset
|
2270 } |
13074
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2271 // Switch to 'vm_state'. This ensures that possibly_sweep() can be called |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2272 // without having to consider the state in which the current thread is. |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2273 ThreadInVMfromUnknown in_vm; |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2274 NMethodSweeper::possibly_sweep(); |
1202 | 2275 } else { |
12880
469216acdb28
8023014: CodeSweeperSweepNoFlushTest.java fails with HS crash
anoll
parents:
12861
diff
changeset
|
2276 disable_compilation_forever(); |
1202 | 2277 } |
13074
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2278 |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2279 // Print warning only once |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2280 if (should_print_compiler_warning()) { |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2281 warning("CodeCache is full. Compiler has been disabled."); |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2282 warning("Try increasing the code cache size using -XX:ReservedCodeCacheSize="); |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2283 codecache_print(/* detailed= */ true); |
78da3894b86f
8027593: performance drop with constrained codecache starting with hs25 b111
anoll
parents:
13016
diff
changeset
|
2284 } |
1202 | 2285 } |
2286 } | |
2287 | |
2288 // ------------------------------------------------------------------ | |
0 | 2289 // CompileBroker::set_last_compile |
2290 // | |
2291 // Record this compilation for debugging purposes. | |
2292 void CompileBroker::set_last_compile(CompilerThread* thread, methodHandle method, bool is_osr, int comp_level) { | |
2293 ResourceMark rm; | |
2294 char* method_name = method->name()->as_C_string(); | |
2295 strncpy(_last_method_compiled, method_name, CompileBroker::name_buffer_length); | |
17567 | 2296 _last_method_compiled[CompileBroker::name_buffer_length - 1] = '\0'; // ensure null terminated |
0 | 2297 char current_method[CompilerCounters::cmname_buffer_length]; |
2298 size_t maxLen = CompilerCounters::cmname_buffer_length; | |
2299 | |
2300 if (UsePerfData) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
2301 const char* class_name = method->method_holder()->name()->as_C_string(); |
0 | 2302 |
2303 size_t s1len = strlen(class_name); | |
2304 size_t s2len = strlen(method_name); | |
2305 | |
2306 // check if we need to truncate the string | |
2307 if (s1len + s2len + 2 > maxLen) { | |
2308 | |
2309 // the strategy is to lop off the leading characters of the | |
2310 // class name and the trailing characters of the method name. | |
2311 | |
2312 if (s2len + 2 > maxLen) { | |
2313 // lop of the entire class name string, let snprintf handle | |
2314 // truncation of the method name. | |
2315 class_name += s1len; // null string | |
2316 } | |
2317 else { | |
2318 // lop off the extra characters from the front of the class name | |
2319 class_name += ((s1len + s2len + 2) - maxLen); | |
2320 } | |
2321 } | |
2322 | |
2323 jio_snprintf(current_method, maxLen, "%s %s", class_name, method_name); | |
2324 } | |
2325 | |
2326 if (CICountOSR && is_osr) { | |
2327 _last_compile_type = osr_compile; | |
2328 } else { | |
2329 _last_compile_type = normal_compile; | |
2330 } | |
2331 _last_compile_level = comp_level; | |
2332 | |
2333 if (UsePerfData) { | |
2334 CompilerCounters* counters = thread->counters(); | |
2335 counters->set_current_method(current_method); | |
2336 counters->set_compile_type((jlong)_last_compile_type); | |
2337 } | |
2338 } | |
2339 | |
2340 | |
2341 // ------------------------------------------------------------------ | |
2342 // CompileBroker::push_jni_handle_block | |
2343 // | |
2344 // Push on a new block of JNI handles. | |
2345 void CompileBroker::push_jni_handle_block() { | |
2346 JavaThread* thread = JavaThread::current(); | |
2347 | |
2348 // Allocate a new block for JNI handles. | |
2349 // Inlined code from jni_PushLocalFrame() | |
2350 JNIHandleBlock* java_handles = thread->active_handles(); | |
2351 JNIHandleBlock* compile_handles = JNIHandleBlock::allocate_block(thread); | |
2352 assert(compile_handles != NULL && java_handles != NULL, "should not be NULL"); | |
2353 compile_handles->set_pop_frame_link(java_handles); // make sure java handles get gc'd. | |
2354 thread->set_active_handles(compile_handles); | |
2355 } | |
2356 | |
2357 | |
2358 // ------------------------------------------------------------------ | |
2359 // CompileBroker::pop_jni_handle_block | |
2360 // | |
2361 // Pop off the current block of JNI handles. | |
2362 void CompileBroker::pop_jni_handle_block() { | |
2363 JavaThread* thread = JavaThread::current(); | |
2364 | |
2365 // Release our JNI handle block | |
2366 JNIHandleBlock* compile_handles = thread->active_handles(); | |
2367 JNIHandleBlock* java_handles = compile_handles->pop_frame_link(); | |
2368 thread->set_active_handles(java_handles); | |
2369 compile_handles->set_pop_frame_link(NULL); | |
2370 JNIHandleBlock::release_block(compile_handles, thread); // may block | |
2371 } | |
2372 | |
2373 | |
2374 // ------------------------------------------------------------------ | |
2375 // CompileBroker::check_break_at | |
2376 // | |
2377 // Should the compilation break at the current compilation. | |
2378 bool CompileBroker::check_break_at(methodHandle method, int compile_id, bool is_osr) { | |
2379 if (CICountOSR && is_osr && (compile_id == CIBreakAtOSR)) { | |
2380 return true; | |
2381 } else if( CompilerOracle::should_break_at(method) ) { // break when compiling | |
2382 return true; | |
2383 } else { | |
2384 return (compile_id == CIBreakAt); | |
2385 } | |
2386 } | |
2387 | |
2388 // ------------------------------------------------------------------ | |
2389 // CompileBroker::collect_statistics | |
2390 // | |
2391 // Collect statistics about the compilation. | |
2392 | |
2393 void CompileBroker::collect_statistics(CompilerThread* thread, elapsedTimer time, CompileTask* task) { | |
2394 bool success = task->is_success(); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
2395 methodHandle method (thread, task->method()); |
0 | 2396 uint compile_id = task->compile_id(); |
2397 bool is_osr = (task->osr_bci() != standard_entry_bci); | |
2398 nmethod* code = task->code(); | |
2399 CompilerCounters* counters = thread->counters(); | |
2400 | |
2401 assert(code == NULL || code->is_locked_by_vm(), "will survive the MutexLocker"); | |
2402 MutexLocker locker(CompileStatistics_lock); | |
2403 | |
2404 // _perf variables are production performance counters which are | |
2405 // updated regardless of the setting of the CITime and CITimeEach flags | |
2406 // | |
2407 if (!success) { | |
2408 _total_bailout_count++; | |
2409 if (UsePerfData) { | |
2410 _perf_last_failed_method->set_value(counters->current_method()); | |
2411 _perf_last_failed_type->set_value(counters->compile_type()); | |
2412 _perf_total_bailout_count->inc(); | |
2413 } | |
2414 } else if (code == NULL) { | |
2415 if (UsePerfData) { | |
2416 _perf_last_invalidated_method->set_value(counters->current_method()); | |
2417 _perf_last_invalidated_type->set_value(counters->compile_type()); | |
2418 _perf_total_invalidated_count->inc(); | |
2419 } | |
2420 _total_invalidated_count++; | |
2421 } else { | |
2422 // Compilation succeeded | |
2423 | |
2424 // update compilation ticks - used by the implementation of | |
2425 // java.lang.management.CompilationMBean | |
2426 _perf_total_compilation->inc(time.ticks()); | |
2427 | |
10405 | 2428 _t_total_compilation.add(time); |
2429 _peak_compilation_time = time.milliseconds() > _peak_compilation_time ? time.milliseconds() : _peak_compilation_time; | |
2430 | |
0 | 2431 if (CITime) { |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2432 int bytes_compiled = method->code_size() + task->num_inlined_bytecodes(); |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2433 JVMCI_ONLY(CompilerStatistics* stats = compiler(task->comp_level())->stats();) |
0 | 2434 if (is_osr) { |
2435 _t_osr_compilation.add(time); | |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2436 _sum_osr_bytes_compiled += bytes_compiled; |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2437 JVMCI_ONLY(stats->_osr.update(time, bytes_compiled);) |
0 | 2438 } else { |
2439 _t_standard_compilation.add(time); | |
2440 _sum_standard_bytes_compiled += method->code_size() + task->num_inlined_bytecodes(); | |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2441 JVMCI_ONLY(stats->_standard.update(time, bytes_compiled);) |
0 | 2442 } |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2443 JVMCI_ONLY(stats->_nmethods_size += code->total_size();) |
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2444 JVMCI_ONLY(stats->_nmethods_code_size += code->insts_size();) |
0 | 2445 } |
2446 | |
2447 if (UsePerfData) { | |
2448 // save the name of the last method compiled | |
2449 _perf_last_method->set_value(counters->current_method()); | |
2450 _perf_last_compile_type->set_value(counters->compile_type()); | |
2451 _perf_last_compile_size->set_value(method->code_size() + | |
2452 task->num_inlined_bytecodes()); | |
2453 if (is_osr) { | |
2454 _perf_osr_compilation->inc(time.ticks()); | |
2455 _perf_sum_osr_bytes_compiled->inc(method->code_size() + task->num_inlined_bytecodes()); | |
2456 } else { | |
2457 _perf_standard_compilation->inc(time.ticks()); | |
2458 _perf_sum_standard_bytes_compiled->inc(method->code_size() + task->num_inlined_bytecodes()); | |
2459 } | |
2460 } | |
2461 | |
2462 if (CITimeEach) { | |
2463 float bytes_per_sec = 1.0 * (method->code_size() + task->num_inlined_bytecodes()) / time.seconds(); | |
2464 tty->print_cr("%3d seconds: %f bytes/sec : %f (bytes %d + %d inlined)", | |
2465 compile_id, time.seconds(), bytes_per_sec, method->code_size(), task->num_inlined_bytecodes()); | |
2466 } | |
2467 | |
2468 // Collect counts of successful compilations | |
1748 | 2469 _sum_nmethod_size += code->total_size(); |
2470 _sum_nmethod_code_size += code->insts_size(); | |
0 | 2471 _total_compile_count++; |
2472 | |
2473 if (UsePerfData) { | |
1748 | 2474 _perf_sum_nmethod_size->inc( code->total_size()); |
2475 _perf_sum_nmethod_code_size->inc(code->insts_size()); | |
0 | 2476 _perf_total_compile_count->inc(); |
2477 } | |
2478 | |
2479 if (is_osr) { | |
2480 if (UsePerfData) _perf_total_osr_compile_count->inc(); | |
2481 _total_osr_compile_count++; | |
2482 } else { | |
2483 if (UsePerfData) _perf_total_standard_compile_count->inc(); | |
2484 _total_standard_compile_count++; | |
2485 } | |
2486 } | |
2487 // set the current method for the thread to null | |
2488 if (UsePerfData) counters->set_current_method(""); | |
2489 } | |
2490 | |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2491 const char* CompileBroker::compiler_name(int comp_level) { |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2492 AbstractCompiler *comp = CompileBroker::compiler(comp_level); |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2493 if (comp == NULL) { |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2494 return "no compiler"; |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2495 } else { |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2496 return (comp->name()); |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2497 } |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2498 } |
0 | 2499 |
22298 | 2500 #if INCLUDE_JVMCI |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2501 void CompileBroker::print_times(AbstractCompiler* comp) { |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2502 CompilerStatistics* stats = comp->stats(); |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2503 tty->print_cr(" %s {speed: %d bytes/s; standard: %6.3f s, %d bytes, %d methods; osr: %6.3f s, %d bytes, %d methods; nmethods_size: %d bytes; nmethods_code_size: %d bytes}", |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2504 comp->name(), stats->bytes_per_second(), |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2505 stats->_standard._time.seconds(), stats->_standard._bytes, stats->_standard._count, |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2506 stats->_osr._time.seconds(), stats->_osr._bytes, stats->_osr._count, |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2507 stats->_nmethods_size, stats->_nmethods_code_size); |
17374
4a6e24a8fc2c
Track code installation time
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
17166
diff
changeset
|
2508 comp->print_timers(); |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2509 } |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2510 #endif |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2511 |
13364
5a4293f24642
added -G:PrintCompRate option for periodically printing out the current compilation rate
Doug Simon <doug.simon@oracle.com>
parents:
13107
diff
changeset
|
2512 void CompileBroker::print_times(bool per_compiler, bool aggregate) { |
22298 | 2513 #if INCLUDE_JVMCI |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2514 elapsedTimer standard_compilation; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2515 elapsedTimer total_compilation; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2516 elapsedTimer osr_compilation; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2517 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2518 int standard_bytes_compiled = 0; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2519 int osr_bytes_compiled = 0; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2520 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2521 int standard_compile_count = 0; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2522 int osr_compile_count = 0; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2523 int total_compile_count = 0; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2524 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2525 int nmethods_size = 0; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2526 int nmethods_code_size = 0; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2527 bool printedHeader = false; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2528 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2529 for (unsigned int i = 0; i < sizeof(_compilers) / sizeof(AbstractCompiler*); i++) { |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2530 AbstractCompiler* comp = _compilers[i]; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2531 if (comp != NULL) { |
13364
5a4293f24642
added -G:PrintCompRate option for periodically printing out the current compilation rate
Doug Simon <doug.simon@oracle.com>
parents:
13107
diff
changeset
|
2532 if (per_compiler && aggregate && !printedHeader) { |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2533 printedHeader = true; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2534 tty->cr(); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2535 tty->print_cr("Individual compiler times (for compiled methods only)"); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2536 tty->print_cr("------------------------------------------------"); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2537 tty->cr(); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2538 } |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2539 CompilerStatistics* stats = comp->stats(); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2540 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2541 standard_compilation.add(stats->_standard._time); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2542 osr_compilation.add(stats->_osr._time); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2543 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2544 standard_bytes_compiled += stats->_standard._bytes; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2545 osr_bytes_compiled += stats->_osr._bytes; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2546 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2547 standard_compile_count += stats->_standard._count; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2548 osr_compile_count += stats->_osr._count; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2549 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2550 nmethods_size += stats->_nmethods_size; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2551 nmethods_code_size += stats->_nmethods_code_size; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2552 |
13364
5a4293f24642
added -G:PrintCompRate option for periodically printing out the current compilation rate
Doug Simon <doug.simon@oracle.com>
parents:
13107
diff
changeset
|
2553 if (per_compiler) { |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15494
diff
changeset
|
2554 print_times(comp); |
13364
5a4293f24642
added -G:PrintCompRate option for periodically printing out the current compilation rate
Doug Simon <doug.simon@oracle.com>
parents:
13107
diff
changeset
|
2555 } |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2556 } |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2557 } |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2558 total_compile_count = osr_compile_count + standard_compile_count; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2559 total_compilation.add(osr_compilation); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2560 total_compilation.add(standard_compilation); |
18171
9bb816169777
Report code install timer in hosted mode
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
18125
diff
changeset
|
2561 |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2562 #ifndef COMPILERJVMCI |
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2563 // In hosted mode, print the JVMCI compiler specific counters manually. |
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
21527
diff
changeset
|
2564 JVMCICompiler::print_compilation_timers(); |
18171
9bb816169777
Report code install timer in hosted mode
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
18125
diff
changeset
|
2565 #endif |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2566 #else |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2567 elapsedTimer standard_compilation = CompileBroker::_t_standard_compilation; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2568 elapsedTimer osr_compilation = CompileBroker::_t_osr_compilation; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2569 elapsedTimer total_compilation = CompileBroker::_t_total_compilation; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2570 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2571 int standard_bytes_compiled = CompileBroker::_sum_standard_bytes_compiled; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2572 int osr_bytes_compiled = CompileBroker::_sum_osr_bytes_compiled; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2573 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2574 int standard_compile_count = CompileBroker::_total_standard_compile_count; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2575 int osr_compile_count = CompileBroker::_total_osr_compile_count; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2576 int total_compile_count = CompileBroker::_total_compile_count; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2577 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2578 int nmethods_size = CompileBroker::_sum_nmethod_code_size; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2579 int nmethods_code_size = CompileBroker::_sum_nmethod_size; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2580 #endif |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2581 |
13364
5a4293f24642
added -G:PrintCompRate option for periodically printing out the current compilation rate
Doug Simon <doug.simon@oracle.com>
parents:
13107
diff
changeset
|
2582 if (!aggregate) { |
5a4293f24642
added -G:PrintCompRate option for periodically printing out the current compilation rate
Doug Simon <doug.simon@oracle.com>
parents:
13107
diff
changeset
|
2583 return; |
5a4293f24642
added -G:PrintCompRate option for periodically printing out the current compilation rate
Doug Simon <doug.simon@oracle.com>
parents:
13107
diff
changeset
|
2584 } |
0 | 2585 tty->cr(); |
2586 tty->print_cr("Accumulated compiler times (for compiled methods only)"); | |
2587 tty->print_cr("------------------------------------------------"); | |
2588 //0000000000111111111122222222223333333333444444444455555555556666666666 | |
2589 //0123456789012345678901234567890123456789012345678901234567890123456789 | |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2590 tty->print_cr(" Total compilation time : %6.3f s", total_compilation.seconds()); |
0 | 2591 tty->print_cr(" Standard compilation : %6.3f s, Average : %2.3f", |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2592 standard_compilation.seconds(), |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2593 standard_compilation.seconds() / standard_compile_count); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2594 tty->print_cr(" On stack replacement : %6.3f s, Average : %2.3f", osr_compilation.seconds(), osr_compilation.seconds() / osr_compile_count); |
1131 | 2595 |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2596 AbstractCompiler *comp = compiler(CompLevel_simple); |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2597 if (comp != NULL) { |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2598 comp->print_timers(); |
0 | 2599 } |
8771
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2600 comp = compiler(CompLevel_full_optimization); |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2601 if (comp != NULL) { |
75a28f465a12
8008663: [parfait] Null pointer deference in hotspot/src/share/vm/compiler/compileBroker.cpp
morris
parents:
8730
diff
changeset
|
2602 comp->print_timers(); |
1783 | 2603 } |
0 | 2604 tty->cr(); |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2605 tty->print_cr(" Total compiled methods : %6d methods", total_compile_count); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2606 tty->print_cr(" Standard compilation : %6d methods", standard_compile_count); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2607 tty->print_cr(" On stack replacement : %6d methods", osr_compile_count); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2608 int tcb = osr_bytes_compiled + standard_bytes_compiled; |
0 | 2609 tty->print_cr(" Total compiled bytecodes : %6d bytes", tcb); |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2610 tty->print_cr(" Standard compilation : %6d bytes", standard_bytes_compiled); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2611 tty->print_cr(" On stack replacement : %6d bytes", osr_bytes_compiled); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2612 double tcs = total_compilation.seconds(); |
7568
140d4d4ab3b9
added per-compiler compilation speed statistics for C1 and C2 (included in output produced by -XX:+CITime)
Doug Simon <doug.simon@oracle.com>
parents:
7482
diff
changeset
|
2613 int bps = tcs == 0.0 ? 0 : (int)(tcb / tcs); |
0 | 2614 tty->print_cr(" Average compilation speed: %6d bytes/s", bps); |
2615 tty->cr(); | |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2616 tty->print_cr(" nmethod code size : %6d bytes", nmethods_code_size); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12356
diff
changeset
|
2617 tty->print_cr(" nmethod total size : %6d bytes", nmethods_size); |
0 | 2618 } |
2619 | |
2620 // Debugging output for failure | |
2621 void CompileBroker::print_last_compile() { | |
2622 if ( _last_compile_level != CompLevel_none && | |
2623 compiler(_last_compile_level) != NULL && | |
2624 _last_method_compiled != NULL && | |
2625 _last_compile_type != no_compile) { | |
2626 if (_last_compile_type == osr_compile) { | |
2627 tty->print_cr("Last parse: [osr]%d+++(%d) %s", | |
2628 _osr_compilation_id, _last_compile_level, _last_method_compiled); | |
2629 } else { | |
2630 tty->print_cr("Last parse: %d+++(%d) %s", | |
2631 _compilation_id, _last_compile_level, _last_method_compiled); | |
2632 } | |
2633 } | |
2634 } | |
2635 | |
2636 | |
2637 void CompileBroker::print_compiler_threads_on(outputStream* st) { | |
2638 #ifndef PRODUCT | |
2639 st->print_cr("Compiler thread printing unimplemented."); | |
2640 st->cr(); | |
2641 #endif | |
2642 } |