author | katleman |
Thu, 16 Oct 2014 12:01:59 -0700 | |
changeset 27002 | 297d6a75d68a |
parent 26160 | aba6b01cb988 |
child 27687 | 3a6367d7110b |
child 27880 | afb974a04396 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
2 |
* Copyright (c) 2000, 2014, Oracle and/or its affiliates. All rights reserved. |
1 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4461
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4461
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:
4461
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "memory/allocation.inline.hpp" |
|
27 |
#include "memory/cardTableModRefBS.hpp" |
|
28 |
#include "memory/cardTableRS.hpp" |
|
29 |
#include "memory/sharedHeap.hpp" |
|
30 |
#include "memory/space.hpp" |
|
31 |
#include "memory/space.inline.hpp" |
|
32 |
#include "memory/universe.hpp" |
|
33 |
#include "runtime/java.hpp" |
|
34 |
#include "runtime/mutexLocker.hpp" |
|
35 |
#include "runtime/virtualspace.hpp" |
|
13195 | 36 |
#include "services/memTracker.hpp" |
15482
470d0b0c09f1
8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents:
13963
diff
changeset
|
37 |
#include "utilities/macros.hpp" |
7397 | 38 |
#ifdef COMPILER1 |
39 |
#include "c1/c1_LIR.hpp" |
|
40 |
#include "c1/c1_LIRGenerator.hpp" |
|
41 |
#endif |
|
42 |
||
1 | 43 |
// This kind of "BarrierSet" allows a "CollectedHeap" to detect and |
44 |
// enumerate ref fields that have been modified (since the last |
|
45 |
// enumeration.) |
|
46 |
||
47 |
size_t CardTableModRefBS::compute_byte_map_size() |
|
48 |
{ |
|
49 |
assert(_guard_index == cards_required(_whole_heap.word_size()) - 1, |
|
22775 | 50 |
"uninitialized, check declaration order"); |
51 |
assert(_page_size != 0, "uninitialized, check declaration order"); |
|
1 | 52 |
const size_t granularity = os::vm_allocation_granularity(); |
53 |
return align_size_up(_guard_index + 1, MAX2(_page_size, granularity)); |
|
54 |
} |
|
55 |
||
56 |
CardTableModRefBS::CardTableModRefBS(MemRegion whole_heap, |
|
57 |
int max_covered_regions): |
|
58 |
ModRefBarrierSet(max_covered_regions), |
|
59 |
_whole_heap(whole_heap), |
|
26160 | 60 |
_guard_index(0), |
61 |
_guard_region(), |
|
62 |
_last_valid_index(0), |
|
194 | 63 |
_page_size(os::vm_page_size()), |
26160 | 64 |
_byte_map_size(0), |
65 |
_covered(NULL), |
|
66 |
_committed(NULL), |
|
67 |
_cur_covered_regions(0), |
|
68 |
_byte_map(NULL), |
|
69 |
byte_map_base(NULL), |
|
70 |
// LNC functionality |
|
71 |
_lowest_non_clean(NULL), |
|
72 |
_lowest_non_clean_chunk_size(NULL), |
|
73 |
_lowest_non_clean_base_chunk_index(NULL), |
|
74 |
_last_LNC_resizing_collection(NULL) |
|
1 | 75 |
{ |
76 |
_kind = BarrierSet::CardTableModRef; |
|
77 |
||
26160 | 78 |
assert((uintptr_t(_whole_heap.start()) & (card_size - 1)) == 0, "heap must start at card boundary"); |
79 |
assert((uintptr_t(_whole_heap.end()) & (card_size - 1)) == 0, "heap must end at card boundary"); |
|
80 |
||
81 |
assert(card_size <= 512, "card_size must be less than 512"); // why? |
|
82 |
||
83 |
_covered = new MemRegion[_max_covered_regions]; |
|
84 |
if (_covered == NULL) { |
|
85 |
vm_exit_during_initialization("Could not allocate card table covered region set."); |
|
86 |
} |
|
87 |
} |
|
88 |
||
89 |
void CardTableModRefBS::initialize() { |
|
90 |
_guard_index = cards_required(_whole_heap.word_size()) - 1; |
|
91 |
_last_valid_index = _guard_index - 1; |
|
92 |
||
93 |
_byte_map_size = compute_byte_map_size(); |
|
94 |
||
1 | 95 |
HeapWord* low_bound = _whole_heap.start(); |
96 |
HeapWord* high_bound = _whole_heap.end(); |
|
97 |
||
26160 | 98 |
_cur_covered_regions = 0; |
99 |
_committed = new MemRegion[_max_covered_regions]; |
|
100 |
if (_committed == NULL) { |
|
101 |
vm_exit_during_initialization("Could not allocate card table committed region set."); |
|
1 | 102 |
} |
17376
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
103 |
|
1 | 104 |
const size_t rs_align = _page_size == (size_t) os::vm_page_size() ? 0 : |
105 |
MAX2(_page_size, (size_t) os::vm_allocation_granularity()); |
|
106 |
ReservedSpace heap_rs(_byte_map_size, rs_align, false); |
|
13195 | 107 |
|
108 |
MemTracker::record_virtual_memory_type((address)heap_rs.base(), mtGC); |
|
109 |
||
1 | 110 |
os::trace_page_sizes("card table", _guard_index + 1, _guard_index + 1, |
111 |
_page_size, heap_rs.base(), heap_rs.size()); |
|
112 |
if (!heap_rs.is_reserved()) { |
|
113 |
vm_exit_during_initialization("Could not reserve enough space for the " |
|
114 |
"card marking array"); |
|
115 |
} |
|
116 |
||
22551 | 117 |
// The assembler store_check code will do an unsigned shift of the oop, |
1 | 118 |
// then add it to byte_map_base, i.e. |
119 |
// |
|
120 |
// _byte_map = byte_map_base + (uintptr_t(low_bound) >> card_shift) |
|
121 |
_byte_map = (jbyte*) heap_rs.base(); |
|
122 |
byte_map_base = _byte_map - (uintptr_t(low_bound) >> card_shift); |
|
123 |
assert(byte_for(low_bound) == &_byte_map[0], "Checking start of map"); |
|
124 |
assert(byte_for(high_bound-1) <= &_byte_map[_last_valid_index], "Checking end of map"); |
|
125 |
||
126 |
jbyte* guard_card = &_byte_map[_guard_index]; |
|
127 |
uintptr_t guard_page = align_size_down((uintptr_t)guard_card, _page_size); |
|
128 |
_guard_region = MemRegion((HeapWord*)guard_page, _page_size); |
|
18069
e6d4971c8650
8013057: assert(_needs_gc || SafepointSynchronize::is_at_safepoint()) failed: only read at safepoint
dcubed
parents:
17376
diff
changeset
|
129 |
os::commit_memory_or_exit((char*)guard_page, _page_size, _page_size, |
e6d4971c8650
8013057: assert(_needs_gc || SafepointSynchronize::is_at_safepoint()) failed: only read at safepoint
dcubed
parents:
17376
diff
changeset
|
130 |
!ExecMem, "card table last card"); |
1 | 131 |
*guard_card = last_card; |
132 |
||
26160 | 133 |
_lowest_non_clean = |
134 |
NEW_C_HEAP_ARRAY(CardArr, _max_covered_regions, mtGC); |
|
1 | 135 |
_lowest_non_clean_chunk_size = |
26160 | 136 |
NEW_C_HEAP_ARRAY(size_t, _max_covered_regions, mtGC); |
1 | 137 |
_lowest_non_clean_base_chunk_index = |
26160 | 138 |
NEW_C_HEAP_ARRAY(uintptr_t, _max_covered_regions, mtGC); |
1 | 139 |
_last_LNC_resizing_collection = |
26160 | 140 |
NEW_C_HEAP_ARRAY(int, _max_covered_regions, mtGC); |
1 | 141 |
if (_lowest_non_clean == NULL |
142 |
|| _lowest_non_clean_chunk_size == NULL |
|
143 |
|| _lowest_non_clean_base_chunk_index == NULL |
|
144 |
|| _last_LNC_resizing_collection == NULL) |
|
145 |
vm_exit_during_initialization("couldn't allocate an LNC array."); |
|
26160 | 146 |
for (int i = 0; i < _max_covered_regions; i++) { |
1 | 147 |
_lowest_non_clean[i] = NULL; |
148 |
_lowest_non_clean_chunk_size[i] = 0; |
|
149 |
_last_LNC_resizing_collection[i] = -1; |
|
150 |
} |
|
151 |
||
152 |
if (TraceCardTableModRefBS) { |
|
153 |
gclog_or_tty->print_cr("CardTableModRefBS::CardTableModRefBS: "); |
|
154 |
gclog_or_tty->print_cr(" " |
|
155 |
" &_byte_map[0]: " INTPTR_FORMAT |
|
156 |
" &_byte_map[_last_valid_index]: " INTPTR_FORMAT, |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
157 |
p2i(&_byte_map[0]), |
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
158 |
p2i(&_byte_map[_last_valid_index])); |
1 | 159 |
gclog_or_tty->print_cr(" " |
160 |
" byte_map_base: " INTPTR_FORMAT, |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
161 |
p2i(byte_map_base)); |
1 | 162 |
} |
163 |
} |
|
164 |
||
17376
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
165 |
CardTableModRefBS::~CardTableModRefBS() { |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
166 |
if (_covered) { |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
167 |
delete[] _covered; |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
168 |
_covered = NULL; |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
169 |
} |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
170 |
if (_committed) { |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
171 |
delete[] _committed; |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
172 |
_committed = NULL; |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
173 |
} |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
174 |
if (_lowest_non_clean) { |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
175 |
FREE_C_HEAP_ARRAY(CardArr, _lowest_non_clean, mtGC); |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
176 |
_lowest_non_clean = NULL; |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
177 |
} |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
178 |
if (_lowest_non_clean_chunk_size) { |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
179 |
FREE_C_HEAP_ARRAY(size_t, _lowest_non_clean_chunk_size, mtGC); |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
180 |
_lowest_non_clean_chunk_size = NULL; |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
181 |
} |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
182 |
if (_lowest_non_clean_base_chunk_index) { |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
183 |
FREE_C_HEAP_ARRAY(uintptr_t, _lowest_non_clean_base_chunk_index, mtGC); |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
184 |
_lowest_non_clean_base_chunk_index = NULL; |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
185 |
} |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
186 |
if (_last_LNC_resizing_collection) { |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
187 |
FREE_C_HEAP_ARRAY(int, _last_LNC_resizing_collection, mtGC); |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
188 |
_last_LNC_resizing_collection = NULL; |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
189 |
} |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
190 |
} |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17087
diff
changeset
|
191 |
|
1 | 192 |
int CardTableModRefBS::find_covering_region_by_base(HeapWord* base) { |
193 |
int i; |
|
194 |
for (i = 0; i < _cur_covered_regions; i++) { |
|
195 |
if (_covered[i].start() == base) return i; |
|
196 |
if (_covered[i].start() > base) break; |
|
197 |
} |
|
198 |
// If we didn't find it, create a new one. |
|
199 |
assert(_cur_covered_regions < _max_covered_regions, |
|
200 |
"too many covered regions"); |
|
201 |
// Move the ones above up, to maintain sorted order. |
|
202 |
for (int j = _cur_covered_regions; j > i; j--) { |
|
203 |
_covered[j] = _covered[j-1]; |
|
204 |
_committed[j] = _committed[j-1]; |
|
205 |
} |
|
206 |
int res = i; |
|
207 |
_cur_covered_regions++; |
|
208 |
_covered[res].set_start(base); |
|
209 |
_covered[res].set_word_size(0); |
|
210 |
jbyte* ct_start = byte_for(base); |
|
211 |
uintptr_t ct_start_aligned = align_size_down((uintptr_t)ct_start, _page_size); |
|
212 |
_committed[res].set_start((HeapWord*)ct_start_aligned); |
|
213 |
_committed[res].set_word_size(0); |
|
214 |
return res; |
|
215 |
} |
|
216 |
||
217 |
int CardTableModRefBS::find_covering_region_containing(HeapWord* addr) { |
|
218 |
for (int i = 0; i < _cur_covered_regions; i++) { |
|
219 |
if (_covered[i].contains(addr)) { |
|
220 |
return i; |
|
221 |
} |
|
222 |
} |
|
223 |
assert(0, "address outside of heap?"); |
|
224 |
return -1; |
|
225 |
} |
|
226 |
||
227 |
HeapWord* CardTableModRefBS::largest_prev_committed_end(int ind) const { |
|
228 |
HeapWord* max_end = NULL; |
|
229 |
for (int j = 0; j < ind; j++) { |
|
230 |
HeapWord* this_end = _committed[j].end(); |
|
231 |
if (this_end > max_end) max_end = this_end; |
|
232 |
} |
|
233 |
return max_end; |
|
234 |
} |
|
235 |
||
236 |
MemRegion CardTableModRefBS::committed_unique_to_self(int self, |
|
237 |
MemRegion mr) const { |
|
238 |
MemRegion result = mr; |
|
239 |
for (int r = 0; r < _cur_covered_regions; r += 1) { |
|
240 |
if (r != self) { |
|
241 |
result = result.minus(_committed[r]); |
|
242 |
} |
|
243 |
} |
|
244 |
// Never include the guard page. |
|
245 |
result = result.minus(_guard_region); |
|
246 |
return result; |
|
247 |
} |
|
248 |
||
249 |
void CardTableModRefBS::resize_covered_region(MemRegion new_region) { |
|
250 |
// We don't change the start of a region, only the end. |
|
251 |
assert(_whole_heap.contains(new_region), |
|
252 |
"attempt to cover area not in reserved area"); |
|
253 |
debug_only(verify_guard();) |
|
754
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
254 |
// collided is true if the expansion would push into another committed region |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
255 |
debug_only(bool collided = false;) |
179
59e3abf83f72
6624765: Guarantee failure "Unexpected dirty card found"
jmasa
parents:
1
diff
changeset
|
256 |
int const ind = find_covering_region_by_base(new_region.start()); |
59e3abf83f72
6624765: Guarantee failure "Unexpected dirty card found"
jmasa
parents:
1
diff
changeset
|
257 |
MemRegion const old_region = _covered[ind]; |
1 | 258 |
assert(old_region.start() == new_region.start(), "just checking"); |
259 |
if (new_region.word_size() != old_region.word_size()) { |
|
260 |
// Commit new or uncommit old pages, if necessary. |
|
261 |
MemRegion cur_committed = _committed[ind]; |
|
22551 | 262 |
// Extend the end of this _committed region |
1 | 263 |
// to cover the end of any lower _committed regions. |
264 |
// This forms overlapping regions, but never interior regions. |
|
179
59e3abf83f72
6624765: Guarantee failure "Unexpected dirty card found"
jmasa
parents:
1
diff
changeset
|
265 |
HeapWord* const max_prev_end = largest_prev_committed_end(ind); |
1 | 266 |
if (max_prev_end > cur_committed.end()) { |
267 |
cur_committed.set_end(max_prev_end); |
|
268 |
} |
|
269 |
// Align the end up to a page size (starts are already aligned). |
|
179
59e3abf83f72
6624765: Guarantee failure "Unexpected dirty card found"
jmasa
parents:
1
diff
changeset
|
270 |
jbyte* const new_end = byte_after(new_region.last()); |
754
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
271 |
HeapWord* new_end_aligned = |
179
59e3abf83f72
6624765: Guarantee failure "Unexpected dirty card found"
jmasa
parents:
1
diff
changeset
|
272 |
(HeapWord*) align_size_up((uintptr_t)new_end, _page_size); |
1 | 273 |
assert(new_end_aligned >= (HeapWord*) new_end, |
274 |
"align up, but less"); |
|
2107
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
275 |
// Check the other regions (excludes "ind") to ensure that |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
276 |
// the new_end_aligned does not intrude onto the committed |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
277 |
// space of another region. |
754
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
278 |
int ri = 0; |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
279 |
for (ri = 0; ri < _cur_covered_regions; ri++) { |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
280 |
if (ri != ind) { |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
281 |
if (_committed[ri].contains(new_end_aligned)) { |
2107
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
282 |
// The prior check included in the assert |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
283 |
// (new_end_aligned >= _committed[ri].start()) |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
284 |
// is redundant with the "contains" test. |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
285 |
// Any region containing the new end |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
286 |
// should start at or beyond the region found (ind) |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
287 |
// for the new end (committed regions are not expected to |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
288 |
// be proper subsets of other committed regions). |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
289 |
assert(_committed[ri].start() >= _committed[ind].start(), |
754
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
290 |
"New end of committed region is inconsistent"); |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
291 |
new_end_aligned = _committed[ri].start(); |
2107
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
292 |
// new_end_aligned can be equal to the start of its |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
293 |
// committed region (i.e., of "ind") if a second |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
294 |
// region following "ind" also start at the same location |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
295 |
// as "ind". |
338528868274
6786346: intermittent Internal Error (src/share/vm/memory/cardTableModRefBS.cpp:226)
jmasa
parents:
1676
diff
changeset
|
296 |
assert(new_end_aligned >= _committed[ind].start(), |
754
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
297 |
"New end of committed region is before start"); |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
298 |
debug_only(collided = true;) |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
299 |
// Should only collide with 1 region |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
300 |
break; |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
301 |
} |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
302 |
} |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
303 |
} |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
304 |
#ifdef ASSERT |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
305 |
for (++ri; ri < _cur_covered_regions; ri++) { |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
306 |
assert(!_committed[ri].contains(new_end_aligned), |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
307 |
"New end of committed region is in a second committed region"); |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
308 |
} |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
309 |
#endif |
1 | 310 |
// The guard page is always committed and should not be committed over. |
3587
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
311 |
// "guarded" is used for assertion checking below and recalls the fact |
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
312 |
// that the would-be end of the new committed region would have |
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
313 |
// penetrated the guard page. |
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
314 |
HeapWord* new_end_for_commit = new_end_aligned; |
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
315 |
|
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
316 |
DEBUG_ONLY(bool guarded = false;) |
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
317 |
if (new_end_for_commit > _guard_region.start()) { |
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
318 |
new_end_for_commit = _guard_region.start(); |
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
319 |
DEBUG_ONLY(guarded = true;) |
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
320 |
} |
754
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
321 |
|
1 | 322 |
if (new_end_for_commit > cur_committed.end()) { |
323 |
// Must commit new pages. |
|
179
59e3abf83f72
6624765: Guarantee failure "Unexpected dirty card found"
jmasa
parents:
1
diff
changeset
|
324 |
MemRegion const new_committed = |
1 | 325 |
MemRegion(cur_committed.end(), new_end_for_commit); |
326 |
||
327 |
assert(!new_committed.is_empty(), "Region should not be empty here"); |
|
18069
e6d4971c8650
8013057: assert(_needs_gc || SafepointSynchronize::is_at_safepoint()) failed: only read at safepoint
dcubed
parents:
17376
diff
changeset
|
328 |
os::commit_memory_or_exit((char*)new_committed.start(), |
e6d4971c8650
8013057: assert(_needs_gc || SafepointSynchronize::is_at_safepoint()) failed: only read at safepoint
dcubed
parents:
17376
diff
changeset
|
329 |
new_committed.byte_size(), _page_size, |
e6d4971c8650
8013057: assert(_needs_gc || SafepointSynchronize::is_at_safepoint()) failed: only read at safepoint
dcubed
parents:
17376
diff
changeset
|
330 |
!ExecMem, "card table expansion"); |
1 | 331 |
// Use new_end_aligned (as opposed to new_end_for_commit) because |
332 |
// the cur_committed region may include the guard region. |
|
333 |
} else if (new_end_aligned < cur_committed.end()) { |
|
334 |
// Must uncommit pages. |
|
179
59e3abf83f72
6624765: Guarantee failure "Unexpected dirty card found"
jmasa
parents:
1
diff
changeset
|
335 |
MemRegion const uncommit_region = |
1 | 336 |
committed_unique_to_self(ind, MemRegion(new_end_aligned, |
337 |
cur_committed.end())); |
|
338 |
if (!uncommit_region.is_empty()) { |
|
5892
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
339 |
// It is not safe to uncommit cards if the boundary between |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
340 |
// the generations is moving. A shrink can uncommit cards |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
341 |
// owned by generation A but being used by generation B. |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
342 |
if (!UseAdaptiveGCBoundary) { |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
343 |
if (!os::uncommit_memory((char*)uncommit_region.start(), |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
344 |
uncommit_region.byte_size())) { |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
345 |
assert(false, "Card table contraction failed"); |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
346 |
// The call failed so don't change the end of the |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
347 |
// committed region. This is better than taking the |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
348 |
// VM down. |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
349 |
new_end_aligned = _committed[ind].end(); |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
350 |
} |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
351 |
} else { |
754
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
352 |
new_end_aligned = _committed[ind].end(); |
1 | 353 |
} |
354 |
} |
|
355 |
} |
|
356 |
// In any case, we can reset the end of the current committed entry. |
|
357 |
_committed[ind].set_end(new_end_aligned); |
|
358 |
||
5892
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
359 |
#ifdef ASSERT |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
360 |
// Check that the last card in the new region is committed according |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
361 |
// to the tables. |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
362 |
bool covered = false; |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
363 |
for (int cr = 0; cr < _cur_covered_regions; cr++) { |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
364 |
if (_committed[cr].contains(new_end - 1)) { |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
365 |
covered = true; |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
366 |
break; |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
367 |
} |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
368 |
} |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
369 |
assert(covered, "Card for end of new region not committed"); |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
370 |
#endif |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
371 |
|
1 | 372 |
// The default of 0 is not necessarily clean cards. |
373 |
jbyte* entry; |
|
374 |
if (old_region.last() < _whole_heap.start()) { |
|
375 |
entry = byte_for(_whole_heap.start()); |
|
376 |
} else { |
|
377 |
entry = byte_after(old_region.last()); |
|
378 |
} |
|
1676
d80e69372634
6653214: MemoryPoolMXBean.setUsageThreshold() does not support large heap sizes.
swamyv
parents:
1388
diff
changeset
|
379 |
assert(index_for(new_region.last()) < _guard_index, |
1 | 380 |
"The guard card will be overwritten"); |
754
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
381 |
// This line commented out cleans the newly expanded region and |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
382 |
// not the aligned up expanded region. |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
383 |
// jbyte* const end = byte_after(new_region.last()); |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
384 |
jbyte* const end = (jbyte*) new_end_for_commit; |
3587
0140e816d000
6843292: "Expect to be beyond new region unless impacting another region" assertion too strong
jmasa
parents:
2154
diff
changeset
|
385 |
assert((end >= byte_after(new_region.last())) || collided || guarded, |
754
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
386 |
"Expect to be beyond new region unless impacting another region"); |
1 | 387 |
// do nothing if we resized downward. |
754
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
388 |
#ifdef ASSERT |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
389 |
for (int ri = 0; ri < _cur_covered_regions; ri++) { |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
390 |
if (ri != ind) { |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
391 |
// The end of the new committed region should not |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
392 |
// be in any existing region unless it matches |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
393 |
// the start of the next region. |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
394 |
assert(!_committed[ri].contains(end) || |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
395 |
(_committed[ri].start() == (HeapWord*) end), |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
396 |
"Overlapping committed regions"); |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
397 |
} |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
398 |
} |
fb9904179b42
6688799: Second fix for Guarantee failure "Unexpected dirty card found"
jmasa
parents:
360
diff
changeset
|
399 |
#endif |
1 | 400 |
if (entry < end) { |
401 |
memset(entry, clean_card, pointer_delta(end, entry, sizeof(jbyte))); |
|
402 |
} |
|
403 |
} |
|
404 |
// In any case, the covered size changes. |
|
405 |
_covered[ind].set_word_size(new_region.word_size()); |
|
406 |
if (TraceCardTableModRefBS) { |
|
407 |
gclog_or_tty->print_cr("CardTableModRefBS::resize_covered_region: "); |
|
408 |
gclog_or_tty->print_cr(" " |
|
409 |
" _covered[%d].start(): " INTPTR_FORMAT |
|
410 |
" _covered[%d].last(): " INTPTR_FORMAT, |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
411 |
ind, p2i(_covered[ind].start()), |
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
412 |
ind, p2i(_covered[ind].last())); |
1 | 413 |
gclog_or_tty->print_cr(" " |
414 |
" _committed[%d].start(): " INTPTR_FORMAT |
|
415 |
" _committed[%d].last(): " INTPTR_FORMAT, |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
416 |
ind, p2i(_committed[ind].start()), |
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
417 |
ind, p2i(_committed[ind].last())); |
1 | 418 |
gclog_or_tty->print_cr(" " |
419 |
" byte_for(start): " INTPTR_FORMAT |
|
420 |
" byte_for(last): " INTPTR_FORMAT, |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
421 |
p2i(byte_for(_covered[ind].start())), |
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
422 |
p2i(byte_for(_covered[ind].last()))); |
1 | 423 |
gclog_or_tty->print_cr(" " |
424 |
" addr_for(start): " INTPTR_FORMAT |
|
425 |
" addr_for(last): " INTPTR_FORMAT, |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
426 |
p2i(addr_for((jbyte*) _committed[ind].start())), |
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
427 |
p2i(addr_for((jbyte*) _committed[ind].last()))); |
1 | 428 |
} |
5892
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
429 |
// Touch the last card of the covered region to show that it |
477b32b9d021
6952853: SIGSEGV with UseAdaptiveGCBoundary on 64b linux running jvm2008
jmasa
parents:
5547
diff
changeset
|
430 |
// is committed (or SEGV). |
18073
f02460441ddc
8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
ccheung
parents:
18069
diff
changeset
|
431 |
debug_only((void) (*byte_for(_covered[ind].last()));) |
1 | 432 |
debug_only(verify_guard();) |
433 |
} |
|
434 |
||
435 |
// Note that these versions are precise! The scanning code has to handle the |
|
436 |
// fact that the write barrier may be either precise or imprecise. |
|
437 |
||
22859
7b88983393b7
8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents:
20309
diff
changeset
|
438 |
void CardTableModRefBS::write_ref_field_work(void* field, oop newVal, bool release) { |
7b88983393b7
8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents:
20309
diff
changeset
|
439 |
inline_write_ref_field(field, newVal, release); |
1 | 440 |
} |
441 |
||
2142
032f4652700c
6720309: G1: don't synchronously update RSet during evacuation pauses
iveresov
parents:
2107
diff
changeset
|
442 |
|
9336
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
443 |
void CardTableModRefBS::non_clean_card_iterate_possibly_parallel(Space* sp, |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
444 |
MemRegion mr, |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
445 |
OopsInGenClosure* cl, |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
446 |
CardTableRS* ct) { |
1 | 447 |
if (!mr.is_empty()) { |
25492
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
448 |
// Caller (process_roots()) claims that all GC threads |
11174
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
449 |
// execute this call. With UseDynamicNumberOfGCThreads now all |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
450 |
// active GC threads execute this call. The number of active GC |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
451 |
// threads needs to be passed to par_non_clean_card_iterate_work() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
452 |
// to get proper partitioning and termination. |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
453 |
// |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
454 |
// This is an example of where n_par_threads() is used instead |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
455 |
// of workers()->active_workers(). n_par_threads can be set to 0 to |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
456 |
// turn off parallelism. For example when this code is called as |
25492
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
457 |
// part of verification and SharedHeap::process_roots() is being |
11174
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
458 |
// used, then n_par_threads() may have been set to 0. active_workers |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
459 |
// is not overloaded with the meaning that it is a switch to disable |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
460 |
// parallelism and so keeps the meaning of the number of |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
461 |
// active gc workers. If parallelism has not been shut off by |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
462 |
// setting n_par_threads to 0, then n_par_threads should be |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
463 |
// equal to active_workers. When a different mechanism for shutting |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
464 |
// off parallelism is used, then active_workers can be used in |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
465 |
// place of n_par_threads. |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
466 |
// This is an example of a path where n_par_threads is |
22551 | 467 |
// set to 0 to turn off parallelism. |
11174
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
468 |
// [7] CardTableModRefBS::non_clean_card_iterate() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
469 |
// [8] CardTableRS::younger_refs_in_space_iterate() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
470 |
// [9] Generation::younger_refs_in_space_iterate() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
471 |
// [10] OneContigSpaceCardGeneration::younger_refs_iterate() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
472 |
// [11] CompactingPermGenGen::younger_refs_iterate() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
473 |
// [12] CardTableRS::younger_refs_iterate() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
474 |
// [13] SharedHeap::process_strong_roots() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
475 |
// [14] G1CollectedHeap::verify() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
476 |
// [15] Universe::verify() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
477 |
// [16] G1CollectedHeap::do_collection_pause_at_safepoint() |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
478 |
// |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
479 |
int n_threads = SharedHeap::heap()->n_par_threads(); |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
480 |
bool is_par = n_threads > 0; |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
481 |
if (is_par) { |
15482
470d0b0c09f1
8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents:
13963
diff
changeset
|
482 |
#if INCLUDE_ALL_GCS |
11174
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
483 |
assert(SharedHeap::heap()->n_par_threads() == |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
484 |
SharedHeap::heap()->workers()->active_workers(), "Mismatch"); |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
485 |
non_clean_card_iterate_parallel_work(sp, mr, cl, ct, n_threads); |
15482
470d0b0c09f1
8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents:
13963
diff
changeset
|
486 |
#else // INCLUDE_ALL_GCS |
1 | 487 |
fatal("Parallel gc not supported here."); |
15482
470d0b0c09f1
8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents:
13963
diff
changeset
|
488 |
#endif // INCLUDE_ALL_GCS |
1 | 489 |
} else { |
9336
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
490 |
// We do not call the non_clean_card_iterate_serial() version below because |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
491 |
// we want to clear the cards (which non_clean_card_iterate_serial() does not |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
492 |
// do for us): clear_cl here does the work of finding contiguous dirty ranges |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
493 |
// of cards to process and clear. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
494 |
|
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
495 |
DirtyCardToOopClosure* dcto_cl = sp->new_dcto_cl(cl, precision(), |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
496 |
cl->gen_boundary()); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
497 |
ClearNoncleanCardWrapper clear_cl(dcto_cl, ct); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
498 |
|
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9418
diff
changeset
|
499 |
clear_cl.do_MemRegion(mr); |
1 | 500 |
} |
501 |
} |
|
502 |
} |
|
503 |
||
9336
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
504 |
// The iterator itself is not MT-aware, but |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
505 |
// MT-aware callers and closures can use this to |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
506 |
// accomplish dirty card iteration in parallel. The |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
507 |
// iterator itself does not clear the dirty cards, or |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
508 |
// change their values in any manner. |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
509 |
void CardTableModRefBS::non_clean_card_iterate_serial(MemRegion mr, |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
510 |
MemRegionClosure* cl) { |
11174
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
511 |
bool is_par = (SharedHeap::heap()->n_par_threads() > 0); |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
512 |
assert(!is_par || |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
513 |
(SharedHeap::heap()->n_par_threads() == |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
9624
diff
changeset
|
514 |
SharedHeap::heap()->workers()->active_workers()), "Mismatch"); |
1 | 515 |
for (int i = 0; i < _cur_covered_regions; i++) { |
516 |
MemRegion mri = mr.intersection(_covered[i]); |
|
517 |
if (mri.word_size() > 0) { |
|
518 |
jbyte* cur_entry = byte_for(mri.last()); |
|
519 |
jbyte* limit = byte_for(mri.start()); |
|
520 |
while (cur_entry >= limit) { |
|
521 |
jbyte* next_entry = cur_entry - 1; |
|
522 |
if (*cur_entry != clean_card) { |
|
523 |
size_t non_clean_cards = 1; |
|
524 |
// Should the next card be included in this range of dirty cards. |
|
525 |
while (next_entry >= limit && *next_entry != clean_card) { |
|
526 |
non_clean_cards++; |
|
527 |
cur_entry = next_entry; |
|
528 |
next_entry--; |
|
529 |
} |
|
530 |
// The memory region may not be on a card boundary. So that |
|
531 |
// objects beyond the end of the region are not processed, make |
|
532 |
// cur_cards precise with regard to the end of the memory region. |
|
533 |
MemRegion cur_cards(addr_for(cur_entry), |
|
534 |
non_clean_cards * card_size_in_words); |
|
535 |
MemRegion dirty_region = cur_cards.intersection(mri); |
|
536 |
cl->do_MemRegion(dirty_region); |
|
537 |
} |
|
538 |
cur_entry = next_entry; |
|
539 |
} |
|
540 |
} |
|
541 |
} |
|
542 |
} |
|
543 |
||
544 |
void CardTableModRefBS::dirty_MemRegion(MemRegion mr) { |
|
4461
c17c526d36ef
6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents:
3695
diff
changeset
|
545 |
assert((HeapWord*)align_size_down((uintptr_t)mr.start(), HeapWordSize) == mr.start(), "Unaligned start"); |
c17c526d36ef
6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents:
3695
diff
changeset
|
546 |
assert((HeapWord*)align_size_up ((uintptr_t)mr.end(), HeapWordSize) == mr.end(), "Unaligned end" ); |
1 | 547 |
jbyte* cur = byte_for(mr.start()); |
548 |
jbyte* last = byte_after(mr.last()); |
|
549 |
while (cur < last) { |
|
550 |
*cur = dirty_card; |
|
551 |
cur++; |
|
552 |
} |
|
553 |
} |
|
554 |
||
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
555 |
void CardTableModRefBS::invalidate(MemRegion mr, bool whole_heap) { |
4461
c17c526d36ef
6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents:
3695
diff
changeset
|
556 |
assert((HeapWord*)align_size_down((uintptr_t)mr.start(), HeapWordSize) == mr.start(), "Unaligned start"); |
c17c526d36ef
6906727: UseCompressedOops: some card-marking fixes related to object arrays
ysr
parents:
3695
diff
changeset
|
557 |
assert((HeapWord*)align_size_up ((uintptr_t)mr.end(), HeapWordSize) == mr.end(), "Unaligned end" ); |
1 | 558 |
for (int i = 0; i < _cur_covered_regions; i++) { |
559 |
MemRegion mri = mr.intersection(_covered[i]); |
|
560 |
if (!mri.is_empty()) dirty_MemRegion(mri); |
|
561 |
} |
|
562 |
} |
|
563 |
||
564 |
void CardTableModRefBS::clear_MemRegion(MemRegion mr) { |
|
565 |
// Be conservative: only clean cards entirely contained within the |
|
566 |
// region. |
|
567 |
jbyte* cur; |
|
568 |
if (mr.start() == _whole_heap.start()) { |
|
569 |
cur = byte_for(mr.start()); |
|
570 |
} else { |
|
571 |
assert(mr.start() > _whole_heap.start(), "mr is not covered."); |
|
572 |
cur = byte_after(mr.start() - 1); |
|
573 |
} |
|
574 |
jbyte* last = byte_after(mr.last()); |
|
575 |
memset(cur, clean_card, pointer_delta(last, cur, sizeof(jbyte))); |
|
576 |
} |
|
577 |
||
578 |
void CardTableModRefBS::clear(MemRegion mr) { |
|
579 |
for (int i = 0; i < _cur_covered_regions; i++) { |
|
580 |
MemRegion mri = mr.intersection(_covered[i]); |
|
581 |
if (!mri.is_empty()) clear_MemRegion(mri); |
|
582 |
} |
|
583 |
} |
|
584 |
||
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
585 |
void CardTableModRefBS::dirty(MemRegion mr) { |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
586 |
jbyte* first = byte_for(mr.start()); |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
587 |
jbyte* last = byte_after(mr.last()); |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
588 |
memset(first, dirty_card, last-first); |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
589 |
} |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
590 |
|
9183
3d0e0687fe28
7036482: clear argument is redundant and unused in cardtable methods
ysr
parents:
7397
diff
changeset
|
591 |
// Unlike several other card table methods, dirty_card_iterate() |
3d0e0687fe28
7036482: clear argument is redundant and unused in cardtable methods
ysr
parents:
7397
diff
changeset
|
592 |
// iterates over dirty cards ranges in increasing address order. |
1 | 593 |
void CardTableModRefBS::dirty_card_iterate(MemRegion mr, |
594 |
MemRegionClosure* cl) { |
|
595 |
for (int i = 0; i < _cur_covered_regions; i++) { |
|
596 |
MemRegion mri = mr.intersection(_covered[i]); |
|
597 |
if (!mri.is_empty()) { |
|
598 |
jbyte *cur_entry, *next_entry, *limit; |
|
599 |
for (cur_entry = byte_for(mri.start()), limit = byte_for(mri.last()); |
|
600 |
cur_entry <= limit; |
|
601 |
cur_entry = next_entry) { |
|
602 |
next_entry = cur_entry + 1; |
|
603 |
if (*cur_entry == dirty_card) { |
|
604 |
size_t dirty_cards; |
|
605 |
// Accumulate maximal dirty card range, starting at cur_entry |
|
606 |
for (dirty_cards = 1; |
|
607 |
next_entry <= limit && *next_entry == dirty_card; |
|
608 |
dirty_cards++, next_entry++); |
|
609 |
MemRegion cur_cards(addr_for(cur_entry), |
|
610 |
dirty_cards*card_size_in_words); |
|
611 |
cl->do_MemRegion(cur_cards); |
|
612 |
} |
|
613 |
} |
|
614 |
} |
|
615 |
} |
|
616 |
} |
|
617 |
||
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
618 |
MemRegion CardTableModRefBS::dirty_card_range_after_reset(MemRegion mr, |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
619 |
bool reset, |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
620 |
int reset_val) { |
1 | 621 |
for (int i = 0; i < _cur_covered_regions; i++) { |
622 |
MemRegion mri = mr.intersection(_covered[i]); |
|
623 |
if (!mri.is_empty()) { |
|
624 |
jbyte* cur_entry, *next_entry, *limit; |
|
625 |
for (cur_entry = byte_for(mri.start()), limit = byte_for(mri.last()); |
|
626 |
cur_entry <= limit; |
|
627 |
cur_entry = next_entry) { |
|
628 |
next_entry = cur_entry + 1; |
|
629 |
if (*cur_entry == dirty_card) { |
|
630 |
size_t dirty_cards; |
|
631 |
// Accumulate maximal dirty card range, starting at cur_entry |
|
632 |
for (dirty_cards = 1; |
|
633 |
next_entry <= limit && *next_entry == dirty_card; |
|
634 |
dirty_cards++, next_entry++); |
|
635 |
MemRegion cur_cards(addr_for(cur_entry), |
|
636 |
dirty_cards*card_size_in_words); |
|
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
637 |
if (reset) { |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
638 |
for (size_t i = 0; i < dirty_cards; i++) { |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
639 |
cur_entry[i] = reset_val; |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
640 |
} |
1 | 641 |
} |
642 |
return cur_cards; |
|
643 |
} |
|
644 |
} |
|
645 |
} |
|
646 |
} |
|
647 |
return MemRegion(mr.end(), mr.end()); |
|
648 |
} |
|
649 |
||
650 |
uintx CardTableModRefBS::ct_max_alignment_constraint() { |
|
651 |
return card_size * os::vm_page_size(); |
|
652 |
} |
|
653 |
||
654 |
void CardTableModRefBS::verify_guard() { |
|
655 |
// For product build verification |
|
656 |
guarantee(_byte_map[_guard_index] == last_card, |
|
657 |
"card table guard has been modified"); |
|
658 |
} |
|
659 |
||
660 |
void CardTableModRefBS::verify() { |
|
661 |
verify_guard(); |
|
662 |
} |
|
663 |
||
664 |
#ifndef PRODUCT |
|
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
665 |
void CardTableModRefBS::verify_region(MemRegion mr, |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
666 |
jbyte val, bool val_equals) { |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
667 |
jbyte* start = byte_for(mr.start()); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
668 |
jbyte* end = byte_for(mr.last()); |
26160 | 669 |
bool failures = false; |
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
670 |
for (jbyte* curr = start; curr <= end; ++curr) { |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
671 |
jbyte curr_val = *curr; |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
672 |
bool failed = (val_equals) ? (curr_val != val) : (curr_val == val); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
673 |
if (failed) { |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
674 |
if (!failures) { |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
675 |
tty->cr(); |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
676 |
tty->print_cr("== CT verification failed: [" INTPTR_FORMAT "," INTPTR_FORMAT "]", p2i(start), p2i(end)); |
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
677 |
tty->print_cr("== %sexpecting value: %d", |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
678 |
(val_equals) ? "" : "not ", val); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
679 |
failures = true; |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
680 |
} |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
681 |
tty->print_cr("== card "PTR_FORMAT" ["PTR_FORMAT","PTR_FORMAT"], " |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
682 |
"val: %d", p2i(curr), p2i(addr_for(curr)), |
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
683 |
p2i((HeapWord*) (((size_t) addr_for(curr)) + card_size)), |
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
684 |
(int) curr_val); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
685 |
} |
1 | 686 |
} |
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
687 |
guarantee(!failures, "there should not have been any failures"); |
1 | 688 |
} |
3695
421cfcc8843c
6841313: G1: dirty cards of survivor regions in parallel
apetrusenko
parents:
3587
diff
changeset
|
689 |
|
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
690 |
void CardTableModRefBS::verify_not_dirty_region(MemRegion mr) { |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
691 |
verify_region(mr, dirty_card, false /* val_equals */); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
692 |
} |
3695
421cfcc8843c
6841313: G1: dirty cards of survivor regions in parallel
apetrusenko
parents:
3587
diff
changeset
|
693 |
|
421cfcc8843c
6841313: G1: dirty cards of survivor regions in parallel
apetrusenko
parents:
3587
diff
changeset
|
694 |
void CardTableModRefBS::verify_dirty_region(MemRegion mr) { |
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
9336
diff
changeset
|
695 |
verify_region(mr, dirty_card, true /* val_equals */); |
3695
421cfcc8843c
6841313: G1: dirty cards of survivor regions in parallel
apetrusenko
parents:
3587
diff
changeset
|
696 |
} |
1 | 697 |
#endif |
698 |
||
12268 | 699 |
void CardTableModRefBS::print_on(outputStream* st) const { |
700 |
st->print_cr("Card table byte_map: [" INTPTR_FORMAT "," INTPTR_FORMAT "] byte_map_base: " INTPTR_FORMAT, |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22876
diff
changeset
|
701 |
p2i(_byte_map), p2i(_byte_map + _byte_map_size), p2i(byte_map_base)); |
12268 | 702 |
} |
703 |
||
1 | 704 |
bool CardTableModRefBSForCTRS::card_will_be_scanned(jbyte cv) { |
705 |
return |
|
706 |
CardTableModRefBS::card_will_be_scanned(cv) || |
|
707 |
_rs->is_prev_nonclean_card_val(cv); |
|
708 |
}; |
|
709 |
||
710 |
bool CardTableModRefBSForCTRS::card_may_have_been_dirty(jbyte cv) { |
|
711 |
return |
|
712 |
cv != clean_card && |
|
713 |
(CardTableModRefBS::card_may_have_been_dirty(cv) || |
|
714 |
CardTableRS::youngergen_may_have_been_dirty(cv)); |
|
715 |
}; |