author | johnc |
Thu, 22 Sep 2011 10:57:37 -0700 | |
changeset 10670 | 4ea0e7d2ffbc |
parent 7920 | 298df61588a2 |
child 11455 | a6ab3d8b9a4c |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
7920 | 2 |
* Copyright (c) 2001, 2011, Oracle and/or its affiliates. All rights reserved. |
1374 | 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:
5082
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5082
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:
5082
diff
changeset
|
21 |
* questions. |
1374 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
7920 | 26 |
#include "gc_implementation/g1/g1CollectedHeap.inline.hpp" |
7397 | 27 |
#include "gc_implementation/g1/satbQueue.hpp" |
28 |
#include "memory/allocation.inline.hpp" |
|
29 |
#include "memory/sharedHeap.hpp" |
|
30 |
#include "runtime/mutexLocker.hpp" |
|
31 |
#include "runtime/thread.hpp" |
|
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
32 |
#include "runtime/vmThread.hpp" |
1374 | 33 |
|
7920 | 34 |
// This method removes entries from an SATB buffer that will not be |
35 |
// useful to the concurrent marking threads. An entry is removed if it |
|
36 |
// satisfies one of the following conditions: |
|
37 |
// |
|
38 |
// * it points to an object outside the G1 heap (G1's concurrent |
|
39 |
// marking only visits objects inside the G1 heap), |
|
40 |
// * it points to an object that has been allocated since marking |
|
41 |
// started (according to SATB those objects do not need to be |
|
42 |
// visited during marking), or |
|
43 |
// * it points to an object that has already been marked (no need to |
|
44 |
// process it again). |
|
45 |
// |
|
46 |
// The rest of the entries will be retained and are compacted towards |
|
47 |
// the top of the buffer. If with this filtering we clear a large |
|
48 |
// enough chunk of the buffer we can re-use it (instead of enqueueing |
|
49 |
// it) and we can just allow the mutator to carry on executing. |
|
50 |
||
51 |
bool ObjPtrQueue::should_enqueue_buffer() { |
|
52 |
assert(_lock == NULL || _lock->owned_by_self(), |
|
53 |
"we should have taken the lock before calling this"); |
|
54 |
||
55 |
// A value of 0 means "don't filter SATB buffers". |
|
56 |
if (G1SATBBufferEnqueueingThresholdPercent == 0) { |
|
57 |
return true; |
|
58 |
} |
|
59 |
||
60 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
|
61 |
||
62 |
// This method should only be called if there is a non-NULL buffer |
|
63 |
// that is full. |
|
64 |
assert(_index == 0, "pre-condition"); |
|
65 |
assert(_buf != NULL, "pre-condition"); |
|
66 |
||
67 |
void** buf = _buf; |
|
68 |
size_t sz = _sz; |
|
69 |
||
70 |
// Used for sanity checking at the end of the loop. |
|
71 |
debug_only(size_t entries = 0; size_t retained = 0;) |
|
72 |
||
73 |
size_t i = sz; |
|
74 |
size_t new_index = sz; |
|
75 |
||
76 |
// Given that we are expecting _index == 0, we could have changed |
|
77 |
// the loop condition to (i > 0). But we are using _index for |
|
78 |
// generality. |
|
79 |
while (i > _index) { |
|
80 |
assert(i > 0, "we should have at least one more entry to process"); |
|
81 |
i -= oopSize; |
|
82 |
debug_only(entries += 1;) |
|
83 |
oop* p = (oop*) &buf[byte_index_to_index((int) i)]; |
|
84 |
oop obj = *p; |
|
85 |
// NULL the entry so that unused parts of the buffer contain NULLs |
|
86 |
// at the end. If we are going to retain it we will copy it to its |
|
87 |
// final place. If we have retained all entries we have visited so |
|
88 |
// far, we'll just end up copying it to the same place. |
|
89 |
*p = NULL; |
|
90 |
||
91 |
bool retain = g1h->is_obj_ill(obj); |
|
92 |
if (retain) { |
|
93 |
assert(new_index > 0, "we should not have already filled up the buffer"); |
|
94 |
new_index -= oopSize; |
|
95 |
assert(new_index >= i, |
|
96 |
"new_index should never be below i, as we alwaysr compact 'up'"); |
|
97 |
oop* new_p = (oop*) &buf[byte_index_to_index((int) new_index)]; |
|
98 |
assert(new_p >= p, "the destination location should never be below " |
|
99 |
"the source as we always compact 'up'"); |
|
100 |
assert(*new_p == NULL, |
|
101 |
"we should have already cleared the destination location"); |
|
102 |
*new_p = obj; |
|
103 |
debug_only(retained += 1;) |
|
104 |
} |
|
105 |
} |
|
106 |
size_t entries_calc = (sz - _index) / oopSize; |
|
107 |
assert(entries == entries_calc, "the number of entries we counted " |
|
108 |
"should match the number of entries we calculated"); |
|
109 |
size_t retained_calc = (sz - new_index) / oopSize; |
|
110 |
assert(retained == retained_calc, "the number of retained entries we counted " |
|
111 |
"should match the number of retained entries we calculated"); |
|
112 |
size_t perc = retained_calc * 100 / entries_calc; |
|
113 |
bool should_enqueue = perc > (size_t) G1SATBBufferEnqueueingThresholdPercent; |
|
114 |
_index = new_index; |
|
115 |
||
116 |
return should_enqueue; |
|
117 |
} |
|
118 |
||
1374 | 119 |
void ObjPtrQueue::apply_closure(ObjectClosure* cl) { |
120 |
if (_buf != NULL) { |
|
121 |
apply_closure_to_buffer(cl, _buf, _index, _sz); |
|
122 |
_index = _sz; |
|
123 |
} |
|
124 |
} |
|
125 |
||
126 |
void ObjPtrQueue::apply_closure_to_buffer(ObjectClosure* cl, |
|
127 |
void** buf, size_t index, size_t sz) { |
|
128 |
if (cl == NULL) return; |
|
129 |
for (size_t i = index; i < sz; i += oopSize) { |
|
130 |
oop obj = (oop)buf[byte_index_to_index((int)i)]; |
|
131 |
// There can be NULL entries because of destructors. |
|
132 |
if (obj != NULL) { |
|
133 |
cl->do_object(obj); |
|
134 |
} |
|
135 |
} |
|
136 |
} |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
137 |
|
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
138 |
#ifdef ASSERT |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
139 |
void ObjPtrQueue::verify_oops_in_buffer() { |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
140 |
if (_buf == NULL) return; |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
141 |
for (size_t i = _index; i < _sz; i += oopSize) { |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
142 |
oop obj = (oop)_buf[byte_index_to_index((int)i)]; |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
143 |
assert(obj != NULL && obj->is_oop(true /* ignore mark word */), |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
144 |
"Not an oop"); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
145 |
} |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
146 |
} |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
147 |
#endif |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
148 |
|
1374 | 149 |
#ifdef _MSC_VER // the use of 'this' below gets a warning, make it go away |
150 |
#pragma warning( disable:4355 ) // 'this' : used in base member initializer list |
|
151 |
#endif // _MSC_VER |
|
152 |
||
153 |
||
154 |
SATBMarkQueueSet::SATBMarkQueueSet() : |
|
155 |
PtrQueueSet(), |
|
156 |
_closure(NULL), _par_closures(NULL), |
|
157 |
_shared_satb_queue(this, true /*perm*/) |
|
158 |
{} |
|
159 |
||
160 |
void SATBMarkQueueSet::initialize(Monitor* cbl_mon, Mutex* fl_lock, |
|
4481 | 161 |
int process_completed_threshold, |
1374 | 162 |
Mutex* lock) { |
4481 | 163 |
PtrQueueSet::initialize(cbl_mon, fl_lock, process_completed_threshold, -1); |
1374 | 164 |
_shared_satb_queue.set_lock(lock); |
165 |
if (ParallelGCThreads > 0) { |
|
166 |
_par_closures = NEW_C_HEAP_ARRAY(ObjectClosure*, ParallelGCThreads); |
|
167 |
} |
|
168 |
} |
|
169 |
||
170 |
||
171 |
void SATBMarkQueueSet::handle_zero_index_for_thread(JavaThread* t) { |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
172 |
DEBUG_ONLY(t->satb_mark_queue().verify_oops_in_buffer();) |
1374 | 173 |
t->satb_mark_queue().handle_zero_index(); |
174 |
} |
|
175 |
||
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
176 |
#ifdef ASSERT |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
177 |
void SATBMarkQueueSet::dump_active_values(JavaThread* first, |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
178 |
bool expected_active) { |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
179 |
gclog_or_tty->print_cr("SATB queue active values for Java Threads"); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
180 |
gclog_or_tty->print_cr(" SATB queue set: active is %s", |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
181 |
(is_active()) ? "TRUE" : "FALSE"); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
182 |
gclog_or_tty->print_cr(" expected_active is %s", |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
183 |
(expected_active) ? "TRUE" : "FALSE"); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
184 |
for (JavaThread* t = first; t; t = t->next()) { |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
185 |
bool active = t->satb_mark_queue().is_active(); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
186 |
gclog_or_tty->print_cr(" thread %s, active is %s", |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
187 |
t->name(), (active) ? "TRUE" : "FALSE"); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
188 |
} |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
189 |
} |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
190 |
#endif // ASSERT |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
191 |
|
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
192 |
void SATBMarkQueueSet::set_active_all_threads(bool b, |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
193 |
bool expected_active) { |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
194 |
assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint."); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
195 |
JavaThread* first = Threads::first(); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
196 |
|
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
197 |
#ifdef ASSERT |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
198 |
if (_all_active != expected_active) { |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
199 |
dump_active_values(first, expected_active); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
200 |
|
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
201 |
// I leave this here as a guarantee, instead of an assert, so |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
202 |
// that it will still be compiled in if we choose to uncomment |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
203 |
// the #ifdef ASSERT in a product build. The whole block is |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
204 |
// within an #ifdef ASSERT so the guarantee will not be compiled |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
205 |
// in a product build anyway. |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
206 |
guarantee(false, |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
207 |
"SATB queue set has an unexpected active value"); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
208 |
} |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
209 |
#endif // ASSERT |
1374 | 210 |
_all_active = b; |
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
211 |
|
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
212 |
for (JavaThread* t = first; t; t = t->next()) { |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
213 |
#ifdef ASSERT |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
214 |
bool active = t->satb_mark_queue().is_active(); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
215 |
if (active != expected_active) { |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
216 |
dump_active_values(first, expected_active); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
217 |
|
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
218 |
// I leave this here as a guarantee, instead of an assert, so |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
219 |
// that it will still be compiled in if we choose to uncomment |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
220 |
// the #ifdef ASSERT in a product build. The whole block is |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
221 |
// within an #ifdef ASSERT so the guarantee will not be compiled |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
222 |
// in a product build anyway. |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
223 |
guarantee(false, |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
224 |
"thread has an unexpected active value in its SATB queue"); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
225 |
} |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
226 |
#endif // ASSERT |
1374 | 227 |
t->satb_mark_queue().set_active(b); |
228 |
} |
|
229 |
} |
|
230 |
||
231 |
void SATBMarkQueueSet::set_closure(ObjectClosure* closure) { |
|
232 |
_closure = closure; |
|
233 |
} |
|
234 |
||
235 |
void SATBMarkQueueSet::set_par_closure(int i, ObjectClosure* par_closure) { |
|
236 |
assert(ParallelGCThreads > 0 && _par_closures != NULL, "Precondition"); |
|
237 |
_par_closures[i] = par_closure; |
|
238 |
} |
|
239 |
||
240 |
void SATBMarkQueueSet::iterate_closure_all_threads() { |
|
241 |
for(JavaThread* t = Threads::first(); t; t = t->next()) { |
|
242 |
t->satb_mark_queue().apply_closure(_closure); |
|
243 |
} |
|
244 |
shared_satb_queue()->apply_closure(_closure); |
|
245 |
} |
|
246 |
||
247 |
void SATBMarkQueueSet::par_iterate_closure_all_threads(int worker) { |
|
248 |
SharedHeap* sh = SharedHeap::heap(); |
|
249 |
int parity = sh->strong_roots_parity(); |
|
250 |
||
251 |
for(JavaThread* t = Threads::first(); t; t = t->next()) { |
|
252 |
if (t->claim_oops_do(true, parity)) { |
|
253 |
t->satb_mark_queue().apply_closure(_par_closures[worker]); |
|
254 |
} |
|
255 |
} |
|
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
256 |
|
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
257 |
// We also need to claim the VMThread so that its parity is updated |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
258 |
// otherwise the next call to Thread::possibly_parallel_oops_do inside |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
259 |
// a StrongRootsScope might skip the VMThread because it has a stale |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
260 |
// parity that matches the parity set by the StrongRootsScope |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
261 |
// |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
262 |
// Whichever worker succeeds in claiming the VMThread gets to do |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
263 |
// the shared queue. |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
264 |
|
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
265 |
VMThread* vmt = VMThread::vm_thread(); |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
266 |
if (vmt->claim_oops_do(true, parity)) { |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
267 |
shared_satb_queue()->apply_closure(_par_closures[worker]); |
1374 | 268 |
} |
269 |
} |
|
270 |
||
271 |
bool SATBMarkQueueSet::apply_closure_to_completed_buffer_work(bool par, |
|
272 |
int worker) { |
|
4481 | 273 |
BufferNode* nd = NULL; |
1374 | 274 |
{ |
275 |
MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag); |
|
276 |
if (_completed_buffers_head != NULL) { |
|
277 |
nd = _completed_buffers_head; |
|
4481 | 278 |
_completed_buffers_head = nd->next(); |
1374 | 279 |
if (_completed_buffers_head == NULL) _completed_buffers_tail = NULL; |
280 |
_n_completed_buffers--; |
|
281 |
if (_n_completed_buffers == 0) _process_completed = false; |
|
282 |
} |
|
283 |
} |
|
284 |
ObjectClosure* cl = (par ? _par_closures[worker] : _closure); |
|
285 |
if (nd != NULL) { |
|
4481 | 286 |
void **buf = BufferNode::make_buffer_from_node(nd); |
287 |
ObjPtrQueue::apply_closure_to_buffer(cl, buf, 0, _sz); |
|
288 |
deallocate_buffer(buf); |
|
1374 | 289 |
return true; |
290 |
} else { |
|
291 |
return false; |
|
292 |
} |
|
293 |
} |
|
294 |
||
295 |
void SATBMarkQueueSet::abandon_partial_marking() { |
|
4481 | 296 |
BufferNode* buffers_to_delete = NULL; |
1374 | 297 |
{ |
298 |
MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag); |
|
299 |
while (_completed_buffers_head != NULL) { |
|
4481 | 300 |
BufferNode* nd = _completed_buffers_head; |
301 |
_completed_buffers_head = nd->next(); |
|
302 |
nd->set_next(buffers_to_delete); |
|
1374 | 303 |
buffers_to_delete = nd; |
304 |
} |
|
305 |
_completed_buffers_tail = NULL; |
|
306 |
_n_completed_buffers = 0; |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
307 |
DEBUG_ONLY(assert_completed_buffer_list_len_correct_locked()); |
1374 | 308 |
} |
309 |
while (buffers_to_delete != NULL) { |
|
4481 | 310 |
BufferNode* nd = buffers_to_delete; |
311 |
buffers_to_delete = nd->next(); |
|
312 |
deallocate_buffer(BufferNode::make_buffer_from_node(nd)); |
|
1374 | 313 |
} |
314 |
assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint."); |
|
315 |
// So we can safely manipulate these queues. |
|
316 |
for (JavaThread* t = Threads::first(); t; t = t->next()) { |
|
317 |
t->satb_mark_queue().reset(); |
|
318 |
} |
|
319 |
shared_satb_queue()->reset(); |
|
320 |
} |