author | tschatzl |
Fri, 02 Jun 2017 13:48:01 +0200 | |
changeset 46520 | de5cb3eed39b |
parent 46502 | 116a09d8f142 |
child 46750 | bcab0128a16f |
permissions | -rw-r--r-- |
9994 | 1 |
/* |
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
2 |
* Copyright (c) 2001, 2017, Oracle and/or its affiliates. All rights reserved. |
9994 | 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 |
* |
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
20 |
* or visit www.oracle.com if you need additional information or have any |
|
21 |
* questions. |
|
22 |
* |
|
23 |
*/ |
|
24 |
||
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
25 |
#ifndef SHARE_VM_GC_G1_G1CONCURRENTMARK_INLINE_HPP |
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
26 |
#define SHARE_VM_GC_G1_G1CONCURRENTMARK_INLINE_HPP |
9994 | 27 |
|
30764 | 28 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
29 |
#include "gc/g1/g1ConcurrentMark.hpp" |
42597
a9611bab7578
8057003: Large reference arrays cause extremely long synchronization times
tschatzl
parents:
41176
diff
changeset
|
30 |
#include "gc/g1/g1ConcurrentMarkObjArrayProcessor.inline.hpp" |
37414
2672ba9af0dc
8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents:
37413
diff
changeset
|
31 |
#include "gc/g1/suspendibleThreadSet.hpp" |
30764 | 32 |
#include "gc/shared/taskqueue.inline.hpp" |
46502
116a09d8f142
8180755: Remove use of bitMap.inline.hpp include from instanceKlass.hpp and c1_ValueSet.hpp
tschatzl
parents:
46328
diff
changeset
|
33 |
#include "utilities/bitMap.inline.hpp" |
9994 | 34 |
|
37413
2f71679d06dd
8077144: Concurrent mark initialization takes too long
tschatzl
parents:
35943
diff
changeset
|
35 |
inline bool G1ConcurrentMark::par_mark(oop obj) { |
2f71679d06dd
8077144: Concurrent mark initialization takes too long
tschatzl
parents:
35943
diff
changeset
|
36 |
return _nextMarkBitMap->parMark((HeapWord*)obj); |
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
37 |
} |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
38 |
|
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
39 |
inline bool G1CMBitMapRO::iterate(BitMapClosure* cl, MemRegion mr) { |
11574
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
40 |
HeapWord* start_addr = MAX2(startWord(), mr.start()); |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
41 |
HeapWord* end_addr = MIN2(endWord(), mr.end()); |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
42 |
|
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
43 |
if (end_addr > start_addr) { |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
44 |
// Right-open interval [start-offset, end-offset). |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
45 |
BitMap::idx_t start_offset = heapWordToOffset(start_addr); |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
46 |
BitMap::idx_t end_offset = heapWordToOffset(end_addr); |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
47 |
|
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
48 |
start_offset = _bm.get_next_one_offset(start_offset, end_offset); |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
49 |
while (start_offset < end_offset) { |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
50 |
if (!cl->do_bit(start_offset)) { |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
51 |
return false; |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
52 |
} |
15949
a55dea304e8c
8008079: G1: Add nextObject routine to CMBitMapRO and replace nextWord
tamao
parents:
14073
diff
changeset
|
53 |
HeapWord* next_addr = MIN2(nextObject(offsetToHeapWord(start_offset)), end_addr); |
11574
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
54 |
BitMap::idx_t next_offset = heapWordToOffset(next_addr); |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
55 |
start_offset = _bm.get_next_one_offset(next_offset, end_offset); |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
56 |
} |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
57 |
} |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
58 |
return true; |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
59 |
} |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
60 |
|
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
61 |
// The argument addr should be the start address of a valid object |
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
62 |
HeapWord* G1CMBitMapRO::nextObject(HeapWord* addr) { |
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
63 |
oop obj = (oop) addr; |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
64 |
HeapWord* res = addr + obj->size(); |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
65 |
assert(offsetToHeapWord(heapWordToOffset(res)) == res, "sanity"); |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
66 |
return res; |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
67 |
} |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
68 |
|
26160 | 69 |
#define check_mark(addr) \ |
70 |
assert(_bmStartWord <= (addr) && (addr) < (_bmStartWord + _bmWordSize), \ |
|
71 |
"outside underlying space?"); \ |
|
72 |
assert(G1CollectedHeap::heap()->is_in_exact(addr), \ |
|
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
73 |
"Trying to access not available bitmap " PTR_FORMAT \ |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
74 |
" corresponding to " PTR_FORMAT " (%u)", \ |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
75 |
p2i(this), p2i(addr), G1CollectedHeap::heap()->addr_to_region(addr)); |
26160 | 76 |
|
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
77 |
inline void G1CMBitMap::mark(HeapWord* addr) { |
26160 | 78 |
check_mark(addr); |
79 |
_bm.set_bit(heapWordToOffset(addr)); |
|
80 |
} |
|
81 |
||
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
82 |
inline void G1CMBitMap::clear(HeapWord* addr) { |
26160 | 83 |
check_mark(addr); |
84 |
_bm.clear_bit(heapWordToOffset(addr)); |
|
85 |
} |
|
86 |
||
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
87 |
inline bool G1CMBitMap::parMark(HeapWord* addr) { |
26160 | 88 |
check_mark(addr); |
89 |
return _bm.par_set_bit(heapWordToOffset(addr)); |
|
90 |
} |
|
91 |
||
92 |
#undef check_mark |
|
93 |
||
41176
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
94 |
#ifndef PRODUCT |
31994
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
95 |
template<typename Fn> |
41176
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
96 |
inline void G1CMMarkStack::iterate(Fn fn) const { |
39966
43c7f35b800e
8160897: Concurrent mark mark stack memory allocation leaks memory
tschatzl
parents:
37414
diff
changeset
|
97 |
assert_at_safepoint(true); |
41176
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
98 |
|
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
99 |
size_t num_chunks = 0; |
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
100 |
|
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
101 |
TaskQueueEntryChunk* cur = _chunk_list; |
41176
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
102 |
while (cur != NULL) { |
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
103 |
guarantee(num_chunks <= _chunks_in_chunk_list, "Found " SIZE_FORMAT " oop chunks which is more than there should be", num_chunks); |
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
104 |
|
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
105 |
for (size_t i = 0; i < EntriesPerChunk; ++i) { |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
106 |
if (cur->data[i].is_null()) { |
41176
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
107 |
break; |
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
108 |
} |
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
109 |
fn(cur->data[i]); |
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
110 |
} |
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
111 |
cur = cur->next; |
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
112 |
num_chunks++; |
31994
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
113 |
} |
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
114 |
} |
41176
ff9f64534cff
8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents:
39966
diff
changeset
|
115 |
#endif |
31994
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
116 |
|
32350
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
117 |
// It scans an object and visits its children. |
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
118 |
inline void G1CMTask::scan_task_entry(G1TaskQueueEntry task_entry) { process_grey_task_entry<true>(task_entry); } |
32350
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
119 |
|
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
120 |
inline void G1CMTask::push(G1TaskQueueEntry task_entry) { |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
121 |
assert(task_entry.is_array_slice() || _g1h->is_in_g1_reserved(task_entry.obj()), "invariant"); |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
122 |
assert(task_entry.is_array_slice() || !_g1h->is_on_master_free_list( |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
123 |
_g1h->heap_region_containing(task_entry.obj())), "invariant"); |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
124 |
assert(task_entry.is_array_slice() || !_g1h->is_obj_ill(task_entry.obj()), "invariant"); // FIXME!!! |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
125 |
assert(task_entry.is_array_slice() || _nextMarkBitMap->isMarked((HeapWord*)task_entry.obj()), "invariant"); |
9994 | 126 |
|
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
127 |
if (!_task_queue->push(task_entry)) { |
9994 | 128 |
// The local task queue looks full. We need to push some entries |
129 |
// to the global stack. |
|
130 |
move_entries_to_global_stack(); |
|
131 |
||
132 |
// this should succeed since, even if we overflow the global |
|
133 |
// stack, we should have definitely removed some entries from the |
|
134 |
// local queue. So, there must be space on it. |
|
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
135 |
bool success = _task_queue->push(task_entry); |
9994 | 136 |
assert(success, "invariant"); |
137 |
} |
|
138 |
} |
|
139 |
||
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
140 |
inline bool G1CMTask::is_below_finger(oop obj, HeapWord* global_finger) const { |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
141 |
// If obj is above the global finger, then the mark bitmap scan |
30162 | 142 |
// will find it later, and no push is needed. Similarly, if we have |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
143 |
// a current region and obj is between the local finger and the |
30162 | 144 |
// end of the current region, then no push is needed. The tradeoff |
145 |
// of checking both vs only checking the global finger is that the |
|
146 |
// local check will be more accurate and so result in fewer pushes, |
|
147 |
// but may also be a little slower. |
|
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
148 |
HeapWord* objAddr = (HeapWord*)obj; |
30162 | 149 |
if (_finger != NULL) { |
150 |
// We have a current region. |
|
9994 | 151 |
|
30162 | 152 |
// Finger and region values are all NULL or all non-NULL. We |
153 |
// use _finger to check since we immediately use its value. |
|
154 |
assert(_curr_region != NULL, "invariant"); |
|
155 |
assert(_region_limit != NULL, "invariant"); |
|
156 |
assert(_region_limit <= global_finger, "invariant"); |
|
157 |
||
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
158 |
// True if obj is less than the local finger, or is between |
30162 | 159 |
// the region limit and the global finger. |
160 |
if (objAddr < _finger) { |
|
161 |
return true; |
|
162 |
} else if (objAddr < _region_limit) { |
|
163 |
return false; |
|
164 |
} // Else check global finger. |
|
165 |
} |
|
166 |
// Check global finger. |
|
167 |
return objAddr < global_finger; |
|
168 |
} |
|
9994 | 169 |
|
32350
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
170 |
template<bool scan> |
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
171 |
inline void G1CMTask::process_grey_task_entry(G1TaskQueueEntry task_entry) { |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
172 |
assert(scan || (task_entry.is_oop() && task_entry.obj()->is_typeArray()), "Skipping scan of grey non-typeArray"); |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
173 |
assert(task_entry.is_array_slice() || _nextMarkBitMap->isMarked((HeapWord*)task_entry.obj()), |
42597
a9611bab7578
8057003: Large reference arrays cause extremely long synchronization times
tschatzl
parents:
41176
diff
changeset
|
174 |
"Any stolen object should be a slice or marked"); |
32350
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
175 |
|
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
176 |
if (scan) { |
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
177 |
if (task_entry.is_array_slice()) { |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
178 |
_words_scanned += _objArray_processor.process_slice(task_entry.slice()); |
42597
a9611bab7578
8057003: Large reference arrays cause extremely long synchronization times
tschatzl
parents:
41176
diff
changeset
|
179 |
} else { |
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
180 |
oop obj = task_entry.obj(); |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
181 |
if (G1CMObjArrayProcessor::should_be_sliced(obj)) { |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
182 |
_words_scanned += _objArray_processor.process_obj(obj); |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
183 |
} else { |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
184 |
_words_scanned += obj->oop_iterate_size(_cm_oop_closure);; |
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
185 |
} |
42597
a9611bab7578
8057003: Large reference arrays cause extremely long synchronization times
tschatzl
parents:
41176
diff
changeset
|
186 |
} |
32350
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
187 |
} |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
188 |
check_limits(); |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
189 |
} |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
190 |
|
42597
a9611bab7578
8057003: Large reference arrays cause extremely long synchronization times
tschatzl
parents:
41176
diff
changeset
|
191 |
inline size_t G1CMTask::scan_objArray(objArrayOop obj, MemRegion mr) { |
a9611bab7578
8057003: Large reference arrays cause extremely long synchronization times
tschatzl
parents:
41176
diff
changeset
|
192 |
obj->oop_iterate(_cm_oop_closure, mr); |
a9611bab7578
8057003: Large reference arrays cause extremely long synchronization times
tschatzl
parents:
41176
diff
changeset
|
193 |
return mr.word_size(); |
a9611bab7578
8057003: Large reference arrays cause extremely long synchronization times
tschatzl
parents:
41176
diff
changeset
|
194 |
} |
a9611bab7578
8057003: Large reference arrays cause extremely long synchronization times
tschatzl
parents:
41176
diff
changeset
|
195 |
|
37413
2f71679d06dd
8077144: Concurrent mark initialization takes too long
tschatzl
parents:
35943
diff
changeset
|
196 |
inline void G1CMTask::make_reference_grey(oop obj) { |
2f71679d06dd
8077144: Concurrent mark initialization takes too long
tschatzl
parents:
35943
diff
changeset
|
197 |
if (_cm->par_mark(obj)) { |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
198 |
// No OrderAccess:store_load() is needed. It is implicit in the |
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
199 |
// CAS done in G1CMBitMap::parMark() call in the routine above. |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
200 |
HeapWord* global_finger = _cm->finger(); |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
201 |
|
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
202 |
// We only need to push a newly grey object on the mark |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
203 |
// stack if it is in a section of memory the mark bitmap |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
204 |
// scan has already examined. Mark bitmap scanning |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
205 |
// maintains progress "fingers" for determining that. |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
206 |
// |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
207 |
// Notice that the global finger might be moving forward |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
208 |
// concurrently. This is not a problem. In the worst case, we |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
209 |
// mark the object while it is above the global finger and, by |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
210 |
// the time we read the global finger, it has moved forward |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
211 |
// past this object. In this case, the object will probably |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
212 |
// be visited when a task is scanning the region and will also |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
213 |
// be pushed on the stack. So, some duplicate work, but no |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
214 |
// correctness problems. |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
215 |
if (is_below_finger(obj, global_finger)) { |
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
216 |
G1TaskQueueEntry entry = G1TaskQueueEntry::from_oop(obj); |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
217 |
if (obj->is_typeArray()) { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
218 |
// Immediately process arrays of primitive types, rather |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
219 |
// than pushing on the mark stack. This keeps us from |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
220 |
// adding humongous objects to the mark stack that might |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
221 |
// be reclaimed before the entry is processed - see |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
222 |
// selection of candidates for eager reclaim of humongous |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
223 |
// objects. The cost of the additional type test is |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
224 |
// mitigated by avoiding a trip through the mark stack, |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
225 |
// by only doing a bookkeeping update and avoiding the |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
226 |
// actual scan of the object - a typeArray contains no |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
227 |
// references, and the metadata is built-in. |
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
228 |
process_grey_task_entry<false>(entry); |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
229 |
} else { |
46328
6061df52d610
8168467: Use TaskEntry as task mark queue elements
tschatzl
parents:
42597
diff
changeset
|
230 |
push(entry); |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
231 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
232 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
233 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
234 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
235 |
|
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
236 |
inline void G1CMTask::deal_with_reference(oop obj) { |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
237 |
increment_refs_reached(); |
9994 | 238 |
|
239 |
HeapWord* objAddr = (HeapWord*) obj; |
|
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
240 |
assert(obj->is_oop_or_null(true /* ignore mark word */), "Expected an oop or NULL at " PTR_FORMAT, p2i(obj)); |
11583
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
241 |
if (_g1h->is_in_g1_reserved(objAddr)) { |
9994 | 242 |
assert(obj != NULL, "null check is implicit"); |
243 |
if (!_nextMarkBitMap->isMarked(objAddr)) { |
|
244 |
// Only get the containing region if the object is not marked on the |
|
245 |
// bitmap (otherwise, it's a waste of time since we won't do |
|
246 |
// anything with it). |
|
33786
ac8da6513351
8139867: Change how startsHumongous and continuesHumongous regions work in G1.
david
parents:
33740
diff
changeset
|
247 |
HeapRegion* hr = _g1h->heap_region_containing(obj); |
9994 | 248 |
if (!hr->obj_allocated_since_next_marking(obj)) { |
37413
2f71679d06dd
8077144: Concurrent mark initialization takes too long
tschatzl
parents:
35943
diff
changeset
|
249 |
make_reference_grey(obj); |
9994 | 250 |
} |
251 |
} |
|
252 |
} |
|
253 |
} |
|
254 |
||
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
255 |
inline void G1ConcurrentMark::markPrev(oop p) { |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
256 |
assert(!_prevMarkBitMap->isMarked((HeapWord*) p), "sanity"); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
257 |
// Note we are overriding the read-only view of the prev map here, via |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
258 |
// the cast. |
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
259 |
((G1CMBitMap*)_prevMarkBitMap)->mark((HeapWord*) p); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
260 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
261 |
|
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
262 |
bool G1ConcurrentMark::isPrevMarked(oop p) const { |
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
263 |
assert(p != NULL && p->is_oop(), "expected an oop"); |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
264 |
HeapWord* addr = (HeapWord*)p; |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
265 |
assert(addr >= _prevMarkBitMap->startWord() || |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
266 |
addr < _prevMarkBitMap->endWord(), "in a region"); |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
267 |
|
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
268 |
return _prevMarkBitMap->isMarked(addr); |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
269 |
} |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
34282
diff
changeset
|
270 |
|
37413
2f71679d06dd
8077144: Concurrent mark initialization takes too long
tschatzl
parents:
35943
diff
changeset
|
271 |
inline void G1ConcurrentMark::grayRoot(oop obj, HeapRegion* hr) { |
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
272 |
assert(obj != NULL, "pre-condition"); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
273 |
HeapWord* addr = (HeapWord*) obj; |
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
274 |
if (hr == NULL) { |
33786
ac8da6513351
8139867: Change how startsHumongous and continuesHumongous regions work in G1.
david
parents:
33740
diff
changeset
|
275 |
hr = _g1h->heap_region_containing(addr); |
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
276 |
} else { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
277 |
assert(hr->is_in(addr), "pre-condition"); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
278 |
} |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
279 |
assert(hr != NULL, "sanity"); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
280 |
// Given that we're looking for a region that contains an object |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
281 |
// header it's impossible to get back a HC region. |
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26831
diff
changeset
|
282 |
assert(!hr->is_continues_humongous(), "sanity"); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
283 |
|
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
284 |
if (addr < hr->next_top_at_mark_start()) { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
285 |
if (!_nextMarkBitMap->isMarked(addr)) { |
37413
2f71679d06dd
8077144: Concurrent mark initialization takes too long
tschatzl
parents:
35943
diff
changeset
|
286 |
par_mark(obj); |
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
287 |
} |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
288 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
289 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
290 |
|
37414
2672ba9af0dc
8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents:
37413
diff
changeset
|
291 |
inline bool G1ConcurrentMark::do_yield_check() { |
2672ba9af0dc
8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents:
37413
diff
changeset
|
292 |
if (SuspendibleThreadSet::should_yield()) { |
2672ba9af0dc
8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents:
37413
diff
changeset
|
293 |
SuspendibleThreadSet::yield(); |
2672ba9af0dc
8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents:
37413
diff
changeset
|
294 |
return true; |
2672ba9af0dc
8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents:
37413
diff
changeset
|
295 |
} else { |
2672ba9af0dc
8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents:
37413
diff
changeset
|
296 |
return false; |
2672ba9af0dc
8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents:
37413
diff
changeset
|
297 |
} |
2672ba9af0dc
8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents:
37413
diff
changeset
|
298 |
} |
2672ba9af0dc
8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents:
37413
diff
changeset
|
299 |
|
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
300 |
#endif // SHARE_VM_GC_G1_G1CONCURRENTMARK_INLINE_HPP |