author | tschatzl |
Wed, 25 Nov 2015 14:43:29 +0100 | |
changeset 34300 | 6075c1e0e913 |
parent 34141 | 1030e4216817 |
child 35040 | d00805788fdd |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
30764 | 2 |
* Copyright (c) 2001, 2015, 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:
4481
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4481
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:
4481
diff
changeset
|
21 |
* questions. |
1374 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
30764 | 26 |
#include "gc/g1/dirtyCardQueue.hpp" |
27 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
|
28 |
#include "gc/g1/heapRegionRemSet.hpp" |
|
29 |
#include "gc/shared/workgroup.hpp" |
|
25351
7c198a690050
8044775: Improve usage of umbrella header atomic.inline.hpp.
goetz
parents:
24104
diff
changeset
|
30 |
#include "runtime/atomic.inline.hpp" |
7397 | 31 |
#include "runtime/mutexLocker.hpp" |
32 |
#include "runtime/safepoint.hpp" |
|
14583
d70ee55535f4
8003935: Simplify the needed includes for using Thread::current()
stefank
parents:
10565
diff
changeset
|
33 |
#include "runtime/thread.inline.hpp" |
1374 | 34 |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
35 |
DirtyCardQueue::DirtyCardQueue(DirtyCardQueueSet* qset, bool permanent) : |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
36 |
// Dirty card queues are always active, so we create them with their |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
37 |
// active field set to true. |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
38 |
PtrQueue(qset, permanent, true /* active */) |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
39 |
{ } |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
40 |
|
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
41 |
DirtyCardQueue::~DirtyCardQueue() { |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
42 |
if (!is_permanent()) { |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
43 |
flush(); |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
44 |
} |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
45 |
} |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
46 |
|
1374 | 47 |
bool DirtyCardQueue::apply_closure(CardTableEntryClosure* cl, |
48 |
bool consume, |
|
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
49 |
uint worker_i) { |
1374 | 50 |
bool res = true; |
51 |
if (_buf != NULL) { |
|
52 |
res = apply_closure_to_buffer(cl, _buf, _index, _sz, |
|
53 |
consume, |
|
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
54 |
worker_i); |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
55 |
if (res && consume) { |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
56 |
_index = _sz; |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
57 |
} |
1374 | 58 |
} |
59 |
return res; |
|
60 |
} |
|
61 |
||
62 |
bool DirtyCardQueue::apply_closure_to_buffer(CardTableEntryClosure* cl, |
|
63 |
void** buf, |
|
64 |
size_t index, size_t sz, |
|
65 |
bool consume, |
|
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
66 |
uint worker_i) { |
1374 | 67 |
if (cl == NULL) return true; |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
68 |
size_t limit = byte_index_to_index(sz); |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
69 |
for (size_t i = byte_index_to_index(index); i < limit; ++i) { |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
70 |
jbyte* card_ptr = static_cast<jbyte*>(buf[i]); |
1374 | 71 |
if (card_ptr != NULL) { |
72 |
// Set the entry to null, so we don't do it again (via the test |
|
73 |
// above) if we reconsider this buffer. |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
74 |
if (consume) { |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
75 |
buf[i] = NULL; |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
76 |
} |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
77 |
if (!cl->do_card_ptr(card_ptr, worker_i)) { |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
78 |
return false; |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
79 |
} |
1374 | 80 |
} |
81 |
} |
|
82 |
return true; |
|
83 |
} |
|
84 |
||
4481 | 85 |
DirtyCardQueueSet::DirtyCardQueueSet(bool notify_when_complete) : |
86 |
PtrQueueSet(notify_when_complete), |
|
24104
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
87 |
_mut_process_closure(NULL), |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
88 |
_shared_dirty_card_queue(this, true /* permanent */), |
1374 | 89 |
_free_ids(NULL), |
90 |
_processed_buffers_mut(0), _processed_buffers_rs_thread(0) |
|
91 |
{ |
|
92 |
_all_active = true; |
|
93 |
} |
|
94 |
||
2882
d508a8bac491
6841831: G1: assert(contains_reference(from),"We just added it!") fires
iveresov
parents:
2881
diff
changeset
|
95 |
// Determines how many mutator threads can process the buffers in parallel. |
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
96 |
uint DirtyCardQueueSet::num_par_ids() { |
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
97 |
return (uint)os::processor_count(); |
1374 | 98 |
} |
99 |
||
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
100 |
void DirtyCardQueueSet::initialize(CardTableEntryClosure* cl, |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
101 |
Monitor* cbl_mon, |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
102 |
Mutex* fl_lock, |
4481 | 103 |
int process_completed_threshold, |
1374 | 104 |
int max_completed_queue, |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
105 |
Mutex* lock, |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
106 |
DirtyCardQueueSet* fl_owner) { |
24104
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
107 |
_mut_process_closure = cl; |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
108 |
PtrQueueSet::initialize(cbl_mon, |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
109 |
fl_lock, |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
110 |
process_completed_threshold, |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
111 |
max_completed_queue, |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
112 |
fl_owner); |
3583
805584e16d0f
6864886: G1: rename -XX parameters related to update buffers
tonyp
parents:
2882
diff
changeset
|
113 |
set_buffer_size(G1UpdateBufferSize); |
1374 | 114 |
_shared_dirty_card_queue.set_lock(lock); |
115 |
_free_ids = new FreeIdSet((int) num_par_ids(), _cbl_mon); |
|
116 |
} |
|
117 |
||
118 |
void DirtyCardQueueSet::handle_zero_index_for_thread(JavaThread* t) { |
|
119 |
t->dirty_card_queue().handle_zero_index(); |
|
120 |
} |
|
121 |
||
122 |
bool DirtyCardQueueSet::mut_process_buffer(void** buf) { |
|
123 |
||
124 |
// Used to determine if we had already claimed a par_id |
|
125 |
// before entering this method. |
|
126 |
bool already_claimed = false; |
|
127 |
||
128 |
// We grab the current JavaThread. |
|
129 |
JavaThread* thread = JavaThread::current(); |
|
130 |
||
131 |
// We get the the number of any par_id that this thread |
|
132 |
// might have already claimed. |
|
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
133 |
uint worker_i = thread->get_claimed_par_id(); |
1374 | 134 |
|
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
135 |
// If worker_i is not UINT_MAX then the thread has already claimed |
1374 | 136 |
// a par_id. We make note of it using the already_claimed value |
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
137 |
if (worker_i != UINT_MAX) { |
1374 | 138 |
already_claimed = true; |
139 |
} else { |
|
140 |
||
141 |
// Otherwise we need to claim a par id |
|
142 |
worker_i = _free_ids->claim_par_id(); |
|
143 |
||
144 |
// And store the par_id value in the thread |
|
145 |
thread->set_claimed_par_id(worker_i); |
|
146 |
} |
|
147 |
||
148 |
bool b = false; |
|
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
149 |
if (worker_i != UINT_MAX) { |
24104
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
150 |
b = DirtyCardQueue::apply_closure_to_buffer(_mut_process_closure, buf, 0, |
1374 | 151 |
_sz, true, worker_i); |
152 |
if (b) Atomic::inc(&_processed_buffers_mut); |
|
153 |
||
154 |
// If we had not claimed an id before entering the method |
|
155 |
// then we must release the id. |
|
156 |
if (!already_claimed) { |
|
157 |
||
158 |
// we release the id |
|
159 |
_free_ids->release_par_id(worker_i); |
|
160 |
||
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
161 |
// and set the claimed_id in the thread to UINT_MAX |
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
162 |
thread->set_claimed_par_id(UINT_MAX); |
1374 | 163 |
} |
164 |
} |
|
165 |
return b; |
|
166 |
} |
|
167 |
||
4481 | 168 |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
169 |
BufferNode* DirtyCardQueueSet::get_completed_buffer(int stop_at) { |
4481 | 170 |
BufferNode* nd = NULL; |
1374 | 171 |
MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag); |
172 |
||
173 |
if ((int)_n_completed_buffers <= stop_at) { |
|
174 |
_process_completed = false; |
|
175 |
return NULL; |
|
176 |
} |
|
177 |
||
178 |
if (_completed_buffers_head != NULL) { |
|
179 |
nd = _completed_buffers_head; |
|
4481 | 180 |
_completed_buffers_head = nd->next(); |
1374 | 181 |
if (_completed_buffers_head == NULL) |
182 |
_completed_buffers_tail = NULL; |
|
183 |
_n_completed_buffers--; |
|
4481 | 184 |
assert(_n_completed_buffers >= 0, "Invariant"); |
1374 | 185 |
} |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
186 |
DEBUG_ONLY(assert_completed_buffer_list_len_correct_locked()); |
1374 | 187 |
return nd; |
188 |
} |
|
189 |
||
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
190 |
bool DirtyCardQueueSet::apply_closure_to_completed_buffer_helper(CardTableEntryClosure* cl, |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
191 |
uint worker_i, |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
192 |
BufferNode* nd) { |
1374 | 193 |
if (nd != NULL) { |
4481 | 194 |
void **buf = BufferNode::make_buffer_from_node(nd); |
195 |
size_t index = nd->index(); |
|
1374 | 196 |
bool b = |
6247 | 197 |
DirtyCardQueue::apply_closure_to_buffer(cl, buf, |
4481 | 198 |
index, _sz, |
1374 | 199 |
true, worker_i); |
200 |
if (b) { |
|
201 |
deallocate_buffer(buf); |
|
202 |
return true; // In normal case, go on to next buffer. |
|
203 |
} else { |
|
4481 | 204 |
enqueue_complete_buffer(buf, index); |
1374 | 205 |
return false; |
206 |
} |
|
207 |
} else { |
|
208 |
return false; |
|
209 |
} |
|
210 |
} |
|
211 |
||
6247 | 212 |
bool DirtyCardQueueSet::apply_closure_to_completed_buffer(CardTableEntryClosure* cl, |
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23543
diff
changeset
|
213 |
uint worker_i, |
6247 | 214 |
int stop_at, |
215 |
bool during_pause) { |
|
216 |
assert(!during_pause || stop_at == 0, "Should not leave any completed buffers during a pause"); |
|
217 |
BufferNode* nd = get_completed_buffer(stop_at); |
|
218 |
bool res = apply_closure_to_completed_buffer_helper(cl, worker_i, nd); |
|
219 |
if (res) Atomic::inc(&_processed_buffers_rs_thread); |
|
220 |
return res; |
|
221 |
} |
|
222 |
||
24104
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
223 |
void DirtyCardQueueSet::apply_closure_to_all_completed_buffers(CardTableEntryClosure* cl) { |
4481 | 224 |
BufferNode* nd = _completed_buffers_head; |
1374 | 225 |
while (nd != NULL) { |
226 |
bool b = |
|
24104
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
227 |
DirtyCardQueue::apply_closure_to_buffer(cl, |
4481 | 228 |
BufferNode::make_buffer_from_node(nd), |
229 |
0, _sz, false); |
|
1374 | 230 |
guarantee(b, "Should not stop early."); |
4481 | 231 |
nd = nd->next(); |
1374 | 232 |
} |
233 |
} |
|
234 |
||
24104
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
235 |
void DirtyCardQueueSet::par_apply_closure_to_all_completed_buffers(CardTableEntryClosure* cl) { |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
236 |
BufferNode* nd = _cur_par_buffer_node; |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
237 |
while (nd != NULL) { |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
238 |
BufferNode* next = (BufferNode*)nd->next(); |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
239 |
BufferNode* actual = (BufferNode*)Atomic::cmpxchg_ptr((void*)next, (volatile void*)&_cur_par_buffer_node, (void*)nd); |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
240 |
if (actual == nd) { |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
241 |
bool b = |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
242 |
DirtyCardQueue::apply_closure_to_buffer(cl, |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
243 |
BufferNode::make_buffer_from_node(actual), |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
244 |
0, _sz, false); |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
245 |
guarantee(b, "Should not stop early."); |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
246 |
nd = next; |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
247 |
} else { |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
248 |
nd = actual; |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
249 |
} |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
250 |
} |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
251 |
} |
febf9363fb68
8019342: G1: High "Other" time most likely due to card redirtying
tschatzl
parents:
23855
diff
changeset
|
252 |
|
6247 | 253 |
// Deallocates any completed log buffers |
254 |
void DirtyCardQueueSet::clear() { |
|
4481 | 255 |
BufferNode* buffers_to_delete = NULL; |
1374 | 256 |
{ |
257 |
MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag); |
|
258 |
while (_completed_buffers_head != NULL) { |
|
4481 | 259 |
BufferNode* nd = _completed_buffers_head; |
260 |
_completed_buffers_head = nd->next(); |
|
261 |
nd->set_next(buffers_to_delete); |
|
1374 | 262 |
buffers_to_delete = nd; |
263 |
} |
|
264 |
_n_completed_buffers = 0; |
|
265 |
_completed_buffers_tail = NULL; |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
266 |
DEBUG_ONLY(assert_completed_buffer_list_len_correct_locked()); |
1374 | 267 |
} |
268 |
while (buffers_to_delete != NULL) { |
|
4481 | 269 |
BufferNode* nd = buffers_to_delete; |
270 |
buffers_to_delete = nd->next(); |
|
271 |
deallocate_buffer(BufferNode::make_buffer_from_node(nd)); |
|
1374 | 272 |
} |
6247 | 273 |
|
274 |
} |
|
275 |
||
276 |
void DirtyCardQueueSet::abandon_logs() { |
|
277 |
assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint."); |
|
278 |
clear(); |
|
1374 | 279 |
// Since abandon is done only at safepoints, we can safely manipulate |
280 |
// these queues. |
|
281 |
for (JavaThread* t = Threads::first(); t; t = t->next()) { |
|
282 |
t->dirty_card_queue().reset(); |
|
283 |
} |
|
284 |
shared_dirty_card_queue()->reset(); |
|
285 |
} |
|
286 |
||
287 |
||
288 |
void DirtyCardQueueSet::concatenate_logs() { |
|
289 |
// Iterate over all the threads, if we find a partial log add it to |
|
290 |
// the global list of logs. Temporarily turn off the limit on the number |
|
291 |
// of outstanding buffers. |
|
292 |
int save_max_completed_queue = _max_completed_queue; |
|
293 |
_max_completed_queue = max_jint; |
|
294 |
assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint."); |
|
295 |
for (JavaThread* t = Threads::first(); t; t = t->next()) { |
|
296 |
DirtyCardQueue& dcq = t->dirty_card_queue(); |
|
297 |
if (dcq.size() != 0) { |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
298 |
void** buf = dcq.get_buf(); |
1374 | 299 |
// We must NULL out the unused entries, then enqueue. |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
300 |
size_t limit = dcq.byte_index_to_index(dcq.get_index()); |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
301 |
for (size_t i = 0; i < limit; ++i) { |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
30764
diff
changeset
|
302 |
buf[i] = NULL; |
1374 | 303 |
} |
304 |
enqueue_complete_buffer(dcq.get_buf(), dcq.get_index()); |
|
305 |
dcq.reinitialize(); |
|
306 |
} |
|
307 |
} |
|
308 |
if (_shared_dirty_card_queue.size() != 0) { |
|
309 |
enqueue_complete_buffer(_shared_dirty_card_queue.get_buf(), |
|
310 |
_shared_dirty_card_queue.get_index()); |
|
311 |
_shared_dirty_card_queue.reinitialize(); |
|
312 |
} |
|
313 |
// Restore the completed buffer queue limit. |
|
314 |
_max_completed_queue = save_max_completed_queue; |
|
315 |
} |