author | sangheki |
Wed, 13 Nov 2019 10:49:32 -0800 | |
changeset 59061 | df6f2350edfa |
parent 58375 | a1eba2e37671 |
child 58679 | 9c3209ff7550 |
child 59247 | 56bf71d64d51 |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52716
diff
changeset
|
2 |
* Copyright (c) 2001, 2019, 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" |
30764 | 26 |
#include "gc/g1/g1BlockOffsetTable.inline.hpp" |
27 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
|
47789
a77a7d3bc4f6
8149127: Rename g1/concurrentMarkThread.* to g1/g1ConcurrentMarkThread.*
tschatzl
parents:
47634
diff
changeset
|
28 |
#include "gc/g1/g1ConcurrentRefine.hpp" |
30764 | 29 |
#include "gc/g1/heapRegionManager.inline.hpp" |
55510
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
30 |
#include "gc/g1/heapRegionRemSet.inline.hpp" |
30764 | 31 |
#include "gc/shared/space.inline.hpp" |
7397 | 32 |
#include "memory/allocation.hpp" |
23452
d7dca4e6b95d
8035815: Cache-align and pad the from card cache
tschatzl
parents:
23451
diff
changeset
|
33 |
#include "memory/padded.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" |
40655
9f644073d3a0
8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents:
38271
diff
changeset
|
35 |
#include "runtime/atomic.hpp" |
7397 | 36 |
#include "utilities/bitMap.inline.hpp" |
46560
388aa8d67c80
8181449: Fix debug.hpp / globalDefinitions.hpp dependency inversion
kbarrett
parents:
43666
diff
changeset
|
37 |
#include "utilities/debug.hpp" |
388aa8d67c80
8181449: Fix debug.hpp / globalDefinitions.hpp dependency inversion
kbarrett
parents:
43666
diff
changeset
|
38 |
#include "utilities/formatBuffer.hpp" |
7397 | 39 |
#include "utilities/globalDefinitions.hpp" |
19339 | 40 |
#include "utilities/growableArray.hpp" |
1374 | 41 |
|
49607
acffe6ff3ae7
8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents:
49164
diff
changeset
|
42 |
const char* HeapRegionRemSet::_state_strings[] = {"Untracked", "Updating", "Complete"}; |
49672
f1d2b6c0693b
8200305: Update gc,liveness output with remset state after rebuild remset concurrently changes
tschatzl
parents:
49632
diff
changeset
|
43 |
const char* HeapRegionRemSet::_short_state_strings[] = {"UNTRA", "UPDAT", "CMPLT"}; |
49607
acffe6ff3ae7
8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents:
49164
diff
changeset
|
44 |
|
55510
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
45 |
PerRegionTable* PerRegionTable::alloc(HeapRegion* hr) { |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
46 |
PerRegionTable* fl = _free_list; |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
47 |
while (fl != NULL) { |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
48 |
PerRegionTable* nxt = fl->next(); |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
49 |
PerRegionTable* res = Atomic::cmpxchg(nxt, &_free_list, fl); |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
50 |
if (res == fl) { |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
51 |
fl->init(hr, true); |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
52 |
return fl; |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
53 |
} else { |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
54 |
fl = _free_list; |
1374 | 55 |
} |
56 |
} |
|
55510
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
57 |
assert(fl == NULL, "Loop condition."); |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
58 |
return new PerRegionTable(hr); |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55148
diff
changeset
|
59 |
} |
1374 | 60 |
|
41283
2615c024f3eb
8033552: Fix missing missing volatile specifiers in CAS operations in GC code
eosterlund
parents:
40655
diff
changeset
|
61 |
PerRegionTable* volatile PerRegionTable::_free_list = NULL; |
1374 | 62 |
|
63 |
size_t OtherRegionsTable::_max_fine_entries = 0; |
|
64 |
size_t OtherRegionsTable::_mod_max_fine_entries_mask = 0; |
|
65 |
size_t OtherRegionsTable::_fine_eviction_stride = 0; |
|
66 |
size_t OtherRegionsTable::_fine_eviction_sample_size = 0; |
|
67 |
||
52347
14ef0f74667b
8211388: Make OtherRegionsTable independent of the region it is for
tschatzl
parents:
52345
diff
changeset
|
68 |
OtherRegionsTable::OtherRegionsTable(Mutex* m) : |
1374 | 69 |
_g1h(G1CollectedHeap::heap()), |
51332 | 70 |
_m(m), |
46745
f7b9bb98bb72
8176571: Fine bitmaps should be allocated as belonging to mtGC, not mtInternal
kbarrett
parents:
46560
diff
changeset
|
71 |
_coarse_map(G1CollectedHeap::heap()->max_regions(), mtGC), |
51332 | 72 |
_n_coarse_entries(0), |
1374 | 73 |
_fine_grain_regions(NULL), |
51332 | 74 |
_n_fine_entries(0), |
75 |
_first_all_fine_prts(NULL), |
|
76 |
_last_all_fine_prts(NULL), |
|
1374 | 77 |
_fine_eviction_start(0), |
52347
14ef0f74667b
8211388: Make OtherRegionsTable independent of the region it is for
tschatzl
parents:
52345
diff
changeset
|
78 |
_sparse_table() |
1374 | 79 |
{ |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
80 |
typedef PerRegionTable* PerRegionTablePtr; |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
81 |
|
1374 | 82 |
if (_max_fine_entries == 0) { |
83 |
assert(_mod_max_fine_entries_mask == 0, "Both or none."); |
|
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
84 |
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
|
85 |
_max_fine_entries = (size_t)1 << max_entries_log; |
1374 | 86 |
_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
|
87 |
|
1374 | 88 |
assert(_fine_eviction_sample_size == 0 |
89 |
&& _fine_eviction_stride == 0, "All init at same time."); |
|
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
90 |
_fine_eviction_sample_size = MAX2((size_t)4, max_entries_log); |
1374 | 91 |
_fine_eviction_stride = _max_fine_entries / _fine_eviction_sample_size; |
92 |
} |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
93 |
|
17376
4ee999c3c007
8012902: remove use of global operator new - take 2
minqi
parents:
17119
diff
changeset
|
94 |
_fine_grain_regions = NEW_C_HEAP_ARRAY3(PerRegionTablePtr, _max_fine_entries, |
25946 | 95 |
mtGC, CURRENT_PC, AllocFailStrategy::RETURN_NULL); |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
96 |
|
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
97 |
if (_fine_grain_regions == NULL) { |
17087
f0b76c4c93a0
8011661: Insufficient memory message says "malloc" when sometimes it should say "mmap"
ccheung
parents:
17031
diff
changeset
|
98 |
vm_exit_out_of_memory(sizeof(void*)*_max_fine_entries, OOM_MALLOC_ERROR, |
1374 | 99 |
"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
|
100 |
} |
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
101 |
|
1374 | 102 |
for (size_t i = 0; i < _max_fine_entries; i++) { |
103 |
_fine_grain_regions[i] = NULL; |
|
104 |
} |
|
105 |
} |
|
106 |
||
13335 | 107 |
void OtherRegionsTable::link_to_all(PerRegionTable* prt) { |
108 |
// We always append to the beginning of the list for convenience; |
|
109 |
// the order of entries in this list does not matter. |
|
110 |
if (_first_all_fine_prts != NULL) { |
|
111 |
assert(_first_all_fine_prts->prev() == NULL, "invariant"); |
|
112 |
_first_all_fine_prts->set_prev(prt); |
|
113 |
prt->set_next(_first_all_fine_prts); |
|
114 |
} else { |
|
115 |
// this is the first element we insert. Adjust the "last" pointer |
|
116 |
_last_all_fine_prts = prt; |
|
117 |
assert(prt->next() == NULL, "just checking"); |
|
118 |
} |
|
119 |
// the new element is always the first element without a predecessor |
|
120 |
prt->set_prev(NULL); |
|
121 |
_first_all_fine_prts = prt; |
|
122 |
||
123 |
assert(prt->prev() == NULL, "just checking"); |
|
124 |
assert(_first_all_fine_prts == prt, "just checking"); |
|
125 |
assert((_first_all_fine_prts == NULL && _last_all_fine_prts == NULL) || |
|
126 |
(_first_all_fine_prts != NULL && _last_all_fine_prts != NULL), |
|
127 |
"just checking"); |
|
128 |
assert(_last_all_fine_prts == NULL || _last_all_fine_prts->next() == NULL, |
|
129 |
"just checking"); |
|
130 |
assert(_first_all_fine_prts == NULL || _first_all_fine_prts->prev() == NULL, |
|
131 |
"just checking"); |
|
132 |
} |
|
133 |
||
134 |
void OtherRegionsTable::unlink_from_all(PerRegionTable* prt) { |
|
135 |
if (prt->prev() != NULL) { |
|
136 |
assert(_first_all_fine_prts != prt, "just checking"); |
|
137 |
prt->prev()->set_next(prt->next()); |
|
138 |
// removing the last element in the list? |
|
139 |
if (_last_all_fine_prts == prt) { |
|
140 |
_last_all_fine_prts = prt->prev(); |
|
141 |
} |
|
142 |
} else { |
|
143 |
assert(_first_all_fine_prts == prt, "just checking"); |
|
144 |
_first_all_fine_prts = prt->next(); |
|
145 |
// list is empty now? |
|
146 |
if (_first_all_fine_prts == NULL) { |
|
147 |
_last_all_fine_prts = NULL; |
|
148 |
} |
|
149 |
} |
|
150 |
||
151 |
if (prt->next() != NULL) { |
|
152 |
prt->next()->set_prev(prt->prev()); |
|
153 |
} |
|
154 |
||
155 |
prt->set_next(NULL); |
|
156 |
prt->set_prev(NULL); |
|
157 |
||
158 |
assert((_first_all_fine_prts == NULL && _last_all_fine_prts == NULL) || |
|
159 |
(_first_all_fine_prts != NULL && _last_all_fine_prts != NULL), |
|
160 |
"just checking"); |
|
161 |
assert(_last_all_fine_prts == NULL || _last_all_fine_prts->next() == NULL, |
|
162 |
"just checking"); |
|
163 |
assert(_first_all_fine_prts == NULL || _first_all_fine_prts->prev() == NULL, |
|
164 |
"just checking"); |
|
165 |
} |
|
166 |
||
49608
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
167 |
CardIdx_t OtherRegionsTable::card_within_region(OopOrNarrowOopStar within_region, HeapRegion* hr) { |
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
168 |
assert(hr->is_in_reserved(within_region), |
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
169 |
"HeapWord " PTR_FORMAT " is outside of region %u [" PTR_FORMAT ", " PTR_FORMAT ")", |
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
170 |
p2i(within_region), hr->hrm_index(), p2i(hr->bottom()), p2i(hr->end())); |
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
171 |
CardIdx_t result = (CardIdx_t)(pointer_delta((HeapWord*)within_region, hr->bottom()) >> (CardTable::card_shift - LogHeapWordSize)); |
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
172 |
return result; |
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
173 |
} |
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
174 |
|
26850
515ff48cf5f0
8036116: Fix thread-id types in G1 remembered set implementations
brutisso
parents:
26424
diff
changeset
|
175 |
void OtherRegionsTable::add_reference(OopOrNarrowOopStar from, uint tid) { |
1374 | 176 |
// Note that this may be a continued H region. |
33786
ac8da6513351
8139867: Change how startsHumongous and continuesHumongous regions work in G1.
david
parents:
33783
diff
changeset
|
177 |
HeapRegion* from_hr = _g1h->heap_region_containing(from); |
26424
639e661f19f3
8057143: Incomplete renaming of variables containing "hrs" to "hrm" related to HeapRegionSeq
tschatzl
parents:
26422
diff
changeset
|
178 |
RegionIdx_t from_hrm_ind = (RegionIdx_t) from_hr->hrm_index(); |
1374 | 179 |
|
180 |
// If the region is already coarsened, return. |
|
26424
639e661f19f3
8057143: Incomplete renaming of variables containing "hrs" to "hrm" related to HeapRegionSeq
tschatzl
parents:
26422
diff
changeset
|
181 |
if (_coarse_map.at(from_hrm_ind)) { |
35875
8ea0b4b0bcc8
8147087: Race when reusing PerRegionTable bitmaps may result in dropped remembered set entries
tschatzl
parents:
35461
diff
changeset
|
182 |
assert(contains_reference(from), "We just found " PTR_FORMAT " in the Coarse table", p2i(from)); |
1374 | 183 |
return; |
184 |
} |
|
185 |
||
186 |
// Otherwise find a per-region table to add it to. |
|
26424
639e661f19f3
8057143: Incomplete renaming of variables containing "hrs" to "hrm" related to HeapRegionSeq
tschatzl
parents:
26422
diff
changeset
|
187 |
size_t ind = from_hrm_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
|
188 |
PerRegionTable* prt = find_region_table(ind, from_hr); |
1374 | 189 |
if (prt == NULL) { |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52716
diff
changeset
|
190 |
MutexLocker x(_m, Mutex::_no_safepoint_check_flag); |
1374 | 191 |
// Confirm that it's really not there... |
192 |
prt = find_region_table(ind, from_hr); |
|
193 |
if (prt == NULL) { |
|
194 |
||
49608
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
195 |
CardIdx_t card_index = card_within_region(from, from_hr); |
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
196 |
|
52656 | 197 |
if (_sparse_table.add_card(from_hrm_ind, card_index)) { |
35875
8ea0b4b0bcc8
8147087: Race when reusing PerRegionTable bitmaps may result in dropped remembered set entries
tschatzl
parents:
35461
diff
changeset
|
198 |
assert(contains_reference_locked(from), "We just added " PTR_FORMAT " to the Sparse table", p2i(from)); |
1374 | 199 |
return; |
200 |
} |
|
201 |
||
202 |
if (_n_fine_entries == _max_fine_entries) { |
|
203 |
prt = delete_region_table(); |
|
13335 | 204 |
// There is no need to clear the links to the 'all' list here: |
205 |
// prt will be reused immediately, i.e. remain in the 'all' list. |
|
206 |
prt->init(from_hr, false /* clear_links_to_all_list */); |
|
1374 | 207 |
} else { |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
208 |
prt = PerRegionTable::alloc(from_hr); |
13335 | 209 |
link_to_all(prt); |
1374 | 210 |
} |
211 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
212 |
PerRegionTable* first_prt = _fine_grain_regions[ind]; |
13335 | 213 |
prt->set_collision_list_next(first_prt); |
30765
32398da8ba8f
8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0
bdelsart
parents:
30764
diff
changeset
|
214 |
// The assignment into _fine_grain_regions allows the prt to |
32398da8ba8f
8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0
bdelsart
parents:
30764
diff
changeset
|
215 |
// start being used concurrently. In addition to |
32398da8ba8f
8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0
bdelsart
parents:
30764
diff
changeset
|
216 |
// collision_list_next which must be visible (else concurrent |
32398da8ba8f
8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0
bdelsart
parents:
30764
diff
changeset
|
217 |
// parsing of the list, if any, may fail to see other entries), |
32398da8ba8f
8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0
bdelsart
parents:
30764
diff
changeset
|
218 |
// the content of the prt must be visible (else for instance |
32398da8ba8f
8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0
bdelsart
parents:
30764
diff
changeset
|
219 |
// some mark bits may not yet seem cleared or a 'later' update |
32398da8ba8f
8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0
bdelsart
parents:
30764
diff
changeset
|
220 |
// performed by a concurrent thread could be undone when the |
32398da8ba8f
8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0
bdelsart
parents:
30764
diff
changeset
|
221 |
// zeroing becomes visible). This requires store ordering. |
47634
6a0c42c40cd1
8188220: Remove Atomic::*_ptr() uses and overloads from hotspot
coleenp
parents:
47216
diff
changeset
|
222 |
OrderAccess::release_store(&_fine_grain_regions[ind], prt); |
1374 | 223 |
_n_fine_entries++; |
224 |
||
52656 | 225 |
// Transfer from sparse to fine-grain. |
226 |
SparsePRTEntry *sprt_entry = _sparse_table.get_entry(from_hrm_ind); |
|
227 |
assert(sprt_entry != NULL, "There should have been an entry"); |
|
228 |
for (int i = 0; i < sprt_entry->num_valid_cards(); i++) { |
|
229 |
CardIdx_t c = sprt_entry->card(i); |
|
230 |
prt->add_card(c); |
|
1374 | 231 |
} |
52656 | 232 |
// Now we can delete the sparse entry. |
233 |
bool res = _sparse_table.delete_entry(from_hrm_ind); |
|
234 |
assert(res, "It should have been there."); |
|
1374 | 235 |
} |
236 |
assert(prt != NULL && prt->hr() == from_hr, "consequence"); |
|
237 |
} |
|
238 |
// Note that we can't assert "prt->hr() == from_hr", because of the |
|
239 |
// possibility of concurrent reuse. But see head comment of |
|
240 |
// OtherRegionsTable for why this is OK. |
|
241 |
assert(prt != NULL, "Inv"); |
|
242 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
243 |
prt->add_reference(from); |
49607
acffe6ff3ae7
8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents:
49164
diff
changeset
|
244 |
assert(contains_reference(from), "We just added " PTR_FORMAT " to the PRT (%d)", p2i(from), prt->contains_reference(from)); |
1374 | 245 |
} |
246 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
247 |
PerRegionTable* |
1374 | 248 |
OtherRegionsTable::find_region_table(size_t ind, HeapRegion* hr) const { |
29580
a67a581cfe11
8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents:
28379
diff
changeset
|
249 |
assert(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
|
250 |
PerRegionTable* prt = _fine_grain_regions[ind]; |
1374 | 251 |
while (prt != NULL && prt->hr() != hr) { |
13335 | 252 |
prt = prt->collision_list_next(); |
1374 | 253 |
} |
254 |
// Loop postcondition is the method postcondition. |
|
255 |
return prt; |
|
256 |
} |
|
257 |
||
258 |
jint OtherRegionsTable::_n_coarsenings = 0; |
|
259 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
260 |
PerRegionTable* OtherRegionsTable::delete_region_table() { |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
261 |
assert(_m->owned_by_self(), "Precondition"); |
1374 | 262 |
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
|
263 |
PerRegionTable* max = NULL; |
1374 | 264 |
jint max_occ = 0; |
33589
7cbd1b2c139b
8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents:
33105
diff
changeset
|
265 |
PerRegionTable** max_prev = NULL; |
1374 | 266 |
size_t max_ind; |
267 |
||
268 |
size_t i = _fine_eviction_start; |
|
269 |
for (size_t k = 0; k < _fine_eviction_sample_size; k++) { |
|
270 |
size_t ii = i; |
|
271 |
// Make sure we get a non-NULL sample. |
|
272 |
while (_fine_grain_regions[ii] == NULL) { |
|
273 |
ii++; |
|
274 |
if (ii == _max_fine_entries) ii = 0; |
|
275 |
guarantee(ii != i, "We must find one."); |
|
276 |
} |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
277 |
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
|
278 |
PerRegionTable* cur = *prev; |
1374 | 279 |
while (cur != NULL) { |
280 |
jint cur_occ = cur->occupied(); |
|
281 |
if (max == NULL || cur_occ > max_occ) { |
|
282 |
max = cur; |
|
283 |
max_prev = prev; |
|
284 |
max_ind = i; |
|
285 |
max_occ = cur_occ; |
|
286 |
} |
|
13335 | 287 |
prev = cur->collision_list_next_addr(); |
288 |
cur = cur->collision_list_next(); |
|
1374 | 289 |
} |
290 |
i = i + _fine_eviction_stride; |
|
291 |
if (i >= _n_fine_entries) i = i - _n_fine_entries; |
|
292 |
} |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
293 |
|
1374 | 294 |
_fine_eviction_start++; |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
295 |
|
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
296 |
if (_fine_eviction_start >= _n_fine_entries) { |
1374 | 297 |
_fine_eviction_start -= _n_fine_entries; |
298 |
} |
|
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
299 |
|
1374 | 300 |
guarantee(max != NULL, "Since _n_fine_entries > 0"); |
33589
7cbd1b2c139b
8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents:
33105
diff
changeset
|
301 |
guarantee(max_prev != NULL, "Since max != NULL."); |
1374 | 302 |
|
303 |
// Set the corresponding coarse bit. |
|
26316
93f6b40c038b
8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents:
26160
diff
changeset
|
304 |
size_t max_hrm_index = (size_t) max->hr()->hrm_index(); |
93f6b40c038b
8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents:
26160
diff
changeset
|
305 |
if (!_coarse_map.at(max_hrm_index)) { |
93f6b40c038b
8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents:
26160
diff
changeset
|
306 |
_coarse_map.at_put(max_hrm_index, true); |
1374 | 307 |
_n_coarse_entries++; |
308 |
} |
|
309 |
||
310 |
// Unsplice. |
|
13335 | 311 |
*max_prev = max->collision_list_next(); |
1374 | 312 |
Atomic::inc(&_n_coarsenings); |
313 |
_n_fine_entries--; |
|
314 |
return max; |
|
315 |
} |
|
316 |
||
28379
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
317 |
bool OtherRegionsTable::occupancy_less_or_equal_than(size_t limit) const { |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
318 |
if (limit <= (size_t)G1RSetSparseRegionEntries) { |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
319 |
return occ_coarse() == 0 && _first_all_fine_prts == NULL && occ_sparse() <= limit; |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
320 |
} else { |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
321 |
// Current uses of this method may only use values less than G1RSetSparseRegionEntries |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
322 |
// for the limit. The solution, comparing against occupied() would be too slow |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
323 |
// at this time. |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
324 |
Unimplemented(); |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
325 |
return false; |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
326 |
} |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
327 |
} |
e6784fc8fff2
8048179: Early reclaim of large objects that are referenced by a few objects
tschatzl
parents:
28170
diff
changeset
|
328 |
|
25889
221296ac4359
8027959: Early reclamation of large objects in G1
tschatzl
parents:
25492
diff
changeset
|
329 |
bool OtherRegionsTable::is_empty() const { |
221296ac4359
8027959: Early reclamation of large objects in G1
tschatzl
parents:
25492
diff
changeset
|
330 |
return occ_sparse() == 0 && occ_coarse() == 0 && _first_all_fine_prts == NULL; |
221296ac4359
8027959: Early reclamation of large objects in G1
tschatzl
parents:
25492
diff
changeset
|
331 |
} |
1374 | 332 |
|
333 |
size_t OtherRegionsTable::occupied() const { |
|
334 |
size_t sum = occ_fine(); |
|
335 |
sum += occ_sparse(); |
|
336 |
sum += occ_coarse(); |
|
337 |
return sum; |
|
338 |
} |
|
339 |
||
340 |
size_t OtherRegionsTable::occ_fine() const { |
|
341 |
size_t sum = 0; |
|
13335 | 342 |
|
343 |
size_t num = 0; |
|
344 |
PerRegionTable * cur = _first_all_fine_prts; |
|
345 |
while (cur != NULL) { |
|
346 |
sum += cur->occupied(); |
|
347 |
cur = cur->next(); |
|
348 |
num++; |
|
1374 | 349 |
} |
13335 | 350 |
guarantee(num == _n_fine_entries, "just checking"); |
1374 | 351 |
return sum; |
352 |
} |
|
353 |
||
354 |
size_t OtherRegionsTable::occ_coarse() const { |
|
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3262
diff
changeset
|
355 |
return (_n_coarse_entries * HeapRegion::CardsPerRegion); |
1374 | 356 |
} |
357 |
||
358 |
size_t OtherRegionsTable::occ_sparse() const { |
|
359 |
return _sparse_table.occupied(); |
|
360 |
} |
|
361 |
||
362 |
size_t OtherRegionsTable::mem_size() const { |
|
363 |
size_t sum = 0; |
|
17395
2cd3ef1be718
8014240: G1: Add remembered set size information to output of G1PrintRegionLivenessInfo
tschatzl
parents:
17119
diff
changeset
|
364 |
// 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
|
365 |
if (_first_all_fine_prts != NULL) { |
2cd3ef1be718
8014240: G1: Add remembered set size information to output of G1PrintRegionLivenessInfo
tschatzl
parents:
17119
diff
changeset
|
366 |
assert(_last_all_fine_prts != NULL && |
2cd3ef1be718
8014240: G1: Add remembered set size information to output of G1PrintRegionLivenessInfo
tschatzl
parents:
17119
diff
changeset
|
367 |
_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
|
368 |
sum += _first_all_fine_prts->mem_size() * _n_fine_entries; |
1374 | 369 |
} |
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
370 |
sum += (sizeof(PerRegionTable*) * _max_fine_entries); |
1374 | 371 |
sum += (_coarse_map.size_in_words() * HeapWordSize); |
372 |
sum += (_sparse_table.mem_size()); |
|
24108
dfc2242fc6b0
8040792: G1: Memory usage calculation uses sizeof(this) instead of sizeof(classname)
tschatzl
parents:
24106
diff
changeset
|
373 |
sum += sizeof(OtherRegionsTable) - sizeof(_sparse_table); // Avoid double counting above. |
1374 | 374 |
return sum; |
375 |
} |
|
376 |
||
377 |
size_t OtherRegionsTable::static_mem_size() { |
|
35200
7802299b31e7
8145671: Rename FromCardCache to G1FromCardCache
tschatzl
parents:
35199
diff
changeset
|
378 |
return G1FromCardCache::static_mem_size(); |
1374 | 379 |
} |
380 |
||
381 |
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
|
382 |
return PerRegionTable::fl_mem_size(); |
1374 | 383 |
} |
384 |
||
385 |
void OtherRegionsTable::clear() { |
|
13335 | 386 |
// if there are no entries, skip this step |
387 |
if (_first_all_fine_prts != NULL) { |
|
388 |
guarantee(_first_all_fine_prts != NULL && _last_all_fine_prts != NULL, "just checking"); |
|
389 |
PerRegionTable::bulk_free(_first_all_fine_prts, _last_all_fine_prts); |
|
390 |
memset(_fine_grain_regions, 0, _max_fine_entries * sizeof(_fine_grain_regions[0])); |
|
391 |
} else { |
|
392 |
guarantee(_first_all_fine_prts == NULL && _last_all_fine_prts == NULL, "just checking"); |
|
1374 | 393 |
} |
13335 | 394 |
|
395 |
_first_all_fine_prts = _last_all_fine_prts = NULL; |
|
1374 | 396 |
_sparse_table.clear(); |
38161
1c7dd1e39fcf
8155811: Remove HeapRegionRemSet::_coarse_dirty flag
tschatzl
parents:
38156
diff
changeset
|
397 |
if (_n_coarse_entries > 0) { |
38156 | 398 |
_coarse_map.clear(); |
399 |
} |
|
1374 | 400 |
_n_fine_entries = 0; |
401 |
_n_coarse_entries = 0; |
|
402 |
} |
|
403 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
404 |
bool OtherRegionsTable::contains_reference(OopOrNarrowOopStar from) const { |
1374 | 405 |
// Cast away const in this case. |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52716
diff
changeset
|
406 |
MutexLocker x((Mutex*)_m, Mutex::_no_safepoint_check_flag); |
1374 | 407 |
return contains_reference_locked(from); |
408 |
} |
|
409 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2996
diff
changeset
|
410 |
bool OtherRegionsTable::contains_reference_locked(OopOrNarrowOopStar from) const { |
33786
ac8da6513351
8139867: Change how startsHumongous and continuesHumongous regions work in G1.
david
parents:
33783
diff
changeset
|
411 |
HeapRegion* hr = _g1h->heap_region_containing(from); |
26316
93f6b40c038b
8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents:
26160
diff
changeset
|
412 |
RegionIdx_t hr_ind = (RegionIdx_t) hr->hrm_index(); |
1374 | 413 |
// Is this region in the coarse map? |
414 |
if (_coarse_map.at(hr_ind)) return true; |
|
415 |
||
13113
ab3870a79b18
6921087: G1: remove per-GC-thread expansion tables from the fine-grain remembered sets
johnc
parents:
12779
diff
changeset
|
416 |
PerRegionTable* prt = find_region_table(hr_ind & _mod_max_fine_entries_mask, |
49608
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
417 |
hr); |
1374 | 418 |
if (prt != NULL) { |
419 |
return prt->contains_reference(from); |
|
420 |
||
421 |
} else { |
|
49608
1852b17b0efc
8196485: FromCardCache default card index can cause crashes
tschatzl
parents:
49607
diff
changeset
|
422 |
CardIdx_t card_index = card_within_region(from, hr); |
2996
1097030e5ec3
6843694: G1: assert(index < _vs.committed_size(),"bad index"), g1BlockOffsetTable.inline.hpp:55
johnc
parents:
2882
diff
changeset
|
423 |
return _sparse_table.contains_card(hr_ind, card_index); |
1374 | 424 |
} |
425 |
} |
|
426 |
||
35461 | 427 |
HeapRegionRemSet::HeapRegionRemSet(G1BlockOffsetTable* bot, |
1374 | 428 |
HeapRegion* hr) |
35461 | 429 |
: _bot(bot), |
51332 | 430 |
_code_roots(), |
58375
a1eba2e37671
8231472: Clean up Monitor::_safepoint_check usage to Mutex::_safepoint_check
coleenp
parents:
55510
diff
changeset
|
431 |
_m(Mutex::leaf, FormatBuffer<128>("HeapRegionRemSet lock #%u", hr->hrm_index()), true, Mutex::_safepoint_check_never), |
52347
14ef0f74667b
8211388: Make OtherRegionsTable independent of the region it is for
tschatzl
parents:
52345
diff
changeset
|
432 |
_other_regions(&_m), |
14ef0f74667b
8211388: Make OtherRegionsTable independent of the region it is for
tschatzl
parents:
52345
diff
changeset
|
433 |
_hr(hr), |
51332 | 434 |
_state(Untracked) |
435 |
{ |
|
10000
5bbb58b0dbb9
7046182: G1: remove unnecessary iterations over the collection set
tonyp
parents:
9989
diff
changeset
|
436 |
} |
1374 | 437 |
|
52347
14ef0f74667b
8211388: Make OtherRegionsTable independent of the region it is for
tschatzl
parents:
52345
diff
changeset
|
438 |
void HeapRegionRemSet::clear_fcc() { |
14ef0f74667b
8211388: Make OtherRegionsTable independent of the region it is for
tschatzl
parents:
52345
diff
changeset
|
439 |
G1FromCardCache::clear(_hr->hrm_index()); |
14ef0f74667b
8211388: Make OtherRegionsTable independent of the region it is for
tschatzl
parents:
52345
diff
changeset
|
440 |
} |
14ef0f74667b
8211388: Make OtherRegionsTable independent of the region it is for
tschatzl
parents:
52345
diff
changeset
|
441 |
|
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
442 |
void HeapRegionRemSet::setup_remset_size() { |
12779
06778de5193e
7171936: LOG_G incorrectly defined in globalDefinitions.hpp
brutisso
parents:
12381
diff
changeset
|
443 |
const int LOG_M = 20; |
55148
0dab93cb3b0c
8223162: Improve ergonomics for Sparse PRT entry sizing
tschatzl
parents:
54623
diff
changeset
|
444 |
guarantee(HeapRegion::LogOfHRGrainBytes >= LOG_M, "Code assumes the region size >= 1M, but is " SIZE_FORMAT "B", HeapRegion::GrainBytes); |
0dab93cb3b0c
8223162: Improve ergonomics for Sparse PRT entry sizing
tschatzl
parents:
54623
diff
changeset
|
445 |
|
0dab93cb3b0c
8223162: Improve ergonomics for Sparse PRT entry sizing
tschatzl
parents:
54623
diff
changeset
|
446 |
int region_size_log_mb = HeapRegion::LogOfHRGrainBytes - LOG_M; |
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
447 |
if (FLAG_IS_DEFAULT(G1RSetSparseRegionEntries)) { |
55148
0dab93cb3b0c
8223162: Improve ergonomics for Sparse PRT entry sizing
tschatzl
parents:
54623
diff
changeset
|
448 |
G1RSetSparseRegionEntries = G1RSetSparseRegionEntriesBase * ((size_t)1 << (region_size_log_mb + 1)); |
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
449 |
} |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
450 |
if (FLAG_IS_DEFAULT(G1RSetRegionEntries)) { |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
451 |
G1RSetRegionEntries = G1RSetRegionEntriesBase * (region_size_log_mb + 1); |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
452 |
} |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
453 |
guarantee(G1RSetSparseRegionEntries > 0 && G1RSetRegionEntries > 0 , "Sanity"); |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
454 |
} |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
4900
diff
changeset
|
455 |
|
49607
acffe6ff3ae7
8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents:
49164
diff
changeset
|
456 |
void HeapRegionRemSet::clear(bool only_cardset) { |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52716
diff
changeset
|
457 |
MutexLocker x(&_m, Mutex::_no_safepoint_check_flag); |
49607
acffe6ff3ae7
8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents:
49164
diff
changeset
|
458 |
clear_locked(only_cardset); |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
459 |
} |
19339 | 460 |
|
49607
acffe6ff3ae7
8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents:
49164
diff
changeset
|
461 |
void HeapRegionRemSet::clear_locked(bool only_cardset) { |
acffe6ff3ae7
8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents:
49164
diff
changeset
|
462 |
if (!only_cardset) { |
acffe6ff3ae7
8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents:
49164
diff
changeset
|
463 |
_code_roots.clear(); |
acffe6ff3ae7
8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents:
49164
diff
changeset
|
464 |
} |
52347
14ef0f74667b
8211388: Make OtherRegionsTable independent of the region it is for
tschatzl
parents:
52345
diff
changeset
|
465 |
clear_fcc(); |
1374 | 466 |
_other_regions.clear(); |
49607
acffe6ff3ae7
8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents:
49164
diff
changeset
|
467 |
set_state_empty(); |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
468 |
assert(occupied_locked() == 0, "Should be clear."); |
1374 | 469 |
} |
470 |
||
19339 | 471 |
// Code roots support |
26422 | 472 |
// |
473 |
// The code root set is protected by two separate locking schemes |
|
474 |
// When at safepoint the per-hrrs lock must be held during modifications |
|
475 |
// except when doing a full gc. |
|
476 |
// When not at safepoint the CodeCache_lock must be held during modifications. |
|
477 |
// When concurrent readers access the contains() function |
|
478 |
// (during the evacuation phase) no removals are allowed. |
|
19339 | 479 |
|
480 |
void HeapRegionRemSet::add_strong_code_root(nmethod* nm) { |
|
481 |
assert(nm != NULL, "sanity"); |
|
35879 | 482 |
assert((!CodeCache_lock->owned_by_self() || SafepointSynchronize::is_at_safepoint()), |
483 |
"should call add_strong_code_root_locked instead. CodeCache_lock->owned_by_self(): %s, is_at_safepoint(): %s", |
|
484 |
BOOL_TO_STR(CodeCache_lock->owned_by_self()), BOOL_TO_STR(SafepointSynchronize::is_at_safepoint())); |
|
26422 | 485 |
// Optimistic unlocked contains-check |
486 |
if (!_code_roots.contains(nm)) { |
|
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52716
diff
changeset
|
487 |
MutexLocker ml(&_m, Mutex::_no_safepoint_check_flag); |
26422 | 488 |
add_strong_code_root_locked(nm); |
489 |
} |
|
490 |
} |
|
491 |
||
492 |
void HeapRegionRemSet::add_strong_code_root_locked(nmethod* nm) { |
|
493 |
assert(nm != NULL, "sanity"); |
|
35879 | 494 |
assert((CodeCache_lock->owned_by_self() || |
495 |
(SafepointSynchronize::is_at_safepoint() && |
|
496 |
(_m.owned_by_self() || Thread::current()->is_VM_thread()))), |
|
497 |
"not safely locked. CodeCache_lock->owned_by_self(): %s, is_at_safepoint(): %s, _m.owned_by_self(): %s, Thread::current()->is_VM_thread(): %s", |
|
498 |
BOOL_TO_STR(CodeCache_lock->owned_by_self()), BOOL_TO_STR(SafepointSynchronize::is_at_safepoint()), |
|
499 |
BOOL_TO_STR(_m.owned_by_self()), BOOL_TO_STR(Thread::current()->is_VM_thread())); |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
500 |
_code_roots.add(nm); |
19339 | 501 |
} |
502 |
||
503 |
void HeapRegionRemSet::remove_strong_code_root(nmethod* nm) { |
|
504 |
assert(nm != NULL, "sanity"); |
|
25492
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
505 |
assert_locked_or_safepoint(CodeCache_lock); |
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
506 |
|
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52716
diff
changeset
|
507 |
MutexLocker ml(CodeCache_lock->owned_by_self() ? NULL : &_m, Mutex::_no_safepoint_check_flag); |
26422 | 508 |
_code_roots.remove(nm); |
25492
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
509 |
|
19339 | 510 |
// Check that there were no duplicates |
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
511 |
guarantee(!_code_roots.contains(nm), "duplicate entry found"); |
19339 | 512 |
} |
513 |
||
26422 | 514 |
void HeapRegionRemSet::strong_code_roots_do(CodeBlobClosure* blk) const { |
515 |
_code_roots.nmethods_do(blk); |
|
19339 | 516 |
} |
517 |
||
26422 | 518 |
void HeapRegionRemSet::clean_strong_code_roots(HeapRegion* hr) { |
519 |
_code_roots.clean(hr); |
|
19339 | 520 |
} |
521 |
||
522 |
size_t HeapRegionRemSet::strong_code_roots_mem_size() { |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
21563
diff
changeset
|
523 |
return _code_roots.mem_size(); |
19339 | 524 |
} |