author | pchilanomate |
Tue, 05 Feb 2019 15:12:13 -0500 | |
changeset 53646 | 043ae846819f |
parent 53582 | 881c5fbeb849 |
child 53775 | 5d20b085d893 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
53582 | 2 |
* Copyright (c) 1997, 2019, Oracle and/or its affiliates. All rights reserved. |
1 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5533
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5533
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5533
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
51959
db0c3952de52
8209645: Split ClassLoaderData and ClassLoaderDataGraph into separate files
coleenp
parents:
51865
diff
changeset
|
26 |
#include "classfile/classLoaderDataGraph.inline.hpp" |
24426
0a69c8cdfca9
8038654: Separate SymbolTable and StringTable code
gziemski
parents:
24096
diff
changeset
|
27 |
#include "classfile/stringTable.hpp" |
34666 | 28 |
#include "classfile/symbolTable.hpp" |
7397 | 29 |
#include "classfile/systemDictionary.hpp" |
30 |
#include "code/codeCache.hpp" |
|
31 |
#include "code/icBuffer.hpp" |
|
32 |
#include "code/nmethod.hpp" |
|
33 |
#include "code/pcDesc.hpp" |
|
34 |
#include "code/scopeDesc.hpp" |
|
30764 | 35 |
#include "gc/shared/collectedHeap.hpp" |
49594
898ef81cbc0e
8200106: Move NoSafepointVerifier out from gcLocker.hpp
stefank
parents:
49449
diff
changeset
|
36 |
#include "gc/shared/gcLocker.hpp" |
46702 | 37 |
#include "gc/shared/strongRootsScope.hpp" |
38 |
#include "gc/shared/workgroup.hpp" |
|
7397 | 39 |
#include "interpreter/interpreter.hpp" |
50113 | 40 |
#include "jfr/jfrEvents.hpp" |
33763
cec2333f839c
8140348: Convert TraceSafepoint to Unified Logging
rprotacio
parents:
33589
diff
changeset
|
41 |
#include "logging/log.hpp" |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46630
diff
changeset
|
42 |
#include "logging/logStream.hpp" |
7397 | 43 |
#include "memory/resourceArea.hpp" |
49359
59f6547e151f
8199264: Remove universe.inline.hpp to simplify include dependencies
stefank
parents:
49169
diff
changeset
|
44 |
#include "memory/universe.hpp" |
7397 | 45 |
#include "oops/oop.inline.hpp" |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
46 |
#include "oops/symbol.hpp" |
40655
9f644073d3a0
8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents:
38151
diff
changeset
|
47 |
#include "runtime/atomic.hpp" |
7397 | 48 |
#include "runtime/compilationPolicy.hpp" |
49 |
#include "runtime/deoptimization.hpp" |
|
50 |
#include "runtime/frame.inline.hpp" |
|
53582 | 51 |
#include "runtime/handles.inline.hpp" |
49449
ef5d5d343e2a
8199263: Split interfaceSupport.hpp to not require including .inline.hpp files
coleenp
parents:
49359
diff
changeset
|
52 |
#include "runtime/interfaceSupport.inline.hpp" |
7397 | 53 |
#include "runtime/mutexLocker.hpp" |
50429
83aec1d357d4
8204301: Make OrderAccess functions available to hpp rather than inline.hpp files
coleenp
parents:
50113
diff
changeset
|
54 |
#include "runtime/orderAccess.hpp" |
7397 | 55 |
#include "runtime/osThread.hpp" |
56 |
#include "runtime/safepoint.hpp" |
|
47881
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
57 |
#include "runtime/safepointMechanism.inline.hpp" |
7397 | 58 |
#include "runtime/signature.hpp" |
59 |
#include "runtime/stubCodeGenerator.hpp" |
|
60 |
#include "runtime/stubRoutines.hpp" |
|
61 |
#include "runtime/sweeper.hpp" |
|
62 |
#include "runtime/synchronizer.hpp" |
|
14583
d70ee55535f4
8003935: Simplify the needed includes for using Thread::current()
stefank
parents:
13195
diff
changeset
|
63 |
#include "runtime/thread.inline.hpp" |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
64 |
#include "runtime/threadSMR.hpp" |
37161
e881f320966e
8150015: Integrate TraceTime with Unified Logging more seamlessly
rehn
parents:
37135
diff
changeset
|
65 |
#include "runtime/timerTrace.hpp" |
7397 | 66 |
#include "services/runtimeService.hpp" |
67 |
#include "utilities/events.hpp" |
|
15482
470d0b0c09f1
8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents:
14583
diff
changeset
|
68 |
#include "utilities/macros.hpp" |
7397 | 69 |
#ifdef COMPILER1 |
70 |
#include "c1/c1_globals.hpp" |
|
71 |
#endif |
|
1 | 72 |
|
50113 | 73 |
template <typename E> |
74 |
static void set_current_safepoint_id(E* event, int adjustment = 0) { |
|
75 |
assert(event != NULL, "invariant"); |
|
76 |
event->set_safepointId(SafepointSynchronize::safepoint_counter() + adjustment); |
|
77 |
} |
|
78 |
||
79 |
static void post_safepoint_begin_event(EventSafepointBegin* event, |
|
80 |
int thread_count, |
|
81 |
int critical_thread_count) { |
|
82 |
assert(event != NULL, "invariant"); |
|
83 |
assert(event->should_commit(), "invariant"); |
|
84 |
set_current_safepoint_id(event); |
|
85 |
event->set_totalThreadCount(thread_count); |
|
86 |
event->set_jniCriticalThreadCount(critical_thread_count); |
|
87 |
event->commit(); |
|
88 |
} |
|
89 |
||
90 |
static void post_safepoint_cleanup_event(EventSafepointCleanup* event) { |
|
91 |
assert(event != NULL, "invariant"); |
|
92 |
assert(event->should_commit(), "invariant"); |
|
93 |
set_current_safepoint_id(event); |
|
94 |
event->commit(); |
|
95 |
} |
|
96 |
||
97 |
static void post_safepoint_synchronize_event(EventSafepointStateSynchronization* event, |
|
98 |
int initial_number_of_threads, |
|
99 |
int threads_waiting_to_block, |
|
100 |
unsigned int iterations) { |
|
101 |
assert(event != NULL, "invariant"); |
|
102 |
if (event->should_commit()) { |
|
103 |
// Group this event together with the ones committed after the counter is increased |
|
104 |
set_current_safepoint_id(event, 1); |
|
105 |
event->set_initialThreadCount(initial_number_of_threads); |
|
106 |
event->set_runningThreadCount(threads_waiting_to_block); |
|
107 |
event->set_iterations(iterations); |
|
108 |
event->commit(); |
|
109 |
} |
|
110 |
} |
|
111 |
||
112 |
static void post_safepoint_wait_blocked_event(EventSafepointWaitBlocked* event, |
|
113 |
int initial_threads_waiting_to_block) { |
|
114 |
assert(event != NULL, "invariant"); |
|
115 |
assert(event->should_commit(), "invariant"); |
|
116 |
set_current_safepoint_id(event); |
|
117 |
event->set_runningThreadCount(initial_threads_waiting_to_block); |
|
118 |
event->commit(); |
|
119 |
} |
|
120 |
||
121 |
static void post_safepoint_cleanup_task_event(EventSafepointCleanupTask* event, |
|
122 |
const char* name) { |
|
123 |
assert(event != NULL, "invariant"); |
|
124 |
if (event->should_commit()) { |
|
125 |
set_current_safepoint_id(event); |
|
126 |
event->set_name(name); |
|
127 |
event->commit(); |
|
128 |
} |
|
129 |
} |
|
130 |
||
131 |
static void post_safepoint_end_event(EventSafepointEnd* event) { |
|
132 |
assert(event != NULL, "invariant"); |
|
133 |
if (event->should_commit()) { |
|
134 |
// Group this event together with the ones committed before the counter increased |
|
135 |
set_current_safepoint_id(event, -1); |
|
136 |
event->commit(); |
|
137 |
} |
|
138 |
} |
|
139 |
||
1 | 140 |
// -------------------------------------------------------------------------------------------------- |
141 |
// Implementation of Safepoint begin/end |
|
142 |
||
143 |
SafepointSynchronize::SynchronizeState volatile SafepointSynchronize::_state = SafepointSynchronize::_not_synchronized; |
|
144 |
volatile int SafepointSynchronize::_waiting_to_block = 0; |
|
52672
bbfa1b3aaf7e
8212108: SafepointSynchronizer never ending counter (big enough)
rehn
parents:
52628
diff
changeset
|
145 |
volatile uint64_t SafepointSynchronize::_safepoint_counter = 0; |
11631
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
146 |
int SafepointSynchronize::_current_jni_active_count = 0; |
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
147 |
long SafepointSynchronize::_end_of_last_safepoint = 0; |
49169
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
148 |
int SafepointSynchronize::_defer_thr_suspend_loop_count = 4000; |
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
149 |
static const int safepoint_spin_before_yield = 2000; |
1 | 150 |
static volatile int PageArmed = 0 ; // safepoint polling page is RO|RW vs PROT_NONE |
151 |
static volatile int TryingToBlock = 0 ; // proximate value -- for advisory use only |
|
152 |
static bool timeout_error_printed = false; |
|
153 |
||
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
154 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
155 |
// Statistic related statics |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
156 |
julong SafepointSynchronize::_coalesced_vmop_count = 0; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
157 |
static jlong _safepoint_begin_time = 0; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
158 |
static float _ts_of_current_safepoint = 0.0f; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
159 |
static volatile int _nof_threads_hit_polling_page = 0; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
160 |
|
1 | 161 |
// Roll all threads forward to a safepoint and suspend them all |
162 |
void SafepointSynchronize::begin() { |
|
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
163 |
EventSafepointBegin begin_event; |
1 | 164 |
Thread* myThread = Thread::current(); |
165 |
assert(myThread->is_VM_thread(), "Only VM thread may execute a safepoint"); |
|
166 |
||
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
167 |
if (log_is_enabled(Debug, safepoint, stats)) { |
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
168 |
_safepoint_begin_time = os::javaTimeNanos(); |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
169 |
_ts_of_current_safepoint = tty->time_stamp().seconds(); |
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
170 |
_nof_threads_hit_polling_page = 0; |
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
171 |
} |
1 | 172 |
|
47636
52d46d7e3454
8188919: Generalize GC thread suspend/resume at safepoints
eosterlund
parents:
47629
diff
changeset
|
173 |
Universe::heap()->safepoint_synchronize_begin(); |
1 | 174 |
|
175 |
// By getting the Threads_lock, we assure that no threads are about to start or |
|
176 |
// exit. It is released again in SafepointSynchronize::end(). |
|
177 |
Threads_lock->lock(); |
|
178 |
||
179 |
assert( _state == _not_synchronized, "trying to safepoint synchronize with wrong state"); |
|
180 |
||
181 |
int nof_threads = Threads::number_of_threads(); |
|
182 |
||
48472
de2e4ff493bf
8152957: Improve specificity of safepoint logging to print safepoint type
coleenp
parents:
48105
diff
changeset
|
183 |
log_debug(safepoint)("Safepoint synchronization initiated. (%d threads)", nof_threads); |
1 | 184 |
|
185 |
RuntimeService::record_safepoint_begin(); |
|
186 |
||
187 |
MutexLocker mu(Safepoint_lock); |
|
188 |
||
11631
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
189 |
// Reset the count of active JNI critical threads |
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
190 |
_current_jni_active_count = 0; |
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
191 |
|
1 | 192 |
// Set number of threads to wait for, before we initiate the callbacks |
193 |
_waiting_to_block = nof_threads; |
|
194 |
TryingToBlock = 0 ; |
|
195 |
int still_running = nof_threads; |
|
196 |
||
197 |
// Save the starting time, so that it can be compared to see if this has taken |
|
198 |
// too long to complete. |
|
33589
7cbd1b2c139b
8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents:
33222
diff
changeset
|
199 |
jlong safepoint_limit_time = 0; |
1 | 200 |
timeout_error_printed = false; |
201 |
||
202 |
// Begin the process of bringing the system to a safepoint. |
|
203 |
// Java threads can be in several different states and are |
|
204 |
// stopped by different mechanisms: |
|
205 |
// |
|
206 |
// 1. Running interpreted |
|
22551 | 207 |
// The interpreter dispatch table is changed to force it to |
1 | 208 |
// check for a safepoint condition between bytecodes. |
209 |
// 2. Running in native code |
|
210 |
// When returning from the native code, a Java thread must check |
|
211 |
// the safepoint _state to see if we must block. If the |
|
212 |
// VM thread sees a Java thread in native, it does |
|
213 |
// not wait for this thread to block. The order of the memory |
|
214 |
// writes and reads of both the safepoint state and the Java |
|
215 |
// threads state is critical. In order to guarantee that the |
|
216 |
// memory writes are serialized with respect to each other, |
|
52460 | 217 |
// the VM thread issues a memory barrier instruction. |
1 | 218 |
// 3. Running compiled Code |
219 |
// Compiled code reads a global (Safepoint Polling) page that |
|
220 |
// is set to fault if we are trying to get to a safepoint. |
|
221 |
// 4. Blocked |
|
222 |
// A thread which is blocked will not be allowed to return from the |
|
223 |
// block condition until the safepoint operation is complete. |
|
224 |
// 5. In VM or Transitioning between states |
|
225 |
// If a Java thread is currently running in the VM or transitioning |
|
226 |
// between states, the safepointing code will wait for the thread to |
|
227 |
// block itself when it attempts transitions to a new state. |
|
228 |
// |
|
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
229 |
{ |
40664
1ec65b303bb7
8164523: Clean up metadata for event based tracing
egahlin
parents:
40655
diff
changeset
|
230 |
EventSafepointStateSynchronization sync_event; |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
231 |
int initial_running = 0; |
1 | 232 |
|
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
233 |
_state = _synchronizing; |
47881
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
234 |
|
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
235 |
if (SafepointMechanism::uses_thread_local_poll()) { |
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
236 |
// Arming the per thread poll while having _state != _not_synchronized means safepointing |
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
237 |
log_trace(safepoint)("Setting thread local yield flag for threads"); |
50921
7f462e8383f6
8206003: SafepointSynchronize with TLH: StoreStore barriers should be moved out of the loop
mdoerr
parents:
50429
diff
changeset
|
238 |
OrderAccess::storestore(); // storestore, global state -> local state |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
239 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *cur = jtiwh.next(); ) { |
47881
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
240 |
// Make sure the threads start polling, it is time to yield. |
50921
7f462e8383f6
8206003: SafepointSynchronize with TLH: StoreStore barriers should be moved out of the loop
mdoerr
parents:
50429
diff
changeset
|
241 |
SafepointMechanism::arm_local_poll(cur); |
47881
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
242 |
} |
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
243 |
} |
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
244 |
OrderAccess::fence(); // storestore|storeload, global state -> local state |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
245 |
|
47881
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
246 |
if (SafepointMechanism::uses_global_page_poll()) { |
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
247 |
// Make interpreter safepoint aware |
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
248 |
Interpreter::notice_safepoints(); |
1 | 249 |
|
49169
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
250 |
// Make polling safepoint aware |
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
251 |
guarantee (PageArmed == 0, "invariant") ; |
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
252 |
PageArmed = 1 ; |
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
253 |
os::make_polling_page_unreadable(); |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
254 |
} |
1 | 255 |
|
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
256 |
// Consider using active_processor_count() ... but that call is expensive. |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
257 |
int ncpus = os::processor_count() ; |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
258 |
unsigned int iterations = 0; |
1 | 259 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
260 |
{ |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
261 |
JavaThreadIteratorWithHandle jtiwh; |
1 | 262 |
#ifdef ASSERT |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
263 |
for (; JavaThread *cur = jtiwh.next(); ) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
264 |
assert(cur->safepoint_state()->is_running(), "Illegal initial state"); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
265 |
// Clear the visited flag to ensure that the critical counts are collected properly. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
266 |
cur->set_visited_for_critical_count(false); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
267 |
} |
1 | 268 |
#endif // ASSERT |
269 |
||
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
270 |
if (SafepointTimeout) |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
271 |
safepoint_limit_time = os::javaTimeNanos() + (jlong)SafepointTimeoutDelay * MICROUNITS; |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
272 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
273 |
// Iterate through all threads until it have been determined how to stop them all at a safepoint |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
274 |
int steps = 0 ; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
275 |
while(still_running > 0) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
276 |
jtiwh.rewind(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
277 |
for (; JavaThread *cur = jtiwh.next(); ) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
278 |
assert(!cur->is_ConcurrentGC_thread(), "A concurrent GC thread is unexpectly being suspended"); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
279 |
ThreadSafepointState *cur_state = cur->safepoint_state(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
280 |
if (cur_state->is_running()) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
281 |
cur_state->examine_state_of_thread(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
282 |
if (!cur_state->is_running()) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
283 |
still_running--; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
284 |
// consider adjusting steps downward: |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
285 |
// steps = 0 |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
286 |
// steps -= NNN |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
287 |
// steps >>= 1 |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
288 |
// steps = MIN(steps, 2000-100) |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
289 |
// if (iterations != 0) steps -= NNN |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
290 |
} |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
291 |
LogTarget(Trace, safepoint) lt; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
292 |
if (lt.is_enabled()) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
293 |
ResourceMark rm; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
294 |
LogStream ls(lt); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
295 |
cur_state->print_on(&ls); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
296 |
} |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
297 |
} |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
298 |
} |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
299 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
300 |
if (iterations == 0) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
301 |
initial_running = still_running; |
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
302 |
if (log_is_enabled(Debug, safepoint, stats)) { |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
303 |
begin_statistics(nof_threads, still_running); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
304 |
} |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
305 |
} |
1 | 306 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
307 |
if (still_running > 0) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
308 |
// Check for if it takes to long |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
309 |
if (SafepointTimeout && safepoint_limit_time < os::javaTimeNanos()) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
310 |
print_safepoint_timeout(_spinning_timeout); |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
311 |
} |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
312 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
313 |
// Spin to avoid context switching. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
314 |
// There's a tension between allowing the mutators to run (and rendezvous) |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
315 |
// vs spinning. As the VM thread spins, wasting cycles, it consumes CPU that |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
316 |
// a mutator might otherwise use profitably to reach a safepoint. Excessive |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
317 |
// spinning by the VM thread on a saturated system can increase rendezvous latency. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
318 |
// Blocking or yielding incur their own penalties in the form of context switching |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
319 |
// and the resultant loss of $ residency. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
320 |
// |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
321 |
// Further complicating matters is that yield() does not work as naively expected |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
322 |
// on many platforms -- yield() does not guarantee that any other ready threads |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
323 |
// will run. As such we revert to naked_short_sleep() after some number of iterations. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
324 |
// nakes_short_sleep() is implemented as a short unconditional sleep. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
325 |
// Typical operating systems round a "short" sleep period up to 10 msecs, so sleeping |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
326 |
// can actually increase the time it takes the VM thread to detect that a system-wide |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
327 |
// stop-the-world safepoint has been reached. In a pathological scenario such as that |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
328 |
// described in CR6415670 the VMthread may sleep just before the mutator(s) become safe. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
329 |
// In that case the mutators will be stalled waiting for the safepoint to complete and the |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
330 |
// the VMthread will be sleeping, waiting for the mutators to rendezvous. The VMthread |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
331 |
// will eventually wake up and detect that all mutators are safe, at which point |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
332 |
// we'll again make progress. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
333 |
// |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
334 |
// Beware too that that the VMThread typically runs at elevated priority. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
335 |
// Its default priority is higher than the default mutator priority. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
336 |
// Obviously, this complicates spinning. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
337 |
// |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
338 |
// Note too that on Windows XP SwitchThreadTo() has quite different behavior than Sleep(0). |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
339 |
// Sleep(0) will _not yield to lower priority threads, while SwitchThreadTo() will. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
340 |
// |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
341 |
// See the comments in synchronizer.cpp for additional remarks on spinning. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
342 |
// |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
343 |
// In the future we might: |
51376
181e6a03249b
8164683: Solaris: JVM abuses thread preemption control
coleenp
parents:
51338
diff
changeset
|
344 |
// -- Modify the safepoint scheme to avoid potentially unbounded spinning. |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
345 |
// This is tricky as the path used by a thread exiting the JVM (say on |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
346 |
// on JNI call-out) simply stores into its state field. The burden |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
347 |
// is placed on the VM thread, which must poll (spin). |
51376
181e6a03249b
8164683: Solaris: JVM abuses thread preemption control
coleenp
parents:
51338
diff
changeset
|
348 |
// -- Find something useful to do while spinning. If the safepoint is GC-related |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
349 |
// we might aggressively scan the stacks of threads that are already safe. |
51376
181e6a03249b
8164683: Solaris: JVM abuses thread preemption control
coleenp
parents:
51338
diff
changeset
|
350 |
// -- YieldTo() any still-running mutators that are ready but OFFPROC. |
181e6a03249b
8164683: Solaris: JVM abuses thread preemption control
coleenp
parents:
51338
diff
changeset
|
351 |
// -- Check system saturation. If the system is not fully saturated then |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
352 |
// simply spin and avoid sleep/yield. |
51376
181e6a03249b
8164683: Solaris: JVM abuses thread preemption control
coleenp
parents:
51338
diff
changeset
|
353 |
// -- As still-running mutators rendezvous they could unpark the sleeping |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
354 |
// VMthread. This works well for still-running mutators that become |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
355 |
// safe. The VMthread must still poll for mutators that call-out. |
51376
181e6a03249b
8164683: Solaris: JVM abuses thread preemption control
coleenp
parents:
51338
diff
changeset
|
356 |
// -- Drive the policy on time-since-begin instead of iterations. |
181e6a03249b
8164683: Solaris: JVM abuses thread preemption control
coleenp
parents:
51338
diff
changeset
|
357 |
// -- Consider making the spin duration a function of the # of CPUs: |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
358 |
// Spin = (((ncpus-1) * M) + K) + F(still_running) |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
359 |
// Alternately, instead of counting iterations of the outer loop |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
360 |
// we could count the # of threads visited in the inner loop, above. |
51376
181e6a03249b
8164683: Solaris: JVM abuses thread preemption control
coleenp
parents:
51338
diff
changeset
|
361 |
// -- On windows consider using the return value from SwitchThreadTo() |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
362 |
// to drive subsequent spin/SwitchThreadTo()/Sleep(N) decisions. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
363 |
|
49169
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
364 |
if (int(iterations) == -1) { // overflow - something is wrong. |
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
365 |
// We can only overflow here when we are using global |
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
366 |
// polling pages. We keep this guarantee in its original |
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
367 |
// form so that searches of the bug database for this |
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
368 |
// failure mode find the right bugs. |
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
369 |
guarantee (PageArmed == 0, "invariant"); |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
370 |
} |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
371 |
|
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
372 |
// Instead of (ncpus > 1) consider either (still_running < (ncpus + EPSILON)) or |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
373 |
// ((still_running + _waiting_to_block - TryingToBlock)) < ncpus) |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
374 |
++steps ; |
49169
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
375 |
if (ncpus > 1 && steps < safepoint_spin_before_yield) { |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
376 |
SpinPause() ; // MP-Polite spin |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
377 |
} else |
49169
af8578e25d17
8194085: Obsolete the deprecated SafepointSynchronize flags and remove related code
hseigel
parents:
49061
diff
changeset
|
378 |
if (steps < _defer_thr_suspend_loop_count) { |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
379 |
os::naked_yield() ; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
380 |
} else { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
381 |
os::naked_short_sleep(1); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
382 |
} |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
383 |
|
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
384 |
iterations ++ ; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
385 |
} |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
386 |
assert(iterations < (uint)max_jint, "We have been iterating in the safepoint loop too long"); |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
387 |
} |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
388 |
} // ThreadsListHandle destroyed here. |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
389 |
assert(still_running == 0, "sanity check"); |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
390 |
|
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
391 |
if (log_is_enabled(Debug, safepoint, stats)) { |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
392 |
update_statistics_on_spin_end(); |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
393 |
} |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
394 |
if (sync_event.should_commit()) { |
50113 | 395 |
post_safepoint_synchronize_event(&sync_event, initial_running, _waiting_to_block, iterations); |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
396 |
} |
50113 | 397 |
} |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
398 |
|
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
399 |
// wait until all threads are stopped |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
400 |
{ |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
401 |
EventSafepointWaitBlocked wait_blocked_event; |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
402 |
int initial_waiting_to_block = _waiting_to_block; |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
403 |
|
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
404 |
while (_waiting_to_block > 0) { |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
405 |
log_debug(safepoint)("Waiting for %d thread(s) to block", _waiting_to_block); |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
406 |
if (!SafepointTimeout || timeout_error_printed) { |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
407 |
Safepoint_lock->wait(true); // true, means with no safepoint checks |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
408 |
} else { |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
409 |
// Compute remaining time |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
410 |
jlong remaining_time = safepoint_limit_time - os::javaTimeNanos(); |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
411 |
|
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
412 |
// If there is no remaining time, then there is an error |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
413 |
if (remaining_time < 0 || Safepoint_lock->wait(true, remaining_time / MICROUNITS)) { |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
414 |
print_safepoint_timeout(_blocking_timeout); |
33763
cec2333f839c
8140348: Convert TraceSafepoint to Unified Logging
rprotacio
parents:
33589
diff
changeset
|
415 |
} |
1 | 416 |
} |
417 |
} |
|
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
418 |
assert(_waiting_to_block == 0, "sanity check"); |
1 | 419 |
|
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
420 |
#ifndef PRODUCT |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
421 |
if (SafepointTimeout) { |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
422 |
jlong current_time = os::javaTimeNanos(); |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
423 |
if (safepoint_limit_time < current_time) { |
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
424 |
log_warning(safepoint)("# SafepointSynchronize: Finished after " |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
425 |
INT64_FORMAT_W(6) " ms", |
46630
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
46541
diff
changeset
|
426 |
(int64_t)((current_time - safepoint_limit_time) / MICROUNITS + |
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
46541
diff
changeset
|
427 |
(jlong)SafepointTimeoutDelay)); |
1 | 428 |
} |
429 |
} |
|
430 |
#endif |
|
431 |
||
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
432 |
assert((_safepoint_counter & 0x1) == 0, "must be even"); |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
433 |
assert(Threads_lock->owned_by_self(), "must hold Threads_lock"); |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
434 |
_safepoint_counter ++; |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
435 |
|
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
436 |
// Record state |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
437 |
_state = _synchronized; |
1 | 438 |
|
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
439 |
OrderAccess::fence(); |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
440 |
if (wait_blocked_event.should_commit()) { |
50113 | 441 |
post_safepoint_wait_blocked_event(&wait_blocked_event, initial_waiting_to_block); |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
442 |
} |
50113 | 443 |
} |
1 | 444 |
|
12108
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
445 |
#ifdef ASSERT |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
446 |
// Make sure all the threads were visited. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
447 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *cur = jtiwh.next(); ) { |
12108
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
448 |
assert(cur->was_visited_for_critical_count(), "missed a thread"); |
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
449 |
} |
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
450 |
#endif // ASSERT |
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
451 |
|
11631
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
452 |
// Update the count of active JNI critical regions |
35492
c8c0273e6b91
8146690: Make all classes in GC follow the naming convention.
david
parents:
35061
diff
changeset
|
453 |
GCLocker::set_jni_lock_count(_current_jni_active_count); |
11631
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
454 |
|
48472
de2e4ff493bf
8152957: Improve specificity of safepoint logging to print safepoint type
coleenp
parents:
48105
diff
changeset
|
455 |
log_info(safepoint)("Entering safepoint region: %s", VMThread::vm_safepoint_description()); |
1 | 456 |
|
457 |
RuntimeService::record_safepoint_synchronized(); |
|
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
458 |
if (log_is_enabled(Debug, safepoint, stats)) { |
1 | 459 |
update_statistics_on_sync_end(os::javaTimeNanos()); |
460 |
} |
|
461 |
||
462 |
// Call stuff that needs to be run when a safepoint is just about to be completed |
|
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
463 |
{ |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
464 |
EventSafepointCleanup cleanup_event; |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
465 |
do_cleanup_tasks(); |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
466 |
if (cleanup_event.should_commit()) { |
50113 | 467 |
post_safepoint_cleanup_event(&cleanup_event); |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
468 |
} |
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
469 |
} |
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
470 |
|
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
471 |
if (log_is_enabled(Debug, safepoint, stats)) { |
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
472 |
// Record how much time spend on the above cleanup tasks |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
473 |
update_statistics_on_cleanup_end(os::javaTimeNanos()); |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
474 |
} |
50113 | 475 |
|
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
476 |
if (begin_event.should_commit()) { |
50113 | 477 |
post_safepoint_begin_event(&begin_event, nof_threads, _current_jni_active_count); |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
478 |
} |
1 | 479 |
} |
480 |
||
481 |
// Wake up all threads, so they are ready to resume execution after the safepoint |
|
482 |
// operation has been carried out |
|
483 |
void SafepointSynchronize::end() { |
|
484 |
assert(Threads_lock->owned_by_self(), "must hold Threads_lock"); |
|
485 |
assert((_safepoint_counter & 0x1) == 1, "must be odd"); |
|
50113 | 486 |
EventSafepointEnd event; |
1 | 487 |
_safepoint_counter ++; |
488 |
// memory fence isn't required here since an odd _safepoint_counter |
|
489 |
// value can do no harm and a fence is issued below anyway. |
|
490 |
||
491 |
DEBUG_ONLY(Thread* myThread = Thread::current();) |
|
492 |
assert(myThread->is_VM_thread(), "Only VM thread can execute a safepoint"); |
|
493 |
||
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
494 |
if (log_is_enabled(Debug, safepoint, stats)) { |
1 | 495 |
end_statistics(os::javaTimeNanos()); |
496 |
} |
|
497 |
||
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
498 |
{ |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
499 |
JavaThreadIteratorWithHandle jtiwh; |
1 | 500 |
#ifdef ASSERT |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
501 |
// A pending_exception cannot be installed during a safepoint. The threads |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
502 |
// may install an async exception after they come back from a safepoint into |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
503 |
// pending_exception after they unblock. But that should happen later. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
504 |
for (; JavaThread *cur = jtiwh.next(); ) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
505 |
assert (!(cur->has_pending_exception() && |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
506 |
cur->safepoint_state()->is_at_poll_safepoint()), |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
507 |
"safepoint installed a pending exception"); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
508 |
} |
1 | 509 |
#endif // ASSERT |
510 |
||
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
511 |
if (PageArmed) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
512 |
assert(SafepointMechanism::uses_global_page_poll(), "sanity"); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
513 |
// Make polling safepoint aware |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
514 |
os::make_polling_page_readable(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
515 |
PageArmed = 0 ; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
516 |
} |
1 | 517 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
518 |
if (SafepointMechanism::uses_global_page_poll()) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
519 |
// Remove safepoint check from interpreter |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
520 |
Interpreter::ignore_safepoints(); |
1 | 521 |
} |
522 |
||
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
523 |
{ |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
524 |
MutexLocker mu(Safepoint_lock); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
525 |
|
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
526 |
assert(_state == _synchronized, "must be synchronized before ending safepoint synchronization"); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
527 |
|
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
528 |
if (SafepointMechanism::uses_thread_local_poll()) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
529 |
_state = _not_synchronized; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
530 |
OrderAccess::storestore(); // global state -> local state |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
531 |
jtiwh.rewind(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
532 |
for (; JavaThread *current = jtiwh.next(); ) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
533 |
ThreadSafepointState* cur_state = current->safepoint_state(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
534 |
cur_state->restart(); // TSS _running |
50921
7f462e8383f6
8206003: SafepointSynchronize with TLH: StoreStore barriers should be moved out of the loop
mdoerr
parents:
50429
diff
changeset
|
535 |
SafepointMechanism::disarm_local_poll(current); |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
536 |
} |
48472
de2e4ff493bf
8152957: Improve specificity of safepoint logging to print safepoint type
coleenp
parents:
48105
diff
changeset
|
537 |
log_info(safepoint)("Leaving safepoint region"); |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
538 |
} else { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
539 |
// Set to not synchronized, so the threads will not go into the signal_thread_blocked method |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
540 |
// when they get restarted. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
541 |
_state = _not_synchronized; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
542 |
OrderAccess::fence(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
543 |
|
48472
de2e4ff493bf
8152957: Improve specificity of safepoint logging to print safepoint type
coleenp
parents:
48105
diff
changeset
|
544 |
log_info(safepoint)("Leaving safepoint region"); |
1 | 545 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
546 |
// Start suspended threads |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
547 |
jtiwh.rewind(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
548 |
for (; JavaThread *current = jtiwh.next(); ) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
549 |
ThreadSafepointState* cur_state = current->safepoint_state(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
550 |
assert(cur_state->type() != ThreadSafepointState::_running, "Thread not suspended at safepoint"); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
551 |
cur_state->restart(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
552 |
assert(cur_state->is_running(), "safepoint state has not been reset"); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
553 |
} |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
554 |
} |
1 | 555 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
556 |
RuntimeService::record_safepoint_end(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
557 |
|
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
558 |
// Release threads lock, so threads can be created/destroyed again. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
559 |
// It will also release all threads blocked in signal_thread_blocked. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
560 |
Threads_lock->unlock(); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
561 |
} |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
562 |
} // ThreadsListHandle destroyed here. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47881
diff
changeset
|
563 |
|
47636
52d46d7e3454
8188919: Generalize GC thread suspend/resume at safepoints
eosterlund
parents:
47629
diff
changeset
|
564 |
Universe::heap()->safepoint_synchronize_end(); |
22551 | 565 |
// record this time so VMThread can keep track how much time has elapsed |
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
566 |
// since last safepoint. |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
567 |
_end_of_last_safepoint = os::javaTimeMillis(); |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
568 |
if (event.should_commit()) { |
50113 | 569 |
post_safepoint_end_event(&event); |
37176
663bdc7d0b86
8149023: Event based tracing should cover safepoint begin and end
dsimms
parents:
37161
diff
changeset
|
570 |
} |
1 | 571 |
} |
572 |
||
573 |
bool SafepointSynchronize::is_cleanup_needed() { |
|
46541
d20828de9e39
8181859: Monitor deflation is not checked in cleanup path
rehn
parents:
46496
diff
changeset
|
574 |
// Need a safepoint if there are many monitors to deflate. |
d20828de9e39
8181859: Monitor deflation is not checked in cleanup path
rehn
parents:
46496
diff
changeset
|
575 |
if (ObjectSynchronizer::is_cleanup_needed()) return true; |
1 | 576 |
// Need a safepoint if some inline cache buffers is non-empty |
577 |
if (!InlineCacheBuffer::is_empty()) return true; |
|
578 |
return false; |
|
579 |
} |
|
580 |
||
46702 | 581 |
class ParallelSPCleanupThreadClosure : public ThreadClosure { |
582 |
private: |
|
583 |
CodeBlobClosure* _nmethod_cl; |
|
584 |
DeflateMonitorCounters* _counters; |
|
585 |
||
586 |
public: |
|
587 |
ParallelSPCleanupThreadClosure(DeflateMonitorCounters* counters) : |
|
51865
eb954a4b6083
8132849: Increased stop time in cleanup phase because of single-threaded walk of thread stacks in NMethodSweeper::mark_active_nmethods()
rkennke
parents:
51598
diff
changeset
|
588 |
_nmethod_cl(UseCodeAging ? NMethodSweeper::prepare_reset_hotness_counters() : NULL), |
eb954a4b6083
8132849: Increased stop time in cleanup phase because of single-threaded walk of thread stacks in NMethodSweeper::mark_active_nmethods()
rkennke
parents:
51598
diff
changeset
|
589 |
_counters(counters) {} |
46702 | 590 |
|
591 |
void do_thread(Thread* thread) { |
|
592 |
ObjectSynchronizer::deflate_thread_local_monitors(thread, _counters); |
|
593 |
if (_nmethod_cl != NULL && thread->is_Java_thread() && |
|
594 |
! thread->is_Code_cache_sweeper_thread()) { |
|
595 |
JavaThread* jt = (JavaThread*) thread; |
|
596 |
jt->nmethods_do(_nmethod_cl); |
|
597 |
} |
|
1 | 598 |
} |
46702 | 599 |
}; |
600 |
||
601 |
class ParallelSPCleanupTask : public AbstractGangTask { |
|
602 |
private: |
|
603 |
SubTasksDone _subtasks; |
|
604 |
ParallelSPCleanupThreadClosure _cleanup_threads_cl; |
|
605 |
uint _num_workers; |
|
606 |
DeflateMonitorCounters* _counters; |
|
607 |
public: |
|
608 |
ParallelSPCleanupTask(uint num_workers, DeflateMonitorCounters* counters) : |
|
609 |
AbstractGangTask("Parallel Safepoint Cleanup"), |
|
51334
cc2c79d22508
8208671: Runtime, JFR, Serviceability changes to allow enabling -Wreorder
tschatzl
parents:
50966
diff
changeset
|
610 |
_subtasks(SubTasksDone(SafepointSynchronize::SAFEPOINT_CLEANUP_NUM_TASKS)), |
46702 | 611 |
_cleanup_threads_cl(ParallelSPCleanupThreadClosure(counters)), |
612 |
_num_workers(num_workers), |
|
613 |
_counters(counters) {} |
|
614 |
||
615 |
void work(uint worker_id) { |
|
616 |
// All threads deflate monitors and mark nmethods (if necessary). |
|
47637
87141990dde5
8185580: Refactor Threads::possibly_parallel_oops_do() to use Threads::parallel_java_threads_do()
rkennke
parents:
47636
diff
changeset
|
617 |
Threads::possibly_parallel_threads_do(true, &_cleanup_threads_cl); |
46702 | 618 |
|
51598 | 619 |
if (_subtasks.try_claim_task(SafepointSynchronize::SAFEPOINT_CLEANUP_DEFLATE_MONITORS)) { |
52703
e7fdc9d9c376
8202415: Incorrect time logged for monitor deflation
dcubed
parents:
52672
diff
changeset
|
620 |
const char* name = "deflating global idle monitors"; |
46702 | 621 |
EventSafepointCleanupTask event; |
622 |
TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup)); |
|
623 |
ObjectSynchronizer::deflate_idle_monitors(_counters); |
|
50113 | 624 |
if (event.should_commit()) { |
625 |
post_safepoint_cleanup_task_event(&event, name); |
|
626 |
} |
|
46702 | 627 |
} |
628 |
||
51598 | 629 |
if (_subtasks.try_claim_task(SafepointSynchronize::SAFEPOINT_CLEANUP_UPDATE_INLINE_CACHES)) { |
46702 | 630 |
const char* name = "updating inline caches"; |
631 |
EventSafepointCleanupTask event; |
|
632 |
TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup)); |
|
633 |
InlineCacheBuffer::update_inline_caches(); |
|
50113 | 634 |
if (event.should_commit()) { |
635 |
post_safepoint_cleanup_task_event(&event, name); |
|
636 |
} |
|
46702 | 637 |
} |
1 | 638 |
|
51598 | 639 |
if (_subtasks.try_claim_task(SafepointSynchronize::SAFEPOINT_CLEANUP_COMPILATION_POLICY)) { |
46702 | 640 |
const char* name = "compilation policy safepoint handler"; |
641 |
EventSafepointCleanupTask event; |
|
642 |
TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup)); |
|
643 |
CompilationPolicy::policy()->do_safepoint_work(); |
|
50113 | 644 |
if (event.should_commit()) { |
645 |
post_safepoint_cleanup_task_event(&event, name); |
|
646 |
} |
|
46702 | 647 |
} |
648 |
||
51598 | 649 |
if (_subtasks.try_claim_task(SafepointSynchronize::SAFEPOINT_CLEANUP_SYMBOL_TABLE_REHASH)) { |
46702 | 650 |
if (SymbolTable::needs_rehashing()) { |
651 |
const char* name = "rehashing symbol table"; |
|
652 |
EventSafepointCleanupTask event; |
|
653 |
TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup)); |
|
654 |
SymbolTable::rehash_table(); |
|
50113 | 655 |
if (event.should_commit()) { |
656 |
post_safepoint_cleanup_task_event(&event, name); |
|
657 |
} |
|
46702 | 658 |
} |
659 |
} |
|
660 |
||
51598 | 661 |
if (_subtasks.try_claim_task(SafepointSynchronize::SAFEPOINT_CLEANUP_STRING_TABLE_REHASH)) { |
46702 | 662 |
if (StringTable::needs_rehashing()) { |
663 |
const char* name = "rehashing string table"; |
|
664 |
EventSafepointCleanupTask event; |
|
665 |
TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup)); |
|
666 |
StringTable::rehash_table(); |
|
50113 | 667 |
if (event.should_commit()) { |
668 |
post_safepoint_cleanup_task_event(&event, name); |
|
669 |
} |
|
46702 | 670 |
} |
671 |
} |
|
672 |
||
51598 | 673 |
if (_subtasks.try_claim_task(SafepointSynchronize::SAFEPOINT_CLEANUP_CLD_PURGE)) { |
46702 | 674 |
// CMS delays purging the CLDG until the beginning of the next safepoint and to |
675 |
// make sure concurrent sweep is done |
|
676 |
const char* name = "purging class loader data graph"; |
|
677 |
EventSafepointCleanupTask event; |
|
678 |
TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup)); |
|
679 |
ClassLoaderDataGraph::purge_if_needed(); |
|
50113 | 680 |
if (event.should_commit()) { |
681 |
post_safepoint_cleanup_task_event(&event, name); |
|
682 |
} |
|
46702 | 683 |
} |
47774 | 684 |
|
51598 | 685 |
if (_subtasks.try_claim_task(SafepointSynchronize::SAFEPOINT_CLEANUP_SYSTEM_DICTIONARY_RESIZE)) { |
47774 | 686 |
const char* name = "resizing system dictionaries"; |
687 |
EventSafepointCleanupTask event; |
|
688 |
TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup)); |
|
689 |
ClassLoaderDataGraph::resize_if_needed(); |
|
50113 | 690 |
if (event.should_commit()) { |
691 |
post_safepoint_cleanup_task_event(&event, name); |
|
692 |
} |
|
47774 | 693 |
} |
51338
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
694 |
|
46702 | 695 |
_subtasks.all_tasks_completed(_num_workers); |
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
696 |
} |
46702 | 697 |
}; |
698 |
||
699 |
// Various cleaning tasks that should be done periodically at safepoints. |
|
700 |
void SafepointSynchronize::do_cleanup_tasks() { |
|
701 |
||
702 |
TraceTime timer("safepoint cleanup tasks", TRACETIME_LOG(Info, safepoint, cleanup)); |
|
703 |
||
704 |
// Prepare for monitor deflation. |
|
705 |
DeflateMonitorCounters deflate_counters; |
|
706 |
ObjectSynchronizer::prepare_deflate_idle_monitors(&deflate_counters); |
|
707 |
||
708 |
CollectedHeap* heap = Universe::heap(); |
|
709 |
assert(heap != NULL, "heap not initialized yet?"); |
|
710 |
WorkGang* cleanup_workers = heap->get_safepoint_workers(); |
|
711 |
if (cleanup_workers != NULL) { |
|
712 |
// Parallel cleanup using GC provided thread pool. |
|
713 |
uint num_cleanup_workers = cleanup_workers->active_workers(); |
|
714 |
ParallelSPCleanupTask cleanup(num_cleanup_workers, &deflate_counters); |
|
715 |
StrongRootsScope srs(num_cleanup_workers); |
|
716 |
cleanup_workers->run_task(&cleanup); |
|
717 |
} else { |
|
718 |
// Serial cleanup using VMThread. |
|
719 |
ParallelSPCleanupTask cleanup(1, &deflate_counters); |
|
720 |
StrongRootsScope srs(1); |
|
721 |
cleanup.work(0); |
|
1 | 722 |
} |
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
723 |
|
51338
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
724 |
// Needs to be done single threaded by the VMThread. This walks |
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
725 |
// the thread stacks looking for references to metadata before |
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
726 |
// deciding to remove it from the metaspaces. |
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
727 |
if (ClassLoaderDataGraph::should_clean_metaspaces_and_reset()) { |
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
728 |
const char* name = "cleanup live ClassLoaderData metaspaces"; |
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
729 |
TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup)); |
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
730 |
ClassLoaderDataGraph::walk_metadata_and_clean_metaspaces(); |
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
731 |
} |
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
732 |
|
46702 | 733 |
// Finish monitor deflation. |
734 |
ObjectSynchronizer::finish_deflate_idle_monitors(&deflate_counters); |
|
51338
aa3bfacc912c
8208677: Move inner metaspace cleaning out of class unloading
coleenp
parents:
51334
diff
changeset
|
735 |
|
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52703
diff
changeset
|
736 |
assert(InlineCacheBuffer::is_empty(), "should have cleaned up ICBuffer"); |
1 | 737 |
} |
738 |
||
739 |
||
740 |
bool SafepointSynchronize::safepoint_safe(JavaThread *thread, JavaThreadState state) { |
|
741 |
switch(state) { |
|
742 |
case _thread_in_native: |
|
743 |
// native threads are safe if they have no java stack or have walkable stack |
|
744 |
return !thread->has_last_Java_frame() || thread->frame_anchor()->walkable(); |
|
745 |
||
746 |
// blocked threads should have already have walkable stack |
|
747 |
case _thread_blocked: |
|
748 |
assert(!thread->has_last_Java_frame() || thread->frame_anchor()->walkable(), "blocked and not walkable"); |
|
749 |
return true; |
|
750 |
||
751 |
default: |
|
752 |
return false; |
|
753 |
} |
|
754 |
} |
|
755 |
||
756 |
||
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
757 |
// See if the thread is running inside a lazy critical native and |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
758 |
// update the thread critical count if so. Also set a suspend flag to |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
759 |
// cause the native wrapper to return into the JVM to do the unlock |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
760 |
// once the native finishes. |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
761 |
void SafepointSynchronize::check_for_lazy_critical_native(JavaThread *thread, JavaThreadState state) { |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
762 |
if (state == _thread_in_native && |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
763 |
thread->has_last_Java_frame() && |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
764 |
thread->frame_anchor()->walkable()) { |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
765 |
// This thread might be in a critical native nmethod so look at |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
766 |
// the top of the stack and increment the critical count if it |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
767 |
// is. |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
768 |
frame wrapper_frame = thread->last_frame(); |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
769 |
CodeBlob* stub_cb = wrapper_frame.cb(); |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
770 |
if (stub_cb != NULL && |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
771 |
stub_cb->is_nmethod() && |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
772 |
stub_cb->as_nmethod_or_null()->is_lazy_critical_native()) { |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
773 |
// A thread could potentially be in a critical native across |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
774 |
// more than one safepoint, so only update the critical state on |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
775 |
// the first one. When it returns it will perform the unlock. |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
776 |
if (!thread->do_critical_native_unlock()) { |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
777 |
#ifdef ASSERT |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
778 |
if (!thread->in_critical()) { |
35492
c8c0273e6b91
8146690: Make all classes in GC follow the naming convention.
david
parents:
35061
diff
changeset
|
779 |
GCLocker::increment_debug_jni_lock_count(); |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
780 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
781 |
#endif |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
782 |
thread->enter_critical(); |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
783 |
// Make sure the native wrapper calls back on return to |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
784 |
// perform the needed critical unlock. |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
785 |
thread->set_critical_native_unlock(); |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
786 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
787 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
788 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
789 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
790 |
|
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
791 |
|
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
11631
diff
changeset
|
792 |
|
1 | 793 |
// ------------------------------------------------------------------------------------------------------- |
794 |
// Implementation of Safepoint callback point |
|
795 |
||
53646
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
796 |
void SafepointSynchronize::block(JavaThread *thread, bool block_in_safepoint_check) { |
1 | 797 |
assert(thread != NULL, "thread must be set"); |
798 |
assert(thread->is_Java_thread(), "not a Java thread"); |
|
799 |
||
800 |
// Threads shouldn't block if they are in the middle of printing, but... |
|
801 |
ttyLocker::break_tty_lock_for_safepoint(os::current_thread_id()); |
|
802 |
||
803 |
// Only bail from the block() call if the thread is gone from the |
|
804 |
// thread list; starting to exit should still block. |
|
805 |
if (thread->is_terminated()) { |
|
806 |
// block current thread if we come here from native code when VM is gone |
|
807 |
thread->block_if_vm_exited(); |
|
808 |
||
809 |
// otherwise do nothing |
|
810 |
return; |
|
811 |
} |
|
812 |
||
813 |
JavaThreadState state = thread->thread_state(); |
|
814 |
thread->frame_anchor()->make_walkable(thread); |
|
815 |
||
816 |
// Check that we have a valid thread_state at this point |
|
817 |
switch(state) { |
|
818 |
case _thread_in_vm_trans: |
|
819 |
case _thread_in_Java: // From compiled code |
|
820 |
||
821 |
// We are highly likely to block on the Safepoint_lock. In order to avoid blocking in this case, |
|
822 |
// we pretend we are still in the VM. |
|
823 |
thread->set_thread_state(_thread_in_vm); |
|
824 |
||
825 |
if (is_synchronizing()) { |
|
826 |
Atomic::inc (&TryingToBlock) ; |
|
827 |
} |
|
828 |
||
829 |
// We will always be holding the Safepoint_lock when we are examine the state |
|
830 |
// of a thread. Hence, the instructions between the Safepoint_lock->lock() and |
|
831 |
// Safepoint_lock->unlock() are happening atomic with regards to the safepoint code |
|
832 |
Safepoint_lock->lock_without_safepoint_check(); |
|
833 |
if (is_synchronizing()) { |
|
834 |
// Decrement the number of threads to wait for and signal vm thread |
|
835 |
assert(_waiting_to_block > 0, "sanity check"); |
|
836 |
_waiting_to_block--; |
|
837 |
thread->safepoint_state()->set_has_called_back(true); |
|
838 |
||
12108
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
839 |
DEBUG_ONLY(thread->set_visited_for_critical_count(true)); |
11631
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
840 |
if (thread->in_critical()) { |
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
841 |
// Notice that this thread is in a critical section |
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
842 |
increment_jni_active_count(); |
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
843 |
} |
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
844 |
|
1 | 845 |
// Consider (_waiting_to_block < 2) to pipeline the wakeup of the VM thread |
846 |
if (_waiting_to_block == 0) { |
|
847 |
Safepoint_lock->notify_all(); |
|
848 |
} |
|
849 |
} |
|
850 |
||
53646
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
851 |
if (block_in_safepoint_check) { |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
852 |
// We transition the thread to state _thread_blocked here, but |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
853 |
// we can't do our usual check for external suspension and then |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
854 |
// self-suspend after the lock_without_safepoint_check() call |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
855 |
// below because we are often called during transitions while |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
856 |
// we hold different locks. That would leave us suspended while |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
857 |
// holding a resource which results in deadlocks. |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
858 |
thread->set_thread_state(_thread_blocked); |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
859 |
Safepoint_lock->unlock(); |
1 | 860 |
|
53646
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
861 |
// We now try to acquire the threads lock. Since this lock is hold by the VM thread during |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
862 |
// the entire safepoint, the threads will all line up here during the safepoint. |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
863 |
Threads_lock->lock_without_safepoint_check(); |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
864 |
// restore original state. This is important if the thread comes from compiled code, so it |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
865 |
// will continue to execute with the _thread_in_Java state. |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
866 |
thread->set_thread_state(state); |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
867 |
Threads_lock->unlock(); |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
868 |
} else { |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
869 |
// We choose not to block in this call since we would be |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
870 |
// caught when transitioning back anyways if the safepoint |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
871 |
// is still going on. |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
872 |
thread->set_thread_state(state); |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
873 |
Safepoint_lock->unlock(); |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
874 |
} |
1 | 875 |
break; |
876 |
||
877 |
case _thread_in_native_trans: |
|
878 |
case _thread_blocked_trans: |
|
879 |
case _thread_new_trans: |
|
53646
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
880 |
if (thread->safepoint_state()->type() == ThreadSafepointState::_call_back && |
043ae846819f
8210832: Remove sneaky locking in class Monitor
pchilanomate
parents:
53582
diff
changeset
|
881 |
block_in_safepoint_check) { |
1 | 882 |
thread->print_thread_state(); |
883 |
fatal("Deadlock in safepoint code. " |
|
884 |
"Should have called back to the VM before blocking."); |
|
885 |
} |
|
886 |
||
887 |
// We transition the thread to state _thread_blocked here, but |
|
888 |
// we can't do our usual check for external suspension and then |
|
889 |
// self-suspend after the lock_without_safepoint_check() call |
|
890 |
// below because we are often called during transitions while |
|
891 |
// we hold different locks. That would leave us suspended while |
|
892 |
// holding a resource which results in deadlocks. |
|
893 |
thread->set_thread_state(_thread_blocked); |
|
894 |
||
895 |
// It is not safe to suspend a thread if we discover it is in _thread_in_native_trans. Hence, |
|
896 |
// the safepoint code might still be waiting for it to block. We need to change the state here, |
|
897 |
// so it can see that it is at a safepoint. |
|
898 |
||
899 |
// Block until the safepoint operation is completed. |
|
900 |
Threads_lock->lock_without_safepoint_check(); |
|
901 |
||
902 |
// Restore state |
|
903 |
thread->set_thread_state(state); |
|
904 |
||
905 |
Threads_lock->unlock(); |
|
906 |
break; |
|
907 |
||
908 |
default: |
|
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
909 |
fatal("Illegal threadstate encountered: %d", state); |
1 | 910 |
} |
911 |
||
912 |
// Check for pending. async. exceptions or suspends - except if the |
|
913 |
// thread was blocked inside the VM. has_special_runtime_exit_condition() |
|
914 |
// is called last since it grabs a lock and we only want to do that when |
|
915 |
// we must. |
|
916 |
// |
|
917 |
// Note: we never deliver an async exception at a polling point as the |
|
918 |
// compiler may not have an exception handler for it. The polling |
|
919 |
// code will notice the async and deoptimize and the exception will |
|
920 |
// be delivered. (Polling at a return point is ok though). Sure is |
|
921 |
// a lot of bother for a deprecated feature... |
|
922 |
// |
|
923 |
// We don't deliver an async exception if the thread state is |
|
924 |
// _thread_in_native_trans so JNI functions won't be called with |
|
925 |
// a surprising pending exception. If the thread state is going back to java, |
|
926 |
// async exception is checked in check_special_condition_for_native_trans(). |
|
927 |
||
928 |
if (state != _thread_blocked_trans && |
|
929 |
state != _thread_in_vm_trans && |
|
930 |
thread->has_special_runtime_exit_condition()) { |
|
931 |
thread->handle_special_runtime_exit_condition( |
|
932 |
!thread->is_at_poll_safepoint() && (state != _thread_in_native_trans)); |
|
933 |
} |
|
934 |
} |
|
935 |
||
936 |
// ------------------------------------------------------------------------------------------------------ |
|
937 |
// Exception handlers |
|
938 |
||
939 |
||
940 |
void SafepointSynchronize::handle_polling_page_exception(JavaThread *thread) { |
|
941 |
assert(thread->is_Java_thread(), "polling reference encountered by VM thread"); |
|
942 |
assert(thread->thread_state() == _thread_in_Java, "should come from Java code"); |
|
47881
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
943 |
if (!ThreadLocalHandshakes) { |
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
944 |
assert(SafepointSynchronize::is_synchronizing(), "polling encountered outside safepoint synchronization"); |
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
945 |
} |
1 | 946 |
|
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
947 |
if (log_is_enabled(Debug, safepoint, stats)) { |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
948 |
Atomic::inc(&_nof_threads_hit_polling_page); |
1 | 949 |
} |
950 |
||
951 |
ThreadSafepointState* state = thread->safepoint_state(); |
|
952 |
||
953 |
state->handle_polling_page_exception(); |
|
954 |
} |
|
955 |
||
956 |
||
957 |
void SafepointSynchronize::print_safepoint_timeout(SafepointTimeoutReason reason) { |
|
958 |
if (!timeout_error_printed) { |
|
959 |
timeout_error_printed = true; |
|
22551 | 960 |
// Print out the thread info which didn't reach the safepoint for debugging |
1 | 961 |
// purposes (useful when there are lots of threads in the debugger). |
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
962 |
LogTarget(Warning, safepoint) lt; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
963 |
if (lt.is_enabled()) { |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
964 |
ResourceMark rm; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
965 |
LogStream ls(lt); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
966 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
967 |
ls.cr(); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
968 |
ls.print_cr("# SafepointSynchronize::begin: Timeout detected:"); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
969 |
if (reason == _spinning_timeout) { |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
970 |
ls.print_cr("# SafepointSynchronize::begin: Timed out while spinning to reach a safepoint."); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
971 |
} else if (reason == _blocking_timeout) { |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
972 |
ls.print_cr("# SafepointSynchronize::begin: Timed out while waiting for threads to stop."); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
973 |
} |
1 | 974 |
|
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
975 |
ls.print_cr("# SafepointSynchronize::begin: Threads which did not reach the safepoint:"); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
976 |
ThreadSafepointState *cur_state; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
977 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *cur_thread = jtiwh.next(); ) { |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
978 |
cur_state = cur_thread->safepoint_state(); |
1 | 979 |
|
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
980 |
if (cur_thread->thread_state() != _thread_blocked && |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
981 |
((reason == _spinning_timeout && cur_state->is_running()) || |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
982 |
(reason == _blocking_timeout && !cur_state->has_called_back()))) { |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
983 |
ls.print("# "); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
984 |
cur_thread->print_on(&ls); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
985 |
ls.cr(); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
986 |
} |
1 | 987 |
} |
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
988 |
ls.print_cr("# SafepointSynchronize::begin: (End of list)"); |
1 | 989 |
} |
990 |
} |
|
991 |
||
52628
639a3e43f5b7
8213992: Rename and make DieOnSafepointTimeout the diagnostic option
shade
parents:
52460
diff
changeset
|
992 |
// To debug the long safepoint, specify both AbortVMOnSafepointTimeout & |
1 | 993 |
// ShowMessageBoxOnError. |
52628
639a3e43f5b7
8213992: Rename and make DieOnSafepointTimeout the diagnostic option
shade
parents:
52460
diff
changeset
|
994 |
if (AbortVMOnSafepointTimeout) { |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
995 |
fatal("Safepoint sync time longer than " INTX_FORMAT "ms detected when executing %s.", |
46496 | 996 |
SafepointTimeoutDelay, VMThread::vm_safepoint_description()); |
1 | 997 |
} |
998 |
} |
|
999 |
||
1000 |
||
1001 |
// ------------------------------------------------------------------------------------------------------- |
|
1002 |
// Implementation of ThreadSafepointState |
|
1003 |
||
1004 |
ThreadSafepointState::ThreadSafepointState(JavaThread *thread) { |
|
1005 |
_thread = thread; |
|
1006 |
_type = _running; |
|
1007 |
_has_called_back = false; |
|
1008 |
_at_poll_safepoint = false; |
|
1009 |
} |
|
1010 |
||
1011 |
void ThreadSafepointState::create(JavaThread *thread) { |
|
1012 |
ThreadSafepointState *state = new ThreadSafepointState(thread); |
|
1013 |
thread->set_safepoint_state(state); |
|
1014 |
} |
|
1015 |
||
1016 |
void ThreadSafepointState::destroy(JavaThread *thread) { |
|
1017 |
if (thread->safepoint_state()) { |
|
1018 |
delete(thread->safepoint_state()); |
|
1019 |
thread->set_safepoint_state(NULL); |
|
1020 |
} |
|
1021 |
} |
|
1022 |
||
1023 |
void ThreadSafepointState::examine_state_of_thread() { |
|
1024 |
assert(is_running(), "better be running or just have hit safepoint poll"); |
|
1025 |
||
1026 |
JavaThreadState state = _thread->thread_state(); |
|
1027 |
||
6269
10e06287c0b0
6975006: assert(check.is_deoptimized_frame()) failed: missed deopt
never
parents:
5547
diff
changeset
|
1028 |
// Save the state at the start of safepoint processing. |
10e06287c0b0
6975006: assert(check.is_deoptimized_frame()) failed: missed deopt
never
parents:
5547
diff
changeset
|
1029 |
_orig_thread_state = state; |
10e06287c0b0
6975006: assert(check.is_deoptimized_frame()) failed: missed deopt
never
parents:
5547
diff
changeset
|
1030 |
|
1 | 1031 |
// Check for a thread that is suspended. Note that thread resume tries |
1032 |
// to grab the Threads_lock which we own here, so a thread cannot be |
|
1033 |
// resumed during safepoint synchronization. |
|
1034 |
||
3826 | 1035 |
// We check to see if this thread is suspended without locking to |
1036 |
// avoid deadlocking with a third thread that is waiting for this |
|
1037 |
// thread to be suspended. The third thread can notice the safepoint |
|
1038 |
// that we're trying to start at the beginning of its SR_lock->wait() |
|
1039 |
// call. If that happens, then the third thread will block on the |
|
1040 |
// safepoint while still holding the underlying SR_lock. We won't be |
|
1041 |
// able to get the SR_lock and we'll deadlock. |
|
1042 |
// |
|
1043 |
// We don't need to grab the SR_lock here for two reasons: |
|
1044 |
// 1) The suspend flags are both volatile and are set with an |
|
1045 |
// Atomic::cmpxchg() call so we should see the suspended |
|
1046 |
// state right away. |
|
1047 |
// 2) We're being called from the safepoint polling loop; if |
|
1048 |
// we don't see the suspended state on this iteration, then |
|
1049 |
// we'll come around again. |
|
1050 |
// |
|
1051 |
bool is_suspended = _thread->is_ext_suspended(); |
|
1 | 1052 |
if (is_suspended) { |
1053 |
roll_forward(_at_safepoint); |
|
1054 |
return; |
|
1055 |
} |
|
1056 |
||
1057 |
// Some JavaThread states have an initial safepoint state of |
|
1058 |
// running, but are actually at a safepoint. We will happily |
|
1059 |
// agree and update the safepoint state here. |
|
1060 |
if (SafepointSynchronize::safepoint_safe(_thread, state)) { |
|
12108
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
1061 |
SafepointSynchronize::check_for_lazy_critical_native(_thread, state); |
11631
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
1062 |
roll_forward(_at_safepoint); |
33813f69207b
7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents:
10565
diff
changeset
|
1063 |
return; |
1 | 1064 |
} |
1065 |
||
1066 |
if (state == _thread_in_vm) { |
|
1067 |
roll_forward(_call_back); |
|
1068 |
return; |
|
1069 |
} |
|
1070 |
||
1071 |
// All other thread states will continue to run until they |
|
1072 |
// transition and self-block in state _blocked |
|
1073 |
// Safepoint polling in compiled code causes the Java threads to do the same. |
|
1074 |
// Note: new threads may require a malloc so they must be allowed to finish |
|
1075 |
||
1076 |
assert(is_running(), "examine_state_of_thread on non-running thread"); |
|
1077 |
return; |
|
1078 |
} |
|
1079 |
||
1080 |
// Returns true is thread could not be rolled forward at present position. |
|
1081 |
void ThreadSafepointState::roll_forward(suspend_type type) { |
|
1082 |
_type = type; |
|
1083 |
||
1084 |
switch(_type) { |
|
1085 |
case _at_safepoint: |
|
1086 |
SafepointSynchronize::signal_thread_at_safepoint(); |
|
12108
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
1087 |
DEBUG_ONLY(_thread->set_visited_for_critical_count(true)); |
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
1088 |
if (_thread->in_critical()) { |
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
1089 |
// Notice that this thread is in a critical section |
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
1090 |
SafepointSynchronize::increment_jni_active_count(); |
b2ee55606008
7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents:
11637
diff
changeset
|
1091 |
} |
1 | 1092 |
break; |
1093 |
||
1094 |
case _call_back: |
|
1095 |
set_has_called_back(false); |
|
1096 |
break; |
|
1097 |
||
1098 |
case _running: |
|
1099 |
default: |
|
1100 |
ShouldNotReachHere(); |
|
1101 |
} |
|
1102 |
} |
|
1103 |
||
1104 |
void ThreadSafepointState::restart() { |
|
1105 |
switch(type()) { |
|
1106 |
case _at_safepoint: |
|
1107 |
case _call_back: |
|
1108 |
break; |
|
1109 |
||
1110 |
case _running: |
|
1111 |
default: |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
30764
diff
changeset
|
1112 |
tty->print_cr("restart thread " INTPTR_FORMAT " with state %d", |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
33105
diff
changeset
|
1113 |
p2i(_thread), _type); |
1 | 1114 |
_thread->print(); |
1115 |
ShouldNotReachHere(); |
|
1116 |
} |
|
1117 |
_type = _running; |
|
1118 |
set_has_called_back(false); |
|
1119 |
} |
|
1120 |
||
1121 |
||
1122 |
void ThreadSafepointState::print_on(outputStream *st) const { |
|
33589
7cbd1b2c139b
8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents:
33222
diff
changeset
|
1123 |
const char *s = NULL; |
1 | 1124 |
|
1125 |
switch(_type) { |
|
1126 |
case _running : s = "_running"; break; |
|
1127 |
case _at_safepoint : s = "_at_safepoint"; break; |
|
1128 |
case _call_back : s = "_call_back"; break; |
|
1129 |
default: |
|
1130 |
ShouldNotReachHere(); |
|
1131 |
} |
|
1132 |
||
1133 |
st->print_cr("Thread: " INTPTR_FORMAT |
|
1134 |
" [0x%2x] State: %s _has_called_back %d _at_poll_safepoint %d", |
|
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
33105
diff
changeset
|
1135 |
p2i(_thread), _thread->osthread()->thread_id(), s, _has_called_back, |
1 | 1136 |
_at_poll_safepoint); |
1137 |
||
1138 |
_thread->print_thread_state_on(st); |
|
1139 |
} |
|
1140 |
||
1141 |
// --------------------------------------------------------------------------------------------------------------------- |
|
1142 |
||
47881
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
1143 |
// Block the thread at poll or poll return for safepoint/handshake. |
1 | 1144 |
void ThreadSafepointState::handle_polling_page_exception() { |
1145 |
||
1146 |
// Check state. block() will set thread state to thread_in_vm which will |
|
1147 |
// cause the safepoint state _type to become _call_back. |
|
47881
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
1148 |
suspend_type t = type(); |
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
1149 |
assert(!SafepointMechanism::uses_global_page_poll() || t == ThreadSafepointState::_running, |
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
1150 |
"polling page exception on thread not running state: %u", uint(t)); |
1 | 1151 |
|
1152 |
// Step 1: Find the nmethod from the return address |
|
1153 |
address real_return_addr = thread()->saved_exception_pc(); |
|
1154 |
||
1155 |
CodeBlob *cb = CodeCache::find_blob(real_return_addr); |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
37242
diff
changeset
|
1156 |
assert(cb != NULL && cb->is_compiled(), "return address should be in nmethod"); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
37242
diff
changeset
|
1157 |
CompiledMethod* nm = (CompiledMethod*)cb; |
1 | 1158 |
|
1159 |
// Find frame of caller |
|
1160 |
frame stub_fr = thread()->last_frame(); |
|
1161 |
CodeBlob* stub_cb = stub_fr.cb(); |
|
1162 |
assert(stub_cb->is_safepoint_stub(), "must be a safepoint stub"); |
|
1163 |
RegisterMap map(thread(), true); |
|
1164 |
frame caller_fr = stub_fr.sender(&map); |
|
1165 |
||
1166 |
// Should only be poll_return or poll |
|
1167 |
assert( nm->is_at_poll_or_poll_return(real_return_addr), "should not be at call" ); |
|
1168 |
||
1169 |
// This is a poll immediately before a return. The exception handling code |
|
1170 |
// has already had the effect of causing the return to occur, so the execution |
|
1171 |
// will continue immediately after the call. In addition, the oopmap at the |
|
1172 |
// return point does not mark the return value as an oop (if it is), so |
|
1173 |
// it needs a handle here to be updated. |
|
1174 |
if( nm->is_at_poll_return(real_return_addr) ) { |
|
1175 |
// See if return type is an oop. |
|
1176 |
bool return_oop = nm->method()->is_returning_oop(); |
|
1177 |
Handle return_value; |
|
1178 |
if (return_oop) { |
|
1179 |
// The oop result has been saved on the stack together with all |
|
1180 |
// the other registers. In order to preserve it over GCs we need |
|
1181 |
// to keep it in a handle. |
|
1182 |
oop result = caller_fr.saved_oop_result(&map); |
|
46968
9119841280f4
8160399: is_oop_or_null involves undefined behavior
coleenp
parents:
46702
diff
changeset
|
1183 |
assert(oopDesc::is_oop_or_null(result), "must be oop"); |
1 | 1184 |
return_value = Handle(thread(), result); |
1185 |
assert(Universe::heap()->is_in_or_null(result), "must be heap pointer"); |
|
1186 |
} |
|
1187 |
||
1188 |
// Block the thread |
|
47881
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
1189 |
SafepointMechanism::block_if_requested(thread()); |
1 | 1190 |
|
1191 |
// restore oop result, if any |
|
1192 |
if (return_oop) { |
|
1193 |
caller_fr.set_saved_oop_result(&map, return_value()); |
|
1194 |
} |
|
1195 |
} |
|
1196 |
||
1197 |
// This is a safepoint poll. Verify the return address and block. |
|
1198 |
else { |
|
1199 |
set_at_poll_safepoint(true); |
|
1200 |
||
1201 |
// verify the blob built the "return address" correctly |
|
1202 |
assert(real_return_addr == caller_fr.pc(), "must match"); |
|
1203 |
||
1204 |
// Block the thread |
|
47881
0ce0ac68ace7
8189941: Implementation JEP 312: Thread-local handshake
rehn
parents:
47774
diff
changeset
|
1205 |
SafepointMechanism::block_if_requested(thread()); |
1 | 1206 |
set_at_poll_safepoint(false); |
1207 |
||
1208 |
// If we have a pending async exception deoptimize the frame |
|
1209 |
// as otherwise we may never deliver it. |
|
1210 |
if (thread()->has_async_condition()) { |
|
1211 |
ThreadInVMfromJavaNoAsyncException __tiv(thread()); |
|
7106
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
6453
diff
changeset
|
1212 |
Deoptimization::deoptimize_frame(thread(), caller_fr.id()); |
1 | 1213 |
} |
1214 |
||
1215 |
// If an exception has been installed we must check for a pending deoptimization |
|
1216 |
// Deoptimize frame if exception has been thrown. |
|
1217 |
||
1218 |
if (thread()->has_pending_exception() ) { |
|
1219 |
RegisterMap map(thread(), true); |
|
1220 |
frame caller_fr = stub_fr.sender(&map); |
|
1221 |
if (caller_fr.is_deoptimized_frame()) { |
|
1222 |
// The exception patch will destroy registers that are still |
|
1223 |
// live and will be needed during deoptimization. Defer the |
|
22551 | 1224 |
// Async exception should have deferred the exception until the |
1 | 1225 |
// next safepoint which will be detected when we get into |
1226 |
// the interpreter so if we have an exception now things |
|
1227 |
// are messed up. |
|
1228 |
||
1229 |
fatal("Exception installed and deoptimization is pending"); |
|
1230 |
} |
|
1231 |
} |
|
1232 |
} |
|
1233 |
} |
|
1234 |
||
1235 |
||
1236 |
// |
|
1237 |
// Statistics & Instrumentations |
|
1238 |
// |
|
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1239 |
struct SafepointStats { |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1240 |
float _time_stamp; // record when the current safepoint occurs in seconds |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1241 |
int _vmop_type; // tyep of VM operation triggers the safepoint |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1242 |
int _nof_total_threads; // total number of Java threads |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1243 |
int _nof_initial_running_threads; // total number of initially seen running threads |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1244 |
int _nof_threads_wait_to_block; // total number of threads waiting for to block |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1245 |
bool _page_armed; // true if polling page is armed, false otherwise |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1246 |
int _nof_threads_hit_page_trap; // total number of threads hitting the page trap |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1247 |
jlong _time_to_spin; // total time in millis spent in spinning |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1248 |
jlong _time_to_wait_to_block; // total time in millis spent in waiting for to block |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1249 |
jlong _time_to_do_cleanups; // total time in millis spent in performing cleanups |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1250 |
jlong _time_to_sync; // total time in millis spent in getting to _synchronized |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1251 |
jlong _time_to_exec_vmop; // total time in millis spent in vm operation itself |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1252 |
}; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1253 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1254 |
static const int _statistics_header_count = 30; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1255 |
static int _cur_stat_index = 0; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1256 |
static SafepointStats safepoint_stats = {0}; // zero initialize |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1257 |
static SafepointStats* spstat = &safepoint_stats; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1258 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1259 |
static julong _safepoint_reasons[VM_Operation::VMOp_Terminating]; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1260 |
static jlong _max_sync_time = 0; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1261 |
static jlong _max_vmop_time = 0; |
1 | 1262 |
|
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1263 |
static jlong cleanup_end_time = 0; |
1 | 1264 |
|
1265 |
void SafepointSynchronize::begin_statistics(int nof_threads, int nof_running) { |
|
1266 |
||
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1267 |
spstat->_time_stamp = _ts_of_current_safepoint; |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1268 |
|
1 | 1269 |
VM_Operation *op = VMThread::vm_operation(); |
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1270 |
spstat->_vmop_type = op != NULL ? op->type() : VM_Operation::VMOp_None; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1271 |
_safepoint_reasons[spstat->_vmop_type]++; |
1 | 1272 |
|
1273 |
spstat->_nof_total_threads = nof_threads; |
|
1274 |
spstat->_nof_initial_running_threads = nof_running; |
|
1275 |
||
1276 |
// Records the start time of spinning. The real time spent on spinning |
|
1277 |
// will be adjusted when spin is done. Same trick is applied for time |
|
1278 |
// spent on waiting for threads to block. |
|
1279 |
if (nof_running != 0) { |
|
1280 |
spstat->_time_to_spin = os::javaTimeNanos(); |
|
1281 |
} else { |
|
1282 |
spstat->_time_to_spin = 0; |
|
1283 |
} |
|
1284 |
} |
|
1285 |
||
1286 |
void SafepointSynchronize::update_statistics_on_spin_end() { |
|
1287 |
jlong cur_time = os::javaTimeNanos(); |
|
1288 |
||
1289 |
spstat->_nof_threads_wait_to_block = _waiting_to_block; |
|
1290 |
if (spstat->_nof_initial_running_threads != 0) { |
|
1291 |
spstat->_time_to_spin = cur_time - spstat->_time_to_spin; |
|
1292 |
} |
|
1293 |
||
1294 |
// Records the start time of waiting for to block. Updated when block is done. |
|
1295 |
if (_waiting_to_block != 0) { |
|
1296 |
spstat->_time_to_wait_to_block = cur_time; |
|
1297 |
} else { |
|
1298 |
spstat->_time_to_wait_to_block = 0; |
|
1299 |
} |
|
1300 |
} |
|
1301 |
||
1302 |
void SafepointSynchronize::update_statistics_on_sync_end(jlong end_time) { |
|
1303 |
||
1304 |
if (spstat->_nof_threads_wait_to_block != 0) { |
|
1305 |
spstat->_time_to_wait_to_block = end_time - |
|
1306 |
spstat->_time_to_wait_to_block; |
|
1307 |
} |
|
1308 |
||
1309 |
// Records the end time of sync which will be used to calculate the total |
|
1310 |
// vm operation time. Again, the real time spending in syncing will be deducted |
|
1311 |
// from the start of the sync time later when end_statistics is called. |
|
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1312 |
spstat->_time_to_sync = end_time - _safepoint_begin_time; |
1 | 1313 |
if (spstat->_time_to_sync > _max_sync_time) { |
1314 |
_max_sync_time = spstat->_time_to_sync; |
|
1315 |
} |
|
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1316 |
|
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1317 |
spstat->_time_to_do_cleanups = end_time; |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1318 |
} |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1319 |
|
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1320 |
void SafepointSynchronize::update_statistics_on_cleanup_end(jlong end_time) { |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1321 |
|
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1322 |
// Record how long spent in cleanup tasks. |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1323 |
spstat->_time_to_do_cleanups = end_time - spstat->_time_to_do_cleanups; |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1324 |
cleanup_end_time = end_time; |
1 | 1325 |
} |
1326 |
||
1327 |
void SafepointSynchronize::end_statistics(jlong vmop_end_time) { |
|
1328 |
||
1329 |
// Update the vm operation time. |
|
5042
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1330 |
spstat->_time_to_exec_vmop = vmop_end_time - cleanup_end_time; |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1331 |
if (spstat->_time_to_exec_vmop > _max_vmop_time) { |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1332 |
_max_vmop_time = spstat->_time_to_exec_vmop; |
f86707fd195a
6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents:
4006
diff
changeset
|
1333 |
} |
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1334 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1335 |
spstat->_nof_threads_hit_page_trap = _nof_threads_hit_polling_page; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1336 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1337 |
print_statistics(); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1338 |
} |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1339 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1340 |
// Helper method to print the header. |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1341 |
static void print_header(outputStream* st) { |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1342 |
// The number of spaces is significant here, and should match the format |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1343 |
// specifiers in print_statistics(). |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1344 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1345 |
st->print(" vmop " |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1346 |
"[ threads: total initially_running wait_to_block ]" |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1347 |
"[ time: spin block sync cleanup vmop ] "); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1348 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1349 |
st->print_cr("page_trap_count"); |
1 | 1350 |
} |
1351 |
||
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1352 |
// This prints a nice table. To get the statistics to not shift due to the logging uptime |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1353 |
// decorator, use the option as: -Xlog:safepoint+stats=debug:[outputfile]:none |
1 | 1354 |
void SafepointSynchronize::print_statistics() { |
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1355 |
LogTarget(Debug, safepoint, stats) lt; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1356 |
assert (lt.is_enabled(), "should only be called when printing statistics is enabled"); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1357 |
LogStream ls(lt); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1358 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1359 |
// Print header every 30 entries |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1360 |
if ((_cur_stat_index % _statistics_header_count) == 0) { |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1361 |
print_header(&ls); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1362 |
_cur_stat_index = 1; // wrap |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1363 |
} else { |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1364 |
_cur_stat_index++; |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1365 |
} |
1 | 1366 |
|
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1367 |
ls.print("%8.3f: ", spstat->_time_stamp); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1368 |
ls.print("%-28s [ " |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1369 |
INT32_FORMAT_W(8) " " INT32_FORMAT_W(17) " " INT32_FORMAT_W(13) " " |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1370 |
"]", |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1371 |
VM_Operation::name(spstat->_vmop_type), |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1372 |
spstat->_nof_total_threads, |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1373 |
spstat->_nof_initial_running_threads, |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1374 |
spstat->_nof_threads_wait_to_block); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1375 |
// "/ MICROUNITS " is to convert the unit from nanos to millis. |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1376 |
ls.print("[ " |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1377 |
INT64_FORMAT_W(7) " " INT64_FORMAT_W(7) " " |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1378 |
INT64_FORMAT_W(7) " " INT64_FORMAT_W(7) " " |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1379 |
INT64_FORMAT_W(7) " ] ", |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1380 |
(int64_t)(spstat->_time_to_spin / MICROUNITS), |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1381 |
(int64_t)(spstat->_time_to_wait_to_block / MICROUNITS), |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1382 |
(int64_t)(spstat->_time_to_sync / MICROUNITS), |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1383 |
(int64_t)(spstat->_time_to_do_cleanups / MICROUNITS), |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1384 |
(int64_t)(spstat->_time_to_exec_vmop / MICROUNITS)); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1385 |
|
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1386 |
ls.print_cr(INT32_FORMAT_W(15) " ", spstat->_nof_threads_hit_page_trap); |
1 | 1387 |
} |
1388 |
||
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1389 |
// This method will be called when VM exits. This tries to summarize the sampling. |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1390 |
// Current thread may already be deleted, so don't use ResourceMark. |
1 | 1391 |
void SafepointSynchronize::print_stat_on_exit() { |
1392 |
||
1393 |
for (int index = 0; index < VM_Operation::VMOp_Terminating; index++) { |
|
1394 |
if (_safepoint_reasons[index] != 0) { |
|
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1395 |
log_debug(safepoint, stats)("%-28s" UINT64_FORMAT_W(10), VM_Operation::name(index), |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1396 |
_safepoint_reasons[index]); |
1 | 1397 |
} |
1398 |
} |
|
1399 |
||
50966
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1400 |
log_debug(safepoint, stats)("VM operations coalesced during safepoint " INT64_FORMAT, |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1401 |
_coalesced_vmop_count); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1402 |
log_debug(safepoint, stats)("Maximum sync time " INT64_FORMAT" ms", |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1403 |
(int64_t)(_max_sync_time / MICROUNITS)); |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1404 |
log_debug(safepoint, stats)("Maximum vm operation time (except for Exit VM operation) " |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1405 |
INT64_FORMAT " ms", |
f939a67fea30
8198720: Obsolete PrintSafepointStatistics, PrintSafepointStatisticsTimeout and PrintSafepointStatisticsCount options
coleenp
parents:
50921
diff
changeset
|
1406 |
(int64_t)(_max_vmop_time / MICROUNITS)); |
1 | 1407 |
} |