author | kbarrett |
Tue, 30 Aug 2016 23:48:16 -0400 | |
changeset 40892 | 330a02d935ad |
parent 40655 | 9f644073d3a0 |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
37143
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
2 |
* Copyright (c) 2001, 2016, Oracle and/or its affiliates. All rights reserved. |
1374 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4473
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4473
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4473
diff
changeset
|
21 |
* questions. |
1374 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
30764 | 26 |
#include "gc/g1/collectionSetChooser.hpp" |
27 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
|
28 |
#include "gc/shared/space.inline.hpp" |
|
40655
9f644073d3a0
8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents:
37985
diff
changeset
|
29 |
#include "runtime/atomic.hpp" |
1374 | 30 |
|
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
31 |
// Even though we don't use the GC efficiency in our heuristics as |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
32 |
// much as we used to, we still order according to GC efficiency. This |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
33 |
// will cause regions with a lot of live objects and large RSets to |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
34 |
// end up at the end of the array. Given that we might skip collecting |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
35 |
// the last few old regions, if after a few mixed GCs the remaining |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
36 |
// have reclaimable bytes under a certain threshold, the hope is that |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
37 |
// the ones we'll skip are ones with both large RSets and a lot of |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
38 |
// live objects, not the ones with just a lot of live objects if we |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
39 |
// ordered according to the amount of reclaimable bytes per region. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
40 |
static int order_regions(HeapRegion* hr1, HeapRegion* hr2) { |
1374 | 41 |
if (hr1 == NULL) { |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
42 |
if (hr2 == NULL) { |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
43 |
return 0; |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
44 |
} else { |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
45 |
return 1; |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
46 |
} |
1374 | 47 |
} else if (hr2 == NULL) { |
48 |
return -1; |
|
49 |
} |
|
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
50 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
51 |
double gc_eff1 = hr1->gc_efficiency(); |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
52 |
double gc_eff2 = hr2->gc_efficiency(); |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
53 |
if (gc_eff1 > gc_eff2) { |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
54 |
return -1; |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
55 |
} if (gc_eff1 < gc_eff2) { |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
56 |
return 1; |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
57 |
} else { |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
58 |
return 0; |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
59 |
} |
1374 | 60 |
} |
61 |
||
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
62 |
static int order_regions(HeapRegion** hr1p, HeapRegion** hr2p) { |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
63 |
return order_regions(*hr1p, *hr2p); |
1374 | 64 |
} |
65 |
||
66 |
CollectionSetChooser::CollectionSetChooser() : |
|
67 |
// The line below is the worst bit of C++ hackery I've ever written |
|
68 |
// (Detlefs, 11/23). You should think of it as equivalent to |
|
69 |
// "_regions(100, true)": initialize the growable array and inform it |
|
6183
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
70 |
// that it should allocate its elem array(s) on the C heap. |
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
71 |
// |
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
72 |
// The first argument, however, is actually a comma expression |
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
73 |
// (set_allocation_type(this, C_HEAP), 100). The purpose of the |
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
74 |
// set_allocation_type() call is to replace the default allocation |
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
75 |
// type for embedded objects STACK_OR_EMBEDDED with C_HEAP. It will |
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
76 |
// allow to pass the assert in GenericGrowableArray() which checks |
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
77 |
// that a growable array object must be on C heap if elements are. |
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
78 |
// |
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
79 |
// Note: containing object is allocated on C heap since it is CHeapObj. |
4c74cfe14f20
6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents:
5547
diff
changeset
|
80 |
// |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
81 |
_regions((ResourceObj::set_allocation_type((address) &_regions, |
1374 | 82 |
ResourceObj::C_HEAP), |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
83 |
100), true /* C_Heap */), |
32745 | 84 |
_front(0), _end(0), _first_par_unreserved_idx(0), |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
85 |
_region_live_threshold_bytes(0), _remaining_reclaimable_bytes(0) { |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
86 |
_region_live_threshold_bytes = |
15089 | 87 |
HeapRegion::GrainBytes * (size_t) G1MixedGCLiveThresholdPercent / 100; |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
88 |
} |
1374 | 89 |
|
90 |
#ifndef PRODUCT |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
91 |
void CollectionSetChooser::verify() { |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
92 |
guarantee(_end <= regions_length(), "_end: %u regions length: %u", _end, regions_length()); |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
93 |
guarantee(_front <= _end, "_front: %u _end: %u", _front, _end); |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
94 |
uint index = 0; |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
95 |
size_t sum_of_reclaimable_bytes = 0; |
32745 | 96 |
while (index < _front) { |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
97 |
guarantee(regions_at(index) == NULL, |
32745 | 98 |
"all entries before _front should be NULL"); |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
99 |
index += 1; |
1374 | 100 |
} |
101 |
HeapRegion *prev = NULL; |
|
32745 | 102 |
while (index < _end) { |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
103 |
HeapRegion *curr = regions_at(index++); |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
104 |
guarantee(curr != NULL, "Regions in _regions array cannot be NULL"); |
10680
f7bdba11999b
7095236: G1: _markedRegions never contains NULL regions
ysr
parents:
10523
diff
changeset
|
105 |
guarantee(!curr->is_young(), "should not be young!"); |
31346
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
30874
diff
changeset
|
106 |
guarantee(!curr->is_pinned(), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
107 |
"Pinned region should not be in collection set (index %u)", curr->hrm_index()); |
10680
f7bdba11999b
7095236: G1: _markedRegions never contains NULL regions
ysr
parents:
10523
diff
changeset
|
108 |
if (prev != NULL) { |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
109 |
guarantee(order_regions(prev, curr) != 1, |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
110 |
"GC eff prev: %1.4f GC eff curr: %1.4f", |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
111 |
prev->gc_efficiency(), curr->gc_efficiency()); |
1374 | 112 |
} |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
113 |
sum_of_reclaimable_bytes += curr->reclaimable_bytes(); |
10680
f7bdba11999b
7095236: G1: _markedRegions never contains NULL regions
ysr
parents:
10523
diff
changeset
|
114 |
prev = curr; |
1374 | 115 |
} |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
116 |
guarantee(sum_of_reclaimable_bytes == _remaining_reclaimable_bytes, |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
117 |
"reclaimable bytes inconsistent, " |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
118 |
"remaining: " SIZE_FORMAT " sum: " SIZE_FORMAT, |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
119 |
_remaining_reclaimable_bytes, sum_of_reclaimable_bytes); |
1374 | 120 |
} |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
121 |
#endif // !PRODUCT |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
122 |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
123 |
void CollectionSetChooser::sort_regions() { |
1374 | 124 |
// First trim any unused portion of the top in the parallel case. |
125 |
if (_first_par_unreserved_idx > 0) { |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
126 |
assert(_first_par_unreserved_idx <= regions_length(), |
1374 | 127 |
"Or we didn't reserved enough length"); |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
128 |
regions_trunc_to(_first_par_unreserved_idx); |
1374 | 129 |
} |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
130 |
_regions.sort(order_regions); |
32745 | 131 |
assert(_end <= regions_length(), "Requirement"); |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
132 |
#ifdef ASSERT |
32745 | 133 |
for (uint i = 0; i < _end; i++) { |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
134 |
assert(regions_at(i) != NULL, "Should be true by sorting!"); |
1374 | 135 |
} |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
136 |
#endif // ASSERT |
35061 | 137 |
if (log_is_enabled(Trace, gc, liveness)) { |
138 |
G1PrintRegionLivenessInfoClosure cl("Post-Sorting"); |
|
32745 | 139 |
for (uint i = 0; i < _end; ++i) { |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
140 |
HeapRegion* r = regions_at(i); |
8930
52368505ee8e
7027766: G1: introduce flag to dump the liveness information per region at the end of marking
tonyp
parents:
7397
diff
changeset
|
141 |
cl.doHeapRegion(r); |
1374 | 142 |
} |
143 |
} |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
144 |
verify(); |
1374 | 145 |
} |
146 |
||
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
147 |
void CollectionSetChooser::add_region(HeapRegion* hr) { |
31346
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
30874
diff
changeset
|
148 |
assert(!hr->is_pinned(), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
149 |
"Pinned region shouldn't be added to the collection set (index %u)", hr->hrm_index()); |
1374 | 150 |
assert(!hr->is_young(), "should not be young!"); |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
151 |
_regions.append(hr); |
32745 | 152 |
_end++; |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
153 |
_remaining_reclaimable_bytes += hr->reclaimable_bytes(); |
1374 | 154 |
hr->calc_gc_efficiency(); |
155 |
} |
|
156 |
||
32745 | 157 |
void CollectionSetChooser::push(HeapRegion* hr) { |
158 |
assert(hr != NULL, "Can't put back a NULL region"); |
|
159 |
assert(_front >= 1, "Too many regions have been put back"); |
|
160 |
_front--; |
|
161 |
regions_at_put(_front, hr); |
|
162 |
_remaining_reclaimable_bytes += hr->reclaimable_bytes(); |
|
163 |
} |
|
164 |
||
30874
18714bae50db
8080837: Move number of workers calculation out of CollectionSetChooser::prepare_for_par_region_addition
stefank
parents:
30764
diff
changeset
|
165 |
void CollectionSetChooser::prepare_for_par_region_addition(uint n_threads, |
18714bae50db
8080837: Move number of workers calculation out of CollectionSetChooser::prepare_for_par_region_addition
stefank
parents:
30764
diff
changeset
|
166 |
uint n_regions, |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
167 |
uint chunk_size) { |
1374 | 168 |
_first_par_unreserved_idx = 0; |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
169 |
uint max_waste = n_threads * chunk_size; |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
170 |
// it should be aligned with respect to chunk_size |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
171 |
uint aligned_n_regions = (n_regions + chunk_size - 1) / chunk_size * chunk_size; |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
172 |
assert(aligned_n_regions % chunk_size == 0, "should be aligned"); |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
173 |
regions_at_put_grow(aligned_n_regions + max_waste - 1, NULL); |
1374 | 174 |
} |
175 |
||
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
176 |
uint CollectionSetChooser::claim_array_chunk(uint chunk_size) { |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
177 |
uint res = (uint) Atomic::add((jint) chunk_size, |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
178 |
(volatile jint*) &_first_par_unreserved_idx); |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
179 |
assert(regions_length() > res + chunk_size - 1, |
1374 | 180 |
"Should already have been expanded"); |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
181 |
return res - chunk_size; |
1374 | 182 |
} |
183 |
||
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
184 |
void CollectionSetChooser::set_region(uint index, HeapRegion* hr) { |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
185 |
assert(regions_at(index) == NULL, "precondition"); |
1374 | 186 |
assert(!hr->is_young(), "should not be young!"); |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
187 |
regions_at_put(index, hr); |
1374 | 188 |
hr->calc_gc_efficiency(); |
189 |
} |
|
190 |
||
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
191 |
void CollectionSetChooser::update_totals(uint region_num, |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
192 |
size_t reclaimable_bytes) { |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
193 |
// Only take the lock if we actually need to update the totals. |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
194 |
if (region_num > 0) { |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
195 |
assert(reclaimable_bytes > 0, "invariant"); |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
196 |
// We could have just used atomics instead of taking the |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
197 |
// lock. However, we currently don't have an atomic add for size_t. |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
198 |
MutexLockerEx x(ParGCRareEvent_lock, Mutex::_no_safepoint_check_flag); |
32745 | 199 |
_end += region_num; |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
200 |
_remaining_reclaimable_bytes += reclaimable_bytes; |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
201 |
} else { |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
202 |
assert(reclaimable_bytes == 0, "invariant"); |
1374 | 203 |
} |
204 |
} |
|
205 |
||
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
206 |
void CollectionSetChooser::clear() { |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
207 |
_regions.clear(); |
32745 | 208 |
_front = 0; |
209 |
_end = 0; |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
210 |
_remaining_reclaimable_bytes = 0; |
37143
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
211 |
} |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
212 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
213 |
class ParKnownGarbageHRClosure: public HeapRegionClosure { |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
214 |
G1CollectedHeap* _g1h; |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
215 |
CSetChooserParUpdater _cset_updater; |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
216 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
217 |
public: |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
218 |
ParKnownGarbageHRClosure(CollectionSetChooser* hrSorted, |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
219 |
uint chunk_size) : |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
220 |
_g1h(G1CollectedHeap::heap()), |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
221 |
_cset_updater(hrSorted, true /* parallel */, chunk_size) { } |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
222 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
223 |
bool doHeapRegion(HeapRegion* r) { |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
224 |
// Do we have any marking information for this region? |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
225 |
if (r->is_marked()) { |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
226 |
// We will skip any region that's currently used as an old GC |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
227 |
// alloc region (we should not consider those for collection |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
228 |
// before we fill them up). |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
229 |
if (_cset_updater.should_add(r) && !_g1h->is_old_gc_alloc_region(r)) { |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
230 |
_cset_updater.add_region(r); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
231 |
} |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
232 |
} |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
233 |
return false; |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
234 |
} |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11396
diff
changeset
|
235 |
}; |
37143
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
236 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
237 |
class ParKnownGarbageTask: public AbstractGangTask { |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
238 |
CollectionSetChooser* _hrSorted; |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
239 |
uint _chunk_size; |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
240 |
G1CollectedHeap* _g1; |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
241 |
HeapRegionClaimer _hrclaimer; |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
242 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
243 |
public: |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
244 |
ParKnownGarbageTask(CollectionSetChooser* hrSorted, uint chunk_size, uint n_workers) : |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
245 |
AbstractGangTask("ParKnownGarbageTask"), |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
246 |
_hrSorted(hrSorted), _chunk_size(chunk_size), |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
247 |
_g1(G1CollectedHeap::heap()), _hrclaimer(n_workers) {} |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
248 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
249 |
void work(uint worker_id) { |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
250 |
ParKnownGarbageHRClosure parKnownGarbageCl(_hrSorted, _chunk_size); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
251 |
_g1->heap_region_par_iterate(&parKnownGarbageCl, worker_id, &_hrclaimer); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
252 |
} |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
253 |
}; |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
254 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
255 |
uint CollectionSetChooser::calculate_parallel_work_chunk_size(uint n_workers, uint n_regions) const { |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
256 |
assert(n_workers > 0, "Active gc workers should be greater than 0"); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
257 |
const uint overpartition_factor = 4; |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
258 |
const uint min_chunk_size = MAX2(n_regions / n_workers, 1U); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
259 |
return MAX2(n_regions / (n_workers * overpartition_factor), min_chunk_size); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
260 |
} |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
261 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
262 |
void CollectionSetChooser::rebuild(WorkGang* workers, uint n_regions) { |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
263 |
clear(); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
264 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
265 |
uint n_workers = workers->active_workers(); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
266 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
267 |
uint chunk_size = calculate_parallel_work_chunk_size(n_workers, n_regions); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
268 |
prepare_for_par_region_addition(n_workers, n_regions, chunk_size); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
269 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
270 |
ParKnownGarbageTask par_known_garbage_task(this, chunk_size, n_workers); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
271 |
workers->run_task(&par_known_garbage_task); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
272 |
|
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
273 |
sort_regions(); |
345ad6728be3
8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents:
35061
diff
changeset
|
274 |
} |