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