annotate src/share/vm/gc_implementation/g1/heapRegionSet.cpp @ 20481:c02ec279b062

8057768: Make heap region region type in G1 HeapRegion explicit Reviewed-by: brutisso, tschatzl
author brutisso
date Tue, 16 Sep 2014 14:27:40 +0200
parents a8ea2f110d87
children 7848fc12602b
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
1 /*
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
2 * Copyright (c) 2011, 2014, Oracle and/or its affiliates. All rights reserved.
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
4 *
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
7 * published by the Free Software Foundation.
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
8 *
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
13 * accompanied this code).
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
14 *
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
18 *
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
21 * questions.
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
22 *
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
23 */
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
24
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
25 #include "precompiled.hpp"
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
26 #include "gc_implementation/g1/g1CollectedHeap.inline.hpp"
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
27 #include "gc_implementation/g1/heapRegionRemSet.hpp"
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
28 #include "gc_implementation/g1/heapRegionSet.inline.hpp"
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
29
17937
78bbf4d43a14 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 17773
diff changeset
30 PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC
78bbf4d43a14 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 17773
diff changeset
31
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
32 uint FreeRegionList::_unrealistically_long_length = 0;
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
33
2361
1216415d8e35 7014923: G1: code cleanup
tonyp
parents: 2152
diff changeset
34 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
35 msg->append("[%s] %s ln: %u cy: "SIZE_FORMAT,
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
36 name(), message, length(), total_capacity_bytes());
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
37 fill_in_ext_msg_extra(msg);
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
38 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
39
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
40 #ifndef PRODUCT
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
41 void HeapRegionSetBase::verify_region(HeapRegion* hr) {
20377
a8ea2f110d87 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 20337
diff changeset
42 assert(hr->containing_set() == this, err_msg("Inconsistent containing set for %u", hr->hrm_index()));
a8ea2f110d87 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 20337
diff changeset
43 assert(!hr->is_young(), err_msg("Adding young region %u", hr->hrm_index())); // currently we don't use these sets for young regions
a8ea2f110d87 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 20337
diff changeset
44 assert(hr->isHumongous() == regions_humongous(), err_msg("Wrong humongous state for region %u and set %s", hr->hrm_index(), name()));
20481
c02ec279b062 8057768: Make heap region region type in G1 HeapRegion explicit
brutisso
parents: 20377
diff changeset
45 assert(hr->is_free() == regions_free(), err_msg("Wrong free state for region %u and set %s", hr->hrm_index(), name()));
c02ec279b062 8057768: Make heap region region type in G1 HeapRegion explicit
brutisso
parents: 20377
diff changeset
46 assert(!hr->is_free() || hr->is_empty(), err_msg("Free region %u is not empty for set %s", hr->hrm_index(), name()));
c02ec279b062 8057768: Make heap region region type in G1 HeapRegion explicit
brutisso
parents: 20377
diff changeset
47 assert(!hr->is_empty() || hr->is_free(), err_msg("Empty region %u is not free for set %s", hr->hrm_index(), name()));
20377
a8ea2f110d87 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 20337
diff changeset
48 assert(hr->rem_set()->verify_ready_for_par_iteration(), err_msg("Wrong iteration state %u", hr->hrm_index()));
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
49 }
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
50 #endif
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
51
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
52 void HeapRegionSetBase::verify() {
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
53 // It's important that we also observe the MT safety protocol even
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
54 // for the verification calls. If we do verification without the
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
55 // appropriate locks and the set changes underneath our feet
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
56 // verification might fail and send us on a wild goose chase.
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
57 check_mt_safety();
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
58
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
59 guarantee(( is_empty() && length() == 0 && total_capacity_bytes() == 0) ||
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
60 (!is_empty() && length() >= 0 && total_capacity_bytes() >= 0),
2361
1216415d8e35 7014923: G1: code cleanup
tonyp
parents: 2152
diff changeset
61 hrs_ext_msg(this, "invariant"));
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
62 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
63
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
64 void HeapRegionSetBase::verify_start() {
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
65 // See comment in verify() about MT safety and verification.
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
66 check_mt_safety();
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
67 assert(!_verify_in_progress,
2361
1216415d8e35 7014923: G1: code cleanup
tonyp
parents: 2152
diff changeset
68 hrs_ext_msg(this, "verification should not be in progress"));
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
69
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
70 // Do the basic verification first before we do the checks over the regions.
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
71 HeapRegionSetBase::verify();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
72
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
73 _verify_in_progress = true;
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
74 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
75
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
76 void HeapRegionSetBase::verify_end() {
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
77 // See comment in verify() about MT safety and verification.
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
78 check_mt_safety();
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
79 assert(_verify_in_progress,
2361
1216415d8e35 7014923: G1: code cleanup
tonyp
parents: 2152
diff changeset
80 hrs_ext_msg(this, "verification should be in progress"));
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
81
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
82 _verify_in_progress = false;
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
83 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
84
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
85 void HeapRegionSetBase::print_on(outputStream* out, bool print_contents) {
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
86 out->cr();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
87 out->print_cr("Set: %s ("PTR_FORMAT")", name(), this);
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
88 out->print_cr(" Region Assumptions");
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
89 out->print_cr(" humongous : %s", BOOL_TO_STR(regions_humongous()));
20481
c02ec279b062 8057768: Make heap region region type in G1 HeapRegion explicit
brutisso
parents: 20377
diff changeset
90 out->print_cr(" free : %s", BOOL_TO_STR(regions_free()));
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
91 out->print_cr(" Attributes");
6010
720b6a76dd9d 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 4072
diff changeset
92 out->print_cr(" length : %14u", length());
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
93 out->print_cr(" total capacity : "SIZE_FORMAT_W(14)" bytes",
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
94 total_capacity_bytes());
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
95 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
96
20481
c02ec279b062 8057768: Make heap region region type in G1 HeapRegion explicit
brutisso
parents: 20377
diff changeset
97 HeapRegionSetBase::HeapRegionSetBase(const char* name, bool humongous, bool free, HRSMtSafeChecker* mt_safety_checker)
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
98 : _name(name), _verify_in_progress(false),
20481
c02ec279b062 8057768: Make heap region region type in G1 HeapRegion explicit
brutisso
parents: 20377
diff changeset
99 _is_humongous(humongous), _is_free(free), _mt_safety_checker(mt_safety_checker),
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
100 _count()
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
101 { }
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
102
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
103 void FreeRegionList::set_unrealistically_long_length(uint len) {
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
104 guarantee(_unrealistically_long_length == 0, "should only be set once");
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
105 _unrealistically_long_length = len;
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
106 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
107
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
108 void FreeRegionList::fill_in_ext_msg_extra(hrs_ext_msg* msg) {
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
109 msg->append(" hd: "PTR_FORMAT" tl: "PTR_FORMAT, _head, _tail);
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
110 }
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
111
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
112 void FreeRegionList::remove_all() {
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
113 check_mt_safety();
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
114 verify_optional();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
115
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
116 HeapRegion* curr = _head;
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
117 while (curr != NULL) {
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
118 verify_region(curr);
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
119
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
120 HeapRegion* next = curr->next();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
121 curr->set_next(NULL);
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
122 curr->set_prev(NULL);
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
123 curr->set_containing_set(NULL);
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
124 curr = next;
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
125 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
126 clear();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
127
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
128 verify_optional();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
129 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
130
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
131 void FreeRegionList::add_ordered(FreeRegionList* from_list) {
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
132 check_mt_safety();
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
133 from_list->check_mt_safety();
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
134
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
135 verify_optional();
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
136 from_list->verify_optional();
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
137
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
138 if (from_list->is_empty()) {
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
139 return;
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
140 }
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
141
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
142 #ifdef ASSERT
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
143 FreeRegionListIterator iter(from_list);
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
144 while (iter.more_available()) {
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
145 HeapRegion* hr = iter.get_next();
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
146 // In set_containing_set() we check that we either set the value
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
147 // from NULL to non-NULL or vice versa to catch bugs. So, we have
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
148 // to NULL it first before setting it to the value.
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
149 hr->set_containing_set(NULL);
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
150 hr->set_containing_set(this);
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
151 }
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
152 #endif // ASSERT
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
153
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
154 if (is_empty()) {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
155 assert(length() == 0 && _tail == NULL, hrs_ext_msg(this, "invariant"));
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
156 _head = from_list->_head;
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
157 _tail = from_list->_tail;
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
158 } else {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
159 HeapRegion* curr_to = _head;
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
160 HeapRegion* curr_from = from_list->_head;
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
161
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
162 while (curr_from != NULL) {
20377
a8ea2f110d87 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 20337
diff changeset
163 while (curr_to != NULL && curr_to->hrm_index() < curr_from->hrm_index()) {
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
164 curr_to = curr_to->next();
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
165 }
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
166
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
167 if (curr_to == NULL) {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
168 // The rest of the from list should be added as tail
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
169 _tail->set_next(curr_from);
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
170 curr_from->set_prev(_tail);
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
171 curr_from = NULL;
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
172 } else {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
173 HeapRegion* next_from = curr_from->next();
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
174
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
175 curr_from->set_next(curr_to);
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
176 curr_from->set_prev(curr_to->prev());
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
177 if (curr_to->prev() == NULL) {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
178 _head = curr_from;
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
179 } else {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
180 curr_to->prev()->set_next(curr_from);
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
181 }
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
182 curr_to->set_prev(curr_from);
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
183
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
184 curr_from = next_from;
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
185 }
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
186 }
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
187
20377
a8ea2f110d87 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 20337
diff changeset
188 if (_tail->hrm_index() < from_list->_tail->hrm_index()) {
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
189 _tail = from_list->_tail;
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
190 }
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
191 }
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
192
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
193 _count.increment(from_list->length(), from_list->total_capacity_bytes());
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
194 from_list->clear();
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
195
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
196 verify_optional();
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
197 from_list->verify_optional();
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
198 }
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
199
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
200 void FreeRegionList::remove_starting_at(HeapRegion* first, uint num_regions) {
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
201 check_mt_safety();
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
202 assert(num_regions >= 1, hrs_ext_msg(this, "pre-condition"));
2361
1216415d8e35 7014923: G1: code cleanup
tonyp
parents: 2152
diff changeset
203 assert(!is_empty(), hrs_ext_msg(this, "pre-condition"));
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
204
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
205 verify_optional();
6010
720b6a76dd9d 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 4072
diff changeset
206 DEBUG_ONLY(uint old_length = length();)
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
207
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
208 HeapRegion* curr = first;
6010
720b6a76dd9d 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 4072
diff changeset
209 uint count = 0;
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
210 while (count < num_regions) {
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
211 verify_region(curr);
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
212 HeapRegion* next = curr->next();
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
213 HeapRegion* prev = curr->prev();
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
214
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
215 assert(count < num_regions,
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
216 hrs_err_msg("[%s] should not come across more regions "
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
217 "pending for removal than num_regions: %u",
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
218 name(), num_regions));
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
219
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
220 if (prev == NULL) {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
221 assert(_head == curr, hrs_ext_msg(this, "invariant"));
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
222 _head = next;
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
223 } else {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
224 assert(_head != curr, hrs_ext_msg(this, "invariant"));
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
225 prev->set_next(next);
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
226 }
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
227 if (next == NULL) {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
228 assert(_tail == curr, hrs_ext_msg(this, "invariant"));
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
229 _tail = prev;
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
230 } else {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
231 assert(_tail != curr, hrs_ext_msg(this, "invariant"));
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
232 next->set_prev(prev);
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
233 }
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
234 if (_last = curr) {
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
235 _last = NULL;
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
236 }
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
237
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
238 curr->set_next(NULL);
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
239 curr->set_prev(NULL);
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
240 remove(curr);
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
241
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
242 count++;
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
243 curr = next;
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
244 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
245
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
246 assert(count == num_regions,
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
247 hrs_err_msg("[%s] count: %u should be == num_regions: %u",
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
248 name(), count, num_regions));
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
249 assert(length() + num_regions == old_length,
2361
1216415d8e35 7014923: G1: code cleanup
tonyp
parents: 2152
diff changeset
250 hrs_err_msg("[%s] new length should be consistent "
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
251 "new length: %u old length: %u num_regions: %u",
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
252 name(), length(), old_length, num_regions));
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
253
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
254 verify_optional();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
255 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
256
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
257 void FreeRegionList::verify() {
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
258 // See comment in HeapRegionSetBase::verify() about MT safety and
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
259 // verification.
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
260 check_mt_safety();
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
261
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
262 // This will also do the basic verification too.
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
263 verify_start();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
264
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
265 verify_list();
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
266
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
267 verify_end();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
268 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
269
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
270 void FreeRegionList::clear() {
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
271 _count = HeapRegionSetCount();
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
272 _head = NULL;
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
273 _tail = NULL;
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
274 _last = NULL;
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
275 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
276
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
277 void FreeRegionList::print_on(outputStream* out, bool print_contents) {
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
278 HeapRegionSetBase::print_on(out, print_contents);
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
279 out->print_cr(" Linking");
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
280 out->print_cr(" head : "PTR_FORMAT, _head);
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
281 out->print_cr(" tail : "PTR_FORMAT, _tail);
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
282
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
283 if (print_contents) {
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
284 out->print_cr(" Contents");
17736
58fc1b1523dc 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 6254
diff changeset
285 FreeRegionListIterator iter(this);
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
286 while (iter.more_available()) {
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
287 HeapRegion* hr = iter.get_next();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
288 hr->print_on(out);
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
289 }
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
290 }
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
291
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
292 out->cr();
2152
0fa27f37d4d4 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
293 }
17737
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
294
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
295 void FreeRegionList::verify_list() {
20336
6701abbc4441 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 17937
diff changeset
296 HeapRegion* curr = _head;
17737
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
297 HeapRegion* prev1 = NULL;
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
298 HeapRegion* prev0 = NULL;
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
299 uint count = 0;
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
300 size_t capacity = 0;
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
301 uint last_index = 0;
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
302
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
303 guarantee(_head == NULL || _head->prev() == NULL, "_head should not have a prev");
17737
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
304 while (curr != NULL) {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
305 verify_region(curr);
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
306
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
307 count++;
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
308 guarantee(count < _unrealistically_long_length,
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
309 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()));
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
310
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
311 if (curr->next() != NULL) {
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
312 guarantee(curr->next()->prev() == curr, "Next or prev pointers messed up");
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
313 }
20377
a8ea2f110d87 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 20337
diff changeset
314 guarantee(curr->hrm_index() == 0 || curr->hrm_index() > last_index, "List should be sorted");
a8ea2f110d87 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 20337
diff changeset
315 last_index = curr->hrm_index();
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
316
17737
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
317 capacity += curr->capacity();
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
318
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
319 prev1 = prev0;
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
320 prev0 = curr;
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
321 curr = curr->next();
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
322 }
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
323
20377
a8ea2f110d87 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 20337
diff changeset
324 guarantee(_tail == prev0, err_msg("Expected %s to end with %u but it ended with %u.", name(), _tail->hrm_index(), prev0->hrm_index()));
17773
8ee855b4e667 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 17737
diff changeset
325 guarantee(_tail == NULL || _tail->next() == NULL, "_tail should not have a next");
17737
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
326 guarantee(length() == count, err_msg("%s count mismatch. Expected %u, actual %u.", name(), length(), count));
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
327 guarantee(total_capacity_bytes() == capacity, err_msg("%s capacity mismatch. Expected " SIZE_FORMAT ", actual " SIZE_FORMAT,
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
328 name(), total_capacity_bytes(), capacity));
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
329 }
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
330
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
331 // Note on the check_mt_safety() methods below:
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
332 //
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
333 // Verification of the "master" heap region sets / lists that are
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
334 // maintained by G1CollectedHeap is always done during a STW pause and
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
335 // by the VM thread at the start / end of the pause. The standard
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
336 // verification methods all assert check_mt_safety(). This is
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
337 // important as it ensures that verification is done without
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
338 // concurrent updates taking place at the same time. It follows, that,
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
339 // for the "master" heap region sets / lists, the check_mt_safety()
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
340 // method should include the VM thread / STW case.
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
341
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
342 void MasterFreeRegionListMtSafeChecker::check() {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
343 // Master Free List MT safety protocol:
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
344 // (a) If we're at a safepoint, operations on the master free list
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
345 // should be invoked by either the VM thread (which will serialize
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
346 // them) or by the GC workers while holding the
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
347 // FreeList_lock.
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
348 // (b) If we're not at a safepoint, operations on the master free
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
349 // list should be invoked while holding the Heap_lock.
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
350
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
351 if (SafepointSynchronize::is_at_safepoint()) {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
352 guarantee(Thread::current()->is_VM_thread() ||
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
353 FreeList_lock->owned_by_self(), "master free list MT safety protocol at a safepoint");
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
354 } else {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
355 guarantee(Heap_lock->owned_by_self(), "master free list MT safety protocol outside a safepoint");
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
356 }
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
357 }
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
358
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
359 void SecondaryFreeRegionListMtSafeChecker::check() {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
360 // Secondary Free List MT safety protocol:
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
361 // Operations on the secondary free list should always be invoked
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
362 // while holding the SecondaryFreeList_lock.
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
363
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
364 guarantee(SecondaryFreeList_lock->owned_by_self(), "secondary free list MT safety protocol");
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
365 }
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
366
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
367 void OldRegionSetMtSafeChecker::check() {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
368 // Master Old Set MT safety protocol:
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
369 // (a) If we're at a safepoint, operations on the master old set
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
370 // should be invoked:
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
371 // - by the VM thread (which will serialize them), or
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
372 // - by the GC workers while holding the FreeList_lock, if we're
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
373 // at a safepoint for an evacuation pause (this lock is taken
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
374 // anyway when an GC alloc region is retired so that a new one
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
375 // is allocated from the free list), or
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
376 // - by the GC workers while holding the OldSets_lock, if we're at a
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
377 // safepoint for a cleanup pause.
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
378 // (b) If we're not at a safepoint, operations on the master old set
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
379 // should be invoked while holding the Heap_lock.
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
380
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
381 if (SafepointSynchronize::is_at_safepoint()) {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
382 guarantee(Thread::current()->is_VM_thread()
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
383 || FreeList_lock->owned_by_self() || OldSets_lock->owned_by_self(),
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
384 "master old set MT safety protocol at a safepoint");
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
385 } else {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
386 guarantee(Heap_lock->owned_by_self(), "master old set MT safety protocol outside a safepoint");
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
387 }
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
388 }
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
389
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
390 void HumongousRegionSetMtSafeChecker::check() {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
391 // Humongous Set MT safety protocol:
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
392 // (a) If we're at a safepoint, operations on the master humongous
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
393 // set should be invoked by either the VM thread (which will
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
394 // serialize them) or by the GC workers while holding the
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
395 // OldSets_lock.
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
396 // (b) If we're not at a safepoint, operations on the master
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
397 // humongous set should be invoked while holding the Heap_lock.
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
398
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
399 if (SafepointSynchronize::is_at_safepoint()) {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
400 guarantee(Thread::current()->is_VM_thread() ||
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
401 OldSets_lock->owned_by_self(),
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
402 "master humongous set MT safety protocol at a safepoint");
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
403 } else {
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
404 guarantee(Heap_lock->owned_by_self(),
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
405 "master humongous set MT safety protocol outside a safepoint");
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
406 }
0d2ce7411240 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 17736
diff changeset
407 }
20337
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
408
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
409 void FreeRegionList_test() {
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
410 FreeRegionList l("test");
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
411
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
412 const uint num_regions_in_test = 5;
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
413 // Create a fake heap. It does not need to be valid, as the HeapRegion constructor
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
414 // does not access it.
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
415 MemRegion heap(NULL, num_regions_in_test * HeapRegion::GrainWords);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
416 // Allocate a fake BOT because the HeapRegion constructor initializes
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
417 // the BOT.
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
418 size_t bot_size = G1BlockOffsetSharedArray::compute_size(heap.word_size());
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
419 HeapWord* bot_data = NEW_C_HEAP_ARRAY(HeapWord, bot_size, mtGC);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
420 ReservedSpace bot_rs(G1BlockOffsetSharedArray::compute_size(heap.word_size()));
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
421 G1RegionToSpaceMapper* bot_storage =
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
422 G1RegionToSpaceMapper::create_mapper(bot_rs,
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
423 os::vm_page_size(),
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
424 HeapRegion::GrainBytes,
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
425 G1BlockOffsetSharedArray::N_bytes,
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
426 mtGC);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
427 G1BlockOffsetSharedArray oa(heap, bot_storage);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
428 bot_storage->commit_regions(0, num_regions_in_test);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
429 HeapRegion hr0(0, &oa, heap);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
430 HeapRegion hr1(1, &oa, heap);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
431 HeapRegion hr2(2, &oa, heap);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
432 HeapRegion hr3(3, &oa, heap);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
433 HeapRegion hr4(4, &oa, heap);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
434 l.add_ordered(&hr1);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
435 l.add_ordered(&hr0);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
436 l.add_ordered(&hr3);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
437 l.add_ordered(&hr4);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
438 l.add_ordered(&hr2);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
439 assert(l.length() == num_regions_in_test, "wrong length");
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
440 l.verify_list();
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
441
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
442 bot_storage->uncommit_regions(0, num_regions_in_test);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
443 delete bot_storage;
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
444 FREE_C_HEAP_ARRAY(HeapWord, bot_data, mtGC);
1f1d373cd044 8038423: G1: Decommit memory within heap
tschatzl
parents: 20336
diff changeset
445 }