Mercurial > hg > graal-compiler
annotate src/share/vm/gc_implementation/g1/g1SATBCardTableModRefBS.cpp @ 21143:e13c65f874e5
Make G1 and Graal be friends in JDK9
| author | iveresov |
|---|---|
| date | Tue, 28 Apr 2015 17:08:47 -0700 |
| parents | 7848fc12602b |
| children |
| rev | line source |
|---|---|
| 342 | 1 /* |
|
12316
190899198332
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
11068
diff
changeset
|
2 * Copyright (c) 2001, 2013, Oracle and/or its affiliates. All rights reserved. |
| 342 | 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 * | |
|
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1317
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1317
diff
changeset
|
20 * or visit www.oracle.com if you need additional information or have any |
|
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1317
diff
changeset
|
21 * questions. |
| 342 | 22 * |
| 23 */ | |
| 24 | |
| 1972 | 25 #include "precompiled.hpp" |
| 20337 | 26 #include "gc_implementation/g1/g1CollectedHeap.inline.hpp" |
| 1972 | 27 #include "gc_implementation/g1/g1SATBCardTableModRefBS.hpp" |
| 28 #include "gc_implementation/g1/heapRegion.hpp" | |
| 29 #include "gc_implementation/g1/satbQueue.hpp" | |
| 30 #include "runtime/mutexLocker.hpp" | |
|
20197
ce8f6bb717c9
8042195: Introduce umbrella header orderAccess.inline.hpp.
goetz
parents:
17951
diff
changeset
|
31 #include "runtime/orderAccess.inline.hpp" |
|
7180
f34d701e952e
8003935: Simplify the needed includes for using Thread::current()
stefank
parents:
3960
diff
changeset
|
32 #include "runtime/thread.inline.hpp" |
| 342 | 33 |
| 34 G1SATBCardTableModRefBS::G1SATBCardTableModRefBS(MemRegion whole_heap, | |
| 35 int max_covered_regions) : | |
| 36 CardTableModRefBSForCTRS(whole_heap, max_covered_regions) | |
| 37 { | |
| 38 _kind = G1SATBCT; | |
| 39 } | |
| 40 | |
| 41 void G1SATBCardTableModRefBS::enqueue(oop pre_val) { | |
|
3249
e1162778c1c8
7009266: G1: assert(obj->is_oop_or_null(true )) failed: Error
johnc
parents:
1972
diff
changeset
|
42 // Nulls should have been already filtered. |
|
e1162778c1c8
7009266: G1: assert(obj->is_oop_or_null(true )) failed: Error
johnc
parents:
1972
diff
changeset
|
43 assert(pre_val->is_oop(true), "Error"); |
|
e1162778c1c8
7009266: G1: assert(obj->is_oop_or_null(true )) failed: Error
johnc
parents:
1972
diff
changeset
|
44 |
|
1317
d4197f8d516a
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
845
diff
changeset
|
45 if (!JavaThread::satb_mark_queue_set().is_active()) return; |
| 342 | 46 Thread* thr = Thread::current(); |
| 47 if (thr->is_Java_thread()) { | |
| 48 JavaThread* jt = (JavaThread*)thr; | |
| 49 jt->satb_mark_queue().enqueue(pre_val); | |
| 50 } else { | |
|
11068
b30744960351
8014022: G1: Non Java threads should lock the shared SATB queue lock without safepoint checks.
brutisso
parents:
7180
diff
changeset
|
51 MutexLockerEx x(Shared_SATB_Q_lock, Mutex::_no_safepoint_check_flag); |
| 342 | 52 JavaThread::satb_mark_queue_set().shared_satb_queue()->enqueue(pre_val); |
| 53 } | |
| 54 } | |
| 55 | |
|
845
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
56 template <class T> void |
|
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
57 G1SATBCardTableModRefBS::write_ref_array_pre_work(T* dst, int count) { |
|
1317
d4197f8d516a
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
845
diff
changeset
|
58 if (!JavaThread::satb_mark_queue_set().is_active()) return; |
|
845
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
59 T* elem_ptr = dst; |
|
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
60 for (int i = 0; i < count; i++, elem_ptr++) { |
|
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
61 T heap_oop = oopDesc::load_heap_oop(elem_ptr); |
|
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
62 if (!oopDesc::is_null(heap_oop)) { |
|
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
63 enqueue(oopDesc::decode_heap_oop_not_null(heap_oop)); |
|
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
64 } |
| 342 | 65 } |
| 66 } | |
| 67 | |
|
20275
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
68 void G1SATBCardTableModRefBS::write_ref_array_pre(oop* dst, int count, bool dest_uninitialized) { |
|
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
69 if (!dest_uninitialized) { |
|
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
70 write_ref_array_pre_work(dst, count); |
|
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
71 } |
|
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
72 } |
|
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
73 void G1SATBCardTableModRefBS::write_ref_array_pre(narrowOop* dst, int count, bool dest_uninitialized) { |
|
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
74 if (!dest_uninitialized) { |
|
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
75 write_ref_array_pre_work(dst, count); |
|
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
76 } |
|
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
77 } |
|
e5035defa3c4
8048214: Linker error when compiling G1SATBCardTableModRefBS after include order changes
mgerdin
parents:
20197
diff
changeset
|
78 |
| 12343 | 79 bool G1SATBCardTableModRefBS::mark_card_deferred(size_t card_index) { |
| 80 jbyte val = _byte_map[card_index]; | |
| 81 // It's already processed | |
| 82 if ((val & (clean_card_mask_val() | deferred_card_val())) == deferred_card_val()) { | |
| 83 return false; | |
| 84 } | |
|
12835
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
85 |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
86 if (val == g1_young_gen) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
87 // the card is for a young gen region. We don't need to keep track of all pointers into young |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
88 return false; |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
89 } |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
90 |
| 12343 | 91 // Cached bit can be installed either on a clean card or on a claimed card. |
| 92 jbyte new_val = val; | |
| 93 if (val == clean_card_val()) { | |
| 94 new_val = (jbyte)deferred_card_val(); | |
| 95 } else { | |
| 96 if (val & claimed_card_val()) { | |
| 97 new_val = val | (jbyte)deferred_card_val(); | |
| 98 } | |
| 99 } | |
| 100 if (new_val != val) { | |
| 101 Atomic::cmpxchg(new_val, &_byte_map[card_index], val); | |
| 102 } | |
| 103 return true; | |
| 104 } | |
| 105 | |
|
12835
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
106 void G1SATBCardTableModRefBS::g1_mark_as_young(const MemRegion& mr) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
107 jbyte *const first = byte_for(mr.start()); |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
108 jbyte *const last = byte_after(mr.last()); |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
109 |
| 17951 | 110 // Below we may use an explicit loop instead of memset() because on |
| 111 // certain platforms memset() can give concurrent readers phantom zeros. | |
| 112 if (UseMemSetInBOT) { | |
| 113 memset(first, g1_young_gen, last - first); | |
| 114 } else { | |
| 115 for (jbyte* i = first; i < last; i++) { | |
| 116 *i = g1_young_gen; | |
| 117 } | |
| 118 } | |
|
12835
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
119 } |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
120 |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
121 #ifndef PRODUCT |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
122 void G1SATBCardTableModRefBS::verify_g1_young_region(MemRegion mr) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
123 verify_region(mr, g1_young_gen, true); |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
124 } |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
125 #endif |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
126 |
| 20543 | 127 void G1SATBCardTableLoggingModRefBSChangedListener::on_commit(uint start_idx, size_t num_regions, bool zero_filled) { |
| 128 // Default value for a clean card on the card table is -1. So we cannot take advantage of the zero_filled parameter. | |
| 20337 | 129 MemRegion mr(G1CollectedHeap::heap()->bottom_addr_for_region(start_idx), num_regions * HeapRegion::GrainWords); |
| 130 _card_table->clear(mr); | |
| 131 } | |
| 132 | |
| 342 | 133 G1SATBCardTableLoggingModRefBS:: |
| 134 G1SATBCardTableLoggingModRefBS(MemRegion whole_heap, | |
| 135 int max_covered_regions) : | |
| 136 G1SATBCardTableModRefBS(whole_heap, max_covered_regions), | |
| 20337 | 137 _dcqs(JavaThread::dirty_card_queue_set()), |
| 138 _listener() | |
| 342 | 139 { |
| 140 _kind = G1SATBCTLogging; | |
| 20337 | 141 _listener.set_card_table(this); |
| 142 } | |
| 143 | |
| 144 void G1SATBCardTableLoggingModRefBS::initialize(G1RegionToSpaceMapper* mapper) { | |
| 145 mapper->set_mapping_changed_listener(&_listener); | |
| 146 | |
| 147 _byte_map_size = mapper->reserved().byte_size(); | |
| 148 | |
| 149 _guard_index = cards_required(_whole_heap.word_size()) - 1; | |
| 150 _last_valid_index = _guard_index - 1; | |
| 151 | |
| 152 HeapWord* low_bound = _whole_heap.start(); | |
| 153 HeapWord* high_bound = _whole_heap.end(); | |
| 154 | |
| 155 _cur_covered_regions = 1; | |
| 156 _covered[0] = _whole_heap; | |
| 157 | |
| 158 _byte_map = (jbyte*) mapper->reserved().start(); | |
| 159 byte_map_base = _byte_map - (uintptr_t(low_bound) >> card_shift); | |
| 160 assert(byte_for(low_bound) == &_byte_map[0], "Checking start of map"); | |
| 161 assert(byte_for(high_bound-1) <= &_byte_map[_last_valid_index], "Checking end of map"); | |
| 162 | |
| 163 if (TraceCardTableModRefBS) { | |
| 164 gclog_or_tty->print_cr("G1SATBCardTableModRefBS::G1SATBCardTableModRefBS: "); | |
| 165 gclog_or_tty->print_cr(" " | |
| 166 " &_byte_map[0]: " INTPTR_FORMAT | |
| 167 " &_byte_map[_last_valid_index]: " INTPTR_FORMAT, | |
| 168 p2i(&_byte_map[0]), | |
| 169 p2i(&_byte_map[_last_valid_index])); | |
| 170 gclog_or_tty->print_cr(" " | |
| 171 " byte_map_base: " INTPTR_FORMAT, | |
| 172 p2i(byte_map_base)); | |
| 173 } | |
| 342 | 174 } |
| 175 | |
| 176 void | |
| 177 G1SATBCardTableLoggingModRefBS::write_ref_field_work(void* field, | |
|
14443
3205e78d8193
8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents:
12840
diff
changeset
|
178 oop new_val, |
|
3205e78d8193
8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents:
12840
diff
changeset
|
179 bool release) { |
|
12835
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
180 volatile jbyte* byte = byte_for(field); |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
181 if (*byte == g1_young_gen) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
182 return; |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
183 } |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
184 OrderAccess::storeload(); |
| 342 | 185 if (*byte != dirty_card) { |
| 186 *byte = dirty_card; | |
| 187 Thread* thr = Thread::current(); | |
| 188 if (thr->is_Java_thread()) { | |
| 189 JavaThread* jt = (JavaThread*)thr; | |
| 190 jt->dirty_card_queue().enqueue(byte); | |
| 191 } else { | |
| 192 MutexLockerEx x(Shared_DirtyCardQ_lock, | |
| 193 Mutex::_no_safepoint_check_flag); | |
| 194 _dcqs.shared_dirty_card_queue()->enqueue(byte); | |
| 195 } | |
| 196 } | |
| 197 } | |
| 198 | |
| 199 void | |
| 200 G1SATBCardTableLoggingModRefBS::write_ref_field_static(void* field, | |
| 201 oop new_val) { | |
| 202 uintptr_t field_uint = (uintptr_t)field; | |
|
12316
190899198332
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
11068
diff
changeset
|
203 uintptr_t new_val_uint = cast_from_oop<uintptr_t>(new_val); |
| 342 | 204 uintptr_t comb = field_uint ^ new_val_uint; |
| 205 comb = comb >> HeapRegion::LogOfHRGrainBytes; | |
| 206 if (comb == 0) return; | |
| 207 if (new_val == NULL) return; | |
| 208 // Otherwise, log it. | |
| 209 G1SATBCardTableLoggingModRefBS* g1_bs = | |
| 210 (G1SATBCardTableLoggingModRefBS*)Universe::heap()->barrier_set(); | |
| 211 g1_bs->write_ref_field_work(field, new_val); | |
| 212 } | |
| 213 | |
| 214 void | |
| 215 G1SATBCardTableLoggingModRefBS::invalidate(MemRegion mr, bool whole_heap) { | |
|
12835
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
216 volatile jbyte* byte = byte_for(mr.start()); |
| 342 | 217 jbyte* last_byte = byte_for(mr.last()); |
| 218 Thread* thr = Thread::current(); | |
| 219 if (whole_heap) { | |
| 220 while (byte <= last_byte) { | |
| 221 *byte = dirty_card; | |
| 222 byte++; | |
| 223 } | |
| 224 } else { | |
|
12835
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
225 // skip all consecutive young cards |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
226 for (; byte <= last_byte && *byte == g1_young_gen; byte++); |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
227 |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
228 if (byte <= last_byte) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
229 OrderAccess::storeload(); |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
230 // Enqueue if necessary. |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
231 if (thr->is_Java_thread()) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
232 JavaThread* jt = (JavaThread*)thr; |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
233 for (; byte <= last_byte; byte++) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
234 if (*byte == g1_young_gen) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
235 continue; |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
236 } |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
237 if (*byte != dirty_card) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
238 *byte = dirty_card; |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
239 jt->dirty_card_queue().enqueue(byte); |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
240 } |
| 342 | 241 } |
|
12835
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
242 } else { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
243 MutexLockerEx x(Shared_DirtyCardQ_lock, |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
244 Mutex::_no_safepoint_check_flag); |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
245 for (; byte <= last_byte; byte++) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
246 if (*byte == g1_young_gen) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
247 continue; |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
248 } |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
249 if (*byte != dirty_card) { |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
250 *byte = dirty_card; |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
251 _dcqs.shared_dirty_card_queue()->enqueue(byte); |
|
69944b868a32
8014555: G1: Memory ordering problem with Conc refinement and card marking
mgerdin
parents:
12343
diff
changeset
|
252 } |
| 342 | 253 } |
| 254 } | |
| 255 } | |
| 256 } | |
| 257 } | |
| 21143 | 258 |
| 259 void G1SATBCardTableModRefBS::write_ref_nmethod_pre(oop* dst, nmethod* nm) { | |
| 260 oop obj = oopDesc::load_heap_oop(dst); | |
| 261 if (obj != NULL) { | |
| 262 G1CollectedHeap* g1h = (G1CollectedHeap*)Universe::heap(); | |
| 263 HeapRegion* hr = g1h->heap_region_containing(obj); | |
| 264 assert(!hr->continuesHumongous(), | |
| 265 err_msg("trying to add code root "INTPTR_FORMAT" in continuation of humongous region "HR_FORMAT | |
| 266 " starting at "HR_FORMAT, | |
| 267 (intptr_t)nm, HR_FORMAT_PARAMS(hr), HR_FORMAT_PARAMS(hr->humongous_start_region()))); | |
| 268 hr->add_strong_code_root(nm); | |
| 269 } | |
| 270 } | |
| 271 | |
| 272 class G1EnsureLastRefToRegion : public OopClosure { | |
| 273 G1CollectedHeap* _g1h; | |
| 274 HeapRegion* _hr; | |
| 275 oop* _dst; | |
| 276 | |
| 277 bool _value; | |
| 278 public: | |
| 279 G1EnsureLastRefToRegion(G1CollectedHeap* g1h, HeapRegion* hr, oop* dst) : | |
| 280 _g1h(g1h), _hr(hr), _dst(dst), _value(true) {} | |
| 281 | |
| 282 void do_oop(oop* p) { | |
| 283 if (_value && p != _dst) { | |
| 284 oop obj = oopDesc::load_heap_oop(p); | |
| 285 if (obj != NULL) { | |
| 286 HeapRegion* hr = _g1h->heap_region_containing(obj); | |
| 287 if (hr == _hr) { | |
| 288 // Another reference to the same region. | |
| 289 _value = false; | |
| 290 } | |
| 291 } | |
| 292 } | |
| 293 } | |
| 294 void do_oop(narrowOop* p) { ShouldNotReachHere(); } | |
| 295 bool value() const { return _value; } | |
| 296 }; | |
| 297 | |
| 298 void G1SATBCardTableModRefBS::write_ref_nmethod_post(oop* dst, nmethod* nm) { | |
| 299 oop obj = oopDesc::load_heap_oop(dst); | |
| 300 if (obj != NULL) { | |
| 301 G1CollectedHeap* g1h = (G1CollectedHeap*)Universe::heap(); | |
| 302 HeapRegion* hr = g1h->heap_region_containing(obj); | |
| 303 assert(!hr->continuesHumongous(), | |
| 304 err_msg("trying to remove code root "INTPTR_FORMAT" in continuation of humongous region "HR_FORMAT | |
| 305 " starting at "HR_FORMAT, | |
| 306 (intptr_t)nm, HR_FORMAT_PARAMS(hr), HR_FORMAT_PARAMS(hr->humongous_start_region()))); | |
| 307 G1EnsureLastRefToRegion ensure_last_ref(g1h, hr, dst); | |
| 308 nm->oops_do(&ensure_last_ref); | |
| 309 if (ensure_last_ref.value()) { | |
| 310 // Last reference to this region, remove the nmethod from the rset. | |
| 311 hr->remove_strong_code_root(nm); | |
| 312 } | |
| 313 } | |
| 314 } |