author | drchase |
Fri, 09 May 2014 16:50:54 -0400 | |
changeset 24424 | 2658d7834c6e |
parent 24108 | dfc2242fc6b0 |
child 25351 | 7c198a690050 |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
2 |
* Copyright (c) 2001, 2014, 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:
5346
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5346
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:
5346
diff
changeset
|
21 |
* questions. |
1374 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "gc_implementation/g1/concurrentG1Refine.hpp" |
|
27 |
#include "gc_implementation/g1/g1BlockOffsetTable.inline.hpp" |
|
28 |
#include "gc_implementation/g1/g1CollectedHeap.inline.hpp" |
|
29 |
#include "gc_implementation/g1/heapRegionRemSet.hpp" |
|
30 |
#include "gc_implementation/g1/heapRegionSeq.inline.hpp" |
|
31 |
#include "memory/allocation.hpp" |
|
23452
d7dca4e6b95d
8035815: Cache-align and pad the from card cache
tschatzl
parents:
23451
diff
changeset
|
32 |
#include "memory/padded.inline.hpp" |
7397 | 33 |
#include "memory/space.inline.hpp" |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
34 |
#include "oops/oop.inline.hpp" |
7397 | 35 |
#include "utilities/bitMap.inline.hpp" |
36 |
#include "utilities/globalDefinitions.hpp" |
|
19339 | 37 |
#include "utilities/growableArray.hpp" |
1374 | 38 |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
24108
diff
changeset
|
39 |
PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC |
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
24108
diff
changeset
|
40 |
|
13195 | 41 |
class PerRegionTable: public CHeapObj<mtGC> { |
1374 | 42 |
friend class OtherRegionsTable; |
43 |
friend class HeapRegionRemSetIterator; |
|
44 |
||
45 |
HeapRegion* _hr; |
|
46 |
BitMap _bm; |
|
47 |
jint _occupied; |
|
48 |
||
13335 | 49 |
// next pointer for free/allocated 'all' list |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
50 |
PerRegionTable* _next; |
1374 | 51 |
|
13335 | 52 |
// prev pointer for the allocated 'all' list |
53 |
PerRegionTable* _prev; |
|
1374 | 54 |
|
13335 | 55 |
// next pointer in collision list |
56 |
PerRegionTable * _collision_list_next; |
|
1374 | 57 |
|
13335 | 58 |
// Global free list of PRTs |
59 |
static PerRegionTable* _free_list; |
|
1374 | 60 |
|
61 |
protected: |
|
62 |
// We need access in order to union things into the base table. |
|
63 |
BitMap* bm() { return &_bm; } |
|
64 |
||
65 |
void recount_occupied() { |
|
66 |
_occupied = (jint) bm()->count_one_bits(); |
|
67 |
} |
|
68 |
||
69 |
PerRegionTable(HeapRegion* hr) : |
|
70 |
_hr(hr), |
|
71 |
_occupied(0), |
|
13335 | 72 |
_bm(HeapRegion::CardsPerRegion, false /* in-resource-area */), |
73 |
_collision_list_next(NULL), _next(NULL), _prev(NULL) |
|
1374 | 74 |
{} |
75 |
||
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
76 |
void add_card_work(CardIdx_t from_card, bool par) { |
1374 | 77 |
if (!_bm.at(from_card)) { |
78 |
if (par) { |
|
79 |
if (_bm.par_at_put(from_card, 1)) { |
|
80 |
Atomic::inc(&_occupied); |
|
81 |
} |
|
82 |
} else { |
|
83 |
_bm.at_put(from_card, 1); |
|
84 |
_occupied++; |
|
85 |
} |
|
86 |
} |
|
87 |
} |
|
88 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
89 |
void add_reference_work(OopOrNarrowOopStar from, bool par) { |
1374 | 90 |
// Must make this robust in case "from" is not in "_hr", because of |
91 |
// concurrency. |
|
92 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
93 |
if (G1TraceHeapRegionRememberedSet) { |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
94 |
gclog_or_tty->print_cr(" PRT::Add_reference_work(" PTR_FORMAT "->" PTR_FORMAT").", |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
95 |
from, |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
96 |
UseCompressedOops |
20282
7f9cbdf89af2
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
19339
diff
changeset
|
97 |
? (void *)oopDesc::load_decode_heap_oop((narrowOop*)from) |
7f9cbdf89af2
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
19339
diff
changeset
|
98 |
: (void *)oopDesc::load_decode_heap_oop((oop*)from)); |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
99 |
} |
1374 | 100 |
|
101 |
HeapRegion* loc_hr = hr(); |
|
102 |
// If the test below fails, then this table was reused concurrently |
|
103 |
// with this operation. This is OK, since the old table was coarsened, |
|
104 |
// and adding a bit to the new table is never incorrect. |
|
10767
4ce4a9b10523
7097516: G1: assert(0<= from_card && from_card<HeapRegion::CardsPerRegion) failed: Must be in range.
brutisso
parents:
10677
diff
changeset
|
105 |
// If the table used to belong to a continues humongous region and is |
4ce4a9b10523
7097516: G1: assert(0<= from_card && from_card<HeapRegion::CardsPerRegion) failed: Must be in range.
brutisso
parents:
10677
diff
changeset
|
106 |
// now reused for the corresponding start humongous region, we need to |
4ce4a9b10523
7097516: G1: assert(0<= from_card && from_card<HeapRegion::CardsPerRegion) failed: Must be in range.
brutisso
parents:
10677
diff
changeset
|
107 |
// make sure that we detect this. Thus, we call is_in_reserved_raw() |
4ce4a9b10523
7097516: G1: assert(0<= from_card && from_card<HeapRegion::CardsPerRegion) failed: Must be in range.
brutisso
parents:
10677
diff
changeset
|
108 |
// instead of just is_in_reserved() here. |
4ce4a9b10523
7097516: G1: assert(0<= from_card && from_card<HeapRegion::CardsPerRegion) failed: Must be in range.
brutisso
parents:
10677
diff
changeset
|
109 |
if (loc_hr->is_in_reserved_raw(from)) { |
1374 | 110 |
size_t hw_offset = pointer_delta((HeapWord*)from, loc_hr->bottom()); |
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
111 |
CardIdx_t from_card = (CardIdx_t) |
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
112 |
hw_offset >> (CardTableModRefBS::card_shift - LogHeapWordSize); |
1374 | 113 |
|
10677
370a8da2d63f
7095194: G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
johnc
parents:
10243
diff
changeset
|
114 |
assert(0 <= from_card && (size_t)from_card < HeapRegion::CardsPerRegion, |
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3262
diff
changeset
|
115 |
"Must be in range."); |
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
116 |
add_card_work(from_card, par); |
1374 | 117 |
} |
118 |
} |
|
119 |
||
120 |
public: |
|
121 |
||
122 |
HeapRegion* hr() const { return _hr; } |
|
123 |
||
124 |
jint occupied() const { |
|
125 |
// Overkill, but if we ever need it... |
|
126 |
// guarantee(_occupied == _bm.count_one_bits(), "Check"); |
|
127 |
return _occupied; |
|
128 |
} |
|
129 |
||
13335 | 130 |
void init(HeapRegion* hr, bool clear_links_to_all_list) { |
131 |
if (clear_links_to_all_list) { |
|
132 |
set_next(NULL); |
|
133 |
set_prev(NULL); |
|
134 |
} |
|
1374 | 135 |
_hr = hr; |
13335 | 136 |
_collision_list_next = NULL; |
1374 | 137 |
_occupied = 0; |
138 |
_bm.clear(); |
|
139 |
} |
|
140 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
141 |
void add_reference(OopOrNarrowOopStar from) { |
1374 | 142 |
add_reference_work(from, /*parallel*/ true); |
143 |
} |
|
144 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
145 |
void seq_add_reference(OopOrNarrowOopStar from) { |
1374 | 146 |
add_reference_work(from, /*parallel*/ false); |
147 |
} |
|
148 |
||
149 |
void scrub(CardTableModRefBS* ctbs, BitMap* card_bm) { |
|
150 |
HeapWord* hr_bot = hr()->bottom(); |
|
1676
d80e69372634
6653214: MemoryPoolMXBean.setUsageThreshold() does not support large heap sizes.
swamyv
parents:
1374
diff
changeset
|
151 |
size_t hr_first_card_index = ctbs->index_for(hr_bot); |
1374 | 152 |
bm()->set_intersection_at_offset(*card_bm, hr_first_card_index); |
153 |
recount_occupied(); |
|
154 |
} |
|
155 |
||
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
156 |
void add_card(CardIdx_t from_card_index) { |
1374 | 157 |
add_card_work(from_card_index, /*parallel*/ true); |
158 |
} |
|
159 |
||
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
160 |
void seq_add_card(CardIdx_t from_card_index) { |
1374 | 161 |
add_card_work(from_card_index, /*parallel*/ false); |
162 |
} |
|
163 |
||
164 |
// (Destructively) union the bitmap of the current table into the given |
|
165 |
// bitmap (which is assumed to be of the same size.) |
|
166 |
void union_bitmap_into(BitMap* bm) { |
|
167 |
bm->set_union(_bm); |
|
168 |
} |
|
169 |
||
170 |
// Mem size in bytes. |
|
171 |
size_t mem_size() const { |
|
24108
dfc2242fc6b0
8040792: G1: Memory usage calculation uses sizeof(this) instead of sizeof(classname)
tschatzl
parents:
24106
diff
changeset
|
172 |
return sizeof(PerRegionTable) + _bm.size_in_words() * HeapWordSize; |
1374 | 173 |
} |
174 |
||
175 |
// Requires "from" to be in "hr()". |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
176 |
bool contains_reference(OopOrNarrowOopStar from) const { |
1374 | 177 |
assert(hr()->is_in_reserved(from), "Precondition."); |
178 |
size_t card_ind = pointer_delta(from, hr()->bottom(), |
|
179 |
CardTableModRefBS::card_size); |
|
180 |
return _bm.at(card_ind); |
|
181 |
} |
|
182 |
||
13335 | 183 |
// Bulk-free the PRTs from prt to last, assumes that they are |
184 |
// linked together using their _next field. |
|
185 |
static void bulk_free(PerRegionTable* prt, PerRegionTable* last) { |
|
1374 | 186 |
while (true) { |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
187 |
PerRegionTable* fl = _free_list; |
13335 | 188 |
last->set_next(fl); |
189 |
PerRegionTable* res = (PerRegionTable*) Atomic::cmpxchg_ptr(prt, &_free_list, fl); |
|
190 |
if (res == fl) { |
|
191 |
return; |
|
192 |
} |
|
1374 | 193 |
} |
194 |
ShouldNotReachHere(); |
|
195 |
} |
|
196 |
||
13335 | 197 |
static void free(PerRegionTable* prt) { |
198 |
bulk_free(prt, prt); |
|
199 |
} |
|
200 |
||
201 |
// Returns an initialized PerRegionTable instance. |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
202 |
static PerRegionTable* alloc(HeapRegion* hr) { |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
203 |
PerRegionTable* fl = _free_list; |
1374 | 204 |
while (fl != NULL) { |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
205 |
PerRegionTable* nxt = fl->next(); |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
206 |
PerRegionTable* res = |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
207 |
(PerRegionTable*) |
1374 | 208 |
Atomic::cmpxchg_ptr(nxt, &_free_list, fl); |
209 |
if (res == fl) { |
|
13335 | 210 |
fl->init(hr, true); |
1374 | 211 |
return fl; |
212 |
} else { |
|
213 |
fl = _free_list; |
|
214 |
} |
|
215 |
} |
|
216 |
assert(fl == NULL, "Loop condition."); |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
217 |
return new PerRegionTable(hr); |
1374 | 218 |
} |
219 |
||
13335 | 220 |
PerRegionTable* next() const { return _next; } |
221 |
void set_next(PerRegionTable* next) { _next = next; } |
|
222 |
PerRegionTable* prev() const { return _prev; } |
|
223 |
void set_prev(PerRegionTable* prev) { _prev = prev; } |
|
224 |
||
225 |
// Accessor and Modification routines for the pointer for the |
|
226 |
// singly linked collision list that links the PRTs within the |
|
227 |
// OtherRegionsTable::_fine_grain_regions hash table. |
|
228 |
// |
|
229 |
// It might be useful to also make the collision list doubly linked |
|
230 |
// to avoid iteration over the collisions list during scrubbing/deletion. |
|
231 |
// OTOH there might not be many collisions. |
|
232 |
||
233 |
PerRegionTable* collision_list_next() const { |
|
234 |
return _collision_list_next; |
|
235 |
} |
|
236 |
||
237 |
void set_collision_list_next(PerRegionTable* next) { |
|
238 |
_collision_list_next = next; |
|
239 |
} |
|
240 |
||
241 |
PerRegionTable** collision_list_next_addr() { |
|
242 |
return &_collision_list_next; |
|
243 |
} |
|
244 |
||
1374 | 245 |
static size_t fl_mem_size() { |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
246 |
PerRegionTable* cur = _free_list; |
1374 | 247 |
size_t res = 0; |
248 |
while (cur != NULL) { |
|
17631
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
249 |
res += cur->mem_size(); |
1374 | 250 |
cur = cur->next(); |
251 |
} |
|
252 |
return res; |
|
253 |
} |
|
17631
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
254 |
|
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
255 |
static void test_fl_mem_size(); |
1374 | 256 |
}; |
257 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
258 |
PerRegionTable* PerRegionTable::_free_list = NULL; |
1374 | 259 |
|
260 |
size_t OtherRegionsTable::_max_fine_entries = 0; |
|
261 |
size_t OtherRegionsTable::_mod_max_fine_entries_mask = 0; |
|
262 |
size_t OtherRegionsTable::_fine_eviction_stride = 0; |
|
263 |
size_t OtherRegionsTable::_fine_eviction_sample_size = 0; |
|
264 |
||
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
265 |
OtherRegionsTable::OtherRegionsTable(HeapRegion* hr, Mutex* m) : |
1374 | 266 |
_g1h(G1CollectedHeap::heap()), |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
267 |
_hr(hr), _m(m), |
1374 | 268 |
_coarse_map(G1CollectedHeap::heap()->max_regions(), |
269 |
false /* in-resource-area */), |
|
270 |
_fine_grain_regions(NULL), |
|
13335 | 271 |
_first_all_fine_prts(NULL), _last_all_fine_prts(NULL), |
1374 | 272 |
_n_fine_entries(0), _n_coarse_entries(0), |
273 |
_fine_eviction_start(0), |
|
274 |
_sparse_table(hr) |
|
275 |
{ |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
276 |
typedef PerRegionTable* PerRegionTablePtr; |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
277 |
|
1374 | 278 |
if (_max_fine_entries == 0) { |
279 |
assert(_mod_max_fine_entries_mask == 0, "Both or none."); |
|
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
280 |
size_t max_entries_log = (size_t)log2_long((jlong)G1RSetRegionEntries); |
13752
92f03fcf979a
7197906: BlockOffsetArray::power_to_cards_back() needs to handle > 32 bit shifts
brutisso
parents:
13335
diff
changeset
|
281 |
_max_fine_entries = (size_t)1 << max_entries_log; |
1374 | 282 |
_mod_max_fine_entries_mask = _max_fine_entries - 1; |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
283 |
|
1374 | 284 |
assert(_fine_eviction_sample_size == 0 |
285 |
&& _fine_eviction_stride == 0, "All init at same time."); |
|
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
286 |
_fine_eviction_sample_size = MAX2((size_t)4, max_entries_log); |
1374 | 287 |
_fine_eviction_stride = _max_fine_entries / _fine_eviction_sample_size; |
288 |
} |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
289 |
|
17376
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17119
diff
changeset
|
290 |
_fine_grain_regions = NEW_C_HEAP_ARRAY3(PerRegionTablePtr, _max_fine_entries, |
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17119
diff
changeset
|
291 |
mtGC, 0, AllocFailStrategy::RETURN_NULL); |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
292 |
|
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
293 |
if (_fine_grain_regions == NULL) { |
17087
f0b76c4c93a0
8011661: Insufficient memory message says "malloc" when sometimes it should say "mmap"
ccheung
parents:
17031
diff
changeset
|
294 |
vm_exit_out_of_memory(sizeof(void*)*_max_fine_entries, OOM_MALLOC_ERROR, |
1374 | 295 |
"Failed to allocate _fine_grain_entries."); |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
296 |
} |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
297 |
|
1374 | 298 |
for (size_t i = 0; i < _max_fine_entries; i++) { |
299 |
_fine_grain_regions[i] = NULL; |
|
300 |
} |
|
301 |
} |
|
302 |
||
13335 | 303 |
void OtherRegionsTable::link_to_all(PerRegionTable* prt) { |
304 |
// We always append to the beginning of the list for convenience; |
|
305 |
// the order of entries in this list does not matter. |
|
306 |
if (_first_all_fine_prts != NULL) { |
|
307 |
assert(_first_all_fine_prts->prev() == NULL, "invariant"); |
|
308 |
_first_all_fine_prts->set_prev(prt); |
|
309 |
prt->set_next(_first_all_fine_prts); |
|
310 |
} else { |
|
311 |
// this is the first element we insert. Adjust the "last" pointer |
|
312 |
_last_all_fine_prts = prt; |
|
313 |
assert(prt->next() == NULL, "just checking"); |
|
314 |
} |
|
315 |
// the new element is always the first element without a predecessor |
|
316 |
prt->set_prev(NULL); |
|
317 |
_first_all_fine_prts = prt; |
|
318 |
||
319 |
assert(prt->prev() == NULL, "just checking"); |
|
320 |
assert(_first_all_fine_prts == prt, "just checking"); |
|
321 |
assert((_first_all_fine_prts == NULL && _last_all_fine_prts == NULL) || |
|
322 |
(_first_all_fine_prts != NULL && _last_all_fine_prts != NULL), |
|
323 |
"just checking"); |
|
324 |
assert(_last_all_fine_prts == NULL || _last_all_fine_prts->next() == NULL, |
|
325 |
"just checking"); |
|
326 |
assert(_first_all_fine_prts == NULL || _first_all_fine_prts->prev() == NULL, |
|
327 |
"just checking"); |
|
328 |
} |
|
329 |
||
330 |
void OtherRegionsTable::unlink_from_all(PerRegionTable* prt) { |
|
331 |
if (prt->prev() != NULL) { |
|
332 |
assert(_first_all_fine_prts != prt, "just checking"); |
|
333 |
prt->prev()->set_next(prt->next()); |
|
334 |
// removing the last element in the list? |
|
335 |
if (_last_all_fine_prts == prt) { |
|
336 |
_last_all_fine_prts = prt->prev(); |
|
337 |
} |
|
338 |
} else { |
|
339 |
assert(_first_all_fine_prts == prt, "just checking"); |
|
340 |
_first_all_fine_prts = prt->next(); |
|
341 |
// list is empty now? |
|
342 |
if (_first_all_fine_prts == NULL) { |
|
343 |
_last_all_fine_prts = NULL; |
|
344 |
} |
|
345 |
} |
|
346 |
||
347 |
if (prt->next() != NULL) { |
|
348 |
prt->next()->set_prev(prt->prev()); |
|
349 |
} |
|
350 |
||
351 |
prt->set_next(NULL); |
|
352 |
prt->set_prev(NULL); |
|
353 |
||
354 |
assert((_first_all_fine_prts == NULL && _last_all_fine_prts == NULL) || |
|
355 |
(_first_all_fine_prts != NULL && _last_all_fine_prts != NULL), |
|
356 |
"just checking"); |
|
357 |
assert(_last_all_fine_prts == NULL || _last_all_fine_prts->next() == NULL, |
|
358 |
"just checking"); |
|
359 |
assert(_first_all_fine_prts == NULL || _first_all_fine_prts->prev() == NULL, |
|
360 |
"just checking"); |
|
361 |
} |
|
362 |
||
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
363 |
int** FromCardCache::_cache = NULL; |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
364 |
uint FromCardCache::_max_regions = 0; |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
365 |
size_t FromCardCache::_static_mem_size = 0; |
1374 | 366 |
|
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
367 |
void FromCardCache::initialize(uint n_par_rs, uint max_num_regions) { |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
368 |
guarantee(_cache == NULL, "Should not call this multiple times"); |
1374 | 369 |
|
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
370 |
_max_regions = max_num_regions; |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
371 |
_cache = Padded2DArray<int, mtGC>::create_unfreeable(n_par_rs, |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
372 |
_max_regions, |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
373 |
&_static_mem_size); |
23452
d7dca4e6b95d
8035815: Cache-align and pad the from card cache
tschatzl
parents:
23451
diff
changeset
|
374 |
|
d7dca4e6b95d
8035815: Cache-align and pad the from card cache
tschatzl
parents:
23451
diff
changeset
|
375 |
for (uint i = 0; i < n_par_rs; i++) { |
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
376 |
for (uint j = 0; j < _max_regions; j++) { |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
377 |
set(i, j, InvalidCard); |
1374 | 378 |
} |
379 |
} |
|
380 |
} |
|
381 |
||
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
382 |
void FromCardCache::shrink(uint new_num_regions) { |
23452
d7dca4e6b95d
8035815: Cache-align and pad the from card cache
tschatzl
parents:
23451
diff
changeset
|
383 |
for (uint i = 0; i < HeapRegionRemSet::num_par_rem_sets(); i++) { |
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
384 |
assert(new_num_regions <= _max_regions, "Must be within max."); |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
385 |
for (uint j = new_num_regions; j < _max_regions; j++) { |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
386 |
set(i, j, InvalidCard); |
1374 | 387 |
} |
388 |
} |
|
389 |
} |
|
390 |
||
391 |
#ifndef PRODUCT |
|
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
392 |
void FromCardCache::print(outputStream* out) { |
23452
d7dca4e6b95d
8035815: Cache-align and pad the from card cache
tschatzl
parents:
23451
diff
changeset
|
393 |
for (uint i = 0; i < HeapRegionRemSet::num_par_rem_sets(); i++) { |
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
394 |
for (uint j = 0; j < _max_regions; j++) { |
23858
dae377f5a7c7
8039244: Don't use UINT32_FORMAT and INT32_FORMAT when printing uints and ints in the GC code
stefank
parents:
23855
diff
changeset
|
395 |
out->print_cr("_from_card_cache[%u][%u] = %d.", |
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
396 |
i, j, at(i, j)); |
1374 | 397 |
} |
398 |
} |
|
399 |
} |
|
400 |
#endif |
|
401 |
||
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
402 |
void FromCardCache::clear(uint region_idx) { |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
403 |
uint num_par_remsets = HeapRegionRemSet::num_par_rem_sets(); |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
404 |
for (uint i = 0; i < num_par_remsets; i++) { |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
405 |
set(i, region_idx, InvalidCard); |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
406 |
} |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
407 |
} |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
408 |
|
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
409 |
void OtherRegionsTable::init_from_card_cache(uint max_regions) { |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
410 |
FromCardCache::initialize(HeapRegionRemSet::num_par_rem_sets(), max_regions); |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
411 |
} |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
412 |
|
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
413 |
void OtherRegionsTable::shrink_from_card_cache(uint new_num_regions) { |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
414 |
FromCardCache::shrink(new_num_regions); |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
415 |
} |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
416 |
|
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
417 |
void OtherRegionsTable::print_from_card_cache() { |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
418 |
FromCardCache::print(); |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
419 |
} |
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
420 |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
421 |
void OtherRegionsTable::add_reference(OopOrNarrowOopStar from, int tid) { |
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
422 |
uint cur_hrs_ind = hr()->hrs_index(); |
1374 | 423 |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
424 |
if (G1TraceHeapRegionRememberedSet) { |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
425 |
gclog_or_tty->print_cr("ORT::add_reference_work(" PTR_FORMAT "->" PTR_FORMAT ").", |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
426 |
from, |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
427 |
UseCompressedOops |
20282
7f9cbdf89af2
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
19339
diff
changeset
|
428 |
? (void *)oopDesc::load_decode_heap_oop((narrowOop*)from) |
7f9cbdf89af2
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
19339
diff
changeset
|
429 |
: (void *)oopDesc::load_decode_heap_oop((oop*)from)); |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
430 |
} |
1374 | 431 |
|
432 |
int from_card = (int)(uintptr_t(from) >> CardTableModRefBS::card_shift); |
|
433 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
434 |
if (G1TraceHeapRegionRememberedSet) { |
23858
dae377f5a7c7
8039244: Don't use UINT32_FORMAT and INT32_FORMAT when printing uints and ints in the GC code
stefank
parents:
23855
diff
changeset
|
435 |
gclog_or_tty->print_cr("Table for [" PTR_FORMAT "...): card %d (cache = %d)", |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
436 |
hr()->bottom(), from_card, |
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
437 |
FromCardCache::at((uint)tid, cur_hrs_ind)); |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
438 |
} |
1374 | 439 |
|
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
440 |
if (FromCardCache::contains_or_replace((uint)tid, cur_hrs_ind, from_card)) { |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
441 |
if (G1TraceHeapRegionRememberedSet) { |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
442 |
gclog_or_tty->print_cr(" from-card cache hit."); |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
443 |
} |
1374 | 444 |
assert(contains_reference(from), "We just added it!"); |
445 |
return; |
|
446 |
} |
|
447 |
||
448 |
// Note that this may be a continued H region. |
|
449 |
HeapRegion* from_hr = _g1h->heap_region_containing_raw(from); |
|
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
450 |
RegionIdx_t from_hrs_ind = (RegionIdx_t) from_hr->hrs_index(); |
1374 | 451 |
|
452 |
// If the region is already coarsened, return. |
|
453 |
if (_coarse_map.at(from_hrs_ind)) { |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
454 |
if (G1TraceHeapRegionRememberedSet) { |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
455 |
gclog_or_tty->print_cr(" coarse map hit."); |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
456 |
} |
1374 | 457 |
assert(contains_reference(from), "We just added it!"); |
458 |
return; |
|
459 |
} |
|
460 |
||
461 |
// Otherwise find a per-region table to add it to. |
|
462 |
size_t ind = from_hrs_ind & _mod_max_fine_entries_mask; |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
463 |
PerRegionTable* prt = find_region_table(ind, from_hr); |
1374 | 464 |
if (prt == NULL) { |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
465 |
MutexLockerEx x(_m, Mutex::_no_safepoint_check_flag); |
1374 | 466 |
// Confirm that it's really not there... |
467 |
prt = find_region_table(ind, from_hr); |
|
468 |
if (prt == NULL) { |
|
469 |
||
470 |
uintptr_t from_hr_bot_card_index = |
|
471 |
uintptr_t(from_hr->bottom()) |
|
472 |
>> CardTableModRefBS::card_shift; |
|
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
473 |
CardIdx_t card_index = from_card - from_hr_bot_card_index; |
10677
370a8da2d63f
7095194: G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
johnc
parents:
10243
diff
changeset
|
474 |
assert(0 <= card_index && (size_t)card_index < HeapRegion::CardsPerRegion, |
1374 | 475 |
"Must be in range."); |
476 |
if (G1HRRSUseSparseTable && |
|
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
477 |
_sparse_table.add_card(from_hrs_ind, card_index)) { |
1374 | 478 |
if (G1RecordHRRSOops) { |
479 |
HeapRegionRemSet::record(hr(), from); |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
480 |
if (G1TraceHeapRegionRememberedSet) { |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
481 |
gclog_or_tty->print(" Added card " PTR_FORMAT " to region " |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
482 |
"[" PTR_FORMAT "...) for ref " PTR_FORMAT ".\n", |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
483 |
align_size_down(uintptr_t(from), |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
484 |
CardTableModRefBS::card_size), |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
485 |
hr()->bottom(), from); |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
486 |
} |
1374 | 487 |
} |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
488 |
if (G1TraceHeapRegionRememberedSet) { |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
489 |
gclog_or_tty->print_cr(" added card to sparse table."); |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
490 |
} |
1374 | 491 |
assert(contains_reference_locked(from), "We just added it!"); |
492 |
return; |
|
493 |
} else { |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
494 |
if (G1TraceHeapRegionRememberedSet) { |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
495 |
gclog_or_tty->print_cr(" [tid %d] sparse table entry " |
24092
e274d864545a
8039743: Use correct format specifier to print size_t values and pointers in the GC code
stefank
parents:
23858
diff
changeset
|
496 |
"overflow(f: %d, t: %u)", |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
497 |
tid, from_hrs_ind, cur_hrs_ind); |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
498 |
} |
1374 | 499 |
} |
500 |
||
501 |
if (_n_fine_entries == _max_fine_entries) { |
|
502 |
prt = delete_region_table(); |
|
13335 | 503 |
// There is no need to clear the links to the 'all' list here: |
504 |
// prt will be reused immediately, i.e. remain in the 'all' list. |
|
505 |
prt->init(from_hr, false /* clear_links_to_all_list */); |
|
1374 | 506 |
} else { |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
507 |
prt = PerRegionTable::alloc(from_hr); |
13335 | 508 |
link_to_all(prt); |
1374 | 509 |
} |
510 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
511 |
PerRegionTable* first_prt = _fine_grain_regions[ind]; |
13335 | 512 |
prt->set_collision_list_next(first_prt); |
1374 | 513 |
_fine_grain_regions[ind] = prt; |
514 |
_n_fine_entries++; |
|
515 |
||
516 |
if (G1HRRSUseSparseTable) { |
|
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
517 |
// Transfer from sparse to fine-grain. |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
518 |
SparsePRTEntry *sprt_entry = _sparse_table.get_entry(from_hrs_ind); |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
519 |
assert(sprt_entry != NULL, "There should have been an entry"); |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
520 |
for (int i = 0; i < SparsePRTEntry::cards_num(); i++) { |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
521 |
CardIdx_t c = sprt_entry->card(i); |
1374 | 522 |
if (c != SparsePRTEntry::NullEntry) { |
523 |
prt->add_card(c); |
|
524 |
} |
|
525 |
} |
|
526 |
// Now we can delete the sparse entry. |
|
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
527 |
bool res = _sparse_table.delete_entry(from_hrs_ind); |
1374 | 528 |
assert(res, "It should have been there."); |
529 |
} |
|
530 |
} |
|
531 |
assert(prt != NULL && prt->hr() == from_hr, "consequence"); |
|
532 |
} |
|
533 |
// Note that we can't assert "prt->hr() == from_hr", because of the |
|
534 |
// possibility of concurrent reuse. But see head comment of |
|
535 |
// OtherRegionsTable for why this is OK. |
|
536 |
assert(prt != NULL, "Inv"); |
|
537 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
538 |
prt->add_reference(from); |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
539 |
|
1374 | 540 |
if (G1RecordHRRSOops) { |
541 |
HeapRegionRemSet::record(hr(), from); |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
542 |
if (G1TraceHeapRegionRememberedSet) { |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
543 |
gclog_or_tty->print("Added card " PTR_FORMAT " to region " |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
544 |
"[" PTR_FORMAT "...) for ref " PTR_FORMAT ".\n", |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
545 |
align_size_down(uintptr_t(from), |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
546 |
CardTableModRefBS::card_size), |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
547 |
hr()->bottom(), from); |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
548 |
} |
1374 | 549 |
} |
550 |
assert(contains_reference(from), "We just added it!"); |
|
551 |
} |
|
552 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
553 |
PerRegionTable* |
1374 | 554 |
OtherRegionsTable::find_region_table(size_t ind, HeapRegion* hr) const { |
555 |
assert(0 <= ind && ind < _max_fine_entries, "Preconditions."); |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
556 |
PerRegionTable* prt = _fine_grain_regions[ind]; |
1374 | 557 |
while (prt != NULL && prt->hr() != hr) { |
13335 | 558 |
prt = prt->collision_list_next(); |
1374 | 559 |
} |
560 |
// Loop postcondition is the method postcondition. |
|
561 |
return prt; |
|
562 |
} |
|
563 |
||
564 |
jint OtherRegionsTable::_n_coarsenings = 0; |
|
565 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
566 |
PerRegionTable* OtherRegionsTable::delete_region_table() { |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
567 |
assert(_m->owned_by_self(), "Precondition"); |
1374 | 568 |
assert(_n_fine_entries == _max_fine_entries, "Precondition"); |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
569 |
PerRegionTable* max = NULL; |
1374 | 570 |
jint max_occ = 0; |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
571 |
PerRegionTable** max_prev; |
1374 | 572 |
size_t max_ind; |
573 |
||
574 |
size_t i = _fine_eviction_start; |
|
575 |
for (size_t k = 0; k < _fine_eviction_sample_size; k++) { |
|
576 |
size_t ii = i; |
|
577 |
// Make sure we get a non-NULL sample. |
|
578 |
while (_fine_grain_regions[ii] == NULL) { |
|
579 |
ii++; |
|
580 |
if (ii == _max_fine_entries) ii = 0; |
|
581 |
guarantee(ii != i, "We must find one."); |
|
582 |
} |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
583 |
PerRegionTable** prev = &_fine_grain_regions[ii]; |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
584 |
PerRegionTable* cur = *prev; |
1374 | 585 |
while (cur != NULL) { |
586 |
jint cur_occ = cur->occupied(); |
|
587 |
if (max == NULL || cur_occ > max_occ) { |
|
588 |
max = cur; |
|
589 |
max_prev = prev; |
|
590 |
max_ind = i; |
|
591 |
max_occ = cur_occ; |
|
592 |
} |
|
13335 | 593 |
prev = cur->collision_list_next_addr(); |
594 |
cur = cur->collision_list_next(); |
|
1374 | 595 |
} |
596 |
i = i + _fine_eviction_stride; |
|
597 |
if (i >= _n_fine_entries) i = i - _n_fine_entries; |
|
598 |
} |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
599 |
|
1374 | 600 |
_fine_eviction_start++; |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
601 |
|
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
602 |
if (_fine_eviction_start >= _n_fine_entries) { |
1374 | 603 |
_fine_eviction_start -= _n_fine_entries; |
604 |
} |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
605 |
|
1374 | 606 |
guarantee(max != NULL, "Since _n_fine_entries > 0"); |
607 |
||
608 |
// Set the corresponding coarse bit. |
|
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
609 |
size_t max_hrs_index = (size_t) max->hr()->hrs_index(); |
1374 | 610 |
if (!_coarse_map.at(max_hrs_index)) { |
611 |
_coarse_map.at_put(max_hrs_index, true); |
|
612 |
_n_coarse_entries++; |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
613 |
if (G1TraceHeapRegionRememberedSet) { |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
614 |
gclog_or_tty->print("Coarsened entry in region [" PTR_FORMAT "...] " |
24092
e274d864545a
8039743: Use correct format specifier to print size_t values and pointers in the GC code
stefank
parents:
23858
diff
changeset
|
615 |
"for region [" PTR_FORMAT "...] (" SIZE_FORMAT " coarse entries).\n", |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
616 |
hr()->bottom(), |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
617 |
max->hr()->bottom(), |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
618 |
_n_coarse_entries); |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
619 |
} |
1374 | 620 |
} |
621 |
||
622 |
// Unsplice. |
|
13335 | 623 |
*max_prev = max->collision_list_next(); |
1374 | 624 |
Atomic::inc(&_n_coarsenings); |
625 |
_n_fine_entries--; |
|
626 |
return max; |
|
627 |
} |
|
628 |
||
629 |
||
630 |
// At present, this must be called stop-world single-threaded. |
|
631 |
void OtherRegionsTable::scrub(CardTableModRefBS* ctbs, |
|
632 |
BitMap* region_bm, BitMap* card_bm) { |
|
633 |
// First eliminated garbage regions from the coarse map. |
|
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
634 |
if (G1RSScrubVerbose) { |
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
635 |
gclog_or_tty->print_cr("Scrubbing region %u:", hr()->hrs_index()); |
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
636 |
} |
1374 | 637 |
|
638 |
assert(_coarse_map.size() == region_bm->size(), "Precondition"); |
|
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
639 |
if (G1RSScrubVerbose) { |
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
640 |
gclog_or_tty->print(" Coarse map: before = "SIZE_FORMAT"...", |
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
641 |
_n_coarse_entries); |
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
642 |
} |
1374 | 643 |
_coarse_map.set_intersection(*region_bm); |
644 |
_n_coarse_entries = _coarse_map.count_one_bits(); |
|
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
645 |
if (G1RSScrubVerbose) { |
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
646 |
gclog_or_tty->print_cr(" after = "SIZE_FORMAT".", _n_coarse_entries); |
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
647 |
} |
1374 | 648 |
|
649 |
// Now do the fine-grained maps. |
|
650 |
for (size_t i = 0; i < _max_fine_entries; i++) { |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
651 |
PerRegionTable* cur = _fine_grain_regions[i]; |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
652 |
PerRegionTable** prev = &_fine_grain_regions[i]; |
1374 | 653 |
while (cur != NULL) { |
13335 | 654 |
PerRegionTable* nxt = cur->collision_list_next(); |
1374 | 655 |
// If the entire region is dead, eliminate. |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
656 |
if (G1RSScrubVerbose) { |
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
657 |
gclog_or_tty->print_cr(" For other region %u:", |
9989 | 658 |
cur->hr()->hrs_index()); |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
659 |
} |
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
660 |
if (!region_bm->at((size_t) cur->hr()->hrs_index())) { |
1374 | 661 |
*prev = nxt; |
13335 | 662 |
cur->set_collision_list_next(NULL); |
1374 | 663 |
_n_fine_entries--; |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
664 |
if (G1RSScrubVerbose) { |
1374 | 665 |
gclog_or_tty->print_cr(" deleted via region map."); |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
666 |
} |
13335 | 667 |
unlink_from_all(cur); |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
668 |
PerRegionTable::free(cur); |
1374 | 669 |
} else { |
670 |
// Do fine-grain elimination. |
|
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
671 |
if (G1RSScrubVerbose) { |
1374 | 672 |
gclog_or_tty->print(" occ: before = %4d.", cur->occupied()); |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
673 |
} |
1374 | 674 |
cur->scrub(ctbs, card_bm); |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
675 |
if (G1RSScrubVerbose) { |
1374 | 676 |
gclog_or_tty->print_cr(" after = %4d.", cur->occupied()); |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
677 |
} |
1374 | 678 |
// Did that empty the table completely? |
679 |
if (cur->occupied() == 0) { |
|
680 |
*prev = nxt; |
|
13335 | 681 |
cur->set_collision_list_next(NULL); |
1374 | 682 |
_n_fine_entries--; |
13335 | 683 |
unlink_from_all(cur); |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
684 |
PerRegionTable::free(cur); |
1374 | 685 |
} else { |
13335 | 686 |
prev = cur->collision_list_next_addr(); |
1374 | 687 |
} |
688 |
} |
|
689 |
cur = nxt; |
|
690 |
} |
|
691 |
} |
|
692 |
// Since we may have deleted a from_card_cache entry from the RS, clear |
|
693 |
// the FCC. |
|
694 |
clear_fcc(); |
|
695 |
} |
|
696 |
||
697 |
||
698 |
size_t OtherRegionsTable::occupied() const { |
|
699 |
size_t sum = occ_fine(); |
|
700 |
sum += occ_sparse(); |
|
701 |
sum += occ_coarse(); |
|
702 |
return sum; |
|
703 |
} |
|
704 |
||
705 |
size_t OtherRegionsTable::occ_fine() const { |
|
706 |
size_t sum = 0; |
|
13335 | 707 |
|
708 |
size_t num = 0; |
|
709 |
PerRegionTable * cur = _first_all_fine_prts; |
|
710 |
while (cur != NULL) { |
|
711 |
sum += cur->occupied(); |
|
712 |
cur = cur->next(); |
|
713 |
num++; |
|
1374 | 714 |
} |
13335 | 715 |
guarantee(num == _n_fine_entries, "just checking"); |
1374 | 716 |
return sum; |
717 |
} |
|
718 |
||
719 |
size_t OtherRegionsTable::occ_coarse() const { |
|
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3262
diff
changeset
|
720 |
return (_n_coarse_entries * HeapRegion::CardsPerRegion); |
1374 | 721 |
} |
722 |
||
723 |
size_t OtherRegionsTable::occ_sparse() const { |
|
724 |
return _sparse_table.occupied(); |
|
725 |
} |
|
726 |
||
727 |
size_t OtherRegionsTable::mem_size() const { |
|
728 |
size_t sum = 0; |
|
17395
2cd3ef1be718
8014240: G1: Add remembered set size information to output of G1PrintRegionLivenessInfo
tschatzl
parents:
17119
diff
changeset
|
729 |
// all PRTs are of the same size so it is sufficient to query only one of them. |
2cd3ef1be718
8014240: G1: Add remembered set size information to output of G1PrintRegionLivenessInfo
tschatzl
parents:
17119
diff
changeset
|
730 |
if (_first_all_fine_prts != NULL) { |
2cd3ef1be718
8014240: G1: Add remembered set size information to output of G1PrintRegionLivenessInfo
tschatzl
parents:
17119
diff
changeset
|
731 |
assert(_last_all_fine_prts != NULL && |
2cd3ef1be718
8014240: G1: Add remembered set size information to output of G1PrintRegionLivenessInfo
tschatzl
parents:
17119
diff
changeset
|
732 |
_first_all_fine_prts->mem_size() == _last_all_fine_prts->mem_size(), "check that mem_size() is constant"); |
2cd3ef1be718
8014240: G1: Add remembered set size information to output of G1PrintRegionLivenessInfo
tschatzl
parents:
17119
diff
changeset
|
733 |
sum += _first_all_fine_prts->mem_size() * _n_fine_entries; |
1374 | 734 |
} |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
735 |
sum += (sizeof(PerRegionTable*) * _max_fine_entries); |
1374 | 736 |
sum += (_coarse_map.size_in_words() * HeapWordSize); |
737 |
sum += (_sparse_table.mem_size()); |
|
24108
dfc2242fc6b0
8040792: G1: Memory usage calculation uses sizeof(this) instead of sizeof(classname)
tschatzl
parents:
24106
diff
changeset
|
738 |
sum += sizeof(OtherRegionsTable) - sizeof(_sparse_table); // Avoid double counting above. |
1374 | 739 |
return sum; |
740 |
} |
|
741 |
||
742 |
size_t OtherRegionsTable::static_mem_size() { |
|
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
743 |
return FromCardCache::static_mem_size(); |
1374 | 744 |
} |
745 |
||
746 |
size_t OtherRegionsTable::fl_mem_size() { |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
747 |
return PerRegionTable::fl_mem_size(); |
1374 | 748 |
} |
749 |
||
750 |
void OtherRegionsTable::clear_fcc() { |
|
23456
9a9485a32cb3
8034868: Extract G1 From Card Cache into separate class
tschatzl
parents:
23453
diff
changeset
|
751 |
FromCardCache::clear(hr()->hrs_index()); |
1374 | 752 |
} |
753 |
||
754 |
void OtherRegionsTable::clear() { |
|
13335 | 755 |
// if there are no entries, skip this step |
756 |
if (_first_all_fine_prts != NULL) { |
|
757 |
guarantee(_first_all_fine_prts != NULL && _last_all_fine_prts != NULL, "just checking"); |
|
758 |
PerRegionTable::bulk_free(_first_all_fine_prts, _last_all_fine_prts); |
|
759 |
memset(_fine_grain_regions, 0, _max_fine_entries * sizeof(_fine_grain_regions[0])); |
|
760 |
} else { |
|
761 |
guarantee(_first_all_fine_prts == NULL && _last_all_fine_prts == NULL, "just checking"); |
|
1374 | 762 |
} |
13335 | 763 |
|
764 |
_first_all_fine_prts = _last_all_fine_prts = NULL; |
|
1374 | 765 |
_sparse_table.clear(); |
766 |
_coarse_map.clear(); |
|
767 |
_n_fine_entries = 0; |
|
768 |
_n_coarse_entries = 0; |
|
769 |
||
770 |
clear_fcc(); |
|
771 |
} |
|
772 |
||
773 |
bool OtherRegionsTable::del_single_region_table(size_t ind, |
|
774 |
HeapRegion* hr) { |
|
775 |
assert(0 <= ind && ind < _max_fine_entries, "Preconditions."); |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
776 |
PerRegionTable** prev_addr = &_fine_grain_regions[ind]; |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
777 |
PerRegionTable* prt = *prev_addr; |
1374 | 778 |
while (prt != NULL && prt->hr() != hr) { |
13335 | 779 |
prev_addr = prt->collision_list_next_addr(); |
780 |
prt = prt->collision_list_next(); |
|
1374 | 781 |
} |
782 |
if (prt != NULL) { |
|
783 |
assert(prt->hr() == hr, "Loop postcondition."); |
|
13335 | 784 |
*prev_addr = prt->collision_list_next(); |
785 |
unlink_from_all(prt); |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
786 |
PerRegionTable::free(prt); |
1374 | 787 |
_n_fine_entries--; |
788 |
return true; |
|
789 |
} else { |
|
790 |
return false; |
|
791 |
} |
|
792 |
} |
|
793 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
794 |
bool OtherRegionsTable::contains_reference(OopOrNarrowOopStar from) const { |
1374 | 795 |
// Cast away const in this case. |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
796 |
MutexLockerEx x((Mutex*)_m, Mutex::_no_safepoint_check_flag); |
1374 | 797 |
return contains_reference_locked(from); |
798 |
} |
|
799 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
800 |
bool OtherRegionsTable::contains_reference_locked(OopOrNarrowOopStar from) const { |
1374 | 801 |
HeapRegion* hr = _g1h->heap_region_containing_raw(from); |
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
802 |
RegionIdx_t hr_ind = (RegionIdx_t) hr->hrs_index(); |
1374 | 803 |
// Is this region in the coarse map? |
804 |
if (_coarse_map.at(hr_ind)) return true; |
|
805 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
806 |
PerRegionTable* prt = find_region_table(hr_ind & _mod_max_fine_entries_mask, |
1374 | 807 |
hr); |
808 |
if (prt != NULL) { |
|
809 |
return prt->contains_reference(from); |
|
810 |
||
811 |
} else { |
|
812 |
uintptr_t from_card = |
|
813 |
(uintptr_t(from) >> CardTableModRefBS::card_shift); |
|
814 |
uintptr_t hr_bot_card_index = |
|
815 |
uintptr_t(hr->bottom()) >> CardTableModRefBS::card_shift; |
|
816 |
assert(from_card >= hr_bot_card_index, "Inv"); |
|
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
817 |
CardIdx_t card_index = from_card - hr_bot_card_index; |
10677
370a8da2d63f
7095194: G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
johnc
parents:
10243
diff
changeset
|
818 |
assert(0 <= card_index && (size_t)card_index < HeapRegion::CardsPerRegion, |
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3262
diff
changeset
|
819 |
"Must be in range."); |
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
820 |
return _sparse_table.contains_card(hr_ind, card_index); |
1374 | 821 |
} |
822 |
} |
|
823 |
||
8072 | 824 |
void |
825 |
OtherRegionsTable::do_cleanup_work(HRRSCleanupTask* hrrs_cleanup_task) { |
|
826 |
_sparse_table.do_cleanup_work(hrrs_cleanup_task); |
|
827 |
} |
|
828 |
||
2882
d508a8bac491
6841831: G1: assert(contains_reference(from),"We just added it!") fires
iveresov
parents:
2881
diff
changeset
|
829 |
// Determines how many threads can add records to an rset in parallel. |
d508a8bac491
6841831: G1: assert(contains_reference(from),"We just added it!") fires
iveresov
parents:
2881
diff
changeset
|
830 |
// This can be done by either mutator threads together with the |
d508a8bac491
6841831: G1: assert(contains_reference(from),"We just added it!") fires
iveresov
parents:
2881
diff
changeset
|
831 |
// concurrent refinement threads or GC threads. |
23452
d7dca4e6b95d
8035815: Cache-align and pad the from card cache
tschatzl
parents:
23451
diff
changeset
|
832 |
uint HeapRegionRemSet::num_par_rem_sets() { |
23855
c4574075402c
8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents:
23456
diff
changeset
|
833 |
return MAX2(DirtyCardQueueSet::num_par_ids() + ConcurrentG1Refine::thread_num(), (uint)ParallelGCThreads); |
1374 | 834 |
} |
835 |
||
836 |
HeapRegionRemSet::HeapRegionRemSet(G1BlockOffsetSharedArray* bosa, |
|
837 |
HeapRegion* hr) |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
838 |
: _bosa(bosa), |
23858
dae377f5a7c7
8039244: Don't use UINT32_FORMAT and INT32_FORMAT when printing uints and ints in the GC code
stefank
parents:
23855
diff
changeset
|
839 |
_m(Mutex::leaf, FormatBuffer<128>("HeapRegionRemSet lock #%u", hr->hrs_index()), true), |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
840 |
_code_roots(), _other_regions(hr, &_m) { |
10000
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
841 |
reset_for_par_iteration(); |
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
842 |
} |
1374 | 843 |
|
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
844 |
void HeapRegionRemSet::setup_remset_size() { |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
845 |
// Setup sparse and fine-grain tables sizes. |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
846 |
// table_size = base * (log(region_size / 1M) + 1) |
12779
06778de5193e
7171936: LOG_G incorrectly defined in globalDefinitions.hpp
brutisso
parents:
12381
diff
changeset
|
847 |
const int LOG_M = 20; |
06778de5193e
7171936: LOG_G incorrectly defined in globalDefinitions.hpp
brutisso
parents:
12381
diff
changeset
|
848 |
int region_size_log_mb = MAX2(HeapRegion::LogOfHRGrainBytes - LOG_M, 0); |
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
849 |
if (FLAG_IS_DEFAULT(G1RSetSparseRegionEntries)) { |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
850 |
G1RSetSparseRegionEntries = G1RSetSparseRegionEntriesBase * (region_size_log_mb + 1); |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
851 |
} |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
852 |
if (FLAG_IS_DEFAULT(G1RSetRegionEntries)) { |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
853 |
G1RSetRegionEntries = G1RSetRegionEntriesBase * (region_size_log_mb + 1); |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
854 |
} |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
855 |
guarantee(G1RSetSparseRegionEntries > 0 && G1RSetRegionEntries > 0 , "Sanity"); |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
856 |
} |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
857 |
|
1374 | 858 |
bool HeapRegionRemSet::claim_iter() { |
859 |
if (_iter_state != Unclaimed) return false; |
|
860 |
jint res = Atomic::cmpxchg(Claimed, (jint*)(&_iter_state), Unclaimed); |
|
861 |
return (res == Unclaimed); |
|
862 |
} |
|
863 |
||
864 |
void HeapRegionRemSet::set_iter_complete() { |
|
865 |
_iter_state = Complete; |
|
866 |
} |
|
867 |
||
868 |
bool HeapRegionRemSet::iter_is_complete() { |
|
869 |
return _iter_state == Complete; |
|
870 |
} |
|
871 |
||
872 |
#ifndef PRODUCT |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
873 |
void HeapRegionRemSet::print() { |
17108
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
874 |
HeapRegionRemSetIterator iter(this); |
1374 | 875 |
size_t card_index; |
876 |
while (iter.has_next(card_index)) { |
|
877 |
HeapWord* card_start = |
|
878 |
G1CollectedHeap::heap()->bot_shared()->address_for_index(card_index); |
|
10000
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
879 |
gclog_or_tty->print_cr(" Card " PTR_FORMAT, card_start); |
1374 | 880 |
} |
881 |
if (iter.n_yielded() != occupied()) { |
|
882 |
gclog_or_tty->print_cr("Yielded disagrees with occupied:"); |
|
24092
e274d864545a
8039743: Use correct format specifier to print size_t values and pointers in the GC code
stefank
parents:
23858
diff
changeset
|
883 |
gclog_or_tty->print_cr(" " SIZE_FORMAT_W(6) " yielded (" SIZE_FORMAT_W(6) |
e274d864545a
8039743: Use correct format specifier to print size_t values and pointers in the GC code
stefank
parents:
23858
diff
changeset
|
884 |
" coarse, " SIZE_FORMAT_W(6) " fine).", |
1374 | 885 |
iter.n_yielded(), |
886 |
iter.n_yielded_coarse(), iter.n_yielded_fine()); |
|
24092
e274d864545a
8039743: Use correct format specifier to print size_t values and pointers in the GC code
stefank
parents:
23858
diff
changeset
|
887 |
gclog_or_tty->print_cr(" " SIZE_FORMAT_W(6) " occ (" SIZE_FORMAT_W(6) |
e274d864545a
8039743: Use correct format specifier to print size_t values and pointers in the GC code
stefank
parents:
23858
diff
changeset
|
888 |
" coarse, " SIZE_FORMAT_W(6) " fine).", |
1374 | 889 |
occupied(), occ_coarse(), occ_fine()); |
890 |
} |
|
891 |
guarantee(iter.n_yielded() == occupied(), |
|
892 |
"We should have yielded all the represented cards."); |
|
893 |
} |
|
894 |
#endif |
|
895 |
||
896 |
void HeapRegionRemSet::cleanup() { |
|
897 |
SparsePRT::cleanup_all(); |
|
898 |
} |
|
899 |
||
900 |
void HeapRegionRemSet::clear() { |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
901 |
MutexLockerEx x(&_m, Mutex::_no_safepoint_check_flag); |
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
902 |
clear_locked(); |
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
903 |
} |
19339 | 904 |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
905 |
void HeapRegionRemSet::clear_locked() { |
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
906 |
_code_roots.clear(); |
1374 | 907 |
_other_regions.clear(); |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
908 |
assert(occupied_locked() == 0, "Should be clear."); |
10000
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
909 |
reset_for_par_iteration(); |
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
910 |
} |
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
911 |
|
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
912 |
void HeapRegionRemSet::reset_for_par_iteration() { |
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
913 |
_iter_state = Unclaimed; |
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
914 |
_iter_claimed = 0; |
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
915 |
// It's good to check this to make sure that the two methods are in sync. |
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
916 |
assert(verify_ready_for_par_iteration(), "post-condition"); |
1374 | 917 |
} |
918 |
||
919 |
void HeapRegionRemSet::scrub(CardTableModRefBS* ctbs, |
|
920 |
BitMap* region_bm, BitMap* card_bm) { |
|
921 |
_other_regions.scrub(ctbs, region_bm, card_bm); |
|
922 |
} |
|
923 |
||
19339 | 924 |
// Code roots support |
925 |
||
926 |
void HeapRegionRemSet::add_strong_code_root(nmethod* nm) { |
|
927 |
assert(nm != NULL, "sanity"); |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
928 |
_code_roots.add(nm); |
19339 | 929 |
} |
930 |
||
931 |
void HeapRegionRemSet::remove_strong_code_root(nmethod* nm) { |
|
932 |
assert(nm != NULL, "sanity"); |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
933 |
_code_roots.remove(nm); |
19339 | 934 |
// Check that there were no duplicates |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
935 |
guarantee(!_code_roots.contains(nm), "duplicate entry found"); |
19339 | 936 |
} |
937 |
||
938 |
class NMethodMigrationOopClosure : public OopClosure { |
|
939 |
G1CollectedHeap* _g1h; |
|
940 |
HeapRegion* _from; |
|
941 |
nmethod* _nm; |
|
942 |
||
943 |
uint _num_self_forwarded; |
|
944 |
||
945 |
template <class T> void do_oop_work(T* p) { |
|
946 |
T heap_oop = oopDesc::load_heap_oop(p); |
|
947 |
if (!oopDesc::is_null(heap_oop)) { |
|
948 |
oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
|
949 |
if (_from->is_in(obj)) { |
|
950 |
// Reference still points into the source region. |
|
951 |
// Since roots are immediately evacuated this means that |
|
952 |
// we must have self forwarded the object |
|
953 |
assert(obj->is_forwarded(), |
|
954 |
err_msg("code roots should be immediately evacuated. " |
|
955 |
"Ref: "PTR_FORMAT", " |
|
956 |
"Obj: "PTR_FORMAT", " |
|
957 |
"Region: "HR_FORMAT, |
|
958 |
p, (void*) obj, HR_FORMAT_PARAMS(_from))); |
|
959 |
assert(obj->forwardee() == obj, |
|
960 |
err_msg("not self forwarded? obj = "PTR_FORMAT, (void*)obj)); |
|
961 |
||
962 |
// The object has been self forwarded. |
|
963 |
// Note, if we're during an initial mark pause, there is |
|
964 |
// no need to explicitly mark object. It will be marked |
|
965 |
// during the regular evacuation failure handling code. |
|
966 |
_num_self_forwarded++; |
|
967 |
} else { |
|
968 |
// The reference points into a promotion or to-space region |
|
969 |
HeapRegion* to = _g1h->heap_region_containing(obj); |
|
970 |
to->rem_set()->add_strong_code_root(_nm); |
|
971 |
} |
|
972 |
} |
|
973 |
} |
|
974 |
||
975 |
public: |
|
976 |
NMethodMigrationOopClosure(G1CollectedHeap* g1h, HeapRegion* from, nmethod* nm): |
|
977 |
_g1h(g1h), _from(from), _nm(nm), _num_self_forwarded(0) {} |
|
978 |
||
979 |
void do_oop(narrowOop* p) { do_oop_work(p); } |
|
980 |
void do_oop(oop* p) { do_oop_work(p); } |
|
981 |
||
982 |
uint retain() { return _num_self_forwarded > 0; } |
|
983 |
}; |
|
984 |
||
985 |
void HeapRegionRemSet::migrate_strong_code_roots() { |
|
986 |
assert(hr()->in_collection_set(), "only collection set regions"); |
|
21563
ccbd86010788
8027756: assert(!hr->isHumongous()) failed: code root in humongous region?
tschatzl
parents:
20282
diff
changeset
|
987 |
assert(!hr()->isHumongous(), |
ccbd86010788
8027756: assert(!hr->isHumongous()) failed: code root in humongous region?
tschatzl
parents:
20282
diff
changeset
|
988 |
err_msg("humongous region "HR_FORMAT" should not have been added to the collection set", |
ccbd86010788
8027756: assert(!hr->isHumongous()) failed: code root in humongous region?
tschatzl
parents:
20282
diff
changeset
|
989 |
HR_FORMAT_PARAMS(hr()))); |
19339 | 990 |
|
991 |
ResourceMark rm; |
|
992 |
||
993 |
// List of code blobs to retain for this region |
|
994 |
GrowableArray<nmethod*> to_be_retained(10); |
|
995 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
|
996 |
||
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
997 |
while (!_code_roots.is_empty()) { |
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
998 |
nmethod *nm = _code_roots.pop(); |
19339 | 999 |
if (nm != NULL) { |
1000 |
NMethodMigrationOopClosure oop_cl(g1h, hr(), nm); |
|
1001 |
nm->oops_do(&oop_cl); |
|
1002 |
if (oop_cl.retain()) { |
|
1003 |
to_be_retained.push(nm); |
|
1004 |
} |
|
1005 |
} |
|
1006 |
} |
|
1007 |
||
1008 |
// Now push any code roots we need to retain |
|
1009 |
assert(to_be_retained.is_empty() || hr()->evacuation_failed(), |
|
1010 |
"Retained nmethod list must be empty or " |
|
1011 |
"evacuation of this region failed"); |
|
1012 |
||
1013 |
while (to_be_retained.is_nonempty()) { |
|
1014 |
nmethod* nm = to_be_retained.pop(); |
|
1015 |
assert(nm != NULL, "sanity"); |
|
1016 |
add_strong_code_root(nm); |
|
1017 |
} |
|
1018 |
} |
|
1019 |
||
1020 |
void HeapRegionRemSet::strong_code_roots_do(CodeBlobClosure* blk) const { |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
1021 |
_code_roots.nmethods_do(blk); |
19339 | 1022 |
} |
1023 |
||
1024 |
size_t HeapRegionRemSet::strong_code_roots_mem_size() { |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
1025 |
return _code_roots.mem_size(); |
19339 | 1026 |
} |
1027 |
||
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
1028 |
HeapRegionRemSetIterator:: HeapRegionRemSetIterator(HeapRegionRemSet* hrrs) : |
17108
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1029 |
_hrrs(hrrs), |
1374 | 1030 |
_g1h(G1CollectedHeap::heap()), |
17108
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1031 |
_coarse_map(&hrrs->_other_regions._coarse_map), |
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1032 |
_bosa(hrrs->bosa()), |
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1033 |
_is(Sparse), |
1374 | 1034 |
// Set these values so that we increment to the first region. |
17108
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1035 |
_coarse_cur_region_index(-1), |
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1036 |
_coarse_cur_region_cur_card(HeapRegion::CardsPerRegion-1), |
24101
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1037 |
_cur_card_in_prt(HeapRegion::CardsPerRegion), |
17108
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1038 |
_fine_cur_prt(NULL), |
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1039 |
_n_yielded_coarse(0), |
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1040 |
_n_yielded_fine(0), |
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1041 |
_n_yielded_sparse(0), |
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1042 |
_sparse_iter(&hrrs->_other_regions._sparse_table) {} |
1374 | 1043 |
|
1044 |
bool HeapRegionRemSetIterator::coarse_has_next(size_t& card_index) { |
|
1045 |
if (_hrrs->_other_regions._n_coarse_entries == 0) return false; |
|
1046 |
// Go to the next card. |
|
1047 |
_coarse_cur_region_cur_card++; |
|
1048 |
// Was the last the last card in the current region? |
|
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3262
diff
changeset
|
1049 |
if (_coarse_cur_region_cur_card == HeapRegion::CardsPerRegion) { |
1374 | 1050 |
// Yes: find the next region. This may leave _coarse_cur_region_index |
1051 |
// Set to the last index, in which case there are no more coarse |
|
1052 |
// regions. |
|
1053 |
_coarse_cur_region_index = |
|
1054 |
(int) _coarse_map->get_next_one_offset(_coarse_cur_region_index + 1); |
|
1055 |
if ((size_t)_coarse_cur_region_index < _coarse_map->size()) { |
|
1056 |
_coarse_cur_region_cur_card = 0; |
|
1057 |
HeapWord* r_bot = |
|
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10767
diff
changeset
|
1058 |
_g1h->region_at((uint) _coarse_cur_region_index)->bottom(); |
1374 | 1059 |
_cur_region_card_offset = _bosa->index_for(r_bot); |
1060 |
} else { |
|
1061 |
return false; |
|
1062 |
} |
|
1063 |
} |
|
1064 |
// If we didn't return false above, then we can yield a card. |
|
1065 |
card_index = _cur_region_card_offset + _coarse_cur_region_cur_card; |
|
1066 |
return true; |
|
1067 |
} |
|
1068 |
||
1069 |
bool HeapRegionRemSetIterator::fine_has_next(size_t& card_index) { |
|
1070 |
if (fine_has_next()) { |
|
24101
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1071 |
_cur_card_in_prt = |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1072 |
_fine_cur_prt->_bm.get_next_one_offset(_cur_card_in_prt + 1); |
1374 | 1073 |
} |
24101
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1074 |
if (_cur_card_in_prt == HeapRegion::CardsPerRegion) { |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1075 |
// _fine_cur_prt may still be NULL in case if there are not PRTs at all for |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1076 |
// the remembered set. |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1077 |
if (_fine_cur_prt == NULL || _fine_cur_prt->next() == NULL) { |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1078 |
return false; |
1374 | 1079 |
} |
24101
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1080 |
PerRegionTable* next_prt = _fine_cur_prt->next(); |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1081 |
switch_to_prt(next_prt); |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1082 |
_cur_card_in_prt = _fine_cur_prt->_bm.get_next_one_offset(_cur_card_in_prt + 1); |
1374 | 1083 |
} |
24101
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1084 |
|
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1085 |
card_index = _cur_region_card_offset + _cur_card_in_prt; |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1086 |
guarantee(_cur_card_in_prt < HeapRegion::CardsPerRegion, |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1087 |
err_msg("Card index "SIZE_FORMAT" must be within the region", _cur_card_in_prt)); |
1374 | 1088 |
return true; |
1089 |
} |
|
1090 |
||
1091 |
bool HeapRegionRemSetIterator::fine_has_next() { |
|
24101
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1092 |
return _cur_card_in_prt != HeapRegion::CardsPerRegion; |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1093 |
} |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1094 |
|
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1095 |
void HeapRegionRemSetIterator::switch_to_prt(PerRegionTable* prt) { |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1096 |
assert(prt != NULL, "Cannot switch to NULL prt"); |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1097 |
_fine_cur_prt = prt; |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1098 |
|
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1099 |
HeapWord* r_bot = _fine_cur_prt->hr()->bottom(); |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1100 |
_cur_region_card_offset = _bosa->index_for(r_bot); |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1101 |
|
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1102 |
// The bitmap scan for the PRT always scans from _cur_region_cur_card + 1. |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1103 |
// To avoid special-casing this start case, and not miss the first bitmap |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1104 |
// entry, initialize _cur_region_cur_card with -1 instead of 0. |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1105 |
_cur_card_in_prt = (size_t)-1; |
1374 | 1106 |
} |
1107 |
||
1108 |
bool HeapRegionRemSetIterator::has_next(size_t& card_index) { |
|
1109 |
switch (_is) { |
|
24101
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1110 |
case Sparse: { |
1374 | 1111 |
if (_sparse_iter.has_next(card_index)) { |
1112 |
_n_yielded_sparse++; |
|
1113 |
return true; |
|
1114 |
} |
|
1115 |
// Otherwise, deliberate fall-through |
|
1116 |
_is = Fine; |
|
24101
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1117 |
PerRegionTable* initial_fine_prt = _hrrs->_other_regions._first_all_fine_prts; |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1118 |
if (initial_fine_prt != NULL) { |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1119 |
switch_to_prt(_hrrs->_other_regions._first_all_fine_prts); |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1120 |
} |
bd1a2ee20760
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
tschatzl
parents:
24092
diff
changeset
|
1121 |
} |
1374 | 1122 |
case Fine: |
1123 |
if (fine_has_next(card_index)) { |
|
1124 |
_n_yielded_fine++; |
|
1125 |
return true; |
|
1126 |
} |
|
1127 |
// Otherwise, deliberate fall-through |
|
1128 |
_is = Coarse; |
|
1129 |
case Coarse: |
|
1130 |
if (coarse_has_next(card_index)) { |
|
1131 |
_n_yielded_coarse++; |
|
1132 |
return true; |
|
1133 |
} |
|
1134 |
// Otherwise... |
|
1135 |
break; |
|
1136 |
} |
|
1137 |
assert(ParallelGCThreads > 1 || |
|
1138 |
n_yielded() == _hrrs->occupied(), |
|
1139 |
"Should have yielded all the cards in the rem set " |
|
1140 |
"(in the non-par case)."); |
|
1141 |
return false; |
|
1142 |
} |
|
1143 |
||
1144 |
||
1145 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1146 |
OopOrNarrowOopStar* HeapRegionRemSet::_recorded_oops = NULL; |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1147 |
HeapWord** HeapRegionRemSet::_recorded_cards = NULL; |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1148 |
HeapRegion** HeapRegionRemSet::_recorded_regions = NULL; |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1149 |
int HeapRegionRemSet::_n_recorded = 0; |
1374 | 1150 |
|
1151 |
HeapRegionRemSet::Event* HeapRegionRemSet::_recorded_events = NULL; |
|
1152 |
int* HeapRegionRemSet::_recorded_event_index = NULL; |
|
1153 |
int HeapRegionRemSet::_n_recorded_events = 0; |
|
1154 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1155 |
void HeapRegionRemSet::record(HeapRegion* hr, OopOrNarrowOopStar f) { |
1374 | 1156 |
if (_recorded_oops == NULL) { |
1157 |
assert(_n_recorded == 0 |
|
1158 |
&& _recorded_cards == NULL |
|
1159 |
&& _recorded_regions == NULL, |
|
1160 |
"Inv"); |
|
13195 | 1161 |
_recorded_oops = NEW_C_HEAP_ARRAY(OopOrNarrowOopStar, MaxRecorded, mtGC); |
1162 |
_recorded_cards = NEW_C_HEAP_ARRAY(HeapWord*, MaxRecorded, mtGC); |
|
1163 |
_recorded_regions = NEW_C_HEAP_ARRAY(HeapRegion*, MaxRecorded, mtGC); |
|
1374 | 1164 |
} |
1165 |
if (_n_recorded == MaxRecorded) { |
|
1166 |
gclog_or_tty->print_cr("Filled up 'recorded' (%d).", MaxRecorded); |
|
1167 |
} else { |
|
1168 |
_recorded_cards[_n_recorded] = |
|
1169 |
(HeapWord*)align_size_down(uintptr_t(f), |
|
1170 |
CardTableModRefBS::card_size); |
|
1171 |
_recorded_oops[_n_recorded] = f; |
|
1172 |
_recorded_regions[_n_recorded] = hr; |
|
1173 |
_n_recorded++; |
|
1174 |
} |
|
1175 |
} |
|
1176 |
||
1177 |
void HeapRegionRemSet::record_event(Event evnt) { |
|
1178 |
if (!G1RecordHRRSEvents) return; |
|
1179 |
||
1180 |
if (_recorded_events == NULL) { |
|
1181 |
assert(_n_recorded_events == 0 |
|
1182 |
&& _recorded_event_index == NULL, |
|
1183 |
"Inv"); |
|
13195 | 1184 |
_recorded_events = NEW_C_HEAP_ARRAY(Event, MaxRecordedEvents, mtGC); |
1185 |
_recorded_event_index = NEW_C_HEAP_ARRAY(int, MaxRecordedEvents, mtGC); |
|
1374 | 1186 |
} |
1187 |
if (_n_recorded_events == MaxRecordedEvents) { |
|
1188 |
gclog_or_tty->print_cr("Filled up 'recorded_events' (%d).", MaxRecordedEvents); |
|
1189 |
} else { |
|
1190 |
_recorded_events[_n_recorded_events] = evnt; |
|
1191 |
_recorded_event_index[_n_recorded_events] = _n_recorded; |
|
1192 |
_n_recorded_events++; |
|
1193 |
} |
|
1194 |
} |
|
1195 |
||
1196 |
void HeapRegionRemSet::print_event(outputStream* str, Event evnt) { |
|
1197 |
switch (evnt) { |
|
1198 |
case Event_EvacStart: |
|
1199 |
str->print("Evac Start"); |
|
1200 |
break; |
|
1201 |
case Event_EvacEnd: |
|
1202 |
str->print("Evac End"); |
|
1203 |
break; |
|
1204 |
case Event_RSUpdateEnd: |
|
1205 |
str->print("RS Update End"); |
|
1206 |
break; |
|
1207 |
} |
|
1208 |
} |
|
1209 |
||
1210 |
void HeapRegionRemSet::print_recorded() { |
|
1211 |
int cur_evnt = 0; |
|
1212 |
Event cur_evnt_kind; |
|
1213 |
int cur_evnt_ind = 0; |
|
1214 |
if (_n_recorded_events > 0) { |
|
1215 |
cur_evnt_kind = _recorded_events[cur_evnt]; |
|
1216 |
cur_evnt_ind = _recorded_event_index[cur_evnt]; |
|
1217 |
} |
|
1218 |
||
1219 |
for (int i = 0; i < _n_recorded; i++) { |
|
1220 |
while (cur_evnt < _n_recorded_events && i == cur_evnt_ind) { |
|
1221 |
gclog_or_tty->print("Event: "); |
|
1222 |
print_event(gclog_or_tty, cur_evnt_kind); |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
24108
diff
changeset
|
1223 |
gclog_or_tty->cr(); |
1374 | 1224 |
cur_evnt++; |
1225 |
if (cur_evnt < MaxRecordedEvents) { |
|
1226 |
cur_evnt_kind = _recorded_events[cur_evnt]; |
|
1227 |
cur_evnt_ind = _recorded_event_index[cur_evnt]; |
|
1228 |
} |
|
1229 |
} |
|
1230 |
gclog_or_tty->print("Added card " PTR_FORMAT " to region [" PTR_FORMAT "...]" |
|
1231 |
" for ref " PTR_FORMAT ".\n", |
|
1232 |
_recorded_cards[i], _recorded_regions[i]->bottom(), |
|
1233 |
_recorded_oops[i]); |
|
1234 |
} |
|
1235 |
} |
|
1236 |
||
8072 | 1237 |
void HeapRegionRemSet::reset_for_cleanup_tasks() { |
1238 |
SparsePRT::reset_for_cleanup_tasks(); |
|
1239 |
} |
|
1240 |
||
1241 |
void HeapRegionRemSet::do_cleanup_work(HRRSCleanupTask* hrrs_cleanup_task) { |
|
1242 |
_other_regions.do_cleanup_work(hrrs_cleanup_task); |
|
1243 |
} |
|
1244 |
||
1245 |
void |
|
1246 |
HeapRegionRemSet::finish_cleanup_task(HRRSCleanupTask* hrrs_cleanup_task) { |
|
1247 |
SparsePRT::finish_cleanup_task(hrrs_cleanup_task); |
|
1248 |
} |
|
1249 |
||
1374 | 1250 |
#ifndef PRODUCT |
17631
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1251 |
void PerRegionTable::test_fl_mem_size() { |
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1252 |
PerRegionTable* dummy = alloc(NULL); |
24108
dfc2242fc6b0
8040792: G1: Memory usage calculation uses sizeof(this) instead of sizeof(classname)
tschatzl
parents:
24106
diff
changeset
|
1253 |
|
dfc2242fc6b0
8040792: G1: Memory usage calculation uses sizeof(this) instead of sizeof(classname)
tschatzl
parents:
24106
diff
changeset
|
1254 |
size_t min_prt_size = sizeof(void*) + dummy->bm()->size_in_words() * HeapWordSize; |
dfc2242fc6b0
8040792: G1: Memory usage calculation uses sizeof(this) instead of sizeof(classname)
tschatzl
parents:
24106
diff
changeset
|
1255 |
assert(dummy->mem_size() > min_prt_size, |
dfc2242fc6b0
8040792: G1: Memory usage calculation uses sizeof(this) instead of sizeof(classname)
tschatzl
parents:
24106
diff
changeset
|
1256 |
err_msg("PerRegionTable memory usage is suspiciously small, only has "SIZE_FORMAT" bytes. " |
dfc2242fc6b0
8040792: G1: Memory usage calculation uses sizeof(this) instead of sizeof(classname)
tschatzl
parents:
24106
diff
changeset
|
1257 |
"Should be at least "SIZE_FORMAT" bytes.", dummy->mem_size(), min_prt_size)); |
17631
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1258 |
free(dummy); |
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1259 |
guarantee(dummy->mem_size() == fl_mem_size(), "fl_mem_size() does not return the correct element size"); |
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1260 |
// try to reset the state |
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1261 |
_free_list = NULL; |
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1262 |
delete dummy; |
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1263 |
} |
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1264 |
|
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1265 |
void HeapRegionRemSet::test_prt() { |
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1266 |
PerRegionTable::test_fl_mem_size(); |
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1267 |
} |
17992863b0ab
8014405: G1: PerRegionTable::fl_mem_size() calculates size of the free list using wrong element sizes
tschatzl
parents:
17398
diff
changeset
|
1268 |
|
1374 | 1269 |
void HeapRegionRemSet::test() { |
1270 |
os::sleep(Thread::current(), (jlong)5000, false); |
|
1271 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
|
1272 |
||
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
1273 |
// Run with "-XX:G1LogRSetRegionEntries=2", so that 1 and 5 end up in same |
1374 | 1274 |
// hash bucket. |
1275 |
HeapRegion* hr0 = g1h->region_at(0); |
|
1276 |
HeapRegion* hr1 = g1h->region_at(1); |
|
1277 |
HeapRegion* hr2 = g1h->region_at(5); |
|
1278 |
HeapRegion* hr3 = g1h->region_at(6); |
|
1279 |
HeapRegion* hr4 = g1h->region_at(7); |
|
1280 |
HeapRegion* hr5 = g1h->region_at(8); |
|
1281 |
||
1282 |
HeapWord* hr1_start = hr1->bottom(); |
|
1283 |
HeapWord* hr1_mid = hr1_start + HeapRegion::GrainWords/2; |
|
1284 |
HeapWord* hr1_last = hr1->end() - 1; |
|
1285 |
||
1286 |
HeapWord* hr2_start = hr2->bottom(); |
|
1287 |
HeapWord* hr2_mid = hr2_start + HeapRegion::GrainWords/2; |
|
1288 |
HeapWord* hr2_last = hr2->end() - 1; |
|
1289 |
||
1290 |
HeapWord* hr3_start = hr3->bottom(); |
|
1291 |
HeapWord* hr3_mid = hr3_start + HeapRegion::GrainWords/2; |
|
1292 |
HeapWord* hr3_last = hr3->end() - 1; |
|
1293 |
||
1294 |
HeapRegionRemSet* hrrs = hr0->rem_set(); |
|
1295 |
||
1296 |
// Make three references from region 0x101... |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1297 |
hrrs->add_reference((OopOrNarrowOopStar)hr1_start); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1298 |
hrrs->add_reference((OopOrNarrowOopStar)hr1_mid); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1299 |
hrrs->add_reference((OopOrNarrowOopStar)hr1_last); |
1374 | 1300 |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1301 |
hrrs->add_reference((OopOrNarrowOopStar)hr2_start); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1302 |
hrrs->add_reference((OopOrNarrowOopStar)hr2_mid); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1303 |
hrrs->add_reference((OopOrNarrowOopStar)hr2_last); |
1374 | 1304 |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1305 |
hrrs->add_reference((OopOrNarrowOopStar)hr3_start); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1306 |
hrrs->add_reference((OopOrNarrowOopStar)hr3_mid); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1307 |
hrrs->add_reference((OopOrNarrowOopStar)hr3_last); |
1374 | 1308 |
|
1309 |
// Now cause a coarsening. |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1310 |
hrrs->add_reference((OopOrNarrowOopStar)hr4->bottom()); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
1311 |
hrrs->add_reference((OopOrNarrowOopStar)hr5->bottom()); |
1374 | 1312 |
|
1313 |
// Now, does iteration yield these three? |
|
17108
cf72dcf9a8f2
8011724: G1: Stack allocate instances of HeapRegionRemSetIterator
johnc
parents:
13752
diff
changeset
|
1314 |
HeapRegionRemSetIterator iter(hrrs); |
1374 | 1315 |
size_t sum = 0; |
1316 |
size_t card_index; |
|
1317 |
while (iter.has_next(card_index)) { |
|
1318 |
HeapWord* card_start = |
|
1319 |
G1CollectedHeap::heap()->bot_shared()->address_for_index(card_index); |
|
1320 |
gclog_or_tty->print_cr(" Card " PTR_FORMAT ".", card_start); |
|
1321 |
sum++; |
|
1322 |
} |
|
1323 |
guarantee(sum == 11 - 3 + 2048, "Failure"); |
|
1324 |
guarantee(sum == hrrs->occupied(), "Failure"); |
|
1325 |
} |
|
1326 |
#endif |