author | coleenp |
Wed, 06 Jun 2018 10:45:40 -0400 | |
changeset 50429 | 83aec1d357d4 |
parent 49733 | 6f0a3ea5ab75 |
child 51598 | c88019b32bc4 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
49164
7e958a8ebcd3
8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents:
47622
diff
changeset
|
2 |
* Copyright (c) 2007, 2018, Oracle and/or its affiliates. All rights reserved. |
1 | 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:
3262
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
3262
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:
3262
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
49733
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
26 |
#include "gc/cms/cmsCardTable.hpp" |
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
27 |
#include "gc/cms/cmsHeap.hpp" |
49455
848864ed9b17
8199604: Rename CardTableModRefBS to CardTableBarrierSet
eosterlund
parents:
49164
diff
changeset
|
28 |
#include "gc/shared/cardTableBarrierSet.hpp" |
30764 | 29 |
#include "gc/shared/cardTableRS.hpp" |
30 |
#include "gc/shared/collectedHeap.hpp" |
|
31 |
#include "gc/shared/space.inline.hpp" |
|
7397 | 32 |
#include "memory/allocation.inline.hpp" |
30291
54cdc5c1a9cb
8068352: Move virtualspace.* out of src/share/vm/runtime to memory directory
coleenp
parents:
30173
diff
changeset
|
33 |
#include "memory/virtualspace.hpp" |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
34 |
#include "oops/oop.inline.hpp" |
7397 | 35 |
#include "runtime/java.hpp" |
36 |
#include "runtime/mutexLocker.hpp" |
|
50429
83aec1d357d4
8204301: Make OrderAccess functions available to hpp rather than inline.hpp files
coleenp
parents:
49733
diff
changeset
|
37 |
#include "runtime/orderAccess.hpp" |
11174
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
10273
diff
changeset
|
38 |
#include "runtime/vmThread.hpp" |
1 | 39 |
|
49733
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
40 |
CMSCardTable::CMSCardTable(MemRegion whole_heap) : |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
41 |
CardTableRS(whole_heap, CMSPrecleaningEnabled /* scanned_concurrently */) { |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
42 |
} |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
43 |
|
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
44 |
// Returns the number of chunks necessary to cover "mr". |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
45 |
size_t CMSCardTable::chunks_to_cover(MemRegion mr) { |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
46 |
return (size_t)(addr_to_chunk_index(mr.last()) - |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
47 |
addr_to_chunk_index(mr.start()) + 1); |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
48 |
} |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
49 |
|
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
50 |
// Returns the index of the chunk in a stride which |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
51 |
// covers the given address. |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
52 |
uintptr_t CMSCardTable::addr_to_chunk_index(const void* addr) { |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
53 |
uintptr_t card = (uintptr_t) byte_for(addr); |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
54 |
return card / ParGCCardsPerStrideChunk; |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
55 |
} |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
56 |
|
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
57 |
void CMSCardTable:: |
31964 | 58 |
non_clean_card_iterate_parallel_work(Space* sp, MemRegion mr, |
59 |
OopsInGenClosure* cl, |
|
60 |
CardTableRS* ct, |
|
61 |
uint n_threads) { |
|
30870
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
62 |
assert(n_threads > 0, "expected n_threads > 0"); |
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
63 |
assert(n_threads <= ParallelGCThreads, |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
64 |
"n_threads: %u > ParallelGCThreads: %u", n_threads, ParallelGCThreads); |
30870
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
65 |
|
9336
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
66 |
// Make sure the LNC array is valid for the space. |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
67 |
jbyte** lowest_non_clean; |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
68 |
uintptr_t lowest_non_clean_base_chunk_index; |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
69 |
size_t lowest_non_clean_chunk_size; |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
70 |
get_LNC_array_for_space(sp, lowest_non_clean, |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
71 |
lowest_non_clean_base_chunk_index, |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
72 |
lowest_non_clean_chunk_size); |
1 | 73 |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11174
diff
changeset
|
74 |
uint n_strides = n_threads * ParGCStridesPerThread; |
9336
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
75 |
SequentialSubTasksDone* pst = sp->par_seq_tasks(); |
11174
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
10273
diff
changeset
|
76 |
// Sets the condition for completion of the subtask (how many threads |
fccee5238e70
6593758: RFE: Enhance GC ergonomics to dynamically choose ParallelGCThreads
jmasa
parents:
10273
diff
changeset
|
77 |
// need to finish in order to be done). |
9336
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
78 |
pst->set_n_threads(n_threads); |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
79 |
pst->set_n_tasks(n_strides); |
1 | 80 |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11174
diff
changeset
|
81 |
uint stride = 0; |
9336
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
82 |
while (!pst->is_task_claimed(/* reference */ stride)) { |
30870
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
83 |
process_stride(sp, mr, stride, n_strides, |
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
84 |
cl, ct, |
9336
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
85 |
lowest_non_clean, |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
86 |
lowest_non_clean_base_chunk_index, |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
87 |
lowest_non_clean_chunk_size); |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
88 |
} |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
89 |
if (pst->all_tasks_completed()) { |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
90 |
// Clear lowest_non_clean array for next time. |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
91 |
intptr_t first_chunk_index = addr_to_chunk_index(mr.start()); |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
92 |
uintptr_t last_chunk_index = addr_to_chunk_index(mr.last()); |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
93 |
for (uintptr_t ch = first_chunk_index; ch <= last_chunk_index; ch++) { |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
94 |
intptr_t ind = ch - lowest_non_clean_base_chunk_index; |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
95 |
assert(0 <= ind && ind < (intptr_t)lowest_non_clean_chunk_size, |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
96 |
"Bounds error"); |
413920193f83
7037276: Unnecessary double traversal of dirty card windows
ysr
parents:
9183
diff
changeset
|
97 |
lowest_non_clean[ind] = NULL; |
1 | 98 |
} |
99 |
} |
|
100 |
} |
|
101 |
||
102 |
void |
|
49733
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
103 |
CMSCardTable:: |
1 | 104 |
process_stride(Space* sp, |
105 |
MemRegion used, |
|
106 |
jint stride, int n_strides, |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
107 |
OopsInGenClosure* cl, |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
108 |
CardTableRS* ct, |
1 | 109 |
jbyte** lowest_non_clean, |
110 |
uintptr_t lowest_non_clean_base_chunk_index, |
|
111 |
size_t lowest_non_clean_chunk_size) { |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
112 |
// We go from higher to lower addresses here; it wouldn't help that much |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
113 |
// because of the strided parallelism pattern used here. |
1 | 114 |
|
115 |
// Find the first card address of the first chunk in the stride that is |
|
116 |
// at least "bottom" of the used region. |
|
117 |
jbyte* start_card = byte_for(used.start()); |
|
118 |
jbyte* end_card = byte_after(used.last()); |
|
119 |
uintptr_t start_chunk = addr_to_chunk_index(used.start()); |
|
120 |
uintptr_t start_chunk_stride_num = start_chunk % n_strides; |
|
121 |
jbyte* chunk_card_start; |
|
122 |
||
123 |
if ((uintptr_t)stride >= start_chunk_stride_num) { |
|
124 |
chunk_card_start = (jbyte*)(start_card + |
|
125 |
(stride - start_chunk_stride_num) * |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
126 |
ParGCCardsPerStrideChunk); |
1 | 127 |
} else { |
128 |
// Go ahead to the next chunk group boundary, then to the requested stride. |
|
129 |
chunk_card_start = (jbyte*)(start_card + |
|
130 |
(n_strides - start_chunk_stride_num + stride) * |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
131 |
ParGCCardsPerStrideChunk); |
1 | 132 |
} |
133 |
||
134 |
while (chunk_card_start < end_card) { |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
135 |
// Even though we go from lower to higher addresses below, the |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
136 |
// strided parallelism can interleave the actual processing of the |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
137 |
// dirty pages in various ways. For a specific chunk within this |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
138 |
// stride, we take care to avoid double scanning or missing a card |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
139 |
// by suitably initializing the "min_done" field in process_chunk_boundaries() |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
140 |
// below, together with the dirty region extension accomplished in |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
141 |
// DirtyCardToOopClosure::do_MemRegion(). |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
142 |
jbyte* chunk_card_end = chunk_card_start + ParGCCardsPerStrideChunk; |
1 | 143 |
// Invariant: chunk_mr should be fully contained within the "used" region. |
144 |
MemRegion chunk_mr = MemRegion(addr_for(chunk_card_start), |
|
145 |
chunk_card_end >= end_card ? |
|
146 |
used.end() : addr_for(chunk_card_end)); |
|
147 |
assert(chunk_mr.word_size() > 0, "[chunk_card_start > used_end)"); |
|
148 |
assert(used.contains(chunk_mr), "chunk_mr should be subset of used"); |
|
149 |
||
30870
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
150 |
// This function is used by the parallel card table iteration. |
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
151 |
const bool parallel = true; |
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
152 |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
153 |
DirtyCardToOopClosure* dcto_cl = sp->new_dcto_cl(cl, precision(), |
30870
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
154 |
cl->gen_boundary(), |
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
155 |
parallel); |
3050fdcdc60b
8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents:
30764
diff
changeset
|
156 |
ClearNoncleanCardWrapper clear_cl(dcto_cl, ct, parallel); |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
157 |
|
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
158 |
|
1 | 159 |
// Process the chunk. |
160 |
process_chunk_boundaries(sp, |
|
161 |
dcto_cl, |
|
162 |
chunk_mr, |
|
163 |
used, |
|
164 |
lowest_non_clean, |
|
165 |
lowest_non_clean_base_chunk_index, |
|
166 |
lowest_non_clean_chunk_size); |
|
167 |
||
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
168 |
// We want the LNC array updates above in process_chunk_boundaries |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
169 |
// to be visible before any of the card table value changes as a |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
170 |
// result of the dirty card iteration below. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
171 |
OrderAccess::storestore(); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
172 |
|
29805
398335f2abed
8076241: Remove unused methods mod_card_iterate() and non_clean_card_iterate_serial()
brutisso
parents:
29799
diff
changeset
|
173 |
// We want to clear the cards: clear_cl here does the work of finding |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
174 |
// contiguous dirty ranges of cards to process and clear. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
175 |
clear_cl.do_MemRegion(chunk_mr); |
1 | 176 |
|
177 |
// Find the next chunk of the stride. |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
178 |
chunk_card_start += ParGCCardsPerStrideChunk * n_strides; |
1 | 179 |
} |
180 |
} |
|
181 |
||
182 |
void |
|
49733
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
183 |
CMSCardTable:: |
1 | 184 |
process_chunk_boundaries(Space* sp, |
185 |
DirtyCardToOopClosure* dcto_cl, |
|
186 |
MemRegion chunk_mr, |
|
187 |
MemRegion used, |
|
188 |
jbyte** lowest_non_clean, |
|
189 |
uintptr_t lowest_non_clean_base_chunk_index, |
|
190 |
size_t lowest_non_clean_chunk_size) |
|
191 |
{ |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
192 |
// We must worry about non-array objects that cross chunk boundaries, |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
193 |
// because such objects are both precisely and imprecisely marked: |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
194 |
// .. if the head of such an object is dirty, the entire object |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
195 |
// needs to be scanned, under the interpretation that this |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
196 |
// was an imprecise mark |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
197 |
// .. if the head of such an object is not dirty, we can assume |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
198 |
// precise marking and it's efficient to scan just the dirty |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
199 |
// cards. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
200 |
// In either case, each scanned reference must be scanned precisely |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
201 |
// once so as to avoid cloning of a young referent. For efficiency, |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
202 |
// our closures depend on this property and do not protect against |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
203 |
// double scans. |
1 | 204 |
|
29580
a67a581cfe11
8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents:
27880
diff
changeset
|
205 |
uintptr_t start_chunk_index = addr_to_chunk_index(chunk_mr.start()); |
a67a581cfe11
8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents:
27880
diff
changeset
|
206 |
assert(start_chunk_index >= lowest_non_clean_base_chunk_index, "Bounds error."); |
a67a581cfe11
8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents:
27880
diff
changeset
|
207 |
uintptr_t cur_chunk_index = start_chunk_index - lowest_non_clean_base_chunk_index; |
1 | 208 |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
209 |
// First, set "our" lowest_non_clean entry, which would be |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
210 |
// used by the thread scanning an adjoining left chunk with |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
211 |
// a non-array object straddling the mutual boundary. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
212 |
// Find the object that spans our boundary, if one exists. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
213 |
// first_block is the block possibly straddling our left boundary. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
214 |
HeapWord* first_block = sp->block_start(chunk_mr.start()); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
215 |
assert((chunk_mr.start() != used.start()) || (first_block == chunk_mr.start()), |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
216 |
"First chunk should always have a co-initial block"); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
217 |
// Does the block straddle the chunk's left boundary, and is it |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
218 |
// a non-array object? |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
219 |
if (first_block < chunk_mr.start() // first block straddles left bdry |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
220 |
&& sp->block_is_obj(first_block) // first block is an object |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
221 |
&& !(oop(first_block)->is_objArray() // first block is not an array (arrays are precisely dirtied) |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
222 |
|| oop(first_block)->is_typeArray())) { |
22551 | 223 |
// Find our least non-clean card, so that a left neighbor |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
224 |
// does not scan an object straddling the mutual boundary |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
225 |
// too far to the right, and attempt to scan a portion of |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
226 |
// that object twice. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
227 |
jbyte* first_dirty_card = NULL; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
228 |
jbyte* last_card_of_first_obj = |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
229 |
byte_for(first_block + sp->block_size(first_block) - 1); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
230 |
jbyte* first_card_of_cur_chunk = byte_for(chunk_mr.start()); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
231 |
jbyte* last_card_of_cur_chunk = byte_for(chunk_mr.last()); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
232 |
jbyte* last_card_to_check = |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
233 |
(jbyte*) MIN2((intptr_t) last_card_of_cur_chunk, |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
234 |
(intptr_t) last_card_of_first_obj); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
235 |
// Note that this does not need to go beyond our last card |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
236 |
// if our first object completely straddles this chunk. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
237 |
for (jbyte* cur = first_card_of_cur_chunk; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
238 |
cur <= last_card_to_check; cur++) { |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
239 |
jbyte val = *cur; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
240 |
if (card_will_be_scanned(val)) { |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
241 |
first_dirty_card = cur; break; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
242 |
} else { |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
243 |
assert(!card_may_have_been_dirty(val), "Error"); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
244 |
} |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
245 |
} |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
246 |
if (first_dirty_card != NULL) { |
29580
a67a581cfe11
8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents:
27880
diff
changeset
|
247 |
assert(cur_chunk_index < lowest_non_clean_chunk_size, "Bounds error."); |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
248 |
assert(lowest_non_clean[cur_chunk_index] == NULL, |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
249 |
"Write exactly once : value should be stable hereafter for this round"); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
250 |
lowest_non_clean[cur_chunk_index] = first_dirty_card; |
37055
3ea6a0d5a78c
8151514: Remove the noisy NOISY debugging code from parCardTableModRefBS.cpp
brutisso
parents:
33105
diff
changeset
|
251 |
} |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
252 |
} else { |
22551 | 253 |
// In this case we can help our neighbor by just asking them |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
254 |
// to stop at our first card (even though it may not be dirty). |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
255 |
assert(lowest_non_clean[cur_chunk_index] == NULL, "Write once : value should be stable hereafter"); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
256 |
jbyte* first_card_of_cur_chunk = byte_for(chunk_mr.start()); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
257 |
lowest_non_clean[cur_chunk_index] = first_card_of_cur_chunk; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
258 |
} |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
259 |
|
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
260 |
// Next, set our own max_to_do, which will strictly/exclusively bound |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
261 |
// the highest address that we will scan past the right end of our chunk. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
262 |
HeapWord* max_to_do = NULL; |
1 | 263 |
if (chunk_mr.end() < used.end()) { |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
264 |
// This is not the last chunk in the used region. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
265 |
// What is our last block? We check the first block of |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
266 |
// the next (right) chunk rather than strictly check our last block |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
267 |
// because it's potentially more efficient to do so. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
268 |
HeapWord* const last_block = sp->block_start(chunk_mr.end()); |
1 | 269 |
assert(last_block <= chunk_mr.end(), "In case this property changes."); |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
270 |
if ((last_block == chunk_mr.end()) // our last block does not straddle boundary |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
271 |
|| !sp->block_is_obj(last_block) // last_block isn't an object |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
272 |
|| oop(last_block)->is_objArray() // last_block is an array (precisely marked) |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
273 |
|| oop(last_block)->is_typeArray()) { |
1 | 274 |
max_to_do = chunk_mr.end(); |
275 |
} else { |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
276 |
assert(last_block < chunk_mr.end(), "Tautology"); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
277 |
// It is a non-array object that straddles the right boundary of this chunk. |
1 | 278 |
// last_obj_card is the card corresponding to the start of the last object |
279 |
// in the chunk. Note that the last object may not start in |
|
280 |
// the chunk. |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
281 |
jbyte* const last_obj_card = byte_for(last_block); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
282 |
const jbyte val = *last_obj_card; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
283 |
if (!card_will_be_scanned(val)) { |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
284 |
assert(!card_may_have_been_dirty(val), "Error"); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
285 |
// The card containing the head is not dirty. Any marks on |
1 | 286 |
// subsequent cards still in this chunk must have been made |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
287 |
// precisely; we can cap processing at the end of our chunk. |
1 | 288 |
max_to_do = chunk_mr.end(); |
289 |
} else { |
|
290 |
// The last object must be considered dirty, and extends onto the |
|
291 |
// following chunk. Look for a dirty card in that chunk that will |
|
292 |
// bound our processing. |
|
293 |
jbyte* limit_card = NULL; |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
294 |
const size_t last_block_size = sp->block_size(last_block); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
295 |
jbyte* const last_card_of_last_obj = |
1 | 296 |
byte_for(last_block + last_block_size - 1); |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
297 |
jbyte* const first_card_of_next_chunk = byte_for(chunk_mr.end()); |
1 | 298 |
// This search potentially goes a long distance looking |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
299 |
// for the next card that will be scanned, terminating |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
300 |
// at the end of the last_block, if no earlier dirty card |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
301 |
// is found. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
302 |
assert(byte_for(chunk_mr.end()) - byte_for(chunk_mr.start()) == ParGCCardsPerStrideChunk, |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
303 |
"last card of next chunk may be wrong"); |
1 | 304 |
for (jbyte* cur = first_card_of_next_chunk; |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
305 |
cur <= last_card_of_last_obj; cur++) { |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
306 |
const jbyte val = *cur; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
307 |
if (card_will_be_scanned(val)) { |
1 | 308 |
limit_card = cur; break; |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
309 |
} else { |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
310 |
assert(!card_may_have_been_dirty(val), "Error: card can't be skipped"); |
1 | 311 |
} |
312 |
} |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
313 |
if (limit_card != NULL) { |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
314 |
max_to_do = addr_for(limit_card); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
315 |
assert(limit_card != NULL && max_to_do != NULL, "Error"); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
316 |
} else { |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
317 |
// The following is a pessimistic value, because it's possible |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
318 |
// that a dirty card on a subsequent chunk has been cleared by |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
319 |
// the time we get to look at it; we'll correct for that further below, |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
320 |
// using the LNC array which records the least non-clean card |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
321 |
// before cards were cleared in a particular chunk. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
322 |
limit_card = last_card_of_last_obj; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
323 |
max_to_do = last_block + last_block_size; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
324 |
assert(limit_card != NULL && max_to_do != NULL, "Error"); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
325 |
} |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
326 |
assert(0 < cur_chunk_index+1 && cur_chunk_index+1 < lowest_non_clean_chunk_size, |
1 | 327 |
"Bounds error."); |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
328 |
// It is possible that a dirty card for the last object may have been |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
329 |
// cleared before we had a chance to examine it. In that case, the value |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
330 |
// will have been logged in the LNC for that chunk. |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
331 |
// We need to examine as many chunks to the right as this object |
9985
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
332 |
// covers. However, we need to bound this checking to the largest |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
333 |
// entry in the LNC array: this is because the heap may expand |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
334 |
// after the LNC array has been created but before we reach this point, |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
335 |
// and the last block in our chunk may have been expanded to include |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
336 |
// the expansion delta (and possibly subsequently allocated from, so |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
337 |
// it wouldn't be sufficient to check whether that last block was |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
338 |
// or was not an object at this point). |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
339 |
uintptr_t last_chunk_index_to_check = addr_to_chunk_index(last_block + last_block_size - 1) |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
340 |
- lowest_non_clean_base_chunk_index; |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
341 |
const uintptr_t last_chunk_index = addr_to_chunk_index(used.last()) |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
342 |
- lowest_non_clean_base_chunk_index; |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
343 |
if (last_chunk_index_to_check > last_chunk_index) { |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
344 |
assert(last_block + last_block_size > used.end(), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
345 |
"Inconsistency detected: last_block [" PTR_FORMAT "," PTR_FORMAT "]" |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
346 |
" does not exceed used.end() = " PTR_FORMAT "," |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
347 |
" yet last_chunk_index_to_check " INTPTR_FORMAT |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
348 |
" exceeds last_chunk_index " INTPTR_FORMAT, |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
349 |
p2i(last_block), p2i(last_block + last_block_size), |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
350 |
p2i(used.end()), |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
351 |
last_chunk_index_to_check, last_chunk_index); |
9985
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
352 |
assert(sp->used_region().end() > used.end(), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
353 |
"Expansion did not happen: " |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
354 |
"[" PTR_FORMAT "," PTR_FORMAT ") -> [" PTR_FORMAT "," PTR_FORMAT ")", |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
355 |
p2i(sp->used_region().start()), p2i(sp->used_region().end()), |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31964
diff
changeset
|
356 |
p2i(used.start()), p2i(used.end())); |
9985
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
357 |
last_chunk_index_to_check = last_chunk_index; |
afffb17bb75a
7048782: CMS: assert(last_chunk_index_to_check<= last_chunk_index) failed: parCardTableModRefBS.cpp:359
ysr
parents:
9626
diff
changeset
|
358 |
} |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
359 |
for (uintptr_t lnc_index = cur_chunk_index + 1; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
360 |
lnc_index <= last_chunk_index_to_check; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
361 |
lnc_index++) { |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
362 |
jbyte* lnc_card = lowest_non_clean[lnc_index]; |
1 | 363 |
if (lnc_card != NULL) { |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
364 |
// we can stop at the first non-NULL entry we find |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
365 |
if (lnc_card <= limit_card) { |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
366 |
limit_card = lnc_card; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
367 |
max_to_do = addr_for(limit_card); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
368 |
assert(limit_card != NULL && max_to_do != NULL, "Error"); |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
369 |
} |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
370 |
// In any case, we break now |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
371 |
break; |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
372 |
} // else continue to look for a non-NULL entry if any |
1 | 373 |
} |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
374 |
assert(limit_card != NULL && max_to_do != NULL, "Error"); |
1 | 375 |
} |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
376 |
assert(max_to_do != NULL, "OOPS 1 !"); |
1 | 377 |
} |
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
378 |
assert(max_to_do != NULL, "OOPS 2!"); |
1 | 379 |
} else { |
380 |
max_to_do = used.end(); |
|
381 |
} |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
382 |
assert(max_to_do != NULL, "OOPS 3!"); |
1 | 383 |
// Now we can set the closure we're using so it doesn't to beyond |
384 |
// max_to_do. |
|
385 |
dcto_cl->set_min_done(max_to_do); |
|
386 |
#ifndef PRODUCT |
|
387 |
dcto_cl->set_last_bottom(max_to_do); |
|
388 |
#endif |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
389 |
} |
1 | 390 |
|
391 |
void |
|
49733
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
392 |
CMSCardTable:: |
1 | 393 |
get_LNC_array_for_space(Space* sp, |
394 |
jbyte**& lowest_non_clean, |
|
395 |
uintptr_t& lowest_non_clean_base_chunk_index, |
|
396 |
size_t& lowest_non_clean_chunk_size) { |
|
397 |
||
398 |
int i = find_covering_region_containing(sp->bottom()); |
|
399 |
MemRegion covered = _covered[i]; |
|
400 |
size_t n_chunks = chunks_to_cover(covered); |
|
401 |
||
402 |
// Only the first thread to obtain the lock will resize the |
|
403 |
// LNC array for the covered region. Any later expansion can't affect |
|
404 |
// the used_at_save_marks region. |
|
405 |
// (I observed a bug in which the first thread to execute this would |
|
9624
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
406 |
// resize, and then it would cause "expand_and_allocate" that would |
c3657c3324ee
6883834: ParNew: assert(!_g->to()->is_in_reserved(obj),"Scanning field twice?") with LargeObjects tests
ysr
parents:
9336
diff
changeset
|
407 |
// increase the number of chunks in the covered region. Then a second |
1 | 408 |
// thread would come and execute this, see that the size didn't match, |
409 |
// and free and allocate again. So the first thread would be using a |
|
410 |
// freed "_lowest_non_clean" array.) |
|
411 |
||
412 |
// Do a dirty read here. If we pass the conditional then take the rare |
|
413 |
// event lock and do the read again in case some other thread had already |
|
414 |
// succeeded and done the resize. |
|
47622
817f2a7019e4
8179387: Factor out CMS specific code from GenCollectedHeap into its own subclass
rkennke
parents:
47216
diff
changeset
|
415 |
int cur_collection = CMSHeap::heap()->total_collections(); |
42632
c4f00b77b6d4
8170409: CMS: Crash in CardTableModRefBSForCTRS::process_chunk_boundaries
simonis
parents:
37055
diff
changeset
|
416 |
// Updated _last_LNC_resizing_collection[i] must not be visible before |
c4f00b77b6d4
8170409: CMS: Crash in CardTableModRefBSForCTRS::process_chunk_boundaries
simonis
parents:
37055
diff
changeset
|
417 |
// _lowest_non_clean and friends are visible. Therefore use acquire/release |
c4f00b77b6d4
8170409: CMS: Crash in CardTableModRefBSForCTRS::process_chunk_boundaries
simonis
parents:
37055
diff
changeset
|
418 |
// to guarantee this on non TSO architecures. |
c4f00b77b6d4
8170409: CMS: Crash in CardTableModRefBSForCTRS::process_chunk_boundaries
simonis
parents:
37055
diff
changeset
|
419 |
if (OrderAccess::load_acquire(&_last_LNC_resizing_collection[i]) != cur_collection) { |
1 | 420 |
MutexLocker x(ParGCRareEvent_lock); |
42632
c4f00b77b6d4
8170409: CMS: Crash in CardTableModRefBSForCTRS::process_chunk_boundaries
simonis
parents:
37055
diff
changeset
|
421 |
// This load_acquire is here for clarity only. The MutexLocker already fences. |
c4f00b77b6d4
8170409: CMS: Crash in CardTableModRefBSForCTRS::process_chunk_boundaries
simonis
parents:
37055
diff
changeset
|
422 |
if (OrderAccess::load_acquire(&_last_LNC_resizing_collection[i]) != cur_collection) { |
1 | 423 |
if (_lowest_non_clean[i] == NULL || |
424 |
n_chunks != _lowest_non_clean_chunk_size[i]) { |
|
425 |
||
426 |
// Should we delete the old? |
|
427 |
if (_lowest_non_clean[i] != NULL) { |
|
428 |
assert(n_chunks != _lowest_non_clean_chunk_size[i], |
|
429 |
"logical consequence"); |
|
27880
afb974a04396
8060074: os::free() takes MemoryTrackingLevel but doesn't need it
coleenp
parents:
24424
diff
changeset
|
430 |
FREE_C_HEAP_ARRAY(CardPtr, _lowest_non_clean[i]); |
1 | 431 |
_lowest_non_clean[i] = NULL; |
432 |
} |
|
433 |
// Now allocate a new one if necessary. |
|
434 |
if (_lowest_non_clean[i] == NULL) { |
|
13195 | 435 |
_lowest_non_clean[i] = NEW_C_HEAP_ARRAY(CardPtr, n_chunks, mtGC); |
1 | 436 |
_lowest_non_clean_chunk_size[i] = n_chunks; |
437 |
_lowest_non_clean_base_chunk_index[i] = addr_to_chunk_index(covered.start()); |
|
438 |
for (int j = 0; j < (int)n_chunks; j++) |
|
439 |
_lowest_non_clean[i][j] = NULL; |
|
440 |
} |
|
441 |
} |
|
42632
c4f00b77b6d4
8170409: CMS: Crash in CardTableModRefBSForCTRS::process_chunk_boundaries
simonis
parents:
37055
diff
changeset
|
442 |
// Make sure this gets visible only after _lowest_non_clean* was initialized |
c4f00b77b6d4
8170409: CMS: Crash in CardTableModRefBSForCTRS::process_chunk_boundaries
simonis
parents:
37055
diff
changeset
|
443 |
OrderAccess::release_store(&_last_LNC_resizing_collection[i], cur_collection); |
1 | 444 |
} |
445 |
} |
|
446 |
// In any case, now do the initialization. |
|
447 |
lowest_non_clean = _lowest_non_clean[i]; |
|
448 |
lowest_non_clean_base_chunk_index = _lowest_non_clean_base_chunk_index[i]; |
|
449 |
lowest_non_clean_chunk_size = _lowest_non_clean_chunk_size[i]; |
|
450 |
} |
|
49733
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
451 |
|
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
452 |
#ifdef ASSERT |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
453 |
void CMSCardTable::verify_used_region_at_save_marks(Space* sp) const { |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
454 |
MemRegion ur = sp->used_region(); |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
455 |
MemRegion urasm = sp->used_region_at_save_marks(); |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
456 |
|
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
457 |
if (!ur.contains(urasm)) { |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
458 |
log_warning(gc)("CMS+ParNew: Did you forget to call save_marks()? " |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
459 |
"[" PTR_FORMAT ", " PTR_FORMAT ") is not contained in " |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
460 |
"[" PTR_FORMAT ", " PTR_FORMAT ")", |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
461 |
p2i(urasm.start()), p2i(urasm.end()), p2i(ur.start()), p2i(ur.end())); |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
462 |
MemRegion ur2 = sp->used_region(); |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
463 |
MemRegion urasm2 = sp->used_region_at_save_marks(); |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
464 |
if (!ur.equals(ur2)) { |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
465 |
log_warning(gc)("CMS+ParNew: Flickering used_region()!!"); |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
466 |
} |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
467 |
if (!urasm.equals(urasm2)) { |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
468 |
log_warning(gc)("CMS+ParNew: Flickering used_region_at_save_marks()!!"); |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
469 |
} |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
470 |
ShouldNotReachHere(); |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
471 |
} |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
472 |
} |
6f0a3ea5ab75
8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents:
49488
diff
changeset
|
473 |
#endif // ASSERT |