author | stefank |
Mon, 25 Nov 2019 12:31:39 +0100 | |
changeset 59249 | 29b0d0b61615 |
parent 53725 | 43854454a7db |
permissions | -rw-r--r-- |
53704
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
1 |
/* |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
2 |
* Copyright (c) 2001, 2019, Oracle and/or its affiliates. All rights reserved. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
4 |
* |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
8 |
* |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
13 |
* accompanied this code). |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
14 |
* |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
15 |
* You should have received a copy of the GNU General Public License version |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
18 |
* |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
21 |
* questions. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
22 |
* |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
23 |
*/ |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
24 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
25 |
#include "precompiled.hpp" |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
26 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
27 |
#include "gc/g1/g1CollectionSetCandidates.hpp" |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
28 |
#include "gc/g1/g1CollectionSetChooser.hpp" |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
29 |
#include "gc/g1/heapRegionRemSet.hpp" |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
30 |
#include "gc/shared/space.inline.hpp" |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
31 |
#include "runtime/atomic.hpp" |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
32 |
#include "utilities/quickSort.hpp" |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
33 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
34 |
// Order regions according to GC efficiency. This will cause regions with a lot |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
35 |
// of live objects and large remembered sets to end up at the end of the array. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
36 |
// Given that we might skip collecting the last few old regions, if after a few |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
37 |
// mixed GCs the remaining have reclaimable bytes under a certain threshold, the |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
38 |
// hope is that the ones we'll skip are ones with both large remembered sets and |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
39 |
// a lot of live objects, not the ones with just a lot of live objects if we |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
40 |
// ordered according to the amount of reclaimable bytes per region. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
41 |
static int order_regions(HeapRegion* hr1, HeapRegion* hr2) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
42 |
// Make sure that NULL entries are moved to the end. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
43 |
if (hr1 == NULL) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
44 |
if (hr2 == NULL) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
45 |
return 0; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
46 |
} else { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
47 |
return 1; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
48 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
49 |
} else if (hr2 == NULL) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
50 |
return -1; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
51 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
52 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
53 |
double gc_eff1 = hr1->gc_efficiency(); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
54 |
double gc_eff2 = hr2->gc_efficiency(); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
55 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
56 |
if (gc_eff1 > gc_eff2) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
57 |
return -1; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
58 |
} if (gc_eff1 < gc_eff2) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
59 |
return 1; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
60 |
} else { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
61 |
return 0; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
62 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
63 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
64 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
65 |
// Determine collection set candidates: For all regions determine whether they |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
66 |
// should be a collection set candidates, calculate their efficiency, sort and |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
67 |
// return them as G1CollectionSetCandidates instance. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
68 |
// Threads calculate the GC efficiency of the regions they get to process, and |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
69 |
// put them into some work area unsorted. At the end the array is sorted and |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
70 |
// copied into the G1CollectionSetCandidates instance; the caller will be the new |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
71 |
// owner of this object. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
72 |
class G1BuildCandidateRegionsTask : public AbstractGangTask { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
73 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
74 |
// Work area for building the set of collection set candidates. Contains references |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
75 |
// to heap regions with their GC efficiencies calculated. To reduce contention |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
76 |
// on claiming array elements, worker threads claim parts of this array in chunks; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
77 |
// Array elements may be NULL as threads might not get enough regions to fill |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
78 |
// up their chunks completely. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
79 |
// Final sorting will remove them. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
80 |
class G1BuildCandidateArray : public StackObj { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
81 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
82 |
uint const _max_size; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
83 |
uint const _chunk_size; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
84 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
85 |
HeapRegion** _data; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
86 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
87 |
uint volatile _cur_claim_idx; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
88 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
89 |
// Calculates the maximum array size that will be used. |
53725
43854454a7db
8218680: G1 crashes during calculation of old collection set candidates
tschatzl
parents:
53704
diff
changeset
|
90 |
static uint required_array_size(uint num_regions, uint chunk_size, uint num_workers) { |
53704
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
91 |
uint const max_waste = num_workers * chunk_size; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
92 |
// The array should be aligned with respect to chunk_size. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
93 |
uint const aligned_num_regions = ((num_regions + chunk_size - 1) / chunk_size) * chunk_size; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
94 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
95 |
return aligned_num_regions + max_waste; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
96 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
97 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
98 |
public: |
53725
43854454a7db
8218680: G1 crashes during calculation of old collection set candidates
tschatzl
parents:
53704
diff
changeset
|
99 |
G1BuildCandidateArray(uint max_num_regions, uint chunk_size, uint num_workers) : |
43854454a7db
8218680: G1 crashes during calculation of old collection set candidates
tschatzl
parents:
53704
diff
changeset
|
100 |
_max_size(required_array_size(max_num_regions, chunk_size, num_workers)), |
53704
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
101 |
_chunk_size(chunk_size), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
102 |
_data(NEW_C_HEAP_ARRAY(HeapRegion*, _max_size, mtGC)), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
103 |
_cur_claim_idx(0) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
104 |
for (uint i = 0; i < _max_size; i++) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
105 |
_data[i] = NULL; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
106 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
107 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
108 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
109 |
~G1BuildCandidateArray() { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
110 |
FREE_C_HEAP_ARRAY(HeapRegion*, _data); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
111 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
112 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
113 |
// Claim a new chunk, returning its bounds [from, to[. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
114 |
void claim_chunk(uint& from, uint& to) { |
59249
29b0d0b61615
8234737: Harmonize parameter order in Atomic - add
stefank
parents:
53725
diff
changeset
|
115 |
uint result = Atomic::add(&_cur_claim_idx, _chunk_size); |
53704
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
116 |
assert(_max_size > result - 1, |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
117 |
"Array too small, is %u should be %u with chunk size %u.", |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
118 |
_max_size, result, _chunk_size); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
119 |
from = result - _chunk_size; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
120 |
to = result; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
121 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
122 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
123 |
// Set element in array. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
124 |
void set(uint idx, HeapRegion* hr) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
125 |
assert(idx < _max_size, "Index %u out of bounds %u", idx, _max_size); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
126 |
assert(_data[idx] == NULL, "Value must not have been set."); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
127 |
_data[idx] = hr; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
128 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
129 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
130 |
void sort_and_copy_into(HeapRegion** dest, uint num_regions) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
131 |
if (_cur_claim_idx == 0) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
132 |
return; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
133 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
134 |
for (uint i = _cur_claim_idx; i < _max_size; i++) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
135 |
assert(_data[i] == NULL, "must be"); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
136 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
137 |
QuickSort::sort(_data, _cur_claim_idx, order_regions, true); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
138 |
for (uint i = num_regions; i < _max_size; i++) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
139 |
assert(_data[i] == NULL, "must be"); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
140 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
141 |
for (uint i = 0; i < num_regions; i++) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
142 |
dest[i] = _data[i]; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
143 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
144 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
145 |
}; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
146 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
147 |
// Per-region closure. In addition to determining whether a region should be |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
148 |
// added to the candidates, and calculating those regions' gc efficiencies, also |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
149 |
// gather additional statistics. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
150 |
class G1BuildCandidateRegionsClosure : public HeapRegionClosure { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
151 |
G1BuildCandidateArray* _array; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
152 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
153 |
uint _cur_chunk_idx; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
154 |
uint _cur_chunk_end; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
155 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
156 |
uint _regions_added; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
157 |
size_t _reclaimable_bytes_added; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
158 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
159 |
void add_region(HeapRegion* hr) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
160 |
if (_cur_chunk_idx == _cur_chunk_end) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
161 |
_array->claim_chunk(_cur_chunk_idx, _cur_chunk_end); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
162 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
163 |
assert(_cur_chunk_idx < _cur_chunk_end, "Must be"); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
164 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
165 |
hr->calc_gc_efficiency(); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
166 |
_array->set(_cur_chunk_idx, hr); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
167 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
168 |
_cur_chunk_idx++; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
169 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
170 |
_regions_added++; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
171 |
_reclaimable_bytes_added += hr->reclaimable_bytes(); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
172 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
173 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
174 |
bool should_add(HeapRegion* hr) { return G1CollectionSetChooser::should_add(hr); } |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
175 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
176 |
public: |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
177 |
G1BuildCandidateRegionsClosure(G1BuildCandidateArray* array) : |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
178 |
_array(array), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
179 |
_cur_chunk_idx(0), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
180 |
_cur_chunk_end(0), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
181 |
_regions_added(0), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
182 |
_reclaimable_bytes_added(0) { } |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
183 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
184 |
bool do_heap_region(HeapRegion* r) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
185 |
// We will skip any region that's currently used as an old GC |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
186 |
// alloc region (we should not consider those for collection |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
187 |
// before we fill them up). |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
188 |
if (should_add(r) && !G1CollectedHeap::heap()->is_old_gc_alloc_region(r)) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
189 |
add_region(r); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
190 |
} else if (r->is_old()) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
191 |
// Keep remembered sets for humongous regions, otherwise clean out remembered |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
192 |
// sets for old regions. |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
193 |
r->rem_set()->clear(true /* only_cardset */); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
194 |
} else { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
195 |
assert(r->is_archive() || !r->is_old() || !r->rem_set()->is_tracked(), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
196 |
"Missed to clear unused remembered set of region %u (%s) that is %s", |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
197 |
r->hrm_index(), r->get_type_str(), r->rem_set()->get_state_str()); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
198 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
199 |
return false; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
200 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
201 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
202 |
uint regions_added() const { return _regions_added; } |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
203 |
size_t reclaimable_bytes_added() const { return _reclaimable_bytes_added; } |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
204 |
}; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
205 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
206 |
G1CollectedHeap* _g1h; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
207 |
HeapRegionClaimer _hrclaimer; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
208 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
209 |
uint volatile _num_regions_added; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
210 |
size_t volatile _reclaimable_bytes_added; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
211 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
212 |
G1BuildCandidateArray _result; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
213 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
214 |
void update_totals(uint num_regions, size_t reclaimable_bytes) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
215 |
if (num_regions > 0) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
216 |
assert(reclaimable_bytes > 0, "invariant"); |
59249
29b0d0b61615
8234737: Harmonize parameter order in Atomic - add
stefank
parents:
53725
diff
changeset
|
217 |
Atomic::add(&_num_regions_added, num_regions); |
29b0d0b61615
8234737: Harmonize parameter order in Atomic - add
stefank
parents:
53725
diff
changeset
|
218 |
Atomic::add(&_reclaimable_bytes_added, reclaimable_bytes); |
53704
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
219 |
} else { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
220 |
assert(reclaimable_bytes == 0, "invariant"); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
221 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
222 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
223 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
224 |
public: |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
225 |
G1BuildCandidateRegionsTask(uint max_num_regions, uint chunk_size, uint num_workers) : |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
226 |
AbstractGangTask("G1 Build Candidate Regions"), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
227 |
_g1h(G1CollectedHeap::heap()), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
228 |
_hrclaimer(num_workers), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
229 |
_num_regions_added(0), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
230 |
_reclaimable_bytes_added(0), |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
231 |
_result(max_num_regions, chunk_size, num_workers) { } |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
232 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
233 |
void work(uint worker_id) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
234 |
G1BuildCandidateRegionsClosure cl(&_result); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
235 |
_g1h->heap_region_par_iterate_from_worker_offset(&cl, &_hrclaimer, worker_id); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
236 |
update_totals(cl.regions_added(), cl.reclaimable_bytes_added()); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
237 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
238 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
239 |
G1CollectionSetCandidates* get_sorted_candidates() { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
240 |
HeapRegion** regions = NEW_C_HEAP_ARRAY(HeapRegion*, _num_regions_added, mtGC); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
241 |
_result.sort_and_copy_into(regions, _num_regions_added); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
242 |
return new G1CollectionSetCandidates(regions, |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
243 |
_num_regions_added, |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
244 |
_reclaimable_bytes_added); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
245 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
246 |
}; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
247 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
248 |
uint G1CollectionSetChooser::calculate_work_chunk_size(uint num_workers, uint num_regions) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
249 |
assert(num_workers > 0, "Active gc workers should be greater than 0"); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
250 |
return MAX2(num_regions / num_workers, 1U); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
251 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
252 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
253 |
bool G1CollectionSetChooser::should_add(HeapRegion* hr) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
254 |
return !hr->is_young() && |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
255 |
!hr->is_pinned() && |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
256 |
region_occupancy_low_enough_for_evac(hr->live_bytes()) && |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
257 |
hr->rem_set()->is_complete(); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
258 |
} |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
259 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
260 |
G1CollectionSetCandidates* G1CollectionSetChooser::build(WorkGang* workers, uint max_num_regions) { |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
261 |
uint num_workers = workers->active_workers(); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
262 |
uint chunk_size = calculate_work_chunk_size(num_workers, max_num_regions); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
263 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
264 |
G1BuildCandidateRegionsTask cl(max_num_regions, chunk_size, num_workers); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
265 |
workers->run_task(&cl, num_workers); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
266 |
|
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
267 |
G1CollectionSetCandidates* result = cl.get_sorted_candidates(); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
268 |
result->verify(); |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
269 |
return result; |
ef72c85a0534
8217328: Rename CollectionSetChooser to G1CollectionSetChooser
tschatzl
parents:
diff
changeset
|
270 |
} |