author | tschatzl |
Wed, 25 Nov 2015 14:43:29 +0100 | |
changeset 34300 | 6075c1e0e913 |
parent 33753 | 3add06d0880f |
child 35061 | be6025ebffea |
permissions | -rw-r--r-- |
7923 | 1 |
/* |
29580
a67a581cfe11
8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents:
27880
diff
changeset
|
2 |
* Copyright (c) 2011, 2015, Oracle and/or its affiliates. All rights reserved. |
7923 | 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" |
|
30764 | 26 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
27 |
#include "gc/g1/heapRegionRemSet.hpp" |
|
28 |
#include "gc/g1/heapRegionSet.inline.hpp" |
|
7923 | 29 |
|
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
30 |
uint FreeRegionList::_unrealistically_long_length = 0; |
7923 | 31 |
|
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
32 |
#ifndef PRODUCT |
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
33 |
void HeapRegionSetBase::verify_region(HeapRegion* hr) { |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
34 |
assert(hr->containing_set() == this, "Inconsistent containing set for %u", hr->hrm_index()); |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
35 |
assert(!hr->is_young(), "Adding young region %u", hr->hrm_index()); // currently we don't use these sets for young regions |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
36 |
assert(hr->is_humongous() == regions_humongous(), "Wrong humongous state for region %u and set %s", hr->hrm_index(), name()); |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
37 |
assert(hr->is_free() == regions_free(), "Wrong free state for region %u and set %s", hr->hrm_index(), name()); |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
38 |
assert(!hr->is_free() || hr->is_empty(), "Free region %u is not empty for set %s", hr->hrm_index(), name()); |
31346
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
30764
diff
changeset
|
39 |
assert(!hr->is_empty() || hr->is_free() || hr->is_archive(), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
40 |
"Empty region %u is not free or archive for set %s", hr->hrm_index(), name()); |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
41 |
assert(hr->rem_set()->verify_ready_for_par_iteration(), "Wrong iteration state %u", hr->hrm_index()); |
7923 | 42 |
} |
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
43 |
#endif |
7923 | 44 |
|
45 |
void HeapRegionSetBase::verify() { |
|
46 |
// It's important that we also observe the MT safety protocol even |
|
47 |
// for the verification calls. If we do verification without the |
|
48 |
// appropriate locks and the set changes underneath our feet |
|
49 |
// verification might fail and send us on a wild goose chase. |
|
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
50 |
check_mt_safety(); |
7923 | 51 |
|
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
52 |
guarantee_heap_region_set(( is_empty() && length() == 0 && total_capacity_bytes() == 0) || |
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
53 |
(!is_empty() && length() > 0 && total_capacity_bytes() > 0) , |
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
54 |
"invariant"); |
7923 | 55 |
} |
56 |
||
57 |
void HeapRegionSetBase::verify_start() { |
|
58 |
// See comment in verify() about MT safety and verification. |
|
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
59 |
check_mt_safety(); |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
60 |
assert_heap_region_set(!_verify_in_progress, "verification should not be in progress"); |
7923 | 61 |
|
62 |
// Do the basic verification first before we do the checks over the regions. |
|
63 |
HeapRegionSetBase::verify(); |
|
64 |
||
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
65 |
_verify_in_progress = true; |
7923 | 66 |
} |
67 |
||
68 |
void HeapRegionSetBase::verify_end() { |
|
69 |
// See comment in verify() about MT safety and verification. |
|
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
70 |
check_mt_safety(); |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
71 |
assert_heap_region_set(_verify_in_progress, "verification should be in progress"); |
7923 | 72 |
|
73 |
_verify_in_progress = false; |
|
74 |
} |
|
75 |
||
76 |
void HeapRegionSetBase::print_on(outputStream* out, bool print_contents) { |
|
77 |
out->cr(); |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
78 |
out->print_cr("Set: %s (" PTR_FORMAT ")", name(), p2i(this)); |
7923 | 79 |
out->print_cr(" Region Assumptions"); |
80 |
out->print_cr(" humongous : %s", BOOL_TO_STR(regions_humongous())); |
|
26696
623a25e6c686
8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents:
26316
diff
changeset
|
81 |
out->print_cr(" free : %s", BOOL_TO_STR(regions_free())); |
7923 | 82 |
out->print_cr(" Attributes"); |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10996
diff
changeset
|
83 |
out->print_cr(" length : %14u", length()); |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
84 |
out->print_cr(" total capacity : " SIZE_FORMAT_W(14) " bytes", |
7923 | 85 |
total_capacity_bytes()); |
86 |
} |
|
87 |
||
26696
623a25e6c686
8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents:
26316
diff
changeset
|
88 |
HeapRegionSetBase::HeapRegionSetBase(const char* name, bool humongous, bool free, HRSMtSafeChecker* mt_safety_checker) |
7923 | 89 |
: _name(name), _verify_in_progress(false), |
26696
623a25e6c686
8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents:
26316
diff
changeset
|
90 |
_is_humongous(humongous), _is_free(free), _mt_safety_checker(mt_safety_checker), |
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
91 |
_count() |
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
92 |
{ } |
7923 | 93 |
|
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
94 |
void FreeRegionList::set_unrealistically_long_length(uint len) { |
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
95 |
guarantee(_unrealistically_long_length == 0, "should only be set once"); |
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
96 |
_unrealistically_long_length = len; |
7923 | 97 |
} |
98 |
||
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
99 |
void FreeRegionList::remove_all() { |
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
100 |
check_mt_safety(); |
7923 | 101 |
verify_optional(); |
102 |
||
103 |
HeapRegion* curr = _head; |
|
104 |
while (curr != NULL) { |
|
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
105 |
verify_region(curr); |
7923 | 106 |
|
107 |
HeapRegion* next = curr->next(); |
|
108 |
curr->set_next(NULL); |
|
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
109 |
curr->set_prev(NULL); |
7923 | 110 |
curr->set_containing_set(NULL); |
111 |
curr = next; |
|
112 |
} |
|
113 |
clear(); |
|
114 |
||
115 |
verify_optional(); |
|
116 |
} |
|
117 |
||
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
118 |
void FreeRegionList::add_ordered(FreeRegionList* from_list) { |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
119 |
check_mt_safety(); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
120 |
from_list->check_mt_safety(); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
121 |
|
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
122 |
verify_optional(); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
123 |
from_list->verify_optional(); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
124 |
|
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
125 |
if (from_list->is_empty()) { |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
126 |
return; |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
127 |
} |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
128 |
|
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
129 |
#ifdef ASSERT |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
130 |
FreeRegionListIterator iter(from_list); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
131 |
while (iter.more_available()) { |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
132 |
HeapRegion* hr = iter.get_next(); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
133 |
// In set_containing_set() we check that we either set the value |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
134 |
// from NULL to non-NULL or vice versa to catch bugs. So, we have |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
135 |
// to NULL it first before setting it to the value. |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
136 |
hr->set_containing_set(NULL); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
137 |
hr->set_containing_set(this); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
138 |
} |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
139 |
#endif // ASSERT |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
140 |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
141 |
if (is_empty()) { |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
142 |
assert_free_region_list(length() == 0 && _tail == NULL, "invariant"); |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
143 |
_head = from_list->_head; |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
144 |
_tail = from_list->_tail; |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
145 |
} else { |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
146 |
HeapRegion* curr_to = _head; |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
147 |
HeapRegion* curr_from = from_list->_head; |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
148 |
|
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
149 |
while (curr_from != NULL) { |
26316
93f6b40c038b
8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents:
26160
diff
changeset
|
150 |
while (curr_to != NULL && curr_to->hrm_index() < curr_from->hrm_index()) { |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
151 |
curr_to = curr_to->next(); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
152 |
} |
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
153 |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
154 |
if (curr_to == NULL) { |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
155 |
// The rest of the from list should be added as tail |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
156 |
_tail->set_next(curr_from); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
157 |
curr_from->set_prev(_tail); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
158 |
curr_from = NULL; |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
159 |
} else { |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
160 |
HeapRegion* next_from = curr_from->next(); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
161 |
|
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
162 |
curr_from->set_next(curr_to); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
163 |
curr_from->set_prev(curr_to->prev()); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
164 |
if (curr_to->prev() == NULL) { |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
165 |
_head = curr_from; |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
166 |
} else { |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
167 |
curr_to->prev()->set_next(curr_from); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
168 |
} |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
169 |
curr_to->set_prev(curr_from); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
170 |
|
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
171 |
curr_from = next_from; |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
172 |
} |
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
173 |
} |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
174 |
|
26316
93f6b40c038b
8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents:
26160
diff
changeset
|
175 |
if (_tail->hrm_index() < from_list->_tail->hrm_index()) { |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
176 |
_tail = from_list->_tail; |
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
177 |
} |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
178 |
} |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
179 |
|
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
180 |
_count.increment(from_list->length(), from_list->total_capacity_bytes()); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
181 |
from_list->clear(); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
182 |
|
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
183 |
verify_optional(); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
184 |
from_list->verify_optional(); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
185 |
} |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
186 |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
187 |
void FreeRegionList::remove_starting_at(HeapRegion* first, uint num_regions) { |
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
188 |
check_mt_safety(); |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
189 |
assert_free_region_list(num_regions >= 1, "pre-condition"); |
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
190 |
assert_free_region_list(!is_empty(), "pre-condition"); |
7923 | 191 |
|
192 |
verify_optional(); |
|
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10996
diff
changeset
|
193 |
DEBUG_ONLY(uint old_length = length();) |
7923 | 194 |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
195 |
HeapRegion* curr = first; |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
10996
diff
changeset
|
196 |
uint count = 0; |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
197 |
while (count < num_regions) { |
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
198 |
verify_region(curr); |
7923 | 199 |
HeapRegion* next = curr->next(); |
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
200 |
HeapRegion* prev = curr->prev(); |
7923 | 201 |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
202 |
assert(count < num_regions, |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
203 |
"[%s] should not come across more regions " |
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
204 |
"pending for removal than num_regions: %u", |
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
205 |
name(), num_regions); |
7923 | 206 |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
207 |
if (prev == NULL) { |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
208 |
assert_free_region_list(_head == curr, "invariant"); |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
209 |
_head = next; |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
210 |
} else { |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
211 |
assert_free_region_list(_head != curr, "invariant"); |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
212 |
prev->set_next(next); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
213 |
} |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
214 |
if (next == NULL) { |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
215 |
assert_free_region_list(_tail == curr, "invariant"); |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
216 |
_tail = prev; |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
217 |
} else { |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
218 |
assert_free_region_list(_tail != curr, "invariant"); |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
219 |
next->set_prev(prev); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
220 |
} |
33752 | 221 |
if (_last == curr) { |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
222 |
_last = NULL; |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
223 |
} |
7923 | 224 |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
225 |
curr->set_next(NULL); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
226 |
curr->set_prev(NULL); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
227 |
remove(curr); |
7923 | 228 |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
229 |
count++; |
7923 | 230 |
curr = next; |
231 |
} |
|
232 |
||
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
233 |
assert(count == num_regions, |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
234 |
"[%s] count: %u should be == num_regions: %u", |
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
235 |
name(), count, num_regions); |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
236 |
assert(length() + num_regions == old_length, |
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
237 |
"[%s] new length should be consistent " |
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
238 |
"new length: %u old length: %u num_regions: %u", |
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
239 |
name(), length(), old_length, num_regions); |
7923 | 240 |
|
241 |
verify_optional(); |
|
242 |
} |
|
243 |
||
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
244 |
void FreeRegionList::verify() { |
7923 | 245 |
// See comment in HeapRegionSetBase::verify() about MT safety and |
246 |
// verification. |
|
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
247 |
check_mt_safety(); |
7923 | 248 |
|
249 |
// This will also do the basic verification too. |
|
250 |
verify_start(); |
|
251 |
||
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
252 |
verify_list(); |
7923 | 253 |
|
254 |
verify_end(); |
|
255 |
} |
|
256 |
||
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
257 |
void FreeRegionList::clear() { |
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
258 |
_count = HeapRegionSetCount(); |
7923 | 259 |
_head = NULL; |
260 |
_tail = NULL; |
|
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
261 |
_last = NULL; |
7923 | 262 |
} |
263 |
||
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
264 |
void FreeRegionList::print_on(outputStream* out, bool print_contents) { |
7923 | 265 |
HeapRegionSetBase::print_on(out, print_contents); |
266 |
out->print_cr(" Linking"); |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
267 |
out->print_cr(" head : " PTR_FORMAT, p2i(_head)); |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
268 |
out->print_cr(" tail : " PTR_FORMAT, p2i(_tail)); |
7923 | 269 |
|
270 |
if (print_contents) { |
|
271 |
out->print_cr(" Contents"); |
|
23450
c7c6202fc7e2
8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents:
13336
diff
changeset
|
272 |
FreeRegionListIterator iter(this); |
7923 | 273 |
while (iter.more_available()) { |
274 |
HeapRegion* hr = iter.get_next(); |
|
275 |
hr->print_on(out); |
|
276 |
} |
|
277 |
} |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
278 |
|
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
279 |
out->cr(); |
7923 | 280 |
} |
23459 | 281 |
|
282 |
void FreeRegionList::verify_list() { |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
24424
diff
changeset
|
283 |
HeapRegion* curr = _head; |
23459 | 284 |
HeapRegion* prev1 = NULL; |
285 |
HeapRegion* prev0 = NULL; |
|
286 |
uint count = 0; |
|
287 |
size_t capacity = 0; |
|
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
288 |
uint last_index = 0; |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
289 |
|
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
290 |
guarantee(_head == NULL || _head->prev() == NULL, "_head should not have a prev"); |
23459 | 291 |
while (curr != NULL) { |
292 |
verify_region(curr); |
|
293 |
||
294 |
count++; |
|
295 |
guarantee(count < _unrealistically_long_length, |
|
33753
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
296 |
"[%s] the calculated length: %u seems very long, is there maybe a cycle? curr: " PTR_FORMAT " prev0: " PTR_FORMAT " " "prev1: " PTR_FORMAT " length: %u", |
3add06d0880f
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents:
33752
diff
changeset
|
297 |
name(), count, p2i(curr), p2i(prev0), p2i(prev1), length()); |
23459 | 298 |
|
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
299 |
if (curr->next() != NULL) { |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
300 |
guarantee(curr->next()->prev() == curr, "Next or prev pointers messed up"); |
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
301 |
} |
26316
93f6b40c038b
8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents:
26160
diff
changeset
|
302 |
guarantee(curr->hrm_index() == 0 || curr->hrm_index() > last_index, "List should be sorted"); |
93f6b40c038b
8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents:
26160
diff
changeset
|
303 |
last_index = curr->hrm_index(); |
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
304 |
|
23459 | 305 |
capacity += curr->capacity(); |
306 |
||
307 |
prev1 = prev0; |
|
308 |
prev0 = curr; |
|
309 |
curr = curr->next(); |
|
310 |
} |
|
311 |
||
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
312 |
guarantee(_tail == prev0, "Expected %s to end with %u but it ended with %u.", name(), _tail->hrm_index(), prev0->hrm_index()); |
23471
ec9427262f0a
8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents:
23459
diff
changeset
|
313 |
guarantee(_tail == NULL || _tail->next() == NULL, "_tail should not have a next"); |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
314 |
guarantee(length() == count, "%s count mismatch. Expected %u, actual %u.", name(), length(), count); |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
315 |
guarantee(total_capacity_bytes() == capacity, "%s capacity mismatch. Expected " SIZE_FORMAT ", actual " SIZE_FORMAT, |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
316 |
name(), total_capacity_bytes(), capacity); |
23459 | 317 |
} |
318 |
||
319 |
// Note on the check_mt_safety() methods below: |
|
320 |
// |
|
321 |
// Verification of the "master" heap region sets / lists that are |
|
322 |
// maintained by G1CollectedHeap is always done during a STW pause and |
|
323 |
// by the VM thread at the start / end of the pause. The standard |
|
324 |
// verification methods all assert check_mt_safety(). This is |
|
325 |
// important as it ensures that verification is done without |
|
326 |
// concurrent updates taking place at the same time. It follows, that, |
|
327 |
// for the "master" heap region sets / lists, the check_mt_safety() |
|
328 |
// method should include the VM thread / STW case. |
|
329 |
||
330 |
void MasterFreeRegionListMtSafeChecker::check() { |
|
331 |
// Master Free List MT safety protocol: |
|
332 |
// (a) If we're at a safepoint, operations on the master free list |
|
333 |
// should be invoked by either the VM thread (which will serialize |
|
334 |
// them) or by the GC workers while holding the |
|
335 |
// FreeList_lock. |
|
336 |
// (b) If we're not at a safepoint, operations on the master free |
|
337 |
// list should be invoked while holding the Heap_lock. |
|
338 |
||
339 |
if (SafepointSynchronize::is_at_safepoint()) { |
|
340 |
guarantee(Thread::current()->is_VM_thread() || |
|
341 |
FreeList_lock->owned_by_self(), "master free list MT safety protocol at a safepoint"); |
|
342 |
} else { |
|
343 |
guarantee(Heap_lock->owned_by_self(), "master free list MT safety protocol outside a safepoint"); |
|
344 |
} |
|
345 |
} |
|
346 |
||
347 |
void SecondaryFreeRegionListMtSafeChecker::check() { |
|
348 |
// Secondary Free List MT safety protocol: |
|
349 |
// Operations on the secondary free list should always be invoked |
|
350 |
// while holding the SecondaryFreeList_lock. |
|
351 |
||
352 |
guarantee(SecondaryFreeList_lock->owned_by_self(), "secondary free list MT safety protocol"); |
|
353 |
} |
|
354 |
||
355 |
void OldRegionSetMtSafeChecker::check() { |
|
356 |
// Master Old Set MT safety protocol: |
|
357 |
// (a) If we're at a safepoint, operations on the master old set |
|
358 |
// should be invoked: |
|
359 |
// - by the VM thread (which will serialize them), or |
|
360 |
// - by the GC workers while holding the FreeList_lock, if we're |
|
361 |
// at a safepoint for an evacuation pause (this lock is taken |
|
362 |
// anyway when an GC alloc region is retired so that a new one |
|
363 |
// is allocated from the free list), or |
|
364 |
// - by the GC workers while holding the OldSets_lock, if we're at a |
|
365 |
// safepoint for a cleanup pause. |
|
366 |
// (b) If we're not at a safepoint, operations on the master old set |
|
367 |
// should be invoked while holding the Heap_lock. |
|
368 |
||
369 |
if (SafepointSynchronize::is_at_safepoint()) { |
|
370 |
guarantee(Thread::current()->is_VM_thread() |
|
371 |
|| FreeList_lock->owned_by_self() || OldSets_lock->owned_by_self(), |
|
372 |
"master old set MT safety protocol at a safepoint"); |
|
373 |
} else { |
|
374 |
guarantee(Heap_lock->owned_by_self(), "master old set MT safety protocol outside a safepoint"); |
|
375 |
} |
|
376 |
} |
|
377 |
||
378 |
void HumongousRegionSetMtSafeChecker::check() { |
|
379 |
// Humongous Set MT safety protocol: |
|
380 |
// (a) If we're at a safepoint, operations on the master humongous |
|
381 |
// set should be invoked by either the VM thread (which will |
|
382 |
// serialize them) or by the GC workers while holding the |
|
383 |
// OldSets_lock. |
|
384 |
// (b) If we're not at a safepoint, operations on the master |
|
385 |
// humongous set should be invoked while holding the Heap_lock. |
|
386 |
||
387 |
if (SafepointSynchronize::is_at_safepoint()) { |
|
388 |
guarantee(Thread::current()->is_VM_thread() || |
|
389 |
OldSets_lock->owned_by_self(), |
|
390 |
"master humongous set MT safety protocol at a safepoint"); |
|
391 |
} else { |
|
392 |
guarantee(Heap_lock->owned_by_self(), |
|
393 |
"master humongous set MT safety protocol outside a safepoint"); |
|
394 |
} |
|
395 |
} |
|
26160 | 396 |
|
397 |
void FreeRegionList_test() { |
|
398 |
FreeRegionList l("test"); |
|
399 |
||
400 |
const uint num_regions_in_test = 5; |
|
401 |
// Create a fake heap. It does not need to be valid, as the HeapRegion constructor |
|
402 |
// does not access it. |
|
403 |
MemRegion heap(NULL, num_regions_in_test * HeapRegion::GrainWords); |
|
404 |
// Allocate a fake BOT because the HeapRegion constructor initializes |
|
405 |
// the BOT. |
|
406 |
size_t bot_size = G1BlockOffsetSharedArray::compute_size(heap.word_size()); |
|
407 |
HeapWord* bot_data = NEW_C_HEAP_ARRAY(HeapWord, bot_size, mtGC); |
|
408 |
ReservedSpace bot_rs(G1BlockOffsetSharedArray::compute_size(heap.word_size())); |
|
409 |
G1RegionToSpaceMapper* bot_storage = |
|
410 |
G1RegionToSpaceMapper::create_mapper(bot_rs, |
|
30158
bd6094906ef8
8058354: SPECjvm2008-Derby -2.7% performance regression on Solaris-X64 starting with 9-b29
tschatzl
parents:
29796
diff
changeset
|
411 |
bot_rs.size(), |
26160 | 412 |
os::vm_page_size(), |
413 |
HeapRegion::GrainBytes, |
|
414 |
G1BlockOffsetSharedArray::N_bytes, |
|
415 |
mtGC); |
|
416 |
G1BlockOffsetSharedArray oa(heap, bot_storage); |
|
417 |
bot_storage->commit_regions(0, num_regions_in_test); |
|
27629
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
418 |
|
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
419 |
// Set up memory regions for the heap regions. |
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
420 |
MemRegion mr0(heap.start(), HeapRegion::GrainWords); |
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
421 |
MemRegion mr1(mr0.end(), HeapRegion::GrainWords); |
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
422 |
MemRegion mr2(mr1.end(), HeapRegion::GrainWords); |
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
423 |
MemRegion mr3(mr2.end(), HeapRegion::GrainWords); |
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
424 |
MemRegion mr4(mr3.end(), HeapRegion::GrainWords); |
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
425 |
|
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
426 |
HeapRegion hr0(0, &oa, mr0); |
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
427 |
HeapRegion hr1(1, &oa, mr1); |
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
428 |
HeapRegion hr2(2, &oa, mr2); |
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
429 |
HeapRegion hr3(3, &oa, mr3); |
81ea316b7244
8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents:
26846
diff
changeset
|
430 |
HeapRegion hr4(4, &oa, mr4); |
26160 | 431 |
l.add_ordered(&hr1); |
432 |
l.add_ordered(&hr0); |
|
433 |
l.add_ordered(&hr3); |
|
434 |
l.add_ordered(&hr4); |
|
435 |
l.add_ordered(&hr2); |
|
436 |
assert(l.length() == num_regions_in_test, "wrong length"); |
|
437 |
l.verify_list(); |
|
438 |
||
439 |
bot_storage->uncommit_regions(0, num_regions_in_test); |
|
440 |
delete bot_storage; |
|
27880
afb974a04396
8060074: os::free() takes MemoryTrackingLevel but doesn't need it
coleenp
parents:
27629
diff
changeset
|
441 |
FREE_C_HEAP_ARRAY(HeapWord, bot_data); |
26160 | 442 |
} |