author | ecaspole |
Mon, 21 Sep 2015 10:36:36 -0400 | |
changeset 33093 | 5aa0f908f8cd |
parent 32612 | 79a64ccc4613 |
child 33160 | c59f1676d27e |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
28830 | 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:
5082
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5082
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:
5082
diff
changeset
|
21 |
* questions. |
1374 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
30764 | 26 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
27 |
#include "gc/g1/g1SATBCardTableModRefBS.hpp" |
|
28 |
#include "gc/g1/heapRegion.hpp" |
|
29 |
#include "gc/g1/satbQueue.hpp" |
|
32598
70b490faa49f
8131330: G1CollectedHeap::verify_dirty_young_list fails with assert
kbarrett
parents:
30764
diff
changeset
|
30 |
#include "gc/shared/memset_with_concurrent_readers.hpp" |
29081
c61eb4914428
8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents:
28830
diff
changeset
|
31 |
#include "oops/oop.inline.hpp" |
25351
7c198a690050
8044775: Improve usage of umbrella header atomic.inline.hpp.
goetz
parents:
24471
diff
changeset
|
32 |
#include "runtime/atomic.inline.hpp" |
7397 | 33 |
#include "runtime/mutexLocker.hpp" |
24351
61b33cc6d3cf
8042195: Introduce umbrella header orderAccess.inline.hpp.
goetz
parents:
22859
diff
changeset
|
34 |
#include "runtime/orderAccess.inline.hpp" |
14583
d70ee55535f4
8003935: Simplify the needed includes for using Thread::current()
stefank
parents:
10565
diff
changeset
|
35 |
#include "runtime/thread.inline.hpp" |
1374 | 36 |
|
29325 | 37 |
G1SATBCardTableModRefBS::G1SATBCardTableModRefBS( |
38 |
MemRegion whole_heap, |
|
39 |
const BarrierSet::FakeRtti& fake_rtti) : |
|
40 |
CardTableModRefBS(whole_heap, fake_rtti.add_tag(BarrierSet::G1SATBCT)) |
|
41 |
{ } |
|
1374 | 42 |
|
43 |
void G1SATBCardTableModRefBS::enqueue(oop pre_val) { |
|
9176
42d9d1010f38
7009266: G1: assert(obj->is_oop_or_null(true )) failed: Error
johnc
parents:
7397
diff
changeset
|
44 |
// Nulls should have been already filtered. |
42d9d1010f38
7009266: G1: assert(obj->is_oop_or_null(true )) failed: Error
johnc
parents:
7397
diff
changeset
|
45 |
assert(pre_val->is_oop(true), "Error"); |
42d9d1010f38
7009266: G1: assert(obj->is_oop_or_null(true )) failed: Error
johnc
parents:
7397
diff
changeset
|
46 |
|
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
3262
diff
changeset
|
47 |
if (!JavaThread::satb_mark_queue_set().is_active()) return; |
1374 | 48 |
Thread* thr = Thread::current(); |
49 |
if (thr->is_Java_thread()) { |
|
50 |
JavaThread* jt = (JavaThread*)thr; |
|
51 |
jt->satb_mark_queue().enqueue(pre_val); |
|
52 |
} else { |
|
18495
65a0d2ae4b22
8014022: G1: Non Java threads should lock the shared SATB queue lock without safepoint checks.
brutisso
parents:
14583
diff
changeset
|
53 |
MutexLockerEx x(Shared_SATB_Q_lock, Mutex::_no_safepoint_check_flag); |
1374 | 54 |
JavaThread::satb_mark_queue_set().shared_satb_queue()->enqueue(pre_val); |
55 |
} |
|
56 |
} |
|
57 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
58 |
template <class T> void |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
59 |
G1SATBCardTableModRefBS::write_ref_array_pre_work(T* dst, int count) { |
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
3262
diff
changeset
|
60 |
if (!JavaThread::satb_mark_queue_set().is_active()) return; |
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
61 |
T* elem_ptr = dst; |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
62 |
for (int i = 0; i < count; i++, elem_ptr++) { |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
63 |
T heap_oop = oopDesc::load_heap_oop(elem_ptr); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
64 |
if (!oopDesc::is_null(heap_oop)) { |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
65 |
enqueue(oopDesc::decode_heap_oop_not_null(heap_oop)); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
66 |
} |
1374 | 67 |
} |
68 |
} |
|
69 |
||
25480
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
70 |
void G1SATBCardTableModRefBS::write_ref_array_pre(oop* dst, int count, bool dest_uninitialized) { |
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
71 |
if (!dest_uninitialized) { |
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
72 |
write_ref_array_pre_work(dst, count); |
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
73 |
} |
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
74 |
} |
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
75 |
void G1SATBCardTableModRefBS::write_ref_array_pre(narrowOop* dst, int count, bool dest_uninitialized) { |
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
76 |
if (!dest_uninitialized) { |
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
77 |
write_ref_array_pre_work(dst, count); |
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
78 |
} |
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
79 |
} |
190facf4e732
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
25351
diff
changeset
|
80 |
|
20309 | 81 |
bool G1SATBCardTableModRefBS::mark_card_deferred(size_t card_index) { |
82 |
jbyte val = _byte_map[card_index]; |
|
83 |
// It's already processed |
|
84 |
if ((val & (clean_card_mask_val() | deferred_card_val())) == deferred_card_val()) { |
|
85 |
return false; |
|
86 |
} |
|
20403
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
87 |
|
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
88 |
if (val == g1_young_gen) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
89 |
// the card is for a young gen region. We don't need to keep track of all pointers into young |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
90 |
return false; |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
91 |
} |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
92 |
|
20309 | 93 |
// Cached bit can be installed either on a clean card or on a claimed card. |
94 |
jbyte new_val = val; |
|
95 |
if (val == clean_card_val()) { |
|
96 |
new_val = (jbyte)deferred_card_val(); |
|
97 |
} else { |
|
98 |
if (val & claimed_card_val()) { |
|
99 |
new_val = val | (jbyte)deferred_card_val(); |
|
100 |
} |
|
101 |
} |
|
102 |
if (new_val != val) { |
|
103 |
Atomic::cmpxchg(new_val, &_byte_map[card_index], val); |
|
104 |
} |
|
105 |
return true; |
|
106 |
} |
|
107 |
||
20403
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
108 |
void G1SATBCardTableModRefBS::g1_mark_as_young(const MemRegion& mr) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
109 |
jbyte *const first = byte_for(mr.start()); |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
110 |
jbyte *const last = byte_after(mr.last()); |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
111 |
|
32598
70b490faa49f
8131330: G1CollectedHeap::verify_dirty_young_list fails with assert
kbarrett
parents:
30764
diff
changeset
|
112 |
memset_with_concurrent_readers(first, g1_young_gen, last - first); |
20403
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
113 |
} |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
114 |
|
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
115 |
#ifndef PRODUCT |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
116 |
void G1SATBCardTableModRefBS::verify_g1_young_region(MemRegion mr) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
117 |
verify_region(mr, g1_young_gen, true); |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
118 |
} |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
119 |
#endif |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
120 |
|
27149 | 121 |
void G1SATBCardTableLoggingModRefBSChangedListener::on_commit(uint start_idx, size_t num_regions, bool zero_filled) { |
122 |
// Default value for a clean card on the card table is -1. So we cannot take advantage of the zero_filled parameter. |
|
26160 | 123 |
MemRegion mr(G1CollectedHeap::heap()->bottom_addr_for_region(start_idx), num_regions * HeapRegion::GrainWords); |
124 |
_card_table->clear(mr); |
|
125 |
} |
|
126 |
||
1374 | 127 |
G1SATBCardTableLoggingModRefBS:: |
27687
3a6367d7110b
8064721: The card tables only ever need two covering regions
ehelin
parents:
27149
diff
changeset
|
128 |
G1SATBCardTableLoggingModRefBS(MemRegion whole_heap) : |
29325 | 129 |
G1SATBCardTableModRefBS(whole_heap, BarrierSet::FakeRtti(G1SATBCTLogging)), |
26160 | 130 |
_dcqs(JavaThread::dirty_card_queue_set()), |
131 |
_listener() |
|
1374 | 132 |
{ |
26160 | 133 |
_listener.set_card_table(this); |
134 |
} |
|
135 |
||
136 |
void G1SATBCardTableLoggingModRefBS::initialize(G1RegionToSpaceMapper* mapper) { |
|
137 |
mapper->set_mapping_changed_listener(&_listener); |
|
138 |
||
139 |
_byte_map_size = mapper->reserved().byte_size(); |
|
140 |
||
141 |
_guard_index = cards_required(_whole_heap.word_size()) - 1; |
|
142 |
_last_valid_index = _guard_index - 1; |
|
143 |
||
144 |
HeapWord* low_bound = _whole_heap.start(); |
|
145 |
HeapWord* high_bound = _whole_heap.end(); |
|
146 |
||
147 |
_cur_covered_regions = 1; |
|
148 |
_covered[0] = _whole_heap; |
|
149 |
||
150 |
_byte_map = (jbyte*) mapper->reserved().start(); |
|
151 |
byte_map_base = _byte_map - (uintptr_t(low_bound) >> card_shift); |
|
152 |
assert(byte_for(low_bound) == &_byte_map[0], "Checking start of map"); |
|
153 |
assert(byte_for(high_bound-1) <= &_byte_map[_last_valid_index], "Checking end of map"); |
|
154 |
||
155 |
if (TraceCardTableModRefBS) { |
|
156 |
gclog_or_tty->print_cr("G1SATBCardTableModRefBS::G1SATBCardTableModRefBS: "); |
|
157 |
gclog_or_tty->print_cr(" " |
|
158 |
" &_byte_map[0]: " INTPTR_FORMAT |
|
159 |
" &_byte_map[_last_valid_index]: " INTPTR_FORMAT, |
|
160 |
p2i(&_byte_map[0]), |
|
161 |
p2i(&_byte_map[_last_valid_index])); |
|
162 |
gclog_or_tty->print_cr(" " |
|
163 |
" byte_map_base: " INTPTR_FORMAT, |
|
164 |
p2i(byte_map_base)); |
|
165 |
} |
|
1374 | 166 |
} |
167 |
||
168 |
void |
|
169 |
G1SATBCardTableLoggingModRefBS::write_ref_field_work(void* field, |
|
22859
7b88983393b7
8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents:
20408
diff
changeset
|
170 |
oop new_val, |
7b88983393b7
8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents:
20408
diff
changeset
|
171 |
bool release) { |
20403
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
172 |
volatile jbyte* byte = byte_for(field); |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
173 |
if (*byte == g1_young_gen) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
174 |
return; |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
175 |
} |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
176 |
OrderAccess::storeload(); |
1374 | 177 |
if (*byte != dirty_card) { |
178 |
*byte = dirty_card; |
|
179 |
Thread* thr = Thread::current(); |
|
180 |
if (thr->is_Java_thread()) { |
|
181 |
JavaThread* jt = (JavaThread*)thr; |
|
182 |
jt->dirty_card_queue().enqueue(byte); |
|
183 |
} else { |
|
184 |
MutexLockerEx x(Shared_DirtyCardQ_lock, |
|
185 |
Mutex::_no_safepoint_check_flag); |
|
186 |
_dcqs.shared_dirty_card_queue()->enqueue(byte); |
|
187 |
} |
|
188 |
} |
|
189 |
} |
|
190 |
||
191 |
void |
|
192 |
G1SATBCardTableLoggingModRefBS::write_ref_field_static(void* field, |
|
193 |
oop new_val) { |
|
194 |
uintptr_t field_uint = (uintptr_t)field; |
|
20282
7f9cbdf89af2
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
18495
diff
changeset
|
195 |
uintptr_t new_val_uint = cast_from_oop<uintptr_t>(new_val); |
1374 | 196 |
uintptr_t comb = field_uint ^ new_val_uint; |
197 |
comb = comb >> HeapRegion::LogOfHRGrainBytes; |
|
198 |
if (comb == 0) return; |
|
199 |
if (new_val == NULL) return; |
|
200 |
// Otherwise, log it. |
|
201 |
G1SATBCardTableLoggingModRefBS* g1_bs = |
|
30173
13cf7580b000
8077413: Avoid use of Universe::heap() inside collectors
pliden
parents:
29325
diff
changeset
|
202 |
barrier_set_cast<G1SATBCardTableLoggingModRefBS>(G1CollectedHeap::heap()->barrier_set()); |
32612 | 203 |
g1_bs->write_ref_field_work(field, new_val, false); |
1374 | 204 |
} |
205 |
||
206 |
void |
|
207 |
G1SATBCardTableLoggingModRefBS::invalidate(MemRegion mr, bool whole_heap) { |
|
20403
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
208 |
volatile jbyte* byte = byte_for(mr.start()); |
1374 | 209 |
jbyte* last_byte = byte_for(mr.last()); |
210 |
Thread* thr = Thread::current(); |
|
211 |
if (whole_heap) { |
|
212 |
while (byte <= last_byte) { |
|
213 |
*byte = dirty_card; |
|
214 |
byte++; |
|
215 |
} |
|
216 |
} else { |
|
20403
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
217 |
// skip all consecutive young cards |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
218 |
for (; byte <= last_byte && *byte == g1_young_gen; byte++); |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
219 |
|
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
220 |
if (byte <= last_byte) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
221 |
OrderAccess::storeload(); |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
222 |
// Enqueue if necessary. |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
223 |
if (thr->is_Java_thread()) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
224 |
JavaThread* jt = (JavaThread*)thr; |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
225 |
for (; byte <= last_byte; byte++) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
226 |
if (*byte == g1_young_gen) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
227 |
continue; |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
228 |
} |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
229 |
if (*byte != dirty_card) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
230 |
*byte = dirty_card; |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
231 |
jt->dirty_card_queue().enqueue(byte); |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
232 |
} |
1374 | 233 |
} |
20403
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
234 |
} else { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
235 |
MutexLockerEx x(Shared_DirtyCardQ_lock, |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
236 |
Mutex::_no_safepoint_check_flag); |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
237 |
for (; byte <= last_byte; byte++) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
238 |
if (*byte == g1_young_gen) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
239 |
continue; |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
240 |
} |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
241 |
if (*byte != dirty_card) { |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
242 |
*byte = dirty_card; |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
243 |
_dcqs.shared_dirty_card_queue()->enqueue(byte); |
45a89fbcd8f7
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
20309
diff
changeset
|
244 |
} |
1374 | 245 |
} |
246 |
} |
|
247 |
} |
|
248 |
} |
|
249 |
} |