author | tschatzl |
Thu, 06 Aug 2015 15:49:50 +0200 | |
changeset 32185 | 49a57ff2c3cb |
parent 31592 | 43f48e165466 |
child 32606 | fdaa30d06ada |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
28833
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
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:
5545
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5545
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:
5545
diff
changeset
|
21 |
* questions. |
1374 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
19339 | 26 |
#include "code/nmethod.hpp" |
30764 | 27 |
#include "gc/g1/g1BlockOffsetTable.inline.hpp" |
28 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
|
29 |
#include "gc/g1/g1OopClosures.inline.hpp" |
|
30 |
#include "gc/g1/heapRegion.inline.hpp" |
|
31 |
#include "gc/g1/heapRegionBounds.inline.hpp" |
|
32 |
#include "gc/g1/heapRegionManager.inline.hpp" |
|
33 |
#include "gc/g1/heapRegionRemSet.hpp" |
|
34 |
#include "gc/shared/genOopClosures.inline.hpp" |
|
35 |
#include "gc/shared/liveRange.hpp" |
|
36 |
#include "gc/shared/space.inline.hpp" |
|
7397 | 37 |
#include "memory/iterator.hpp" |
38 |
#include "oops/oop.inline.hpp" |
|
25351
7c198a690050
8044775: Improve usage of umbrella header atomic.inline.hpp.
goetz
parents:
24487
diff
changeset
|
39 |
#include "runtime/atomic.inline.hpp" |
24351
61b33cc6d3cf
8042195: Introduce umbrella header orderAccess.inline.hpp.
goetz
parents:
23538
diff
changeset
|
40 |
#include "runtime/orderAccess.inline.hpp" |
1374 | 41 |
|
10677
370a8da2d63f
7095194: G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
johnc
parents:
10674
diff
changeset
|
42 |
int HeapRegion::LogOfHRGrainBytes = 0; |
370a8da2d63f
7095194: G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
johnc
parents:
10674
diff
changeset
|
43 |
int HeapRegion::LogOfHRGrainWords = 0; |
370a8da2d63f
7095194: G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
johnc
parents:
10674
diff
changeset
|
44 |
size_t HeapRegion::GrainBytes = 0; |
370a8da2d63f
7095194: G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
johnc
parents:
10674
diff
changeset
|
45 |
size_t HeapRegion::GrainWords = 0; |
370a8da2d63f
7095194: G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
johnc
parents:
10674
diff
changeset
|
46 |
size_t HeapRegion::CardsPerRegion = 0; |
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
47 |
|
1374 | 48 |
HeapRegionDCTOC::HeapRegionDCTOC(G1CollectedHeap* g1, |
28207
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
49 |
HeapRegion* hr, |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
50 |
G1ParPushHeapRSClosure* cl, |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
51 |
CardTableModRefBS::PrecisionStyle precision) : |
25360
5b6141bf7564
8047819: G1 HeapRegionDCTOC does not need to inherit ContiguousSpaceDCTOC
mgerdin
parents:
25351
diff
changeset
|
52 |
DirtyCardToOopClosure(hr, cl, precision, NULL), |
28207
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
53 |
_hr(hr), _rs_scan(cl), _g1(g1) { } |
1374 | 54 |
|
55 |
FilterOutOfRegionClosure::FilterOutOfRegionClosure(HeapRegion* r, |
|
56 |
OopClosure* oc) : |
|
13336
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
12508
diff
changeset
|
57 |
_r_bottom(r->bottom()), _r_end(r->end()), _oc(oc) { } |
1374 | 58 |
|
25360
5b6141bf7564
8047819: G1 HeapRegionDCTOC does not need to inherit ContiguousSpaceDCTOC
mgerdin
parents:
25351
diff
changeset
|
59 |
void HeapRegionDCTOC::walk_mem_region(MemRegion mr, |
5b6141bf7564
8047819: G1 HeapRegionDCTOC does not need to inherit ContiguousSpaceDCTOC
mgerdin
parents:
25351
diff
changeset
|
60 |
HeapWord* bottom, |
5b6141bf7564
8047819: G1 HeapRegionDCTOC does not need to inherit ContiguousSpaceDCTOC
mgerdin
parents:
25351
diff
changeset
|
61 |
HeapWord* top) { |
1374 | 62 |
G1CollectedHeap* g1h = _g1; |
25481
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
63 |
size_t oop_size; |
28207
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
64 |
HeapWord* cur = bottom; |
1374 | 65 |
|
66 |
// Start filtering what we add to the remembered set. If the object is |
|
67 |
// not considered dead, either because it is marked (in the mark bitmap) |
|
68 |
// or it was allocated after marking finished, then we add it. Otherwise |
|
69 |
// we can safely ignore the object. |
|
28207
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
70 |
if (!g1h->is_obj_dead(oop(cur), _hr)) { |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
71 |
oop_size = oop(cur)->oop_iterate(_rs_scan, mr); |
1374 | 72 |
} else { |
28207
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
73 |
oop_size = _hr->block_size(cur); |
1374 | 74 |
} |
75 |
||
28207
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
76 |
cur += oop_size; |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10669
diff
changeset
|
77 |
|
28207
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
78 |
if (cur < top) { |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
79 |
oop cur_oop = oop(cur); |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
80 |
oop_size = _hr->block_size(cur); |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
81 |
HeapWord* next_obj = cur + oop_size; |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
82 |
while (next_obj < top) { |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
83 |
// Keep filtering the remembered set. |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
84 |
if (!g1h->is_obj_dead(cur_oop, _hr)) { |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
85 |
// Bottom lies entirely below top, so we can call the |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
86 |
// non-memRegion version of oop_iterate below. |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
87 |
cur_oop->oop_iterate(_rs_scan); |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
88 |
} |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
89 |
cur = next_obj; |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
90 |
cur_oop = oop(cur); |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
91 |
oop_size = _hr->block_size(cur); |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
92 |
next_obj = cur + oop_size; |
1374 | 93 |
} |
94 |
||
95 |
// Last object. Need to do dead-obj filtering here too. |
|
28207
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
96 |
if (!g1h->is_obj_dead(oop(cur), _hr)) { |
6ad23566cbef
8067655: Clean up G1 remembered set oop iteration
mgerdin
parents:
27889
diff
changeset
|
97 |
oop(cur)->oop_iterate(_rs_scan, mr); |
1374 | 98 |
} |
99 |
} |
|
100 |
} |
|
101 |
||
19986
33d188c66ed9
8010722: assert: failed: heap size is too big for compressed oops
tschatzl
parents:
19728
diff
changeset
|
102 |
size_t HeapRegion::max_region_size() { |
26840 | 103 |
return HeapRegionBounds::max_size(); |
19986
33d188c66ed9
8010722: assert: failed: heap size is too big for compressed oops
tschatzl
parents:
19728
diff
changeset
|
104 |
} |
33d188c66ed9
8010722: assert: failed: heap size is too big for compressed oops
tschatzl
parents:
19728
diff
changeset
|
105 |
|
31346
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
31331
diff
changeset
|
106 |
size_t HeapRegion::min_region_size_in_words() { |
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
31331
diff
changeset
|
107 |
return HeapRegionBounds::min_size() >> LogHeapWordSize; |
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
31331
diff
changeset
|
108 |
} |
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
31331
diff
changeset
|
109 |
|
19728
9e1556506d2d
8019902: G1: Use the average heap size rather than the minimum heap size to calculate the region size
brutisso
parents:
19339
diff
changeset
|
110 |
void HeapRegion::setup_heap_region_size(size_t initial_heap_size, size_t max_heap_size) { |
29697
92501504191b
8074459: Flags handling memory sizes should be of type size_t
jwilhelm
parents:
29681
diff
changeset
|
111 |
size_t region_size = G1HeapRegionSize; |
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
112 |
if (FLAG_IS_DEFAULT(G1HeapRegionSize)) { |
19728
9e1556506d2d
8019902: G1: Use the average heap size rather than the minimum heap size to calculate the region size
brutisso
parents:
19339
diff
changeset
|
113 |
size_t average_heap_size = (initial_heap_size + max_heap_size) / 2; |
26840 | 114 |
region_size = MAX2(average_heap_size / HeapRegionBounds::target_number(), |
29697
92501504191b
8074459: Flags handling memory sizes should be of type size_t
jwilhelm
parents:
29681
diff
changeset
|
115 |
HeapRegionBounds::min_size()); |
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
116 |
} |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
117 |
|
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
118 |
int region_size_log = log2_long((jlong) region_size); |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
119 |
// Recalculate the region size to make sure it's a power of |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
120 |
// 2. This means that region_size is the largest power of 2 that's |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
121 |
// <= what we've calculated so far. |
29697
92501504191b
8074459: Flags handling memory sizes should be of type size_t
jwilhelm
parents:
29681
diff
changeset
|
122 |
region_size = ((size_t)1 << region_size_log); |
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
123 |
|
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
124 |
// Now make sure that we don't go over or under our limits. |
26840 | 125 |
if (region_size < HeapRegionBounds::min_size()) { |
126 |
region_size = HeapRegionBounds::min_size(); |
|
127 |
} else if (region_size > HeapRegionBounds::max_size()) { |
|
128 |
region_size = HeapRegionBounds::max_size(); |
|
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
129 |
} |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
130 |
|
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
131 |
// And recalculate the log. |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
132 |
region_size_log = log2_long((jlong) region_size); |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
133 |
|
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
134 |
// Now, set up the globals. |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
135 |
guarantee(LogOfHRGrainBytes == 0, "we should only set it once"); |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
136 |
LogOfHRGrainBytes = region_size_log; |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
137 |
|
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
138 |
guarantee(LogOfHRGrainWords == 0, "we should only set it once"); |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
139 |
LogOfHRGrainWords = LogOfHRGrainBytes - LogHeapWordSize; |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
140 |
|
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
141 |
guarantee(GrainBytes == 0, "we should only set it once"); |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
142 |
// The cast to int is safe, given that we've bounded region_size by |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
143 |
// MIN_REGION_SIZE and MAX_REGION_SIZE. |
29697
92501504191b
8074459: Flags handling memory sizes should be of type size_t
jwilhelm
parents:
29681
diff
changeset
|
144 |
GrainBytes = region_size; |
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
145 |
|
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
146 |
guarantee(GrainWords == 0, "we should only set it once"); |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
147 |
GrainWords = GrainBytes >> LogHeapWordSize; |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
12379
diff
changeset
|
148 |
guarantee((size_t) 1 << LogOfHRGrainWords == GrainWords, "sanity"); |
3697
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
149 |
|
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
150 |
guarantee(CardsPerRegion == 0, "we should only set it once"); |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
151 |
CardsPerRegion = GrainBytes >> CardTableModRefBS::card_shift; |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
152 |
} |
ea9211aa02f5
6819085: G1: use larger and/or user settable region size
tonyp
parents:
3283
diff
changeset
|
153 |
|
1374 | 154 |
void HeapRegion::reset_after_compaction() { |
155 |
G1OffsetTableContigSpace::reset_after_compaction(); |
|
156 |
// After a compaction the mark bitmap is invalid, so we must |
|
157 |
// treat all objects as being inside the unmarked area. |
|
158 |
zero_marked_bytes(); |
|
159 |
init_top_at_mark_start(); |
|
160 |
} |
|
161 |
||
23453
09cfb0164acf
8027295: Free CSet takes ~50% of young pause time
tschatzl
parents:
23451
diff
changeset
|
162 |
void HeapRegion::hr_clear(bool par, bool clear_space, bool locked) { |
7923 | 163 |
assert(_humongous_start_region == NULL, |
164 |
"we should have already filtered out humongous regions"); |
|
26827 | 165 |
assert(_end == orig_end(), |
7923 | 166 |
"we should have already filtered out humongous regions"); |
29470
e34bbcd36e53
8030646: track collection set membership in one place
ehelin
parents:
29325
diff
changeset
|
167 |
assert(!in_collection_set(), |
28633
f346493c0f7d
8068971: A heap region being cleared should not belong to the cset
ehelin
parents:
28207
diff
changeset
|
168 |
err_msg("Should not clear heap region %u in the collection set", hrm_index())); |
1374 | 169 |
|
26837
72a43d3841e7
8057536: Refactor G1 to allow context specific allocations
sjohanss
parents:
26827
diff
changeset
|
170 |
set_allocation_context(AllocationContext::system()); |
1374 | 171 |
set_young_index_in_cset(-1); |
172 |
uninstall_surv_rate_group(); |
|
26696
623a25e6c686
8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents:
26571
diff
changeset
|
173 |
set_free(); |
8928
e5c53268bef5
7023069: G1: Introduce symmetric locking in the slow allocation path
tonyp
parents:
7923
diff
changeset
|
174 |
reset_pre_dummy_top(); |
1374 | 175 |
|
176 |
if (!par) { |
|
177 |
// If this is parallel, this will be done later. |
|
178 |
HeapRegionRemSet* hrrs = rem_set(); |
|
23453
09cfb0164acf
8027295: Free CSet takes ~50% of young pause time
tschatzl
parents:
23451
diff
changeset
|
179 |
if (locked) { |
09cfb0164acf
8027295: Free CSet takes ~50% of young pause time
tschatzl
parents:
23451
diff
changeset
|
180 |
hrrs->clear_locked(); |
09cfb0164acf
8027295: Free CSet takes ~50% of young pause time
tschatzl
parents:
23451
diff
changeset
|
181 |
} else { |
09cfb0164acf
8027295: Free CSet takes ~50% of young pause time
tschatzl
parents:
23451
diff
changeset
|
182 |
hrrs->clear(); |
09cfb0164acf
8027295: Free CSet takes ~50% of young pause time
tschatzl
parents:
23451
diff
changeset
|
183 |
} |
1374 | 184 |
} |
185 |
zero_marked_bytes(); |
|
186 |
||
187 |
_offsets.resize(HeapRegion::GrainWords); |
|
188 |
init_top_at_mark_start(); |
|
1388 | 189 |
if (clear_space) clear(SpaceDecorator::Mangle); |
1374 | 190 |
} |
191 |
||
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
192 |
void HeapRegion::par_clear() { |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
193 |
assert(used() == 0, "the region should have been already cleared"); |
10677
370a8da2d63f
7095194: G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
johnc
parents:
10674
diff
changeset
|
194 |
assert(capacity() == HeapRegion::GrainBytes, "should be back to normal"); |
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
195 |
HeapRegionRemSet* hrrs = rem_set(); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
196 |
hrrs->clear(); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
197 |
CardTableModRefBS* ct_bs = |
29325 | 198 |
barrier_set_cast<CardTableModRefBS>(G1CollectedHeap::heap()->barrier_set()); |
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
199 |
ct_bs->clear(MemRegion(bottom(), end())); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
200 |
} |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
201 |
|
1374 | 202 |
void HeapRegion::calc_gc_efficiency() { |
13482 | 203 |
// GC efficiency is the ratio of how much space would be |
204 |
// reclaimed over how long we predict it would take to reclaim it. |
|
1374 | 205 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
11756
28b6fe22e43d
7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents:
11586
diff
changeset
|
206 |
G1CollectorPolicy* g1p = g1h->g1_policy(); |
13482 | 207 |
|
208 |
// Retrieve a prediction of the elapsed time for this region for |
|
209 |
// a mixed gc because the region will only be evacuated during a |
|
210 |
// mixed gc. |
|
211 |
double region_elapsed_time_ms = |
|
212 |
g1p->predict_region_elapsed_time_ms(this, false /* for_young_gc */); |
|
213 |
_gc_efficiency = (double) reclaimable_bytes() / region_elapsed_time_ms; |
|
1374 | 214 |
} |
215 |
||
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
216 |
void HeapRegion::set_starts_humongous(HeapWord* new_top, HeapWord* new_end) { |
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
217 |
assert(!is_humongous(), "sanity / pre-condition"); |
26827 | 218 |
assert(end() == orig_end(), |
6983
a8c50cedbce9
6991377: G1: race between concurrent refinement and humongous object allocation
tonyp
parents:
6260
diff
changeset
|
219 |
"Should be normal before the humongous object allocation"); |
a8c50cedbce9
6991377: G1: race between concurrent refinement and humongous object allocation
tonyp
parents:
6260
diff
changeset
|
220 |
assert(top() == bottom(), "should be empty"); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
221 |
assert(bottom() <= new_top && new_top <= new_end, "pre-condition"); |
6983
a8c50cedbce9
6991377: G1: race between concurrent refinement and humongous object allocation
tonyp
parents:
6260
diff
changeset
|
222 |
|
26696
623a25e6c686
8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents:
26571
diff
changeset
|
223 |
_type.set_starts_humongous(); |
1374 | 224 |
_humongous_start_region = this; |
6983
a8c50cedbce9
6991377: G1: race between concurrent refinement and humongous object allocation
tonyp
parents:
6260
diff
changeset
|
225 |
|
a8c50cedbce9
6991377: G1: race between concurrent refinement and humongous object allocation
tonyp
parents:
6260
diff
changeset
|
226 |
set_end(new_end); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
227 |
_offsets.set_for_starts_humongous(new_top); |
6983
a8c50cedbce9
6991377: G1: race between concurrent refinement and humongous object allocation
tonyp
parents:
6260
diff
changeset
|
228 |
} |
a8c50cedbce9
6991377: G1: race between concurrent refinement and humongous object allocation
tonyp
parents:
6260
diff
changeset
|
229 |
|
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
230 |
void HeapRegion::set_continues_humongous(HeapRegion* first_hr) { |
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
231 |
assert(!is_humongous(), "sanity / pre-condition"); |
26827 | 232 |
assert(end() == orig_end(), |
6983
a8c50cedbce9
6991377: G1: race between concurrent refinement and humongous object allocation
tonyp
parents:
6260
diff
changeset
|
233 |
"Should be normal before the humongous object allocation"); |
a8c50cedbce9
6991377: G1: race between concurrent refinement and humongous object allocation
tonyp
parents:
6260
diff
changeset
|
234 |
assert(top() == bottom(), "should be empty"); |
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
235 |
assert(first_hr->is_starts_humongous(), "pre-condition"); |
6983
a8c50cedbce9
6991377: G1: race between concurrent refinement and humongous object allocation
tonyp
parents:
6260
diff
changeset
|
236 |
|
26696
623a25e6c686
8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents:
26571
diff
changeset
|
237 |
_type.set_continues_humongous(); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
238 |
_humongous_start_region = first_hr; |
1374 | 239 |
} |
240 |
||
26696
623a25e6c686
8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents:
26571
diff
changeset
|
241 |
void HeapRegion::clear_humongous() { |
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
242 |
assert(is_humongous(), "pre-condition"); |
7923 | 243 |
|
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
244 |
if (is_starts_humongous()) { |
7923 | 245 |
assert(top() <= end(), "pre-condition"); |
26827 | 246 |
set_end(orig_end()); |
7923 | 247 |
if (top() > end()) { |
248 |
// at least one "continues humongous" region after it |
|
249 |
set_top(end()); |
|
250 |
} |
|
251 |
} else { |
|
252 |
// continues humongous |
|
26827 | 253 |
assert(end() == orig_end(), "sanity"); |
7923 | 254 |
} |
255 |
||
10677
370a8da2d63f
7095194: G1: HeapRegion::GrainBytes, GrainWords, and CardsPerRegion should be size_t
johnc
parents:
10674
diff
changeset
|
256 |
assert(capacity() == HeapRegion::GrainBytes, "pre-condition"); |
7923 | 257 |
_humongous_start_region = NULL; |
258 |
} |
|
259 |
||
26316
93f6b40c038b
8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents:
26157
diff
changeset
|
260 |
HeapRegion::HeapRegion(uint hrm_index, |
12381
1438e0fbfa27
7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents:
12379
diff
changeset
|
261 |
G1BlockOffsetSharedArray* sharedOffsetArray, |
26839 | 262 |
MemRegion mr) : |
13756
3b72f3da8cd4
7016955: G1: remove the is_zeroed parameter from the HeapRegion constructor
johnc
parents:
13728
diff
changeset
|
263 |
G1OffsetTableContigSpace(sharedOffsetArray, mr), |
26839 | 264 |
_hrm_index(hrm_index), |
265 |
_allocation_context(AllocationContext::system()), |
|
26696
623a25e6c686
8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents:
26571
diff
changeset
|
266 |
_humongous_start_region(NULL), |
26827 | 267 |
_next_in_special_set(NULL), |
27009
e7e723732b6b
8058298: Separate heap region iterator claim values from the data structures iterated over
mlarsson
parents:
26846
diff
changeset
|
268 |
_evacuation_failed(false), |
12382
6aaecb1cbfe1
7145441: G1: collection set chooser-related cleanup
tonyp
parents:
12381
diff
changeset
|
269 |
_prev_marked_bytes(0), _next_marked_bytes(0), _gc_efficiency(0.0), |
26696
623a25e6c686
8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents:
26571
diff
changeset
|
270 |
_next_young_region(NULL), |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
271 |
_next_dirty_cards_region(NULL), _next(NULL), _prev(NULL), |
7923 | 272 |
#ifdef ASSERT |
273 |
_containing_set(NULL), |
|
274 |
#endif // ASSERT |
|
275 |
_young_index_in_cset(-1), _surv_rate_group(NULL), _age_index(-1), |
|
276 |
_rem_set(NULL), _recorded_rs_length(0), _predicted_elapsed_time_ms(0), |
|
5350
cccf0925702e
6819061: G1: eliminate serial Other times that are proportional to the collection set length
johnc
parents:
5344
diff
changeset
|
277 |
_predicted_bytes_to_copy(0) |
1374 | 278 |
{ |
19339 | 279 |
_rem_set = new HeapRegionRemSet(sharedOffsetArray, this); |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
280 |
assert(HeapRegionRemSet::num_par_rem_sets() > 0, "Invariant."); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
281 |
|
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
282 |
initialize(mr); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
283 |
} |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
284 |
|
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
285 |
void HeapRegion::initialize(MemRegion mr, bool clear_space, bool mangle_space) { |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
286 |
assert(_rem_set->is_empty(), "Remembered set must be empty"); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
287 |
|
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
288 |
G1OffsetTableContigSpace::initialize(mr, clear_space, mangle_space); |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
289 |
|
13756
3b72f3da8cd4
7016955: G1: remove the is_zeroed parameter from the HeapRegion constructor
johnc
parents:
13728
diff
changeset
|
290 |
hr_clear(false /*par*/, false /*clear_space*/); |
1388 | 291 |
set_top(bottom()); |
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
292 |
record_timestamp(); |
26827 | 293 |
|
294 |
assert(mr.end() == orig_end(), |
|
295 |
err_msg("Given region end address " PTR_FORMAT " should match exactly " |
|
296 |
"bottom plus one region size, i.e. " PTR_FORMAT, |
|
297 |
p2i(mr.end()), p2i(orig_end()))); |
|
1374 | 298 |
} |
299 |
||
13336
e582172ff6ff
7114678: G1: various small fixes, code cleanup, and refactoring
tonyp
parents:
12508
diff
changeset
|
300 |
CompactibleSpace* HeapRegion::next_compaction_space() const { |
25730
7eb4e685f739
8048112: G1 Full GC needs to support the case when the very first region is not available
tschatzl
parents:
25492
diff
changeset
|
301 |
return G1CollectedHeap::heap()->next_compaction_region(this); |
1374 | 302 |
} |
303 |
||
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
304 |
void HeapRegion::note_self_forwarding_removal_start(bool during_initial_mark, |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
305 |
bool during_conc_mark) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
306 |
// We always recreate the prev marking info and we'll explicitly |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
307 |
// mark all objects we find to be self-forwarded on the prev |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
308 |
// bitmap. So all objects need to be below PTAMS. |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
309 |
_prev_marked_bytes = 0; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
310 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
311 |
if (during_initial_mark) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
312 |
// During initial-mark, we'll also explicitly mark all objects |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
313 |
// we find to be self-forwarded on the next bitmap. So all |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
314 |
// objects need to be below NTAMS. |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
315 |
_next_top_at_mark_start = top(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
316 |
_next_marked_bytes = 0; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
317 |
} else if (during_conc_mark) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
318 |
// During concurrent mark, all objects in the CSet (including |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
319 |
// the ones we find to be self-forwarded) are implicitly live. |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
320 |
// So all objects need to be above NTAMS. |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
321 |
_next_top_at_mark_start = bottom(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
322 |
_next_marked_bytes = 0; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
323 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
324 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
325 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
326 |
void HeapRegion::note_self_forwarding_removal_end(bool during_initial_mark, |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
327 |
bool during_conc_mark, |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
328 |
size_t marked_bytes) { |
29580
a67a581cfe11
8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents:
29470
diff
changeset
|
329 |
assert(marked_bytes <= used(), |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
330 |
err_msg("marked: " SIZE_FORMAT " used: " SIZE_FORMAT, marked_bytes, used())); |
25492
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
331 |
_prev_top_at_mark_start = top(); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
332 |
_prev_marked_bytes = marked_bytes; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
333 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
11393
diff
changeset
|
334 |
|
1374 | 335 |
HeapWord* |
336 |
HeapRegion::object_iterate_mem_careful(MemRegion mr, |
|
337 |
ObjectClosure* cl) { |
|
338 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
|
339 |
// We used to use "block_start_careful" here. But we're actually happy |
|
340 |
// to update the BOT while we do this... |
|
341 |
HeapWord* cur = block_start(mr.start()); |
|
342 |
mr = mr.intersection(used_region()); |
|
343 |
if (mr.is_empty()) return NULL; |
|
344 |
// Otherwise, find the obj that extends onto mr.start(). |
|
345 |
||
346 |
assert(cur <= mr.start() |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
3191
diff
changeset
|
347 |
&& (oop(cur)->klass_or_null() == NULL || |
1374 | 348 |
cur + oop(cur)->size() > mr.start()), |
349 |
"postcondition of block_start"); |
|
350 |
oop obj; |
|
351 |
while (cur < mr.end()) { |
|
352 |
obj = oop(cur); |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
3191
diff
changeset
|
353 |
if (obj->klass_or_null() == NULL) { |
1374 | 354 |
// Ran into an unparseable point. |
355 |
return cur; |
|
356 |
} else if (!g1h->is_obj_dead(obj)) { |
|
357 |
cl->do_object(obj); |
|
358 |
} |
|
25481
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
359 |
cur += block_size(cur); |
1374 | 360 |
} |
361 |
return NULL; |
|
362 |
} |
|
363 |
||
364 |
HeapWord* |
|
365 |
HeapRegion:: |
|
366 |
oops_on_card_seq_iterate_careful(MemRegion mr, |
|
6068
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
367 |
FilterOutOfRegionClosure* cl, |
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
368 |
bool filter_young, |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
369 |
jbyte* card_ptr) { |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
370 |
// Currently, we should only have to clean the card if filter_young |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
371 |
// is true and vice versa. |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
372 |
if (filter_young) { |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
373 |
assert(card_ptr != NULL, "pre-condition"); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
374 |
} else { |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
375 |
assert(card_ptr == NULL, "pre-condition"); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
376 |
} |
1374 | 377 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
378 |
||
379 |
// If we're within a stop-world GC, then we might look at a card in a |
|
380 |
// GC alloc region that extends onto a GC LAB, which may not be |
|
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
381 |
// parseable. Stop such at the "scan_top" of the region. |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
382 |
if (g1h->is_gc_active()) { |
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
383 |
mr = mr.intersection(MemRegion(bottom(), scan_top())); |
1374 | 384 |
} else { |
385 |
mr = mr.intersection(used_region()); |
|
386 |
} |
|
387 |
if (mr.is_empty()) return NULL; |
|
388 |
// Otherwise, find the obj that extends onto mr.start(). |
|
389 |
||
6068
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
390 |
// The intersection of the incoming mr (for the card) and the |
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
391 |
// allocated part of the region is non-empty. This implies that |
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
392 |
// we have actually allocated into this region. The code in |
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
393 |
// G1CollectedHeap.cpp that allocates a new region sets the |
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
394 |
// is_young tag on the region before allocating. Thus we |
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
395 |
// safely know if this region is young. |
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
396 |
if (is_young() && filter_young) { |
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
397 |
return NULL; |
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
398 |
} |
80ef41e75a2d
6956639: G1: assert(cached_ptr != card_ptr) failed: shouldn't be, concurrentG1Refine.cpp:307
johnc
parents:
5547
diff
changeset
|
399 |
|
6247 | 400 |
assert(!is_young(), "check value of filter_young"); |
401 |
||
9418
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
402 |
// We can only clean the card here, after we make the decision that |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
403 |
// the card is not young. And we only clean the card if we have been |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
404 |
// asked to (i.e., card_ptr != NULL). |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
405 |
if (card_ptr != NULL) { |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
406 |
*card_ptr = CardTableModRefBS::clean_card_val(); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
407 |
// We must complete this write before we do any of the reads below. |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
408 |
OrderAccess::storeload(); |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
409 |
} |
32a87dd6b746
7035144: G1: nightly failure: Non-dirty cards in region that should be dirty (failures still exist...)
tonyp
parents:
8928
diff
changeset
|
410 |
|
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
411 |
// Cache the boundaries of the memory region in some const locals |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
412 |
HeapWord* const start = mr.start(); |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
413 |
HeapWord* const end = mr.end(); |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
414 |
|
1374 | 415 |
// We used to use "block_start_careful" here. But we're actually happy |
416 |
// to update the BOT while we do this... |
|
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
417 |
HeapWord* cur = block_start(start); |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
418 |
assert(cur <= start, "Postcondition"); |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
419 |
|
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
420 |
oop obj; |
1374 | 421 |
|
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
422 |
HeapWord* next = cur; |
28833
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
423 |
do { |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
424 |
cur = next; |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
425 |
obj = oop(cur); |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
426 |
if (obj->klass_or_null() == NULL) { |
1374 | 427 |
// Ran into an unparseable point. |
428 |
return cur; |
|
429 |
} |
|
430 |
// Otherwise... |
|
25481
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
431 |
next = cur + block_size(cur); |
28833
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
432 |
} while (next <= start); |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
433 |
|
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
434 |
// If we finish the above loop...We have a parseable object that |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
435 |
// begins on or before the start of the memory region, and ends |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
436 |
// inside or spans the entire region. |
25481
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
437 |
assert(cur <= start, "Loop postcondition"); |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
438 |
assert(obj->klass_or_null() != NULL, "Loop postcondition"); |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
439 |
|
28833
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
440 |
do { |
1374 | 441 |
obj = oop(cur); |
28833
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
442 |
assert((cur + block_size(cur)) > (HeapWord*)obj, "Loop invariant"); |
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
3191
diff
changeset
|
443 |
if (obj->klass_or_null() == NULL) { |
1374 | 444 |
// Ran into an unparseable point. |
445 |
return cur; |
|
28833
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
446 |
} |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
447 |
|
28833
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
448 |
// Advance the current pointer. "obj" still points to the object to iterate. |
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
449 |
cur = cur + block_size(cur); |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
450 |
|
1374 | 451 |
if (!g1h->is_obj_dead(obj)) { |
28833
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
452 |
// Non-objArrays are sometimes marked imprecise at the object start. We |
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
453 |
// always need to iterate over them in full. |
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
454 |
// We only iterate over object arrays in full if they are completely contained |
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
455 |
// in the memory region. |
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
456 |
if (!obj->is_objArray() || (((HeapWord*)obj) >= start && cur <= end)) { |
1374 | 457 |
obj->oop_iterate(cl); |
458 |
} else { |
|
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11455
diff
changeset
|
459 |
obj->oop_iterate(cl, mr); |
1374 | 460 |
} |
461 |
} |
|
28833
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
462 |
} while (cur < end); |
747e11afce7e
8069760: When iterating over a card, G1 often iterates over much more references than are contained in the card
tschatzl
parents:
28633
diff
changeset
|
463 |
|
1374 | 464 |
return NULL; |
465 |
} |
|
466 |
||
19339 | 467 |
// Code roots support |
468 |
||
469 |
void HeapRegion::add_strong_code_root(nmethod* nm) { |
|
470 |
HeapRegionRemSet* hrrs = rem_set(); |
|
471 |
hrrs->add_strong_code_root(nm); |
|
472 |
} |
|
473 |
||
26422 | 474 |
void HeapRegion::add_strong_code_root_locked(nmethod* nm) { |
475 |
assert_locked_or_safepoint(CodeCache_lock); |
|
476 |
HeapRegionRemSet* hrrs = rem_set(); |
|
477 |
hrrs->add_strong_code_root_locked(nm); |
|
478 |
} |
|
479 |
||
19339 | 480 |
void HeapRegion::remove_strong_code_root(nmethod* nm) { |
481 |
HeapRegionRemSet* hrrs = rem_set(); |
|
482 |
hrrs->remove_strong_code_root(nm); |
|
483 |
} |
|
484 |
||
485 |
void HeapRegion::strong_code_roots_do(CodeBlobClosure* blk) const { |
|
486 |
HeapRegionRemSet* hrrs = rem_set(); |
|
487 |
hrrs->strong_code_roots_do(blk); |
|
488 |
} |
|
489 |
||
490 |
class VerifyStrongCodeRootOopClosure: public OopClosure { |
|
491 |
const HeapRegion* _hr; |
|
492 |
nmethod* _nm; |
|
493 |
bool _failures; |
|
494 |
bool _has_oops_in_region; |
|
495 |
||
496 |
template <class T> void do_oop_work(T* p) { |
|
497 |
T heap_oop = oopDesc::load_heap_oop(p); |
|
498 |
if (!oopDesc::is_null(heap_oop)) { |
|
499 |
oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
|
500 |
||
501 |
// Note: not all the oops embedded in the nmethod are in the |
|
502 |
// current region. We only look at those which are. |
|
503 |
if (_hr->is_in(obj)) { |
|
504 |
// Object is in the region. Check that its less than top |
|
505 |
if (_hr->top() <= (HeapWord*)obj) { |
|
506 |
// Object is above top |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
507 |
gclog_or_tty->print_cr("Object " PTR_FORMAT " in region " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
508 |
"[" PTR_FORMAT ", " PTR_FORMAT ") is above " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
509 |
"top " PTR_FORMAT, |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
510 |
p2i(obj), p2i(_hr->bottom()), p2i(_hr->end()), p2i(_hr->top())); |
19339 | 511 |
_failures = true; |
512 |
return; |
|
513 |
} |
|
514 |
// Nmethod has at least one oop in the current region |
|
515 |
_has_oops_in_region = true; |
|
516 |
} |
|
517 |
} |
|
518 |
} |
|
519 |
||
520 |
public: |
|
521 |
VerifyStrongCodeRootOopClosure(const HeapRegion* hr, nmethod* nm): |
|
522 |
_hr(hr), _failures(false), _has_oops_in_region(false) {} |
|
523 |
||
524 |
void do_oop(narrowOop* p) { do_oop_work(p); } |
|
525 |
void do_oop(oop* p) { do_oop_work(p); } |
|
526 |
||
527 |
bool failures() { return _failures; } |
|
528 |
bool has_oops_in_region() { return _has_oops_in_region; } |
|
529 |
}; |
|
530 |
||
531 |
class VerifyStrongCodeRootCodeBlobClosure: public CodeBlobClosure { |
|
532 |
const HeapRegion* _hr; |
|
533 |
bool _failures; |
|
534 |
public: |
|
535 |
VerifyStrongCodeRootCodeBlobClosure(const HeapRegion* hr) : |
|
536 |
_hr(hr), _failures(false) {} |
|
537 |
||
538 |
void do_code_blob(CodeBlob* cb) { |
|
539 |
nmethod* nm = (cb == NULL) ? NULL : cb->as_nmethod_or_null(); |
|
540 |
if (nm != NULL) { |
|
541 |
// Verify that the nemthod is live |
|
542 |
if (!nm->is_alive()) { |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
543 |
gclog_or_tty->print_cr("region [" PTR_FORMAT "," PTR_FORMAT "] has dead nmethod " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
544 |
PTR_FORMAT " in its strong code roots", |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
545 |
p2i(_hr->bottom()), p2i(_hr->end()), p2i(nm)); |
19339 | 546 |
_failures = true; |
547 |
} else { |
|
548 |
VerifyStrongCodeRootOopClosure oop_cl(_hr, nm); |
|
549 |
nm->oops_do(&oop_cl); |
|
550 |
if (!oop_cl.has_oops_in_region()) { |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
551 |
gclog_or_tty->print_cr("region [" PTR_FORMAT "," PTR_FORMAT "] has nmethod " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
552 |
PTR_FORMAT " in its strong code roots " |
19339 | 553 |
"with no pointers into region", |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
554 |
p2i(_hr->bottom()), p2i(_hr->end()), p2i(nm)); |
19339 | 555 |
_failures = true; |
556 |
} else if (oop_cl.failures()) { |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
557 |
gclog_or_tty->print_cr("region [" PTR_FORMAT "," PTR_FORMAT "] has other " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
558 |
"failures for nmethod " PTR_FORMAT, |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
559 |
p2i(_hr->bottom()), p2i(_hr->end()), p2i(nm)); |
19339 | 560 |
_failures = true; |
561 |
} |
|
562 |
} |
|
563 |
} |
|
564 |
} |
|
565 |
||
566 |
bool failures() { return _failures; } |
|
567 |
}; |
|
568 |
||
569 |
void HeapRegion::verify_strong_code_roots(VerifyOption vo, bool* failures) const { |
|
570 |
if (!G1VerifyHeapRegionCodeRoots) { |
|
571 |
// We're not verifying code roots. |
|
572 |
return; |
|
573 |
} |
|
574 |
if (vo == VerifyOption_G1UseMarkWord) { |
|
575 |
// Marking verification during a full GC is performed after class |
|
576 |
// unloading, code cache unloading, etc so the strong code roots |
|
577 |
// attached to each heap region are in an inconsistent state. They won't |
|
578 |
// be consistent until the strong code roots are rebuilt after the |
|
579 |
// actual GC. Skip verifying the strong code roots in this particular |
|
580 |
// time. |
|
581 |
assert(VerifyDuringGC, "only way to get here"); |
|
582 |
return; |
|
583 |
} |
|
584 |
||
585 |
HeapRegionRemSet* hrrs = rem_set(); |
|
23451
ed2b8bb28fed
8035406: Improve data structure for Code Cache remembered sets
tschatzl
parents:
22551
diff
changeset
|
586 |
size_t strong_code_roots_length = hrrs->strong_code_roots_list_length(); |
19339 | 587 |
|
588 |
// if this region is empty then there should be no entries |
|
589 |
// on its strong code root list |
|
590 |
if (is_empty()) { |
|
591 |
if (strong_code_roots_length > 0) { |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
592 |
gclog_or_tty->print_cr("region [" PTR_FORMAT "," PTR_FORMAT "] is empty " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
593 |
"but has " SIZE_FORMAT " code root entries", |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
594 |
p2i(bottom()), p2i(end()), strong_code_roots_length); |
19339 | 595 |
*failures = true; |
596 |
} |
|
597 |
return; |
|
598 |
} |
|
599 |
||
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
600 |
if (is_continues_humongous()) { |
19339 | 601 |
if (strong_code_roots_length > 0) { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
602 |
gclog_or_tty->print_cr("region " HR_FORMAT " is a continuation of a humongous " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
603 |
"region but has " SIZE_FORMAT " code root entries", |
21563
ccbd86010788
8027756: assert(!hr->isHumongous()) failed: code root in humongous region?
tschatzl
parents:
21561
diff
changeset
|
604 |
HR_FORMAT_PARAMS(this), strong_code_roots_length); |
19339 | 605 |
*failures = true; |
606 |
} |
|
607 |
return; |
|
608 |
} |
|
609 |
||
610 |
VerifyStrongCodeRootCodeBlobClosure cb_cl(this); |
|
611 |
strong_code_roots_do(&cb_cl); |
|
612 |
||
613 |
if (cb_cl.failures()) { |
|
614 |
*failures = true; |
|
615 |
} |
|
616 |
} |
|
617 |
||
1374 | 618 |
void HeapRegion::print() const { print_on(gclog_or_tty); } |
619 |
void HeapRegion::print_on(outputStream* st) const { |
|
26837
72a43d3841e7
8057536: Refactor G1 to allow context specific allocations
sjohanss
parents:
26827
diff
changeset
|
620 |
st->print("AC%4u", allocation_context()); |
72a43d3841e7
8057536: Refactor G1 to allow context specific allocations
sjohanss
parents:
26827
diff
changeset
|
621 |
|
26696
623a25e6c686
8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents:
26571
diff
changeset
|
622 |
st->print(" %2s", get_short_type_str()); |
1374 | 623 |
if (in_collection_set()) |
624 |
st->print(" CS"); |
|
625 |
else |
|
626 |
st->print(" "); |
|
10997
0be4b3be7197
7099849: G1: include heap region information in hs_err files
tonyp
parents:
10770
diff
changeset
|
627 |
st->print(" TS %5d", _gc_time_stamp); |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
628 |
st->print(" PTAMS " PTR_FORMAT " NTAMS " PTR_FORMAT, |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
629 |
p2i(prev_top_at_mark_start()), p2i(next_top_at_mark_start())); |
1374 | 630 |
G1OffsetTableContigSpace::print_on(st); |
631 |
} |
|
632 |
||
19339 | 633 |
class VerifyLiveClosure: public OopClosure { |
634 |
private: |
|
635 |
G1CollectedHeap* _g1h; |
|
636 |
CardTableModRefBS* _bs; |
|
637 |
oop _containing_obj; |
|
638 |
bool _failures; |
|
639 |
int _n_failures; |
|
640 |
VerifyOption _vo; |
|
641 |
public: |
|
642 |
// _vo == UsePrevMarking -> use "prev" marking information, |
|
643 |
// _vo == UseNextMarking -> use "next" marking information, |
|
644 |
// _vo == UseMarkWord -> use mark word from object header. |
|
645 |
VerifyLiveClosure(G1CollectedHeap* g1h, VerifyOption vo) : |
|
29325 | 646 |
_g1h(g1h), _bs(barrier_set_cast<CardTableModRefBS>(g1h->barrier_set())), |
647 |
_containing_obj(NULL), _failures(false), _n_failures(0), _vo(vo) |
|
648 |
{ } |
|
19339 | 649 |
|
650 |
void set_containing_obj(oop obj) { |
|
651 |
_containing_obj = obj; |
|
652 |
} |
|
653 |
||
654 |
bool failures() { return _failures; } |
|
655 |
int n_failures() { return _n_failures; } |
|
656 |
||
657 |
virtual void do_oop(narrowOop* p) { do_oop_work(p); } |
|
658 |
virtual void do_oop( oop* p) { do_oop_work(p); } |
|
659 |
||
660 |
void print_object(outputStream* out, oop obj) { |
|
661 |
#ifdef PRODUCT |
|
662 |
Klass* k = obj->klass(); |
|
663 |
const char* class_name = InstanceKlass::cast(k)->external_name(); |
|
664 |
out->print_cr("class name %s", class_name); |
|
665 |
#else // PRODUCT |
|
666 |
obj->print_on(out); |
|
667 |
#endif // PRODUCT |
|
668 |
} |
|
669 |
||
670 |
template <class T> |
|
671 |
void do_oop_work(T* p) { |
|
672 |
assert(_containing_obj != NULL, "Precondition"); |
|
673 |
assert(!_g1h->is_obj_dead_cond(_containing_obj, _vo), |
|
674 |
"Precondition"); |
|
675 |
T heap_oop = oopDesc::load_heap_oop(p); |
|
676 |
if (!oopDesc::is_null(heap_oop)) { |
|
677 |
oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
|
678 |
bool failed = false; |
|
679 |
if (!_g1h->is_in_closed_subset(obj) || _g1h->is_obj_dead_cond(obj, _vo)) { |
|
680 |
MutexLockerEx x(ParGCRareEvent_lock, |
|
681 |
Mutex::_no_safepoint_check_flag); |
|
682 |
||
683 |
if (!_failures) { |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
24351
diff
changeset
|
684 |
gclog_or_tty->cr(); |
19339 | 685 |
gclog_or_tty->print_cr("----------"); |
686 |
} |
|
687 |
if (!_g1h->is_in_closed_subset(obj)) { |
|
688 |
HeapRegion* from = _g1h->heap_region_containing((HeapWord*)p); |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
689 |
gclog_or_tty->print_cr("Field " PTR_FORMAT |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
690 |
" of live obj " PTR_FORMAT " in region " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
691 |
"[" PTR_FORMAT ", " PTR_FORMAT ")", |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
692 |
p2i(p), p2i(_containing_obj), |
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
693 |
p2i(from->bottom()), p2i(from->end())); |
19339 | 694 |
print_object(gclog_or_tty, _containing_obj); |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
695 |
gclog_or_tty->print_cr("points to obj " PTR_FORMAT " not in the heap", |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
696 |
p2i(obj)); |
19339 | 697 |
} else { |
698 |
HeapRegion* from = _g1h->heap_region_containing((HeapWord*)p); |
|
699 |
HeapRegion* to = _g1h->heap_region_containing((HeapWord*)obj); |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
700 |
gclog_or_tty->print_cr("Field " PTR_FORMAT |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
701 |
" of live obj " PTR_FORMAT " in region " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
702 |
"[" PTR_FORMAT ", " PTR_FORMAT ")", |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
703 |
p2i(p), p2i(_containing_obj), |
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
704 |
p2i(from->bottom()), p2i(from->end())); |
19339 | 705 |
print_object(gclog_or_tty, _containing_obj); |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
706 |
gclog_or_tty->print_cr("points to dead obj " PTR_FORMAT " in region " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
707 |
"[" PTR_FORMAT ", " PTR_FORMAT ")", |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
708 |
p2i(obj), p2i(to->bottom()), p2i(to->end())); |
19339 | 709 |
print_object(gclog_or_tty, obj); |
710 |
} |
|
711 |
gclog_or_tty->print_cr("----------"); |
|
712 |
gclog_or_tty->flush(); |
|
713 |
_failures = true; |
|
714 |
failed = true; |
|
715 |
_n_failures++; |
|
716 |
} |
|
717 |
||
31331
a7c714b6cfb3
7097567: G1: abstract and encapsulate collector phases and transitions between them
drwhite
parents:
30764
diff
changeset
|
718 |
if (!_g1h->collector_state()->full_collection() || G1VerifyRSetsDuringFullGC) { |
19339 | 719 |
HeapRegion* from = _g1h->heap_region_containing((HeapWord*)p); |
720 |
HeapRegion* to = _g1h->heap_region_containing(obj); |
|
721 |
if (from != NULL && to != NULL && |
|
722 |
from != to && |
|
31346
a70d45c06136
8042668: GC Support for shared heap ranges in CDS
jiangli
parents:
31331
diff
changeset
|
723 |
!to->is_pinned()) { |
19339 | 724 |
jbyte cv_obj = *_bs->byte_for_const(_containing_obj); |
725 |
jbyte cv_field = *_bs->byte_for_const(p); |
|
726 |
const jbyte dirty = CardTableModRefBS::dirty_card_val(); |
|
727 |
||
728 |
bool is_bad = !(from->is_young() |
|
729 |
|| to->rem_set()->contains_reference(p) |
|
730 |
|| !G1HRRSFlushLogBuffersOnVerify && // buffers were not flushed |
|
731 |
(_containing_obj->is_objArray() ? |
|
732 |
cv_field == dirty |
|
733 |
: cv_obj == dirty || cv_field == dirty)); |
|
734 |
if (is_bad) { |
|
735 |
MutexLockerEx x(ParGCRareEvent_lock, |
|
736 |
Mutex::_no_safepoint_check_flag); |
|
737 |
||
738 |
if (!_failures) { |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
24351
diff
changeset
|
739 |
gclog_or_tty->cr(); |
19339 | 740 |
gclog_or_tty->print_cr("----------"); |
741 |
} |
|
742 |
gclog_or_tty->print_cr("Missing rem set entry:"); |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
743 |
gclog_or_tty->print_cr("Field " PTR_FORMAT " " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
744 |
"of obj " PTR_FORMAT ", " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
745 |
"in region " HR_FORMAT, |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
746 |
p2i(p), p2i(_containing_obj), |
19339 | 747 |
HR_FORMAT_PARAMS(from)); |
748 |
_containing_obj->print_on(gclog_or_tty); |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
749 |
gclog_or_tty->print_cr("points to obj " PTR_FORMAT " " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
750 |
"in region " HR_FORMAT, |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
751 |
p2i(obj), |
19339 | 752 |
HR_FORMAT_PARAMS(to)); |
753 |
obj->print_on(gclog_or_tty); |
|
754 |
gclog_or_tty->print_cr("Obj head CTE = %d, field CTE = %d.", |
|
755 |
cv_obj, cv_field); |
|
756 |
gclog_or_tty->print_cr("----------"); |
|
757 |
gclog_or_tty->flush(); |
|
758 |
_failures = true; |
|
759 |
if (!failed) _n_failures++; |
|
760 |
} |
|
761 |
} |
|
762 |
} |
|
763 |
} |
|
764 |
} |
|
765 |
}; |
|
3000 | 766 |
|
1374 | 767 |
// This really ought to be commoned up into OffsetTableContigSpace somehow. |
768 |
// We would need a mechanism to make that code skip dead objects. |
|
769 |
||
12379 | 770 |
void HeapRegion::verify(VerifyOption vo, |
4023
6c3401503290
6882730: G1: parallel heap verification messes up region dump
tonyp
parents:
3697
diff
changeset
|
771 |
bool* failures) const { |
1374 | 772 |
G1CollectedHeap* g1 = G1CollectedHeap::heap(); |
4023
6c3401503290
6882730: G1: parallel heap verification messes up region dump
tonyp
parents:
3697
diff
changeset
|
773 |
*failures = false; |
1374 | 774 |
HeapWord* p = bottom(); |
775 |
HeapWord* prev_p = NULL; |
|
9995
290620c08233
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
9989
diff
changeset
|
776 |
VerifyLiveClosure vl_cl(g1, vo); |
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
777 |
bool is_region_humongous = is_humongous(); |
6260
fba83fd3adc7
6974928: G1: sometimes humongous objects are allocated in young regions
tonyp
parents:
6247
diff
changeset
|
778 |
size_t object_num = 0; |
1374 | 779 |
while (p < top()) { |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
780 |
oop obj = oop(p); |
25481
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
781 |
size_t obj_size = block_size(p); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
782 |
object_num += 1; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
783 |
|
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
784 |
if (is_region_humongous != g1->is_humongous(obj_size) && |
25492
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
785 |
!g1->is_obj_dead(obj, this)) { // Dead objects may have bigger block_size since they span several objects. |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
786 |
gclog_or_tty->print_cr("obj " PTR_FORMAT " is of %shumongous size (" |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
787 |
SIZE_FORMAT " words) in a %shumongous region", |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
788 |
p2i(p), g1->is_humongous(obj_size) ? "" : "non-", |
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
789 |
obj_size, is_region_humongous ? "" : "non-"); |
6260
fba83fd3adc7
6974928: G1: sometimes humongous objects are allocated in young regions
tonyp
parents:
6247
diff
changeset
|
790 |
*failures = true; |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
791 |
return; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
792 |
} |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
793 |
|
9995
290620c08233
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
9989
diff
changeset
|
794 |
if (!g1->is_obj_dead_cond(obj, this, vo)) { |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
795 |
if (obj->is_oop()) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13482
diff
changeset
|
796 |
Klass* klass = obj->klass(); |
25492
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
797 |
bool is_metaspace_object = Metaspace::contains(klass) || |
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
798 |
(vo == VerifyOption_G1UsePrevMarking && |
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
799 |
ClassLoaderDataGraph::unload_list_contains(klass)); |
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
800 |
if (!is_metaspace_object) { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
801 |
gclog_or_tty->print_cr("klass " PTR_FORMAT " of object " PTR_FORMAT " " |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
802 |
"not metadata", p2i(klass), p2i(obj)); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
803 |
*failures = true; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
804 |
return; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
805 |
} else if (!klass->is_klass()) { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
806 |
gclog_or_tty->print_cr("klass " PTR_FORMAT " of object " PTR_FORMAT " " |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
807 |
"not a klass", p2i(klass), p2i(obj)); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
808 |
*failures = true; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
809 |
return; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
810 |
} else { |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
811 |
vl_cl.set_containing_obj(obj); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13482
diff
changeset
|
812 |
obj->oop_iterate_no_header(&vl_cl); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
813 |
if (vl_cl.failures()) { |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
814 |
*failures = true; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
815 |
} |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
816 |
if (G1MaxVerifyFailures >= 0 && |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
817 |
vl_cl.n_failures() >= G1MaxVerifyFailures) { |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
818 |
return; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
819 |
} |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
820 |
} |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
821 |
} else { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
822 |
gclog_or_tty->print_cr(PTR_FORMAT " no an oop", p2i(obj)); |
4023
6c3401503290
6882730: G1: parallel heap verification messes up region dump
tonyp
parents:
3697
diff
changeset
|
823 |
*failures = true; |
6c3401503290
6882730: G1: parallel heap verification messes up region dump
tonyp
parents:
3697
diff
changeset
|
824 |
return; |
6c3401503290
6882730: G1: parallel heap verification messes up region dump
tonyp
parents:
3697
diff
changeset
|
825 |
} |
1374 | 826 |
} |
827 |
prev_p = p; |
|
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
828 |
p += obj_size; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
829 |
} |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
830 |
|
26571
5fe42815e865
8025564: gc/memory/UniThread/Linear1 times out during heap verification
brutisso
parents:
26422
diff
changeset
|
831 |
if (!is_young() && !is_empty()) { |
5fe42815e865
8025564: gc/memory/UniThread/Linear1 times out during heap verification
brutisso
parents:
26422
diff
changeset
|
832 |
_offsets.verify(); |
5fe42815e865
8025564: gc/memory/UniThread/Linear1 times out during heap verification
brutisso
parents:
26422
diff
changeset
|
833 |
} |
5fe42815e865
8025564: gc/memory/UniThread/Linear1 times out during heap verification
brutisso
parents:
26422
diff
changeset
|
834 |
|
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
835 |
if (p != top()) { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
836 |
gclog_or_tty->print_cr("end of last object " PTR_FORMAT " " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
837 |
"does not match top " PTR_FORMAT, p2i(p), p2i(top())); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
838 |
*failures = true; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
839 |
return; |
1374 | 840 |
} |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
841 |
|
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
842 |
HeapWord* the_end = end(); |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
843 |
assert(p == top(), "it should still hold"); |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
844 |
// Do some extra BOT consistency checking for addresses in the |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
845 |
// range [top, end). BOT look-ups in this range should yield |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
846 |
// top. No point in doing that if top == end (there's nothing there). |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
847 |
if (p < the_end) { |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
848 |
// Look up top |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
849 |
HeapWord* addr_1 = p; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
850 |
HeapWord* b_start_1 = _offsets.block_start_const(addr_1); |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
851 |
if (b_start_1 != p) { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
852 |
gclog_or_tty->print_cr("BOT look up for top: " PTR_FORMAT " " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
853 |
" yielded " PTR_FORMAT ", expecting " PTR_FORMAT, |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
854 |
p2i(addr_1), p2i(b_start_1), p2i(p)); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
855 |
*failures = true; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
856 |
return; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
857 |
} |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
858 |
|
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
859 |
// Look up top + 1 |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
860 |
HeapWord* addr_2 = p + 1; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
861 |
if (addr_2 < the_end) { |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
862 |
HeapWord* b_start_2 = _offsets.block_start_const(addr_2); |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
863 |
if (b_start_2 != p) { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
864 |
gclog_or_tty->print_cr("BOT look up for top + 1: " PTR_FORMAT " " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
865 |
" yielded " PTR_FORMAT ", expecting " PTR_FORMAT, |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
866 |
p2i(addr_2), p2i(b_start_2), p2i(p)); |
4023
6c3401503290
6882730: G1: parallel heap verification messes up region dump
tonyp
parents:
3697
diff
changeset
|
867 |
*failures = true; |
6c3401503290
6882730: G1: parallel heap verification messes up region dump
tonyp
parents:
3697
diff
changeset
|
868 |
return; |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
869 |
} |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
870 |
} |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
871 |
|
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
872 |
// Look up an address between top and end |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
873 |
size_t diff = pointer_delta(the_end, p) / 2; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
874 |
HeapWord* addr_3 = p + diff; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
875 |
if (addr_3 < the_end) { |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
876 |
HeapWord* b_start_3 = _offsets.block_start_const(addr_3); |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
877 |
if (b_start_3 != p) { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
878 |
gclog_or_tty->print_cr("BOT look up for top + diff: " PTR_FORMAT " " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
879 |
" yielded " PTR_FORMAT ", expecting " PTR_FORMAT, |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
880 |
p2i(addr_3), p2i(b_start_3), p2i(p)); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
881 |
*failures = true; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
882 |
return; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
883 |
} |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
884 |
} |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
885 |
|
22551 | 886 |
// Look up end - 1 |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
887 |
HeapWord* addr_4 = the_end - 1; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
888 |
HeapWord* b_start_4 = _offsets.block_start_const(addr_4); |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
889 |
if (b_start_4 != p) { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
890 |
gclog_or_tty->print_cr("BOT look up for end - 1: " PTR_FORMAT " " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
891 |
" yielded " PTR_FORMAT ", expecting " PTR_FORMAT, |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
892 |
p2i(addr_4), p2i(b_start_4), p2i(p)); |
7904
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
893 |
*failures = true; |
e90e097fced4
7007068: G1: refine the BOT during evac failure handling
tonyp
parents:
7397
diff
changeset
|
894 |
return; |
4023
6c3401503290
6882730: G1: parallel heap verification messes up region dump
tonyp
parents:
3697
diff
changeset
|
895 |
} |
1374 | 896 |
} |
4023
6c3401503290
6882730: G1: parallel heap verification messes up region dump
tonyp
parents:
3697
diff
changeset
|
897 |
|
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26840
diff
changeset
|
898 |
if (is_region_humongous && object_num > 1) { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
899 |
gclog_or_tty->print_cr("region [" PTR_FORMAT "," PTR_FORMAT "] is humongous " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
31346
diff
changeset
|
900 |
"but has " SIZE_FORMAT ", objects", |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
901 |
p2i(bottom()), p2i(end()), object_num); |
6260
fba83fd3adc7
6974928: G1: sometimes humongous objects are allocated in young regions
tonyp
parents:
6247
diff
changeset
|
902 |
*failures = true; |
4023
6c3401503290
6882730: G1: parallel heap verification messes up region dump
tonyp
parents:
3697
diff
changeset
|
903 |
return; |
1374 | 904 |
} |
19339 | 905 |
|
906 |
verify_strong_code_roots(vo, failures); |
|
907 |
} |
|
908 |
||
909 |
void HeapRegion::verify() const { |
|
910 |
bool dummy = false; |
|
911 |
verify(VerifyOption_G1UsePrevMarking, /* failures */ &dummy); |
|
1374 | 912 |
} |
913 |
||
27624
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
27009
diff
changeset
|
914 |
void HeapRegion::prepare_for_compaction(CompactPoint* cp) { |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
27009
diff
changeset
|
915 |
scan_and_forward(this, cp); |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
27009
diff
changeset
|
916 |
} |
fe43edc5046d
8043243: convert SCAN_AND_FORWARD, SCAN_AND_ADJUST_POINTERS, SCAN_AND_COMPACT macros to methods
mlarsson
parents:
27009
diff
changeset
|
917 |
|
1374 | 918 |
// G1OffsetTableContigSpace code; copied from space.cpp. Hope this can go |
919 |
// away eventually. |
|
920 |
||
1388 | 921 |
void G1OffsetTableContigSpace::clear(bool mangle_space) { |
25481
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
922 |
set_top(bottom()); |
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
923 |
_scan_top = bottom(); |
25481
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
924 |
CompactibleSpace::clear(mangle_space); |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
925 |
reset_bot(); |
1374 | 926 |
} |
927 |
||
928 |
void G1OffsetTableContigSpace::set_bottom(HeapWord* new_bottom) { |
|
929 |
Space::set_bottom(new_bottom); |
|
930 |
_offsets.set_bottom(new_bottom); |
|
931 |
} |
|
932 |
||
933 |
void G1OffsetTableContigSpace::set_end(HeapWord* new_end) { |
|
934 |
Space::set_end(new_end); |
|
935 |
_offsets.resize(new_end - bottom()); |
|
936 |
} |
|
937 |
||
29681 | 938 |
#ifndef PRODUCT |
939 |
void G1OffsetTableContigSpace::mangle_unused_area() { |
|
940 |
mangle_unused_area_complete(); |
|
941 |
} |
|
942 |
||
943 |
void G1OffsetTableContigSpace::mangle_unused_area_complete() { |
|
944 |
SpaceMangler::mangle_region(MemRegion(top(), end())); |
|
945 |
} |
|
946 |
#endif |
|
947 |
||
1374 | 948 |
void G1OffsetTableContigSpace::print() const { |
949 |
print_short(); |
|
950 |
gclog_or_tty->print_cr(" [" INTPTR_FORMAT ", " INTPTR_FORMAT ", " |
|
951 |
INTPTR_FORMAT ", " INTPTR_FORMAT ")", |
|
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29698
diff
changeset
|
952 |
p2i(bottom()), p2i(top()), p2i(_offsets.threshold()), p2i(end())); |
1374 | 953 |
} |
954 |
||
955 |
HeapWord* G1OffsetTableContigSpace::initialize_threshold() { |
|
956 |
return _offsets.initialize_threshold(); |
|
957 |
} |
|
958 |
||
959 |
HeapWord* G1OffsetTableContigSpace::cross_threshold(HeapWord* start, |
|
960 |
HeapWord* end) { |
|
961 |
_offsets.alloc_block(start, end); |
|
962 |
return _offsets.threshold(); |
|
963 |
} |
|
964 |
||
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
965 |
HeapWord* G1OffsetTableContigSpace::scan_top() const { |
1374 | 966 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
27681
cc54c45a93ac
8058209: Race in G1 card scanning could allow scanning of memory covered by PLABs
mgerdin
parents:
27624
diff
changeset
|
967 |
HeapWord* local_top = top(); |
cc54c45a93ac
8058209: Race in G1 card scanning could allow scanning of memory covered by PLABs
mgerdin
parents:
27624
diff
changeset
|
968 |
OrderAccess::loadload(); |
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
969 |
const unsigned local_time_stamp = _gc_time_stamp; |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
970 |
assert(local_time_stamp <= g1h->get_gc_time_stamp(), "invariant"); |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
971 |
if (local_time_stamp < g1h->get_gc_time_stamp()) { |
27681
cc54c45a93ac
8058209: Race in G1 card scanning could allow scanning of memory covered by PLABs
mgerdin
parents:
27624
diff
changeset
|
972 |
return local_top; |
cc54c45a93ac
8058209: Race in G1 card scanning could allow scanning of memory covered by PLABs
mgerdin
parents:
27624
diff
changeset
|
973 |
} else { |
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
974 |
return _scan_top; |
27681
cc54c45a93ac
8058209: Race in G1 card scanning could allow scanning of memory covered by PLABs
mgerdin
parents:
27624
diff
changeset
|
975 |
} |
1374 | 976 |
} |
977 |
||
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
978 |
void G1OffsetTableContigSpace::record_timestamp() { |
1374 | 979 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
980 |
unsigned curr_gc_time_stamp = g1h->get_gc_time_stamp(); |
|
981 |
||
982 |
if (_gc_time_stamp < curr_gc_time_stamp) { |
|
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
983 |
// Setting the time stamp here tells concurrent readers to look at |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
984 |
// scan_top to know the maximum allowed address to look at. |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
985 |
|
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
986 |
// scan_top should be bottom for all regions except for the |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
987 |
// retained old alloc region which should have scan_top == top |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
988 |
HeapWord* st = _scan_top; |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
989 |
guarantee(st == _bottom || st == _top, "invariant"); |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
990 |
|
1385
1751733b089b
6723570: G1: assertion failure: p == current_top or oop(p)->is_oop(),"p is not a block start" (revisited!)
iveresov
parents:
1374
diff
changeset
|
991 |
_gc_time_stamp = curr_gc_time_stamp; |
1374 | 992 |
} |
993 |
} |
|
994 |
||
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
995 |
void G1OffsetTableContigSpace::record_retained_region() { |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
996 |
// scan_top is the maximum address where it's safe for the next gc to |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
997 |
// scan this region. |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
998 |
_scan_top = top(); |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
999 |
} |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
1000 |
|
25481
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1001 |
void G1OffsetTableContigSpace::safe_object_iterate(ObjectClosure* blk) { |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1002 |
object_iterate(blk); |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1003 |
} |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1004 |
|
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1005 |
void G1OffsetTableContigSpace::object_iterate(ObjectClosure* blk) { |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1006 |
HeapWord* p = bottom(); |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1007 |
while (p < top()) { |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1008 |
if (block_is_obj(p)) { |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1009 |
blk->do_object(oop(p)); |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1010 |
} |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1011 |
p += block_size(p); |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1012 |
} |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1013 |
} |
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1014 |
|
1374 | 1015 |
G1OffsetTableContigSpace:: |
1016 |
G1OffsetTableContigSpace(G1BlockOffsetSharedArray* sharedOffsetArray, |
|
13756
3b72f3da8cd4
7016955: G1: remove the is_zeroed parameter from the HeapRegion constructor
johnc
parents:
13728
diff
changeset
|
1017 |
MemRegion mr) : |
1374 | 1018 |
_offsets(sharedOffsetArray, mr), |
1019 |
_par_alloc_lock(Mutex::leaf, "OffsetTableContigSpace par alloc lock", true), |
|
1020 |
_gc_time_stamp(0) |
|
1021 |
{ |
|
1022 |
_offsets.set_space(this); |
|
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
1023 |
} |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
1024 |
|
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
1025 |
void G1OffsetTableContigSpace::initialize(MemRegion mr, bool clear_space, bool mangle_space) { |
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
1026 |
CompactibleSpace::initialize(mr, clear_space, mangle_space); |
25481
1427aa24638c
8047818: G1 HeapRegions can no longer be ContiguousSpaces
mgerdin
parents:
25479
diff
changeset
|
1027 |
_top = bottom(); |
27889
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
1028 |
_scan_top = bottom(); |
7d50f95e0076
8065358: Refactor G1s usage of save_marks and reduce related races
mgerdin
parents:
27681
diff
changeset
|
1029 |
set_saved_mark_word(NULL); |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
1030 |
reset_bot(); |
1374 | 1031 |
} |
26157
70eddb655686
8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents:
25730
diff
changeset
|
1032 |