author | stefank |
Thu, 09 May 2019 14:28:30 +0200 | |
changeset 54786 | ebf733a324d4 |
parent 54623 | 1126f0607c70 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52876
diff
changeset
|
2 |
* Copyright (c) 2005, 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:
5433
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5433
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:
5433
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
26 |
#include "gc/cms/cmsHeap.hpp" |
52876
2d17750d41e7
8214791: Consistently name gc files containing VM operations
tschatzl
parents:
49594
diff
changeset
|
27 |
#include "gc/cms/cmsVMOperations.hpp" |
30764 | 28 |
#include "gc/cms/concurrentMarkSweepGeneration.inline.hpp" |
29 |
#include "gc/cms/concurrentMarkSweepThread.hpp" |
|
49594
898ef81cbc0e
8200106: Move NoSafepointVerifier out from gcLocker.hpp
stefank
parents:
49449
diff
changeset
|
30 |
#include "gc/shared/gcLocker.hpp" |
30764 | 31 |
#include "gc/shared/gcTimer.hpp" |
35061 | 32 |
#include "gc/shared/gcTraceTime.inline.hpp" |
30764 | 33 |
#include "gc/shared/isGCActiveMark.hpp" |
54786 | 34 |
#include "memory/universe.hpp" |
49449
ef5d5d343e2a
8199263: Split interfaceSupport.hpp to not require including .inline.hpp files
coleenp
parents:
47622
diff
changeset
|
35 |
#include "runtime/handles.inline.hpp" |
ef5d5d343e2a
8199263: Split interfaceSupport.hpp to not require including .inline.hpp files
coleenp
parents:
47622
diff
changeset
|
36 |
#include "runtime/interfaceSupport.inline.hpp" |
18025 | 37 |
#include "runtime/os.hpp" |
7397 | 38 |
#include "utilities/dtrace.hpp" |
10739 | 39 |
|
1 | 40 |
////////////////////////////////////////////////////////// |
41 |
// Methods in abstract class VM_CMS_Operation |
|
42 |
////////////////////////////////////////////////////////// |
|
43 |
void VM_CMS_Operation::verify_before_gc() { |
|
44 |
if (VerifyBeforeGC && |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
45 |
CMSHeap::heap()->total_collections() >= VerifyGCStartAt) { |
37146 | 46 |
GCTraceTime(Info, gc, phases, verify) tm("Verify Before", _collector->_gc_timer_cm); |
1 | 47 |
HandleMark hm; |
48 |
FreelistLocker x(_collector); |
|
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52876
diff
changeset
|
49 |
MutexLocker y(_collector->bitMapLock(), Mutex::_no_safepoint_check_flag); |
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
50 |
CMSHeap::heap()->prepare_for_verify(); |
14076
84643cfaeaa8
8000831: Heap verification output incorrect/incomplete
johnc
parents:
12630
diff
changeset
|
51 |
Universe::verify(); |
1 | 52 |
} |
53 |
} |
|
54 |
||
55 |
void VM_CMS_Operation::verify_after_gc() { |
|
56 |
if (VerifyAfterGC && |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
57 |
CMSHeap::heap()->total_collections() >= VerifyGCStartAt) { |
37146 | 58 |
GCTraceTime(Info, gc, phases, verify) tm("Verify After", _collector->_gc_timer_cm); |
1 | 59 |
HandleMark hm; |
60 |
FreelistLocker x(_collector); |
|
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52876
diff
changeset
|
61 |
MutexLocker y(_collector->bitMapLock(), Mutex::_no_safepoint_check_flag); |
14076
84643cfaeaa8
8000831: Heap verification output incorrect/incomplete
johnc
parents:
12630
diff
changeset
|
62 |
Universe::verify(); |
1 | 63 |
} |
64 |
} |
|
65 |
||
66 |
bool VM_CMS_Operation::lost_race() const { |
|
67 |
if (CMSCollector::abstract_state() == CMSCollector::Idling) { |
|
68 |
// We lost a race to a foreground collection |
|
69 |
// -- there's nothing to do |
|
70 |
return true; |
|
71 |
} |
|
72 |
assert(CMSCollector::abstract_state() == legal_state(), |
|
73 |
"Inconsistent collector state?"); |
|
74 |
return false; |
|
75 |
} |
|
76 |
||
77 |
bool VM_CMS_Operation::doit_prologue() { |
|
78 |
assert(Thread::current()->is_ConcurrentGC_thread(), "just checking"); |
|
79 |
assert(!CMSCollector::foregroundGCShouldWait(), "Possible deadlock"); |
|
80 |
assert(!ConcurrentMarkSweepThread::cms_thread_has_cms_token(), |
|
81 |
"Possible deadlock"); |
|
82 |
||
83 |
Heap_lock->lock(); |
|
84 |
if (lost_race()) { |
|
85 |
assert(_prologue_succeeded == false, "Initialized in c'tor"); |
|
86 |
Heap_lock->unlock(); |
|
87 |
} else { |
|
88 |
_prologue_succeeded = true; |
|
89 |
} |
|
90 |
return _prologue_succeeded; |
|
91 |
} |
|
92 |
||
93 |
void VM_CMS_Operation::doit_epilogue() { |
|
94 |
assert(Thread::current()->is_ConcurrentGC_thread(), "just checking"); |
|
95 |
assert(!CMSCollector::foregroundGCShouldWait(), "Possible deadlock"); |
|
96 |
assert(!ConcurrentMarkSweepThread::cms_thread_has_cms_token(), |
|
97 |
"Possible deadlock"); |
|
98 |
||
40892
330a02d935ad
8156500: Move Reference pending list into VM to prevent deadlocks
kbarrett
parents:
37146
diff
changeset
|
99 |
if (Universe::has_reference_pending_list()) { |
330a02d935ad
8156500: Move Reference pending list into VM to prevent deadlocks
kbarrett
parents:
37146
diff
changeset
|
100 |
Heap_lock->notify_all(); |
330a02d935ad
8156500: Move Reference pending list into VM to prevent deadlocks
kbarrett
parents:
37146
diff
changeset
|
101 |
} |
1 | 102 |
Heap_lock->unlock(); |
103 |
} |
|
104 |
||
105 |
////////////////////////////////////////////////////////// |
|
106 |
// Methods in class VM_CMS_Initial_Mark |
|
107 |
////////////////////////////////////////////////////////// |
|
108 |
void VM_CMS_Initial_Mark::doit() { |
|
109 |
if (lost_race()) { |
|
110 |
// Nothing to do. |
|
111 |
return; |
|
112 |
} |
|
22520
b9e88eed82d7
8030812: Change the solaris DTrace implementation to use USDT2 instead of USDT1
sla
parents:
21767
diff
changeset
|
113 |
HS_PRIVATE_CMS_INITMARK_BEGIN(); |
33107
77bf0d2069a3
8134953: Make the GC ID available in a central place
brutisso
parents:
31358
diff
changeset
|
114 |
GCIdMark gc_id_mark(_gc_id); |
1 | 115 |
|
21767
41eaa9a17059
8028128: Add a type safe alternative for working with counter based data
mgronlun
parents:
18025
diff
changeset
|
116 |
_collector->_gc_timer_cm->register_gc_pause_start("Initial Mark"); |
18025 | 117 |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
118 |
CMSHeap* heap = CMSHeap::heap(); |
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
119 |
GCCauseSetter gccs(heap, GCCause::_cms_initial_mark); |
1 | 120 |
|
121 |
VM_CMS_Operation::verify_before_gc(); |
|
122 |
||
123 |
IsGCActiveMark x; // stop-world GC active |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
124 |
_collector->do_CMS_operation(CMSCollector::CMS_op_checkpointRootsInitial, heap->gc_cause()); |
1 | 125 |
|
126 |
VM_CMS_Operation::verify_after_gc(); |
|
18025 | 127 |
|
21767
41eaa9a17059
8028128: Add a type safe alternative for working with counter based data
mgronlun
parents:
18025
diff
changeset
|
128 |
_collector->_gc_timer_cm->register_gc_pause_end(); |
18025 | 129 |
|
22520
b9e88eed82d7
8030812: Change the solaris DTrace implementation to use USDT2 instead of USDT1
sla
parents:
21767
diff
changeset
|
130 |
HS_PRIVATE_CMS_INITMARK_END(); |
1 | 131 |
} |
132 |
||
133 |
////////////////////////////////////////////////////////// |
|
134 |
// Methods in class VM_CMS_Final_Remark_Operation |
|
135 |
////////////////////////////////////////////////////////// |
|
136 |
void VM_CMS_Final_Remark::doit() { |
|
137 |
if (lost_race()) { |
|
138 |
// Nothing to do. |
|
139 |
return; |
|
140 |
} |
|
22520
b9e88eed82d7
8030812: Change the solaris DTrace implementation to use USDT2 instead of USDT1
sla
parents:
21767
diff
changeset
|
141 |
HS_PRIVATE_CMS_REMARK_BEGIN(); |
33107
77bf0d2069a3
8134953: Make the GC ID available in a central place
brutisso
parents:
31358
diff
changeset
|
142 |
GCIdMark gc_id_mark(_gc_id); |
1 | 143 |
|
21767
41eaa9a17059
8028128: Add a type safe alternative for working with counter based data
mgronlun
parents:
18025
diff
changeset
|
144 |
_collector->_gc_timer_cm->register_gc_pause_start("Final Mark"); |
18025 | 145 |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
146 |
CMSHeap* heap = CMSHeap::heap(); |
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
147 |
GCCauseSetter gccs(heap, GCCause::_cms_final_remark); |
1 | 148 |
|
149 |
VM_CMS_Operation::verify_before_gc(); |
|
150 |
||
151 |
IsGCActiveMark x; // stop-world GC active |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
152 |
_collector->do_CMS_operation(CMSCollector::CMS_op_checkpointRootsFinal, heap->gc_cause()); |
1 | 153 |
|
154 |
VM_CMS_Operation::verify_after_gc(); |
|
18025 | 155 |
|
156 |
_collector->save_heap_summary(); |
|
21767
41eaa9a17059
8028128: Add a type safe alternative for working with counter based data
mgronlun
parents:
18025
diff
changeset
|
157 |
_collector->_gc_timer_cm->register_gc_pause_end(); |
18025 | 158 |
|
22520
b9e88eed82d7
8030812: Change the solaris DTrace implementation to use USDT2 instead of USDT1
sla
parents:
21767
diff
changeset
|
159 |
HS_PRIVATE_CMS_REMARK_END(); |
1 | 160 |
} |
161 |
||
162 |
// VM operation to invoke a concurrent collection of a |
|
163 |
// GenCollectedHeap heap. |
|
164 |
void VM_GenCollectFullConcurrent::doit() { |
|
165 |
assert(Thread::current()->is_VM_thread(), "Should be VM thread"); |
|
5433
c182d4c3039e
6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents:
1
diff
changeset
|
166 |
assert(GCLockerInvokesConcurrent || ExplicitGCInvokesConcurrent, "Unexpected"); |
1 | 167 |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
168 |
CMSHeap* heap = CMSHeap::heap(); |
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
169 |
if (_gc_count_before == heap->total_collections()) { |
1 | 170 |
// The "full" of do_full_collection call below "forces" |
171 |
// a collection; the second arg, 0, below ensures that |
|
172 |
// only the young gen is collected. XXX In the future, |
|
173 |
// we'll probably need to have something in this interface |
|
174 |
// to say do this only if we are sure we will not bail |
|
175 |
// out to a full collection in this attempt, but that's |
|
176 |
// for the future. |
|
177 |
assert(SafepointSynchronize::is_at_safepoint(), |
|
178 |
"We can only be executing this arm of if at a safepoint"); |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
179 |
GCCauseSetter gccs(heap, _gc_cause); |
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
180 |
heap->do_full_collection(heap->must_clear_all_soft_refs(), GenCollectedHeap::YoungGen); |
1 | 181 |
} // Else no need for a foreground young gc |
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
182 |
assert((_gc_count_before < heap->total_collections()) || |
35492
c8c0273e6b91
8146690: Make all classes in GC follow the naming convention.
david
parents:
35061
diff
changeset
|
183 |
(GCLocker::is_active() /* gc may have been skipped */ |
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
184 |
&& (_gc_count_before == heap->total_collections())), |
1 | 185 |
"total_collections() should be monotonically increasing"); |
186 |
||
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52876
diff
changeset
|
187 |
MutexLocker x(FullGCCount_lock, Mutex::_no_safepoint_check_flag); |
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
188 |
assert(_full_gc_count_before <= heap->total_full_collections(), "Error"); |
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
189 |
if (heap->total_full_collections() == _full_gc_count_before) { |
5433
c182d4c3039e
6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents:
1
diff
changeset
|
190 |
// Nudge the CMS thread to start a concurrent collection. |
18025 | 191 |
CMSCollector::request_full_gc(_full_gc_count_before, _gc_cause); |
1 | 192 |
} else { |
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
193 |
assert(_full_gc_count_before < heap->total_full_collections(), "Error"); |
1 | 194 |
FullGCCount_lock->notify_all(); // Inform the Java thread its work is done |
195 |
} |
|
196 |
} |
|
197 |
||
198 |
bool VM_GenCollectFullConcurrent::evaluate_at_safepoint() const { |
|
199 |
Thread* thr = Thread::current(); |
|
200 |
assert(thr != NULL, "Unexpected tid"); |
|
201 |
if (!thr->is_Java_thread()) { |
|
202 |
assert(thr->is_VM_thread(), "Expected to be evaluated by VM thread"); |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
203 |
CMSHeap* heap = CMSHeap::heap(); |
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
204 |
if (_gc_count_before != heap->total_collections()) { |
1 | 205 |
// No need to do a young gc, we'll just nudge the CMS thread |
206 |
// in the doit() method above, to be executed soon. |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
207 |
assert(_gc_count_before < heap->total_collections(), |
22775 | 208 |
"total_collections() should be monotonically increasing"); |
1 | 209 |
return false; // no need for foreground young gc |
210 |
} |
|
211 |
} |
|
212 |
return true; // may still need foreground young gc |
|
213 |
} |
|
214 |
||
215 |
||
216 |
void VM_GenCollectFullConcurrent::doit_epilogue() { |
|
217 |
Thread* thr = Thread::current(); |
|
218 |
assert(thr->is_Java_thread(), "just checking"); |
|
219 |
JavaThread* jt = (JavaThread*)thr; |
|
40892
330a02d935ad
8156500: Move Reference pending list into VM to prevent deadlocks
kbarrett
parents:
37146
diff
changeset
|
220 |
|
330a02d935ad
8156500: Move Reference pending list into VM to prevent deadlocks
kbarrett
parents:
37146
diff
changeset
|
221 |
if (Universe::has_reference_pending_list()) { |
330a02d935ad
8156500: Move Reference pending list into VM to prevent deadlocks
kbarrett
parents:
37146
diff
changeset
|
222 |
Heap_lock->notify_all(); |
330a02d935ad
8156500: Move Reference pending list into VM to prevent deadlocks
kbarrett
parents:
37146
diff
changeset
|
223 |
} |
1 | 224 |
Heap_lock->unlock(); |
225 |
||
226 |
// It is fine to test whether completed collections has |
|
227 |
// exceeded our request count without locking because |
|
228 |
// the completion count is monotonically increasing; |
|
229 |
// this will break for very long-running apps when the |
|
230 |
// count overflows and wraps around. XXX fix me !!! |
|
231 |
// e.g. at the rate of 1 full gc per ms, this could |
|
232 |
// overflow in about 1000 years. |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
233 |
CMSHeap* heap = CMSHeap::heap(); |
5433
c182d4c3039e
6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents:
1
diff
changeset
|
234 |
if (_gc_cause != GCCause::_gc_locker && |
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
235 |
heap->total_full_collections_completed() <= _full_gc_count_before) { |
6058
9c9aec6ab47d
6944166: G1: explicit GCs are not always handled correctly
tonyp
parents:
5547
diff
changeset
|
236 |
// maybe we should change the condition to test _gc_cause == |
31032
8e72621ca186
8072913: [REDO] GCCause should distinguish jcmd GC.run from System.gc()
ysuenaga
parents:
30764
diff
changeset
|
237 |
// GCCause::_java_lang_system_gc or GCCause::_dcmd_gc_run, |
8e72621ca186
8072913: [REDO] GCCause should distinguish jcmd GC.run from System.gc()
ysuenaga
parents:
30764
diff
changeset
|
238 |
// instead of _gc_cause != GCCause::_gc_locker |
8e72621ca186
8072913: [REDO] GCCause should distinguish jcmd GC.run from System.gc()
ysuenaga
parents:
30764
diff
changeset
|
239 |
assert(GCCause::is_user_requested_gc(_gc_cause), |
6058
9c9aec6ab47d
6944166: G1: explicit GCs are not always handled correctly
tonyp
parents:
5547
diff
changeset
|
240 |
"the only way to get here if this was a System.gc()-induced GC"); |
5433
c182d4c3039e
6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents:
1
diff
changeset
|
241 |
assert(ExplicitGCInvokesConcurrent, "Error"); |
1 | 242 |
// Now, wait for witnessing concurrent gc cycle to complete, |
243 |
// but do so in native mode, because we want to lock the |
|
244 |
// FullGCEvent_lock, which may be needed by the VM thread |
|
245 |
// or by the CMS thread, so we do not want to be suspended |
|
246 |
// while holding that lock. |
|
247 |
ThreadToNativeFromVM native(jt); |
|
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52876
diff
changeset
|
248 |
MutexLocker ml(FullGCCount_lock, Mutex::_no_safepoint_check_flag); |
1 | 249 |
// Either a concurrent or a stop-world full gc is sufficient |
250 |
// witness to our request. |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
251 |
while (heap->total_full_collections_completed() <= _full_gc_count_before) { |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52876
diff
changeset
|
252 |
FullGCCount_lock->wait_without_safepoint_check(); |
1 | 253 |
} |
254 |
} |
|
255 |
} |