author | stefank |
Wed, 10 Apr 2013 14:26:49 +0200 | |
changeset 16685 | 41c34debcde0 |
parent 15861 | 974c3ccaa5d2 |
child 22234 | da823d78ad65 |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
2 |
* Copyright (c) 2001, 2012, 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 |
||
7397 | 25 |
#ifndef SHARE_VM_GC_IMPLEMENTATION_G1_COLLECTIONSETCHOOSER_HPP |
26 |
#define SHARE_VM_GC_IMPLEMENTATION_G1_COLLECTIONSETCHOOSER_HPP |
|
27 |
||
28 |
#include "gc_implementation/g1/heapRegion.hpp" |
|
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. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
51 |
uint _curr_index; |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
52 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
53 |
// The number of candidate old regions added to the CSet chooser. |
15861
974c3ccaa5d2
8007036: G1: Too many old regions added to last mixed GC
johnc
parents:
13336
diff
changeset
|
54 |
// Note: this is not updated when removing a region using |
974c3ccaa5d2
8007036: G1: Too many old regions added to last mixed GC
johnc
parents:
13336
diff
changeset
|
55 |
// remove_and_move_to_next() below. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
56 |
uint _length; |
1374 | 57 |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
58 |
// 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
|
59 |
// parallel GC workers. |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
60 |
uint _first_par_unreserved_idx; |
1374 | 61 |
|
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
62 |
// 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
|
63 |
// 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
|
64 |
// collection. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
65 |
size_t _region_live_threshold_bytes; |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
66 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
67 |
// 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
|
68 |
size_t _remaining_reclaimable_bytes; |
1374 | 69 |
|
70 |
public: |
|
71 |
||
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
72 |
// 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
|
73 |
// 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
|
74 |
HeapRegion* peek() { |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
75 |
HeapRegion* res = NULL; |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
76 |
if (_curr_index < _length) { |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
77 |
res = regions_at(_curr_index); |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
78 |
assert(res != NULL, |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
79 |
err_msg("Unexpected NULL hr in _regions at index %u", |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
80 |
_curr_index)); |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
81 |
} |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
82 |
return res; |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
83 |
} |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
84 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
85 |
// Remove the given region from the CSet chooser and move to the |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
86 |
// next one. The given region should be the current candidate region |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
87 |
// in the CSet chooser. |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
88 |
void remove_and_move_to_next(HeapRegion* hr) { |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
89 |
assert(hr != NULL, "pre-condition"); |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
90 |
assert(_curr_index < _length, "pre-condition"); |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
91 |
assert(regions_at(_curr_index) == hr, "pre-condition"); |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
92 |
regions_at_put(_curr_index, NULL); |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
93 |
assert(hr->reclaimable_bytes() <= _remaining_reclaimable_bytes, |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
94 |
err_msg("remaining reclaimable bytes inconsistent " |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
95 |
"from region: "SIZE_FORMAT" remaining: "SIZE_FORMAT, |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
96 |
hr->reclaimable_bytes(), _remaining_reclaimable_bytes)); |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
97 |
_remaining_reclaimable_bytes -= hr->reclaimable_bytes(); |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
98 |
_curr_index += 1; |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
99 |
} |
1374 | 100 |
|
101 |
CollectionSetChooser(); |
|
102 |
||
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
103 |
void sort_regions(); |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
104 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
105 |
// Determine whether to add the given region to the CSet chooser or |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
106 |
// not. Currently, we skip humongous regions (we never add them to |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
107 |
// the CSet, we only reclaim them during cleanup) and regions whose |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
108 |
// live bytes are over the threshold. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
109 |
bool should_add(HeapRegion* hr) { |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
110 |
assert(hr->is_marked(), "pre-condition"); |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
111 |
assert(!hr->is_young(), "should never consider young regions"); |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
112 |
return !hr->isHumongous() && |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
113 |
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
|
114 |
} |
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
115 |
|
15861
974c3ccaa5d2
8007036: G1: Too many old regions added to last mixed GC
johnc
parents:
13336
diff
changeset
|
116 |
// Returns the number candidate old regions added |
974c3ccaa5d2
8007036: G1: Too many old regions added to last mixed GC
johnc
parents:
13336
diff
changeset
|
117 |
uint length() { return _length; } |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
118 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
119 |
// Serial version. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
120 |
void add_region(HeapRegion *hr); |
1374 | 121 |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
122 |
// Must be called before calls to claim_array_chunk(). |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
123 |
// n_regions is the number of regions, chunk_size the chunk size. |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
124 |
void prepare_for_par_region_addition(uint n_regions, uint chunk_size); |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
125 |
// Returns the first index in a contiguous chunk of chunk_size indexes |
1374 | 126 |
// 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
|
127 |
// calling thread using set_region() (to NULL if necessary). |
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
128 |
uint claim_array_chunk(uint chunk_size); |
1374 | 129 |
// Set the marked array entry at index to hr. Careful to claim the index |
130 |
// first if in parallel. |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
131 |
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
|
132 |
// 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
|
133 |
// and the amount of reclaimable bytes by reclaimable_bytes. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
134 |
void update_totals(uint region_num, size_t reclaimable_bytes); |
1374 | 135 |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
136 |
void clear(); |
1374 | 137 |
|
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
138 |
// Return the number of candidate regions that remain to be collected. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
139 |
uint remaining_regions() { return _length - _curr_index; } |
1374 | 140 |
|
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
141 |
// 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
|
142 |
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
|
143 |
|
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
10680
diff
changeset
|
144 |
// 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
|
145 |
// all the candidate regions in the CSet chooser. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
146 |
size_t remaining_reclaimable_bytes() { return _remaining_reclaimable_bytes; } |
1374 | 147 |
|
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
148 |
// Returns true if the used portion of "_regions" is properly |
1374 | 149 |
// sorted, otherwise asserts false. |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
150 |
void verify() PRODUCT_RETURN; |
1374 | 151 |
}; |
7397 | 152 |
|
13336
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
153 |
class CSetChooserParUpdater : public StackObj { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
154 |
private: |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
155 |
CollectionSetChooser* _chooser; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
156 |
bool _parallel; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
157 |
uint _chunk_size; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
158 |
uint _cur_chunk_idx; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
159 |
uint _cur_chunk_end; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
160 |
uint _regions_added; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
161 |
size_t _reclaimable_bytes_added; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
162 |
|
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
163 |
public: |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
164 |
CSetChooserParUpdater(CollectionSetChooser* chooser, |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
165 |
bool parallel, uint chunk_size) : |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
166 |
_chooser(chooser), _parallel(parallel), _chunk_size(chunk_size), |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
167 |
_cur_chunk_idx(0), _cur_chunk_end(0), |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
168 |
_regions_added(0), _reclaimable_bytes_added(0) { } |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
169 |
|
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
170 |
~CSetChooserParUpdater() { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
171 |
if (_parallel && _regions_added > 0) { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
172 |
_chooser->update_totals(_regions_added, _reclaimable_bytes_added); |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
173 |
} |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
174 |
} |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
175 |
|
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
176 |
void add_region(HeapRegion* hr) { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
177 |
if (_parallel) { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
178 |
if (_cur_chunk_idx == _cur_chunk_end) { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
179 |
_cur_chunk_idx = _chooser->claim_array_chunk(_chunk_size); |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
180 |
_cur_chunk_end = _cur_chunk_idx + _chunk_size; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
181 |
} |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
182 |
assert(_cur_chunk_idx < _cur_chunk_end, "invariant"); |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
183 |
_chooser->set_region(_cur_chunk_idx, hr); |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
184 |
_cur_chunk_idx += 1; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
185 |
} else { |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
186 |
_chooser->add_region(hr); |
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 |
_regions_added += 1; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
189 |
_reclaimable_bytes_added += hr->reclaimable_bytes(); |
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 |
|
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
192 |
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
|
193 |
}; |
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
194 |
|
7397 | 195 |
#endif // SHARE_VM_GC_IMPLEMENTATION_G1_COLLECTIONSETCHOOSER_HPP |
13336
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
13195
diff
changeset
|
196 |