Mercurial > hg > truffle
annotate src/share/vm/gc_implementation/g1/heapRegionSet.cpp @ 17756:a07bea31ef35
8035398: Add card redirty time in "Other" time in G1
Summary: Show the time taken by card redirtying during GC in a new "Redirty Cards" line.
Reviewed-by: jwilhelm, brutisso
author | tschatzl |
---|---|
date | Mon, 24 Mar 2014 15:30:40 +0100 |
parents | 0d2ce7411240 |
children | 8ee855b4e667 |
rev | line source |
---|---|
2152 | 1 /* |
6010
720b6a76dd9d
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
4072
diff
changeset
|
2 * Copyright (c) 2011, 2012, Oracle and/or its affiliates. All rights reserved. |
2152 | 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 * | |
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA | |
20 * or visit www.oracle.com if you need additional information or have any | |
21 * questions. | |
22 * | |
23 */ | |
24 | |
25 #include "precompiled.hpp" | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
26 #include "gc_implementation/g1/heapRegionRemSet.hpp" |
2152 | 27 #include "gc_implementation/g1/heapRegionSet.inline.hpp" |
28 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
29 uint FreeRegionList::_unrealistically_long_length = 0; |
2152 | 30 |
2361 | 31 void HeapRegionSetBase::fill_in_ext_msg(hrs_ext_msg* msg, const char* message) { |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
32 msg->append("[%s] %s ln: %u cy: "SIZE_FORMAT, |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
33 name(), message, length(), total_capacity_bytes()); |
2152 | 34 fill_in_ext_msg_extra(msg); |
35 } | |
36 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
37 #ifndef PRODUCT |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
38 void HeapRegionSetBase::verify_region(HeapRegion* hr) { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
39 assert(hr->containing_set() == this, err_msg("Inconsistent containing set for %u", hr->hrs_index())); |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
40 assert(!hr->is_young(), err_msg("Adding young region %u", hr->hrs_index())); // currently we don't use these sets for young regions |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
41 assert(hr->isHumongous() == regions_humongous(), err_msg("Wrong humongous state for region %u and set %s", hr->hrs_index(), name())); |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
42 assert(hr->is_empty() == regions_empty(), err_msg("Wrong empty state for region %u and set %s", hr->hrs_index(), name())); |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
43 assert(hr->rem_set()->verify_ready_for_par_iteration(), err_msg("Wrong iteration state %u", hr->hrs_index())); |
2152 | 44 } |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
45 #endif |
2152 | 46 |
47 void HeapRegionSetBase::verify() { | |
48 // It's important that we also observe the MT safety protocol even | |
49 // for the verification calls. If we do verification without the | |
50 // appropriate locks and the set changes underneath our feet | |
51 // verification might fail and send us on a wild goose chase. | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
52 check_mt_safety(); |
2152 | 53 |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
54 guarantee(( is_empty() && length() == 0 && total_capacity_bytes() == 0) || |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
55 (!is_empty() && length() >= 0 && total_capacity_bytes() >= 0), |
2361 | 56 hrs_ext_msg(this, "invariant")); |
2152 | 57 } |
58 | |
59 void HeapRegionSetBase::verify_start() { | |
60 // See comment in verify() about MT safety and verification. | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
61 check_mt_safety(); |
2152 | 62 assert(!_verify_in_progress, |
2361 | 63 hrs_ext_msg(this, "verification should not be in progress")); |
2152 | 64 |
65 // Do the basic verification first before we do the checks over the regions. | |
66 HeapRegionSetBase::verify(); | |
67 | |
68 _verify_in_progress = true; | |
69 } | |
70 | |
71 void HeapRegionSetBase::verify_end() { | |
72 // See comment in verify() about MT safety and verification. | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
73 check_mt_safety(); |
2152 | 74 assert(_verify_in_progress, |
2361 | 75 hrs_ext_msg(this, "verification should be in progress")); |
2152 | 76 |
77 _verify_in_progress = false; | |
78 } | |
79 | |
80 void HeapRegionSetBase::print_on(outputStream* out, bool print_contents) { | |
81 out->cr(); | |
82 out->print_cr("Set: %s ("PTR_FORMAT")", name(), this); | |
83 out->print_cr(" Region Assumptions"); | |
84 out->print_cr(" humongous : %s", BOOL_TO_STR(regions_humongous())); | |
85 out->print_cr(" empty : %s", BOOL_TO_STR(regions_empty())); | |
86 out->print_cr(" Attributes"); | |
6010
720b6a76dd9d
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
4072
diff
changeset
|
87 out->print_cr(" length : %14u", length()); |
2152 | 88 out->print_cr(" total capacity : "SIZE_FORMAT_W(14)" bytes", |
89 total_capacity_bytes()); | |
90 } | |
91 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
92 HeapRegionSetBase::HeapRegionSetBase(const char* name, bool humongous, bool empty, HRSMtSafeChecker* mt_safety_checker) |
2152 | 93 : _name(name), _verify_in_progress(false), |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
94 _is_humongous(humongous), _is_empty(empty), _mt_safety_checker(mt_safety_checker), |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
95 _count() |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
96 { } |
2152 | 97 |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
98 void FreeRegionList::set_unrealistically_long_length(uint len) { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
99 guarantee(_unrealistically_long_length == 0, "should only be set once"); |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
100 _unrealistically_long_length = len; |
2152 | 101 } |
102 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
103 void FreeRegionList::fill_in_ext_msg_extra(hrs_ext_msg* msg) { |
2152 | 104 msg->append(" hd: "PTR_FORMAT" tl: "PTR_FORMAT, head(), tail()); |
105 } | |
106 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
107 void FreeRegionList::add_as_head_or_tail(FreeRegionList* from_list, bool as_head) { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
108 check_mt_safety(); |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
109 from_list->check_mt_safety(); |
2432
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
110 |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
111 verify_optional(); |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
112 from_list->verify_optional(); |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
113 |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
114 if (from_list->is_empty()) { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
115 return; |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
116 } |
2432
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
117 |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
118 #ifdef ASSERT |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
119 FreeRegionListIterator iter(from_list); |
2432
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
120 while (iter.more_available()) { |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
121 HeapRegion* hr = iter.get_next(); |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
122 // In set_containing_set() we check that we either set the value |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
123 // from NULL to non-NULL or vice versa to catch bugs. So, we have |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
124 // to NULL it first before setting it to the value. |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
125 hr->set_containing_set(NULL); |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
126 hr->set_containing_set(this); |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
127 } |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
128 #endif // ASSERT |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
129 |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
130 if (_head == NULL) { |
6252
d42fe3c3001d
7184772: G1: Incorrect assert in HeapRegionLinkedList::add_as_head()
johnc
parents:
6010
diff
changeset
|
131 assert(length() == 0 && _tail == NULL, hrs_ext_msg(this, "invariant")); |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
132 _head = from_list->_head; |
2432
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
133 _tail = from_list->_tail; |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
134 } else { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
135 assert(length() > 0 && _tail != NULL, hrs_ext_msg(this, "invariant")); |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
136 if (as_head) { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
137 from_list->_tail->set_next(_head); |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
138 _head = from_list->_head; |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
139 } else { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
140 _tail->set_next(from_list->_head); |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
141 _tail = from_list->_tail; |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
142 } |
2432
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
143 } |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
144 |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
145 _count.increment(from_list->length(), from_list->total_capacity_bytes()); |
2432
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
146 from_list->clear(); |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
147 |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
148 verify_optional(); |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
149 from_list->verify_optional(); |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
150 } |
455328d90876
7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents:
2361
diff
changeset
|
151 |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
152 void FreeRegionList::add_as_head(FreeRegionList* from_list) { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
153 add_as_head_or_tail(from_list, true /* as_head */); |
2152 | 154 } |
155 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
156 void FreeRegionList::add_as_tail(FreeRegionList* from_list) { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
157 add_as_head_or_tail(from_list, false /* as_head */); |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
158 } |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
159 |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
160 void FreeRegionList::remove_all() { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
161 check_mt_safety(); |
2152 | 162 verify_optional(); |
163 | |
164 HeapRegion* curr = _head; | |
165 while (curr != NULL) { | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
166 verify_region(curr); |
2152 | 167 |
168 HeapRegion* next = curr->next(); | |
169 curr->set_next(NULL); | |
170 curr->set_containing_set(NULL); | |
171 curr = next; | |
172 } | |
173 clear(); | |
174 | |
175 verify_optional(); | |
176 } | |
177 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
178 void FreeRegionList::remove_all_pending(uint target_count) { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
179 check_mt_safety(); |
2361 | 180 assert(target_count > 1, hrs_ext_msg(this, "pre-condition")); |
181 assert(!is_empty(), hrs_ext_msg(this, "pre-condition")); | |
2152 | 182 |
183 verify_optional(); | |
6010
720b6a76dd9d
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
4072
diff
changeset
|
184 DEBUG_ONLY(uint old_length = length();) |
2152 | 185 |
186 HeapRegion* curr = _head; | |
187 HeapRegion* prev = NULL; | |
6010
720b6a76dd9d
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
4072
diff
changeset
|
188 uint count = 0; |
2152 | 189 while (curr != NULL) { |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
190 verify_region(curr); |
2152 | 191 HeapRegion* next = curr->next(); |
192 | |
193 if (curr->pending_removal()) { | |
194 assert(count < target_count, | |
2361 | 195 hrs_err_msg("[%s] should not come across more regions " |
6010
720b6a76dd9d
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
4072
diff
changeset
|
196 "pending for removal than target_count: %u", |
2152 | 197 name(), target_count)); |
198 | |
199 if (prev == NULL) { | |
2361 | 200 assert(_head == curr, hrs_ext_msg(this, "invariant")); |
2152 | 201 _head = next; |
202 } else { | |
2361 | 203 assert(_head != curr, hrs_ext_msg(this, "invariant")); |
2152 | 204 prev->set_next(next); |
205 } | |
206 if (next == NULL) { | |
2361 | 207 assert(_tail == curr, hrs_ext_msg(this, "invariant")); |
2152 | 208 _tail = prev; |
209 } else { | |
2361 | 210 assert(_tail != curr, hrs_ext_msg(this, "invariant")); |
2152 | 211 } |
212 | |
213 curr->set_next(NULL); | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
214 remove(curr); |
2152 | 215 curr->set_pending_removal(false); |
216 | |
217 count += 1; | |
218 | |
219 // If we have come across the target number of regions we can | |
220 // just bail out. However, for debugging purposes, we can just | |
221 // carry on iterating to make sure there are not more regions | |
222 // tagged with pending removal. | |
223 DEBUG_ONLY(if (count == target_count) break;) | |
224 } else { | |
225 prev = curr; | |
226 } | |
227 curr = next; | |
228 } | |
229 | |
230 assert(count == target_count, | |
6010
720b6a76dd9d
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
4072
diff
changeset
|
231 hrs_err_msg("[%s] count: %u should be == target_count: %u", |
720b6a76dd9d
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
4072
diff
changeset
|
232 name(), count, target_count)); |
2152 | 233 assert(length() + target_count == old_length, |
2361 | 234 hrs_err_msg("[%s] new length should be consistent " |
6010
720b6a76dd9d
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
4072
diff
changeset
|
235 "new length: %u old length: %u target_count: %u", |
2152 | 236 name(), length(), old_length, target_count)); |
237 | |
238 verify_optional(); | |
239 } | |
240 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
241 void FreeRegionList::verify() { |
2152 | 242 // See comment in HeapRegionSetBase::verify() about MT safety and |
243 // verification. | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
244 check_mt_safety(); |
2152 | 245 |
246 // This will also do the basic verification too. | |
247 verify_start(); | |
248 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
249 verify_list(); |
2152 | 250 |
251 verify_end(); | |
252 } | |
253 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
254 void FreeRegionList::clear() { |
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
255 _count = HeapRegionSetCount(); |
2152 | 256 _head = NULL; |
257 _tail = NULL; | |
258 } | |
259 | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
260 void FreeRegionList::print_on(outputStream* out, bool print_contents) { |
2152 | 261 HeapRegionSetBase::print_on(out, print_contents); |
262 out->print_cr(" Linking"); | |
263 out->print_cr(" head : "PTR_FORMAT, _head); | |
264 out->print_cr(" tail : "PTR_FORMAT, _tail); | |
265 | |
266 if (print_contents) { | |
267 out->print_cr(" Contents"); | |
17736
58fc1b1523dc
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
6254
diff
changeset
|
268 FreeRegionListIterator iter(this); |
2152 | 269 while (iter.more_available()) { |
270 HeapRegion* hr = iter.get_next(); | |
271 hr->print_on(out); | |
272 } | |
273 } | |
274 } | |
17737 | 275 |
276 void FreeRegionList::verify_list() { | |
277 HeapRegion* curr = head(); | |
278 HeapRegion* prev1 = NULL; | |
279 HeapRegion* prev0 = NULL; | |
280 uint count = 0; | |
281 size_t capacity = 0; | |
282 while (curr != NULL) { | |
283 verify_region(curr); | |
284 | |
285 count++; | |
286 guarantee(count < _unrealistically_long_length, | |
287 hrs_err_msg("[%s] the calculated length: %u seems very long, is there maybe a cycle? curr: "PTR_FORMAT" prev0: "PTR_FORMAT" " "prev1: "PTR_FORMAT" length: %u", name(), count, curr, prev0, prev1, length())); | |
288 | |
289 capacity += curr->capacity(); | |
290 | |
291 prev1 = prev0; | |
292 prev0 = curr; | |
293 curr = curr->next(); | |
294 } | |
295 | |
296 guarantee(tail() == prev0, err_msg("Expected %s to end with %u but it ended with %u.", name(), tail()->hrs_index(), prev0->hrs_index())); | |
297 | |
298 guarantee(length() == count, err_msg("%s count mismatch. Expected %u, actual %u.", name(), length(), count)); | |
299 guarantee(total_capacity_bytes() == capacity, err_msg("%s capacity mismatch. Expected " SIZE_FORMAT ", actual " SIZE_FORMAT, | |
300 name(), total_capacity_bytes(), capacity)); | |
301 } | |
302 | |
303 // Note on the check_mt_safety() methods below: | |
304 // | |
305 // Verification of the "master" heap region sets / lists that are | |
306 // maintained by G1CollectedHeap is always done during a STW pause and | |
307 // by the VM thread at the start / end of the pause. The standard | |
308 // verification methods all assert check_mt_safety(). This is | |
309 // important as it ensures that verification is done without | |
310 // concurrent updates taking place at the same time. It follows, that, | |
311 // for the "master" heap region sets / lists, the check_mt_safety() | |
312 // method should include the VM thread / STW case. | |
313 | |
314 void MasterFreeRegionListMtSafeChecker::check() { | |
315 // Master Free List MT safety protocol: | |
316 // (a) If we're at a safepoint, operations on the master free list | |
317 // should be invoked by either the VM thread (which will serialize | |
318 // them) or by the GC workers while holding the | |
319 // FreeList_lock. | |
320 // (b) If we're not at a safepoint, operations on the master free | |
321 // list should be invoked while holding the Heap_lock. | |
322 | |
323 if (SafepointSynchronize::is_at_safepoint()) { | |
324 guarantee(Thread::current()->is_VM_thread() || | |
325 FreeList_lock->owned_by_self(), "master free list MT safety protocol at a safepoint"); | |
326 } else { | |
327 guarantee(Heap_lock->owned_by_self(), "master free list MT safety protocol outside a safepoint"); | |
328 } | |
329 } | |
330 | |
331 void SecondaryFreeRegionListMtSafeChecker::check() { | |
332 // Secondary Free List MT safety protocol: | |
333 // Operations on the secondary free list should always be invoked | |
334 // while holding the SecondaryFreeList_lock. | |
335 | |
336 guarantee(SecondaryFreeList_lock->owned_by_self(), "secondary free list MT safety protocol"); | |
337 } | |
338 | |
339 void OldRegionSetMtSafeChecker::check() { | |
340 // Master Old Set MT safety protocol: | |
341 // (a) If we're at a safepoint, operations on the master old set | |
342 // should be invoked: | |
343 // - by the VM thread (which will serialize them), or | |
344 // - by the GC workers while holding the FreeList_lock, if we're | |
345 // at a safepoint for an evacuation pause (this lock is taken | |
346 // anyway when an GC alloc region is retired so that a new one | |
347 // is allocated from the free list), or | |
348 // - by the GC workers while holding the OldSets_lock, if we're at a | |
349 // safepoint for a cleanup pause. | |
350 // (b) If we're not at a safepoint, operations on the master old set | |
351 // should be invoked while holding the Heap_lock. | |
352 | |
353 if (SafepointSynchronize::is_at_safepoint()) { | |
354 guarantee(Thread::current()->is_VM_thread() | |
355 || FreeList_lock->owned_by_self() || OldSets_lock->owned_by_self(), | |
356 "master old set MT safety protocol at a safepoint"); | |
357 } else { | |
358 guarantee(Heap_lock->owned_by_self(), "master old set MT safety protocol outside a safepoint"); | |
359 } | |
360 } | |
361 | |
362 void HumongousRegionSetMtSafeChecker::check() { | |
363 // Humongous Set MT safety protocol: | |
364 // (a) If we're at a safepoint, operations on the master humongous | |
365 // set should be invoked by either the VM thread (which will | |
366 // serialize them) or by the GC workers while holding the | |
367 // OldSets_lock. | |
368 // (b) If we're not at a safepoint, operations on the master | |
369 // humongous set should be invoked while holding the Heap_lock. | |
370 | |
371 if (SafepointSynchronize::is_at_safepoint()) { | |
372 guarantee(Thread::current()->is_VM_thread() || | |
373 OldSets_lock->owned_by_self(), | |
374 "master humongous set MT safety protocol at a safepoint"); | |
375 } else { | |
376 guarantee(Heap_lock->owned_by_self(), | |
377 "master humongous set MT safety protocol outside a safepoint"); | |
378 } | |
379 } |