author | tschatzl |
Wed, 25 Nov 2015 14:43:29 +0100 | |
changeset 34300 | 6075c1e0e913 |
parent 33105 | 294e48b4f704 |
child 37143 | 345ad6728be3 |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
30764 | 2 |
* Copyright (c) 2001, 2015, 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:
2105
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
2105
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:
2105
diff
changeset
|
21 |
* questions. |
1374 | 22 |
* |
23 |
*/ |
|
24 |
||
30764 | 25 |
#ifndef SHARE_VM_GC_G1_COLLECTIONSETCHOOSER_HPP |
26 |
#define SHARE_VM_GC_G1_COLLECTIONSETCHOOSER_HPP |
|
7397 | 27 |
|
30764 | 28 |
#include "gc/g1/heapRegion.hpp" |
7397 | 29 |
#include "utilities/growableArray.hpp" |
30 |
||
13195 | 31 |
class CollectionSetChooser: public CHeapObj<mtGC> { |
1374 | 32 |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
33 |
GrowableArray<HeapRegion*> _regions; |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
34 |
|
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
35 |
// Unfortunately, GrowableArray uses ints for length and indexes. To |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
36 |
// avoid excessive casting in the rest of the class the following |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
37 |
// wrapper methods are provided that use uints. |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
38 |
|
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
39 |
uint regions_length() { return (uint) _regions.length(); } |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
40 |
HeapRegion* regions_at(uint i) { return _regions.at((int) i); } |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
41 |
void regions_at_put(uint i, HeapRegion* hr) { |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
42 |
_regions.at_put((int) i, hr); |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
43 |
} |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
44 |
void regions_at_put_grow(uint i, HeapRegion* hr) { |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
45 |
_regions.at_put_grow((int) i, hr); |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
46 |
} |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
47 |
void regions_trunc_to(uint i) { _regions.trunc_to((uint) i); } |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
48 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
49 |
// The index of the next candidate old region to be considered for |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
50 |
// addition to the CSet. |
32745 | 51 |
uint _front; |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
52 |
|
32745 | 53 |
// The index of the last candidate old region |
54 |
uint _end; |
|
1374 | 55 |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
56 |
// Keeps track of the start of the next array chunk to be claimed by |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
57 |
// parallel GC workers. |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
58 |
uint _first_par_unreserved_idx; |
1374 | 59 |
|
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
60 |
// If a region has more live bytes than this threshold, it will not |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
61 |
// be added to the CSet chooser and will not be a candidate for |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
62 |
// collection. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
63 |
size_t _region_live_threshold_bytes; |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
64 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
65 |
// The sum of reclaimable bytes over all the regions in the CSet chooser. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
66 |
size_t _remaining_reclaimable_bytes; |
1374 | 67 |
|
68 |
public: |
|
69 |
||
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
70 |
// Return the current candidate region to be considered for |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
71 |
// collection without removing it from the CSet chooser. |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
72 |
HeapRegion* peek() { |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
73 |
HeapRegion* res = NULL; |
32745 | 74 |
if (_front < _end) { |
75 |
res = regions_at(_front); |
|
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
76 |
assert(res != NULL, "Unexpected NULL hr in _regions at index %u", _front); |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
77 |
} |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
78 |
return res; |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
79 |
} |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
80 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
81 |
// Remove the given region from the CSet chooser and move to the |
32745 | 82 |
// next one. |
83 |
HeapRegion* pop() { |
|
84 |
HeapRegion* hr = regions_at(_front); |
|
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
85 |
assert(hr != NULL, "pre-condition"); |
32745 | 86 |
assert(_front < _end, "pre-condition"); |
87 |
regions_at_put(_front, NULL); |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
88 |
assert(hr->reclaimable_bytes() <= _remaining_reclaimable_bytes, |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
89 |
"remaining reclaimable bytes inconsistent " |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
90 |
"from region: " SIZE_FORMAT " remaining: " SIZE_FORMAT, |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32745
diff
changeset
|
91 |
hr->reclaimable_bytes(), _remaining_reclaimable_bytes); |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
92 |
_remaining_reclaimable_bytes -= hr->reclaimable_bytes(); |
32745 | 93 |
_front += 1; |
94 |
return hr; |
|
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
95 |
} |
1374 | 96 |
|
32745 | 97 |
void push(HeapRegion* hr); |
98 |
||
1374 | 99 |
CollectionSetChooser(); |
100 |
||
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
101 |
void sort_regions(); |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
102 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
103 |
// Determine whether to add the given region to the CSet chooser or |
31346
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
30874
diff
changeset
|
104 |
// not. Currently, we skip pinned regions and regions whose live |
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
30874
diff
changeset
|
105 |
// bytes are over the threshold. Humongous regions may be reclaimed during cleanup. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
106 |
bool should_add(HeapRegion* hr) { |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
107 |
assert(hr->is_marked(), "pre-condition"); |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
108 |
assert(!hr->is_young(), "should never consider young regions"); |
31346
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
30874
diff
changeset
|
109 |
return !hr->is_pinned() && |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
110 |
hr->live_bytes() < _region_live_threshold_bytes; |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
111 |
} |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
112 |
|
15861
974c3ccaa5d2
8007036: G1: Too many old regions added to last mixed GC
johnc
parents:
13336
diff
changeset
|
113 |
// Returns the number candidate old regions added |
32745 | 114 |
uint length() { return _end; } |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
115 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
116 |
// Serial version. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
117 |
void add_region(HeapRegion *hr); |
1374 | 118 |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
119 |
// Must be called before calls to claim_array_chunk(). |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
120 |
// n_regions is the number of regions, chunk_size the chunk size. |
30874
18714bae50db
8080837: Move number of workers calculation out of CollectionSetChooser::prepare_for_par_region_addition
stefank
parents:
30764
diff
changeset
|
121 |
void prepare_for_par_region_addition(uint n_threads, uint n_regions, uint chunk_size); |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
122 |
// Returns the first index in a contiguous chunk of chunk_size indexes |
1374 | 123 |
// that the calling thread has reserved. These must be set by the |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
124 |
// calling thread using set_region() (to NULL if necessary). |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
125 |
uint claim_array_chunk(uint chunk_size); |
1374 | 126 |
// Set the marked array entry at index to hr. Careful to claim the index |
127 |
// first if in parallel. |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
128 |
void set_region(uint index, HeapRegion* hr); |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
129 |
// Atomically increment the number of added regions by region_num |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
130 |
// and the amount of reclaimable bytes by reclaimable_bytes. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
131 |
void update_totals(uint region_num, size_t reclaimable_bytes); |
1374 | 132 |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
133 |
void clear(); |
1374 | 134 |
|
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
135 |
// Return the number of candidate regions that remain to be collected. |
32745 | 136 |
uint remaining_regions() { return _end - _front; } |
1374 | 137 |
|
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
138 |
// Determine whether the CSet chooser has more candidate regions or not. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
139 |
bool is_empty() { return remaining_regions() == 0; } |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
140 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
141 |
// Return the reclaimable bytes that remain to be collected on |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
142 |
// all the candidate regions in the CSet chooser. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
143 |
size_t remaining_reclaimable_bytes() { return _remaining_reclaimable_bytes; } |
1374 | 144 |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
145 |
// Returns true if the used portion of "_regions" is properly |
1374 | 146 |
// sorted, otherwise asserts false. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
147 |
void verify() PRODUCT_RETURN; |
1374 | 148 |
}; |
7397 | 149 |
|
13336
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
150 |
class CSetChooserParUpdater : public StackObj { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
151 |
private: |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
152 |
CollectionSetChooser* _chooser; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
153 |
bool _parallel; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
154 |
uint _chunk_size; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
155 |
uint _cur_chunk_idx; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
156 |
uint _cur_chunk_end; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
157 |
uint _regions_added; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
158 |
size_t _reclaimable_bytes_added; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
159 |
|
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
160 |
public: |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
161 |
CSetChooserParUpdater(CollectionSetChooser* chooser, |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
162 |
bool parallel, uint chunk_size) : |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
163 |
_chooser(chooser), _parallel(parallel), _chunk_size(chunk_size), |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
164 |
_cur_chunk_idx(0), _cur_chunk_end(0), |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
165 |
_regions_added(0), _reclaimable_bytes_added(0) { } |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
166 |
|
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
167 |
~CSetChooserParUpdater() { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
168 |
if (_parallel && _regions_added > 0) { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
169 |
_chooser->update_totals(_regions_added, _reclaimable_bytes_added); |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
170 |
} |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
171 |
} |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
172 |
|
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
173 |
void add_region(HeapRegion* hr) { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
174 |
if (_parallel) { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
175 |
if (_cur_chunk_idx == _cur_chunk_end) { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
176 |
_cur_chunk_idx = _chooser->claim_array_chunk(_chunk_size); |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
177 |
_cur_chunk_end = _cur_chunk_idx + _chunk_size; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
178 |
} |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
179 |
assert(_cur_chunk_idx < _cur_chunk_end, "invariant"); |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
180 |
_chooser->set_region(_cur_chunk_idx, hr); |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
181 |
_cur_chunk_idx += 1; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
182 |
} else { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
183 |
_chooser->add_region(hr); |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
184 |
} |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
185 |
_regions_added += 1; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
186 |
_reclaimable_bytes_added += hr->reclaimable_bytes(); |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
187 |
} |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
188 |
|
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
189 |
bool should_add(HeapRegion* hr) { return _chooser->should_add(hr); } |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
190 |
}; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
191 |
|
30764 | 192 |
#endif // SHARE_VM_GC_G1_COLLECTIONSETCHOOSER_HPP |
13336
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
193 |