annotate src/share/vm/gc_implementation/concurrentMarkSweep/vmCMSOperations.cpp @ 6064:9d679effd28c

7166894: Add gc cause to GC logging for all collectors Reviewed-by: mgerdin, johnc
author brutisso
date Tue, 15 May 2012 10:25:06 +0200
parents 436b4a3231bf
children b9a9ed0f8eeb 4202510ee0fe
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
2 * Copyright (c) 2005, 2011, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1520
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1520
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1520
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
26 #include "gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
27 #include "gc_implementation/concurrentMarkSweep/concurrentMarkSweepThread.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
28 #include "gc_implementation/concurrentMarkSweep/vmCMSOperations.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
29 #include "gc_implementation/shared/isGCActiveMark.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
30 #include "memory/gcLocker.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
31 #include "runtime/interfaceSupport.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
32 #include "utilities/dtrace.hpp"
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
33
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
34
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
35 #ifndef USDT2
0
a61af66fc99e Initial load
duke
parents:
diff changeset
36 HS_DTRACE_PROBE_DECL(hs_private, cms__initmark__begin);
a61af66fc99e Initial load
duke
parents:
diff changeset
37 HS_DTRACE_PROBE_DECL(hs_private, cms__initmark__end);
a61af66fc99e Initial load
duke
parents:
diff changeset
38
a61af66fc99e Initial load
duke
parents:
diff changeset
39 HS_DTRACE_PROBE_DECL(hs_private, cms__remark__begin);
a61af66fc99e Initial load
duke
parents:
diff changeset
40 HS_DTRACE_PROBE_DECL(hs_private, cms__remark__end);
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
41 #endif /* !USDT2 */
0
a61af66fc99e Initial load
duke
parents:
diff changeset
42
a61af66fc99e Initial load
duke
parents:
diff changeset
43 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
44 // Methods in abstract class VM_CMS_Operation
a61af66fc99e Initial load
duke
parents:
diff changeset
45 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
46 void VM_CMS_Operation::acquire_pending_list_lock() {
a61af66fc99e Initial load
duke
parents:
diff changeset
47 // The caller may block while communicating
a61af66fc99e Initial load
duke
parents:
diff changeset
48 // with the SLT thread in order to acquire/release the PLL.
a61af66fc99e Initial load
duke
parents:
diff changeset
49 ConcurrentMarkSweepThread::slt()->
a61af66fc99e Initial load
duke
parents:
diff changeset
50 manipulatePLL(SurrogateLockerThread::acquirePLL);
a61af66fc99e Initial load
duke
parents:
diff changeset
51 }
a61af66fc99e Initial load
duke
parents:
diff changeset
52
a61af66fc99e Initial load
duke
parents:
diff changeset
53 void VM_CMS_Operation::release_and_notify_pending_list_lock() {
a61af66fc99e Initial load
duke
parents:
diff changeset
54 // The caller may block while communicating
a61af66fc99e Initial load
duke
parents:
diff changeset
55 // with the SLT thread in order to acquire/release the PLL.
a61af66fc99e Initial load
duke
parents:
diff changeset
56 ConcurrentMarkSweepThread::slt()->
a61af66fc99e Initial load
duke
parents:
diff changeset
57 manipulatePLL(SurrogateLockerThread::releaseAndNotifyPLL);
a61af66fc99e Initial load
duke
parents:
diff changeset
58 }
a61af66fc99e Initial load
duke
parents:
diff changeset
59
a61af66fc99e Initial load
duke
parents:
diff changeset
60 void VM_CMS_Operation::verify_before_gc() {
a61af66fc99e Initial load
duke
parents:
diff changeset
61 if (VerifyBeforeGC &&
a61af66fc99e Initial load
duke
parents:
diff changeset
62 GenCollectedHeap::heap()->total_collections() >= VerifyGCStartAt) {
a61af66fc99e Initial load
duke
parents:
diff changeset
63 HandleMark hm;
a61af66fc99e Initial load
duke
parents:
diff changeset
64 FreelistLocker x(_collector);
a61af66fc99e Initial load
duke
parents:
diff changeset
65 MutexLockerEx y(_collector->bitMapLock(), Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
66 Universe::heap()->prepare_for_verify();
a61af66fc99e Initial load
duke
parents:
diff changeset
67 Universe::verify(true);
a61af66fc99e Initial load
duke
parents:
diff changeset
68 }
a61af66fc99e Initial load
duke
parents:
diff changeset
69 }
a61af66fc99e Initial load
duke
parents:
diff changeset
70
a61af66fc99e Initial load
duke
parents:
diff changeset
71 void VM_CMS_Operation::verify_after_gc() {
a61af66fc99e Initial load
duke
parents:
diff changeset
72 if (VerifyAfterGC &&
a61af66fc99e Initial load
duke
parents:
diff changeset
73 GenCollectedHeap::heap()->total_collections() >= VerifyGCStartAt) {
a61af66fc99e Initial load
duke
parents:
diff changeset
74 HandleMark hm;
a61af66fc99e Initial load
duke
parents:
diff changeset
75 FreelistLocker x(_collector);
a61af66fc99e Initial load
duke
parents:
diff changeset
76 MutexLockerEx y(_collector->bitMapLock(), Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
77 Universe::verify(true);
a61af66fc99e Initial load
duke
parents:
diff changeset
78 }
a61af66fc99e Initial load
duke
parents:
diff changeset
79 }
a61af66fc99e Initial load
duke
parents:
diff changeset
80
a61af66fc99e Initial load
duke
parents:
diff changeset
81 bool VM_CMS_Operation::lost_race() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
82 if (CMSCollector::abstract_state() == CMSCollector::Idling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
83 // We lost a race to a foreground collection
a61af66fc99e Initial load
duke
parents:
diff changeset
84 // -- there's nothing to do
a61af66fc99e Initial load
duke
parents:
diff changeset
85 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
86 }
a61af66fc99e Initial load
duke
parents:
diff changeset
87 assert(CMSCollector::abstract_state() == legal_state(),
a61af66fc99e Initial load
duke
parents:
diff changeset
88 "Inconsistent collector state?");
a61af66fc99e Initial load
duke
parents:
diff changeset
89 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
90 }
a61af66fc99e Initial load
duke
parents:
diff changeset
91
a61af66fc99e Initial load
duke
parents:
diff changeset
92 bool VM_CMS_Operation::doit_prologue() {
a61af66fc99e Initial load
duke
parents:
diff changeset
93 assert(Thread::current()->is_ConcurrentGC_thread(), "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
94 assert(!CMSCollector::foregroundGCShouldWait(), "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
95 assert(!ConcurrentMarkSweepThread::cms_thread_has_cms_token(),
a61af66fc99e Initial load
duke
parents:
diff changeset
96 "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
97
a61af66fc99e Initial load
duke
parents:
diff changeset
98 if (needs_pll()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
99 acquire_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
100 }
a61af66fc99e Initial load
duke
parents:
diff changeset
101 // Get the Heap_lock after the pending_list_lock.
a61af66fc99e Initial load
duke
parents:
diff changeset
102 Heap_lock->lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
103 if (lost_race()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
104 assert(_prologue_succeeded == false, "Initialized in c'tor");
a61af66fc99e Initial load
duke
parents:
diff changeset
105 Heap_lock->unlock();
a61af66fc99e Initial load
duke
parents:
diff changeset
106 if (needs_pll()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
107 release_and_notify_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
108 }
a61af66fc99e Initial load
duke
parents:
diff changeset
109 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
110 _prologue_succeeded = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
111 }
a61af66fc99e Initial load
duke
parents:
diff changeset
112 return _prologue_succeeded;
a61af66fc99e Initial load
duke
parents:
diff changeset
113 }
a61af66fc99e Initial load
duke
parents:
diff changeset
114
a61af66fc99e Initial load
duke
parents:
diff changeset
115 void VM_CMS_Operation::doit_epilogue() {
a61af66fc99e Initial load
duke
parents:
diff changeset
116 assert(Thread::current()->is_ConcurrentGC_thread(), "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
117 assert(!CMSCollector::foregroundGCShouldWait(), "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
118 assert(!ConcurrentMarkSweepThread::cms_thread_has_cms_token(),
a61af66fc99e Initial load
duke
parents:
diff changeset
119 "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
120
a61af66fc99e Initial load
duke
parents:
diff changeset
121 // Release the Heap_lock first.
a61af66fc99e Initial load
duke
parents:
diff changeset
122 Heap_lock->unlock();
a61af66fc99e Initial load
duke
parents:
diff changeset
123 if (needs_pll()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
124 release_and_notify_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
125 }
a61af66fc99e Initial load
duke
parents:
diff changeset
126 }
a61af66fc99e Initial load
duke
parents:
diff changeset
127
a61af66fc99e Initial load
duke
parents:
diff changeset
128 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
129 // Methods in class VM_CMS_Initial_Mark
a61af66fc99e Initial load
duke
parents:
diff changeset
130 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
131 void VM_CMS_Initial_Mark::doit() {
a61af66fc99e Initial load
duke
parents:
diff changeset
132 if (lost_race()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
133 // Nothing to do.
a61af66fc99e Initial load
duke
parents:
diff changeset
134 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
135 }
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
136 #ifndef USDT2
0
a61af66fc99e Initial load
duke
parents:
diff changeset
137 HS_DTRACE_PROBE(hs_private, cms__initmark__begin);
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
138 #else /* USDT2 */
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
139 HS_PRIVATE_CMS_INITMARK_BEGIN(
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
140 );
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
141 #endif /* USDT2 */
0
a61af66fc99e Initial load
duke
parents:
diff changeset
142
a61af66fc99e Initial load
duke
parents:
diff changeset
143 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
144 GCCauseSetter gccs(gch, GCCause::_cms_initial_mark);
a61af66fc99e Initial load
duke
parents:
diff changeset
145
a61af66fc99e Initial load
duke
parents:
diff changeset
146 VM_CMS_Operation::verify_before_gc();
a61af66fc99e Initial load
duke
parents:
diff changeset
147
a61af66fc99e Initial load
duke
parents:
diff changeset
148 IsGCActiveMark x; // stop-world GC active
6064
9d679effd28c 7166894: Add gc cause to GC logging for all collectors
brutisso
parents: 4006
diff changeset
149 _collector->do_CMS_operation(CMSCollector::CMS_op_checkpointRootsInitial, gch->gc_cause());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
150
a61af66fc99e Initial load
duke
parents:
diff changeset
151 VM_CMS_Operation::verify_after_gc();
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
152 #ifndef USDT2
0
a61af66fc99e Initial load
duke
parents:
diff changeset
153 HS_DTRACE_PROBE(hs_private, cms__initmark__end);
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
154 #else /* USDT2 */
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
155 HS_PRIVATE_CMS_INITMARK_END(
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
156 );
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
157 #endif /* USDT2 */
0
a61af66fc99e Initial load
duke
parents:
diff changeset
158 }
a61af66fc99e Initial load
duke
parents:
diff changeset
159
a61af66fc99e Initial load
duke
parents:
diff changeset
160 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
161 // Methods in class VM_CMS_Final_Remark_Operation
a61af66fc99e Initial load
duke
parents:
diff changeset
162 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
163 void VM_CMS_Final_Remark::doit() {
a61af66fc99e Initial load
duke
parents:
diff changeset
164 if (lost_race()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
165 // Nothing to do.
a61af66fc99e Initial load
duke
parents:
diff changeset
166 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
167 }
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
168 #ifndef USDT2
0
a61af66fc99e Initial load
duke
parents:
diff changeset
169 HS_DTRACE_PROBE(hs_private, cms__remark__begin);
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
170 #else /* USDT2 */
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
171 HS_PRIVATE_CMS_REMARK_BEGIN(
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
172 );
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
173 #endif /* USDT2 */
0
a61af66fc99e Initial load
duke
parents:
diff changeset
174
a61af66fc99e Initial load
duke
parents:
diff changeset
175 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
176 GCCauseSetter gccs(gch, GCCause::_cms_final_remark);
a61af66fc99e Initial load
duke
parents:
diff changeset
177
a61af66fc99e Initial load
duke
parents:
diff changeset
178 VM_CMS_Operation::verify_before_gc();
a61af66fc99e Initial load
duke
parents:
diff changeset
179
a61af66fc99e Initial load
duke
parents:
diff changeset
180 IsGCActiveMark x; // stop-world GC active
6064
9d679effd28c 7166894: Add gc cause to GC logging for all collectors
brutisso
parents: 4006
diff changeset
181 _collector->do_CMS_operation(CMSCollector::CMS_op_checkpointRootsFinal, gch->gc_cause());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
182
a61af66fc99e Initial load
duke
parents:
diff changeset
183 VM_CMS_Operation::verify_after_gc();
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
184 #ifndef USDT2
0
a61af66fc99e Initial load
duke
parents:
diff changeset
185 HS_DTRACE_PROBE(hs_private, cms__remark__end);
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
186 #else /* USDT2 */
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
187 HS_PRIVATE_CMS_REMARK_END(
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
188 );
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
189 #endif /* USDT2 */
0
a61af66fc99e Initial load
duke
parents:
diff changeset
190 }
a61af66fc99e Initial load
duke
parents:
diff changeset
191
a61af66fc99e Initial load
duke
parents:
diff changeset
192 // VM operation to invoke a concurrent collection of a
a61af66fc99e Initial load
duke
parents:
diff changeset
193 // GenCollectedHeap heap.
a61af66fc99e Initial load
duke
parents:
diff changeset
194 void VM_GenCollectFullConcurrent::doit() {
a61af66fc99e Initial load
duke
parents:
diff changeset
195 assert(Thread::current()->is_VM_thread(), "Should be VM thread");
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
196 assert(GCLockerInvokesConcurrent || ExplicitGCInvokesConcurrent, "Unexpected");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
197
a61af66fc99e Initial load
duke
parents:
diff changeset
198 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
199 if (_gc_count_before == gch->total_collections()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
200 // The "full" of do_full_collection call below "forces"
a61af66fc99e Initial load
duke
parents:
diff changeset
201 // a collection; the second arg, 0, below ensures that
a61af66fc99e Initial load
duke
parents:
diff changeset
202 // only the young gen is collected. XXX In the future,
a61af66fc99e Initial load
duke
parents:
diff changeset
203 // we'll probably need to have something in this interface
a61af66fc99e Initial load
duke
parents:
diff changeset
204 // to say do this only if we are sure we will not bail
a61af66fc99e Initial load
duke
parents:
diff changeset
205 // out to a full collection in this attempt, but that's
a61af66fc99e Initial load
duke
parents:
diff changeset
206 // for the future.
a61af66fc99e Initial load
duke
parents:
diff changeset
207 assert(SafepointSynchronize::is_at_safepoint(),
a61af66fc99e Initial load
duke
parents:
diff changeset
208 "We can only be executing this arm of if at a safepoint");
a61af66fc99e Initial load
duke
parents:
diff changeset
209 GCCauseSetter gccs(gch, _gc_cause);
a61af66fc99e Initial load
duke
parents:
diff changeset
210 gch->do_full_collection(gch->must_clear_all_soft_refs(),
a61af66fc99e Initial load
duke
parents:
diff changeset
211 0 /* collect only youngest gen */);
a61af66fc99e Initial load
duke
parents:
diff changeset
212 } // Else no need for a foreground young gc
a61af66fc99e Initial load
duke
parents:
diff changeset
213 assert((_gc_count_before < gch->total_collections()) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
214 (GC_locker::is_active() /* gc may have been skipped */
a61af66fc99e Initial load
duke
parents:
diff changeset
215 && (_gc_count_before == gch->total_collections())),
a61af66fc99e Initial load
duke
parents:
diff changeset
216 "total_collections() should be monotonically increasing");
a61af66fc99e Initial load
duke
parents:
diff changeset
217
a61af66fc99e Initial load
duke
parents:
diff changeset
218 MutexLockerEx x(FullGCCount_lock, Mutex::_no_safepoint_check_flag);
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
219 assert(_full_gc_count_before <= gch->total_full_collections(), "Error");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
220 if (gch->total_full_collections() == _full_gc_count_before) {
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
221 // Disable iCMS until the full collection is done, and
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
222 // remember that we did so.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
223 CMSCollector::disable_icms();
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
224 _disabled_icms = true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
225 // In case CMS thread was in icms_wait(), wake it up.
a61af66fc99e Initial load
duke
parents:
diff changeset
226 CMSCollector::start_icms();
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
227 // Nudge the CMS thread to start a concurrent collection.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
228 CMSCollector::request_full_gc(_full_gc_count_before);
a61af66fc99e Initial load
duke
parents:
diff changeset
229 } else {
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
230 assert(_full_gc_count_before < gch->total_full_collections(), "Error");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
231 FullGCCount_lock->notify_all(); // Inform the Java thread its work is done
a61af66fc99e Initial load
duke
parents:
diff changeset
232 }
a61af66fc99e Initial load
duke
parents:
diff changeset
233 }
a61af66fc99e Initial load
duke
parents:
diff changeset
234
a61af66fc99e Initial load
duke
parents:
diff changeset
235 bool VM_GenCollectFullConcurrent::evaluate_at_safepoint() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
236 Thread* thr = Thread::current();
a61af66fc99e Initial load
duke
parents:
diff changeset
237 assert(thr != NULL, "Unexpected tid");
a61af66fc99e Initial load
duke
parents:
diff changeset
238 if (!thr->is_Java_thread()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
239 assert(thr->is_VM_thread(), "Expected to be evaluated by VM thread");
a61af66fc99e Initial load
duke
parents:
diff changeset
240 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
241 if (_gc_count_before != gch->total_collections()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
242 // No need to do a young gc, we'll just nudge the CMS thread
a61af66fc99e Initial load
duke
parents:
diff changeset
243 // in the doit() method above, to be executed soon.
a61af66fc99e Initial load
duke
parents:
diff changeset
244 assert(_gc_count_before < gch->total_collections(),
a61af66fc99e Initial load
duke
parents:
diff changeset
245 "total_collections() should be monotnically increasing");
a61af66fc99e Initial load
duke
parents:
diff changeset
246 return false; // no need for foreground young gc
a61af66fc99e Initial load
duke
parents:
diff changeset
247 }
a61af66fc99e Initial load
duke
parents:
diff changeset
248 }
a61af66fc99e Initial load
duke
parents:
diff changeset
249 return true; // may still need foreground young gc
a61af66fc99e Initial load
duke
parents:
diff changeset
250 }
a61af66fc99e Initial load
duke
parents:
diff changeset
251
a61af66fc99e Initial load
duke
parents:
diff changeset
252
a61af66fc99e Initial load
duke
parents:
diff changeset
253 void VM_GenCollectFullConcurrent::doit_epilogue() {
a61af66fc99e Initial load
duke
parents:
diff changeset
254 Thread* thr = Thread::current();
a61af66fc99e Initial load
duke
parents:
diff changeset
255 assert(thr->is_Java_thread(), "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
256 JavaThread* jt = (JavaThread*)thr;
a61af66fc99e Initial load
duke
parents:
diff changeset
257 // Release the Heap_lock first.
a61af66fc99e Initial load
duke
parents:
diff changeset
258 Heap_lock->unlock();
a61af66fc99e Initial load
duke
parents:
diff changeset
259 release_and_notify_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
260
a61af66fc99e Initial load
duke
parents:
diff changeset
261 // It is fine to test whether completed collections has
a61af66fc99e Initial load
duke
parents:
diff changeset
262 // exceeded our request count without locking because
a61af66fc99e Initial load
duke
parents:
diff changeset
263 // the completion count is monotonically increasing;
a61af66fc99e Initial load
duke
parents:
diff changeset
264 // this will break for very long-running apps when the
a61af66fc99e Initial load
duke
parents:
diff changeset
265 // count overflows and wraps around. XXX fix me !!!
a61af66fc99e Initial load
duke
parents:
diff changeset
266 // e.g. at the rate of 1 full gc per ms, this could
a61af66fc99e Initial load
duke
parents:
diff changeset
267 // overflow in about 1000 years.
a61af66fc99e Initial load
duke
parents:
diff changeset
268 GenCollectedHeap* gch = GenCollectedHeap::heap();
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
269 if (_gc_cause != GCCause::_gc_locker &&
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
270 gch->total_full_collections_completed() <= _full_gc_count_before) {
1656
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
271 // maybe we should change the condition to test _gc_cause ==
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
272 // GCCause::_java_lang_system_gc, instead of
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
273 // _gc_cause != GCCause::_gc_locker
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
274 assert(_gc_cause == GCCause::_java_lang_system_gc,
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
275 "the only way to get here if this was a System.gc()-induced GC");
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
276 assert(ExplicitGCInvokesConcurrent, "Error");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
277 // Now, wait for witnessing concurrent gc cycle to complete,
a61af66fc99e Initial load
duke
parents:
diff changeset
278 // but do so in native mode, because we want to lock the
a61af66fc99e Initial load
duke
parents:
diff changeset
279 // FullGCEvent_lock, which may be needed by the VM thread
a61af66fc99e Initial load
duke
parents:
diff changeset
280 // or by the CMS thread, so we do not want to be suspended
a61af66fc99e Initial load
duke
parents:
diff changeset
281 // while holding that lock.
a61af66fc99e Initial load
duke
parents:
diff changeset
282 ThreadToNativeFromVM native(jt);
a61af66fc99e Initial load
duke
parents:
diff changeset
283 MutexLockerEx ml(FullGCCount_lock, Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
284 // Either a concurrent or a stop-world full gc is sufficient
a61af66fc99e Initial load
duke
parents:
diff changeset
285 // witness to our request.
a61af66fc99e Initial load
duke
parents:
diff changeset
286 while (gch->total_full_collections_completed() <= _full_gc_count_before) {
a61af66fc99e Initial load
duke
parents:
diff changeset
287 FullGCCount_lock->wait(Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
288 }
a61af66fc99e Initial load
duke
parents:
diff changeset
289 }
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
290 // Enable iCMS back if we disabled it earlier.
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
291 if (_disabled_icms) {
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
292 CMSCollector::enable_icms();
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
293 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
294 }