author | eosterlund |
Tue, 16 Oct 2018 13:18:22 +0200 | |
changeset 52142 | ca0c25e01c5b |
parent 51441 | 2e91d927e00c |
child 52582 | 6df094be7f58 |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
49007
82d9d5744e5f
8197994: Move JavaThread::initialize_queues() logic to G1SATBCardTableLoggingModRefBS
pliden
parents:
48105
diff
changeset
|
2 |
* Copyright (c) 2001, 2018, 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" |
51441
2e91d927e00c
8154343: Make SATB related code available to other GCs
kbarrett
parents:
51403
diff
changeset
|
26 |
#include "gc/shared/satbMarkQueue.hpp" |
30764 | 27 |
#include "gc/shared/collectedHeap.hpp" |
51393
cc8d309cd05a
8209345: Merge SATBMarkQueueFilter into SATBMarkQueueSet
kbarrett
parents:
51368
diff
changeset
|
28 |
#include "logging/log.hpp" |
7397 | 29 |
#include "memory/allocation.inline.hpp" |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
30 |
#include "oops/oop.inline.hpp" |
7397 | 31 |
#include "runtime/mutexLocker.hpp" |
51403
0f3bfea3acf1
8209347: SATBMarkQueue.cpp should not need jvm.h
kbarrett
parents:
51401
diff
changeset
|
32 |
#include "runtime/os.hpp" |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
33 |
#include "runtime/safepoint.hpp" |
7397 | 34 |
#include "runtime/thread.hpp" |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47765
diff
changeset
|
35 |
#include "runtime/threadSMR.hpp" |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
36 |
#include "runtime/vmThread.hpp" |
1374 | 37 |
|
33792 | 38 |
SATBMarkQueue::SATBMarkQueue(SATBMarkQueueSet* qset, bool permanent) : |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
39 |
// SATB queues are only active during marking cycles. We create |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
40 |
// them with their active field set to false. If a thread is |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
41 |
// created during a cycle and its SATB queue needs to be activated |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
42 |
// before the thread starts running, we'll need to set its active |
51401
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
43 |
// field to true. This must be done in the collector-specific |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
44 |
// BarrierSet::on_thread_attach() implementation. |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
45 |
PtrQueue(qset, permanent, false /* active */) |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
46 |
{ } |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
47 |
|
33792 | 48 |
void SATBMarkQueue::flush() { |
30255
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
49 |
// Filter now to possibly save work later. If filtering empties the |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
50 |
// buffer then flush_impl can deallocate the buffer. |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
51 |
filter(); |
28507 | 52 |
flush_impl(); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
53 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
54 |
|
51278
d56dd9798d54
8208611: Refactor SATBMarkQueue filtering to allow GC-specific filters
kbarrett
parents:
49752
diff
changeset
|
55 |
// This method will first apply filtering to the buffer. If filtering |
d56dd9798d54
8208611: Refactor SATBMarkQueue filtering to allow GC-specific filters
kbarrett
parents:
49752
diff
changeset
|
56 |
// retains a small enough collection in the buffer, we can continue to |
d56dd9798d54
8208611: Refactor SATBMarkQueue filtering to allow GC-specific filters
kbarrett
parents:
49752
diff
changeset
|
57 |
// use the buffer as-is, instead of enqueueing and replacing it. |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
58 |
|
33792 | 59 |
bool SATBMarkQueue::should_enqueue_buffer() { |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
60 |
assert(_lock == NULL || _lock->owned_by_self(), |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
61 |
"we should have taken the lock before calling this"); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
62 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
63 |
// This method should only be called if there is a non-NULL buffer |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
64 |
// that is full. |
46443 | 65 |
assert(index() == 0, "pre-condition"); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
66 |
assert(_buf != NULL, "pre-condition"); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
67 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
68 |
filter(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
69 |
|
51401
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
70 |
SATBMarkQueueSet* satb_qset = static_cast<SATBMarkQueueSet*>(qset()); |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
71 |
size_t threshold = satb_qset->buffer_enqueue_threshold(); |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
72 |
// Ensure we'll enqueue completely full buffers. |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
73 |
assert(threshold > 0, "enqueue threshold = 0"); |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
74 |
// Ensure we won't enqueue empty buffers. |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
75 |
assert(threshold <= capacity(), |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
76 |
"enqueue threshold " SIZE_FORMAT " exceeds capacity " SIZE_FORMAT, |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
77 |
threshold, capacity()); |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
78 |
return index() < threshold; |
7920 | 79 |
} |
80 |
||
33792 | 81 |
void SATBMarkQueue::apply_closure_and_empty(SATBBufferClosure* cl) { |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
82 |
assert(SafepointSynchronize::is_at_safepoint(), |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
83 |
"SATB queues must only be processed at safepoints"); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
84 |
if (_buf != NULL) { |
46443 | 85 |
cl->do_buffer(&_buf[index()], size()); |
86 |
reset(); |
|
1374 | 87 |
} |
88 |
} |
|
89 |
||
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
90 |
#ifndef PRODUCT |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
91 |
// Helpful for debugging |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
92 |
|
46443 | 93 |
static void print_satb_buffer(const char* name, |
94 |
void** buf, |
|
95 |
size_t index, |
|
96 |
size_t capacity) { |
|
97 |
tty->print_cr(" SATB BUFFER [%s] buf: " PTR_FORMAT " index: " SIZE_FORMAT |
|
98 |
" capacity: " SIZE_FORMAT, |
|
99 |
name, p2i(buf), index, capacity); |
|
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
100 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
101 |
|
46443 | 102 |
void SATBMarkQueue::print(const char* name) { |
103 |
print_satb_buffer(name, _buf, index(), capacity()); |
|
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
104 |
} |
46443 | 105 |
|
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
106 |
#endif // PRODUCT |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
107 |
|
1374 | 108 |
SATBMarkQueueSet::SATBMarkQueueSet() : |
30268
d28a9aa10804
8078021: SATB apply_closure_to_completed_buffer should have closure argument
kbarrett
parents:
30255
diff
changeset
|
109 |
PtrQueueSet(), |
51401
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
110 |
_shared_satb_queue(this, true /* permanent */), |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
111 |
_buffer_enqueue_threshold(0) |
51278
d56dd9798d54
8208611: Refactor SATBMarkQueue filtering to allow GC-specific filters
kbarrett
parents:
49752
diff
changeset
|
112 |
{} |
1374 | 113 |
|
51393
cc8d309cd05a
8209345: Merge SATBMarkQueueFilter into SATBMarkQueueSet
kbarrett
parents:
51368
diff
changeset
|
114 |
void SATBMarkQueueSet::initialize(Monitor* cbl_mon, Mutex* fl_lock, |
4481 | 115 |
int process_completed_threshold, |
51401
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
116 |
uint buffer_enqueue_threshold_percentage, |
1374 | 117 |
Mutex* lock) { |
4481 | 118 |
PtrQueueSet::initialize(cbl_mon, fl_lock, process_completed_threshold, -1); |
1374 | 119 |
_shared_satb_queue.set_lock(lock); |
51401
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
120 |
assert(buffer_size() != 0, "buffer size not initialized"); |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
121 |
// Minimum threshold of 1 ensures enqueuing of completely full buffers. |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
122 |
size_t size = buffer_size(); |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
123 |
size_t enqueue_qty = (size * buffer_enqueue_threshold_percentage) / 100; |
58113ce90caf
8209346: Refactor SATBMarkQueue filter configuration
kbarrett
parents:
51393
diff
changeset
|
124 |
_buffer_enqueue_threshold = MAX2(size - enqueue_qty, (size_t)1); |
1374 | 125 |
} |
126 |
||
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
127 |
#ifdef ASSERT |
22497 | 128 |
void SATBMarkQueueSet::dump_active_states(bool expected_active) { |
36090
cffc1dd98258
8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents:
35061
diff
changeset
|
129 |
log_error(gc, verify)("Expected SATB active state: %s", expected_active ? "ACTIVE" : "INACTIVE"); |
cffc1dd98258
8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents:
35061
diff
changeset
|
130 |
log_error(gc, verify)("Actual SATB active states:"); |
cffc1dd98258
8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents:
35061
diff
changeset
|
131 |
log_error(gc, verify)(" Queue set: %s", is_active() ? "ACTIVE" : "INACTIVE"); |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47765
diff
changeset
|
132 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *t = jtiwh.next(); ) { |
51368
adcb0bb3d1e9
8209118: Abstract SATBMarkQueueSet's ThreadLocalData access
rkennke
parents:
51278
diff
changeset
|
133 |
log_error(gc, verify)(" Thread \"%s\" queue: %s", t->name(), satb_queue_for_thread(t).is_active() ? "ACTIVE" : "INACTIVE"); |
22497 | 134 |
} |
36090
cffc1dd98258
8149541: Use log_error() instead of log_info() when verification reports a problem
brutisso
parents:
35061
diff
changeset
|
135 |
log_error(gc, verify)(" Shared queue: %s", shared_satb_queue()->is_active() ? "ACTIVE" : "INACTIVE"); |
22497 | 136 |
} |
137 |
||
138 |
void SATBMarkQueueSet::verify_active_states(bool expected_active) { |
|
139 |
// Verify queue set state |
|
140 |
if (is_active() != expected_active) { |
|
141 |
dump_active_states(expected_active); |
|
142 |
guarantee(false, "SATB queue set has an unexpected active state"); |
|
143 |
} |
|
144 |
||
145 |
// Verify thread queue states |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47765
diff
changeset
|
146 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *t = jtiwh.next(); ) { |
51368
adcb0bb3d1e9
8209118: Abstract SATBMarkQueueSet's ThreadLocalData access
rkennke
parents:
51278
diff
changeset
|
147 |
if (satb_queue_for_thread(t).is_active() != expected_active) { |
22497 | 148 |
dump_active_states(expected_active); |
149 |
guarantee(false, "Thread SATB queue has an unexpected active state"); |
|
150 |
} |
|
151 |
} |
|
152 |
||
153 |
// Verify shared queue state |
|
154 |
if (shared_satb_queue()->is_active() != expected_active) { |
|
155 |
dump_active_states(expected_active); |
|
156 |
guarantee(false, "Shared SATB queue has an unexpected active state"); |
|
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
157 |
} |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
158 |
} |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
159 |
#endif // ASSERT |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
160 |
|
22497 | 161 |
void SATBMarkQueueSet::set_active_all_threads(bool active, bool expected_active) { |
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
162 |
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
|
163 |
#ifdef ASSERT |
22497 | 164 |
verify_active_states(expected_active); |
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
165 |
#endif // ASSERT |
22497 | 166 |
_all_active = active; |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47765
diff
changeset
|
167 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *t = jtiwh.next(); ) { |
51368
adcb0bb3d1e9
8209118: Abstract SATBMarkQueueSet's ThreadLocalData access
rkennke
parents:
51278
diff
changeset
|
168 |
satb_queue_for_thread(t).set_active(active); |
1374 | 169 |
} |
22497 | 170 |
shared_satb_queue()->set_active(active); |
1374 | 171 |
} |
172 |
||
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
173 |
void SATBMarkQueueSet::filter_thread_buffers() { |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47765
diff
changeset
|
174 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *t = jtiwh.next(); ) { |
51368
adcb0bb3d1e9
8209118: Abstract SATBMarkQueueSet's ThreadLocalData access
rkennke
parents:
51278
diff
changeset
|
175 |
satb_queue_for_thread(t).filter(); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
176 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
177 |
shared_satb_queue()->filter(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
178 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
179 |
|
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
180 |
bool SATBMarkQueueSet::apply_closure_to_completed_buffer(SATBBufferClosure* cl) { |
4481 | 181 |
BufferNode* nd = NULL; |
1374 | 182 |
{ |
183 |
MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag); |
|
184 |
if (_completed_buffers_head != NULL) { |
|
185 |
nd = _completed_buffers_head; |
|
4481 | 186 |
_completed_buffers_head = nd->next(); |
1374 | 187 |
if (_completed_buffers_head == NULL) _completed_buffers_tail = NULL; |
188 |
_n_completed_buffers--; |
|
189 |
if (_n_completed_buffers == 0) _process_completed = false; |
|
190 |
} |
|
191 |
} |
|
192 |
if (nd != NULL) { |
|
4481 | 193 |
void **buf = BufferNode::make_buffer_from_node(nd); |
46443 | 194 |
size_t index = nd->index(); |
195 |
size_t size = buffer_size(); |
|
37065 | 196 |
assert(index <= size, "invariant"); |
197 |
cl->do_buffer(buf + index, size - index); |
|
198 |
deallocate_buffer(nd); |
|
1374 | 199 |
return true; |
200 |
} else { |
|
201 |
return false; |
|
202 |
} |
|
203 |
} |
|
204 |
||
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
205 |
#ifndef PRODUCT |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
206 |
// Helpful for debugging |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
207 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
208 |
#define SATB_PRINTER_BUFFER_SIZE 256 |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
209 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
210 |
void SATBMarkQueueSet::print_all(const char* msg) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
211 |
char buffer[SATB_PRINTER_BUFFER_SIZE]; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
212 |
assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint."); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
213 |
|
35061 | 214 |
tty->cr(); |
215 |
tty->print_cr("SATB BUFFERS [%s]", msg); |
|
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
216 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
217 |
BufferNode* nd = _completed_buffers_head; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
218 |
int i = 0; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
219 |
while (nd != NULL) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
220 |
void** buf = BufferNode::make_buffer_from_node(nd); |
51403
0f3bfea3acf1
8209347: SATBMarkQueue.cpp should not need jvm.h
kbarrett
parents:
51401
diff
changeset
|
221 |
os::snprintf(buffer, SATB_PRINTER_BUFFER_SIZE, "Enqueued: %d", i); |
46443 | 222 |
print_satb_buffer(buffer, buf, nd->index(), buffer_size()); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
223 |
nd = nd->next(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
224 |
i += 1; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
225 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
226 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47765
diff
changeset
|
227 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *t = jtiwh.next(); ) { |
51403
0f3bfea3acf1
8209347: SATBMarkQueue.cpp should not need jvm.h
kbarrett
parents:
51401
diff
changeset
|
228 |
os::snprintf(buffer, SATB_PRINTER_BUFFER_SIZE, "Thread: %s", t->name()); |
51368
adcb0bb3d1e9
8209118: Abstract SATBMarkQueueSet's ThreadLocalData access
rkennke
parents:
51278
diff
changeset
|
229 |
satb_queue_for_thread(t).print(buffer); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
230 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
231 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
232 |
shared_satb_queue()->print("Shared"); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
233 |
|
35061 | 234 |
tty->cr(); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
235 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
236 |
#endif // PRODUCT |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
237 |
|
1374 | 238 |
void SATBMarkQueueSet::abandon_partial_marking() { |
4481 | 239 |
BufferNode* buffers_to_delete = NULL; |
1374 | 240 |
{ |
241 |
MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag); |
|
242 |
while (_completed_buffers_head != NULL) { |
|
4481 | 243 |
BufferNode* nd = _completed_buffers_head; |
244 |
_completed_buffers_head = nd->next(); |
|
245 |
nd->set_next(buffers_to_delete); |
|
1374 | 246 |
buffers_to_delete = nd; |
247 |
} |
|
248 |
_completed_buffers_tail = NULL; |
|
249 |
_n_completed_buffers = 0; |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
250 |
DEBUG_ONLY(assert_completed_buffer_list_len_correct_locked()); |
1374 | 251 |
} |
252 |
while (buffers_to_delete != NULL) { |
|
4481 | 253 |
BufferNode* nd = buffers_to_delete; |
254 |
buffers_to_delete = nd->next(); |
|
37065 | 255 |
deallocate_buffer(nd); |
1374 | 256 |
} |
257 |
assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint."); |
|
258 |
// So we can safely manipulate these queues. |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47765
diff
changeset
|
259 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *t = jtiwh.next(); ) { |
51368
adcb0bb3d1e9
8209118: Abstract SATBMarkQueueSet's ThreadLocalData access
rkennke
parents:
51278
diff
changeset
|
260 |
satb_queue_for_thread(t).reset(); |
1374 | 261 |
} |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47765
diff
changeset
|
262 |
shared_satb_queue()->reset(); |
1374 | 263 |
} |