author | tschatzl |
Thu, 15 Oct 2015 10:15:08 +0200 | |
changeset 33217 | 866a541f5504 |
parent 33105 | 294e48b4f704 |
child 33733 | d2d53d3de9cc |
permissions | -rw-r--r-- |
9994 | 1 |
/* |
29472 | 2 |
* Copyright (c) 2001, 2015, 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 |
||
30764 | 25 |
#ifndef SHARE_VM_GC_G1_CONCURRENTMARK_INLINE_HPP |
26 |
#define SHARE_VM_GC_G1_CONCURRENTMARK_INLINE_HPP |
|
9994 | 27 |
|
30764 | 28 |
#include "gc/g1/concurrentMark.hpp" |
29 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
|
30 |
#include "gc/shared/taskqueue.inline.hpp" |
|
9994 | 31 |
|
13919
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
32 |
// Utility routine to set an exclusive range of cards on the given |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
33 |
// card liveness bitmap |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
34 |
inline void ConcurrentMark::set_card_bitmap_range(BitMap* card_bm, |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
35 |
BitMap::idx_t start_idx, |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
36 |
BitMap::idx_t end_idx, |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
37 |
bool is_par) { |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
38 |
|
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
39 |
// Set the exclusive bit range [start_idx, end_idx). |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
40 |
assert((end_idx - start_idx) > 0, "at least one card"); |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
41 |
assert(end_idx <= card_bm->size(), "sanity"); |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
42 |
|
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
43 |
// Silently clip the end index |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
44 |
end_idx = MIN2(end_idx, card_bm->size()); |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
45 |
|
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
46 |
// For small ranges use a simple loop; otherwise use set_range or |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
47 |
// use par_at_put_range (if parallel). The range is made up of the |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
48 |
// cards that are spanned by an object/mem region so 8 cards will |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
49 |
// allow up to object sizes up to 4K to be handled using the loop. |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
50 |
if ((end_idx - start_idx) <= 8) { |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
51 |
for (BitMap::idx_t i = start_idx; i < end_idx; i += 1) { |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
52 |
if (is_par) { |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
53 |
card_bm->par_set_bit(i); |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
54 |
} else { |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
55 |
card_bm->set_bit(i); |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
56 |
} |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
57 |
} |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
58 |
} else { |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
59 |
// Note BitMap::par_at_put_range() and BitMap::set_range() are exclusive. |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
60 |
if (is_par) { |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
61 |
card_bm->par_at_put_range(start_idx, end_idx, true); |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
62 |
} else { |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
63 |
card_bm->set_range(start_idx, end_idx); |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
64 |
} |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
65 |
} |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
66 |
} |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
67 |
|
11583
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
68 |
// Returns the index in the liveness accounting card bitmap |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
69 |
// for the given address |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
70 |
inline BitMap::idx_t ConcurrentMark::card_bitmap_index_for(HeapWord* addr) { |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
71 |
// Below, the term "card num" means the result of shifting an address |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
72 |
// by the card shift -- address 0 corresponds to card number 0. One |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
73 |
// must subtract the card num of the bottom of the heap to obtain a |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
74 |
// card table index. |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
75 |
intptr_t card_num = intptr_t(uintptr_t(addr) >> CardTableModRefBS::card_shift); |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
76 |
return card_num - heap_bottom_card_num(); |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
77 |
} |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
78 |
|
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
79 |
// Counts the given memory region in the given task/worker |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
80 |
// counting data structures. |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
81 |
inline void ConcurrentMark::count_region(MemRegion mr, HeapRegion* hr, |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
82 |
size_t* marked_bytes_array, |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
83 |
BitMap* task_card_bm) { |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
84 |
G1CollectedHeap* g1h = _g1h; |
20309 | 85 |
CardTableModRefBS* ct_bs = g1h->g1_barrier_set(); |
13919
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
86 |
|
11583
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
87 |
HeapWord* start = mr.start(); |
13919
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
88 |
HeapWord* end = mr.end(); |
11583
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
89 |
size_t region_size_bytes = mr.byte_size(); |
26316
93f6b40c038b
8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents:
26160
diff
changeset
|
90 |
uint index = hr->hrm_index(); |
11583
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
91 |
|
26846
7d4376f8560e
8058495: G1: normalize names for isHumongous() and friends
tonyp
parents:
26831
diff
changeset
|
92 |
assert(!hr->is_continues_humongous(), "should not be HC region"); |
11583
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
93 |
assert(hr == g1h->heap_region_containing(start), "sanity"); |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
94 |
assert(hr == g1h->heap_region_containing(mr.last()), "sanity"); |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
95 |
assert(marked_bytes_array != NULL, "pre-condition"); |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
96 |
assert(task_card_bm != NULL, "pre-condition"); |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
97 |
|
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
98 |
// Add to the task local marked bytes for this region. |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
99 |
marked_bytes_array[index] += region_size_bytes; |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
100 |
|
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
101 |
BitMap::idx_t start_idx = card_bitmap_index_for(start); |
13919
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
102 |
BitMap::idx_t end_idx = card_bitmap_index_for(end); |
11583
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
103 |
|
13919
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
104 |
// Note: if we're looking at the last region in heap - end |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
105 |
// could be actually just beyond the end of the heap; end_idx |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
106 |
// will then correspond to a (non-existent) card that is also |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
107 |
// just beyond the heap. |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
108 |
if (g1h->is_in_g1_reserved(end) && !ct_bs->is_card_aligned(end)) { |
22551 | 109 |
// end of region is not card aligned - increment to cover |
13919
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
110 |
// all the cards spanned by the region. |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
111 |
end_idx += 1; |
11583
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
112 |
} |
13919
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
113 |
// The card bitmap is task/worker specific => no need to use |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
114 |
// the 'par' BitMap routines. |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
115 |
// Set bits in the exclusive bit range [start_idx, end_idx). |
01f6d01a9004
7200261: G1: Liveness counting inconsistencies during marking verification
johnc
parents:
12381
diff
changeset
|
116 |
set_card_bitmap_range(task_card_bm, start_idx, end_idx, false /* is_par */); |
11583
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
117 |
} |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
118 |
|
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
119 |
// Counts the given memory region in the task/worker counting |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
120 |
// data structures for the given worker id. |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
121 |
inline void ConcurrentMark::count_region(MemRegion mr, |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
122 |
HeapRegion* hr, |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
123 |
uint worker_id) { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
124 |
size_t* marked_bytes_array = count_marked_bytes_array_for(worker_id); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
125 |
BitMap* task_card_bm = count_card_bitmap_for(worker_id); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
126 |
count_region(mr, hr, marked_bytes_array, task_card_bm); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
127 |
} |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
128 |
|
11583
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
129 |
// Counts the given object in the given task/worker counting data structures. |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
130 |
inline void ConcurrentMark::count_object(oop obj, |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
131 |
HeapRegion* hr, |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
132 |
size_t* marked_bytes_array, |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
133 |
BitMap* task_card_bm) { |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
134 |
MemRegion mr((HeapWord*)obj, obj->size()); |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
135 |
count_region(mr, hr, marked_bytes_array, task_card_bm); |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
136 |
} |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
137 |
|
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
138 |
// Attempts to mark the given object and, if successful, counts |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
139 |
// the object in the given task/worker counting structures. |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
140 |
inline bool ConcurrentMark::par_mark_and_count(oop obj, |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
141 |
HeapRegion* hr, |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
142 |
size_t* marked_bytes_array, |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
143 |
BitMap* task_card_bm) { |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
144 |
HeapWord* addr = (HeapWord*)obj; |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
145 |
if (_nextMarkBitMap->parMark(addr)) { |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
146 |
// Update the task specific count data for the object. |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
147 |
count_object(obj, hr, marked_bytes_array, task_card_bm); |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
148 |
return true; |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
149 |
} |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
150 |
return false; |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
151 |
} |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
152 |
|
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
153 |
// Attempts to mark the given object and, if successful, counts |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
154 |
// the object in the task/worker counting structures for the |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
155 |
// given worker id. |
83a7383de44c
6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents:
11574
diff
changeset
|
156 |
inline bool ConcurrentMark::par_mark_and_count(oop obj, |
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
157 |
size_t word_size, |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
158 |
HeapRegion* hr, |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
159 |
uint worker_id) { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
160 |
HeapWord* addr = (HeapWord*)obj; |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
161 |
if (_nextMarkBitMap->parMark(addr)) { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
162 |
MemRegion mr(addr, word_size); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
163 |
count_region(mr, hr, worker_id); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
164 |
return true; |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
165 |
} |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
166 |
return false; |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
167 |
} |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
168 |
|
11574
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
169 |
inline bool CMBitMapRO::iterate(BitMapClosure* cl, MemRegion mr) { |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
170 |
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
|
171 |
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
|
172 |
|
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
173 |
if (end_addr > start_addr) { |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
174 |
// 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
|
175 |
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
|
176 |
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
|
177 |
|
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
178 |
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
|
179 |
while (start_offset < end_offset) { |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
180 |
if (!cl->do_bit(start_offset)) { |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
181 |
return false; |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
182 |
} |
15949
a55dea304e8c
8008079: G1: Add nextObject routine to CMBitMapRO and replace nextWord
tamao
parents:
14073
diff
changeset
|
183 |
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
|
184 |
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
|
185 |
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
|
186 |
} |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
187 |
} |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
188 |
return true; |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
189 |
} |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
190 |
|
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
191 |
inline bool CMBitMapRO::iterate(BitMapClosure* cl) { |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
192 |
MemRegion mr(startWord(), sizeInWords()); |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
193 |
return iterate(cl, mr); |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
194 |
} |
8a7fe61966c0
7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents:
11455
diff
changeset
|
195 |
|
26160 | 196 |
#define check_mark(addr) \ |
197 |
assert(_bmStartWord <= (addr) && (addr) < (_bmStartWord + _bmWordSize), \ |
|
198 |
"outside underlying space?"); \ |
|
199 |
assert(G1CollectedHeap::heap()->is_in_exact(addr), \ |
|
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
200 |
"Trying to access not available bitmap " PTR_FORMAT \ |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
201 |
" corresponding to " PTR_FORMAT " (%u)", \ |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
202 |
p2i(this), p2i(addr), G1CollectedHeap::heap()->addr_to_region(addr)); |
26160 | 203 |
|
204 |
inline void CMBitMap::mark(HeapWord* addr) { |
|
205 |
check_mark(addr); |
|
206 |
_bm.set_bit(heapWordToOffset(addr)); |
|
207 |
} |
|
208 |
||
209 |
inline void CMBitMap::clear(HeapWord* addr) { |
|
210 |
check_mark(addr); |
|
211 |
_bm.clear_bit(heapWordToOffset(addr)); |
|
212 |
} |
|
213 |
||
214 |
inline bool CMBitMap::parMark(HeapWord* addr) { |
|
215 |
check_mark(addr); |
|
216 |
return _bm.par_set_bit(heapWordToOffset(addr)); |
|
217 |
} |
|
218 |
||
219 |
inline bool CMBitMap::parClear(HeapWord* addr) { |
|
220 |
check_mark(addr); |
|
221 |
return _bm.par_clear_bit(heapWordToOffset(addr)); |
|
222 |
} |
|
223 |
||
224 |
#undef check_mark |
|
225 |
||
31994
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
226 |
template<typename Fn> |
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
227 |
inline void CMMarkStack::iterate(Fn fn) { |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
228 |
assert(_saved_index == _index, "saved index: %d index: %d", _saved_index, _index); |
31994
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
229 |
for (int i = 0; i < _index; ++i) { |
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
230 |
fn(_base[i]); |
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
231 |
} |
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
232 |
} |
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
31592
diff
changeset
|
233 |
|
32350
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
234 |
// It scans an object and visits its children. |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
235 |
inline void CMTask::scan_object(oop obj) { process_grey_object<true>(obj); } |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
236 |
|
9994 | 237 |
inline void CMTask::push(oop obj) { |
238 |
HeapWord* objAddr = (HeapWord*) obj; |
|
239 |
assert(_g1h->is_in_g1_reserved(objAddr), "invariant"); |
|
240 |
assert(!_g1h->is_on_master_free_list( |
|
241 |
_g1h->heap_region_containing((HeapWord*) objAddr)), "invariant"); |
|
242 |
assert(!_g1h->is_obj_ill(obj), "invariant"); |
|
243 |
assert(_nextMarkBitMap->isMarked(objAddr), "invariant"); |
|
244 |
||
245 |
if (_cm->verbose_high()) { |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22551
diff
changeset
|
246 |
gclog_or_tty->print_cr("[%u] pushing " PTR_FORMAT, _worker_id, p2i((void*) obj)); |
9994 | 247 |
} |
248 |
||
249 |
if (!_task_queue->push(obj)) { |
|
250 |
// The local task queue looks full. We need to push some entries |
|
251 |
// to the global stack. |
|
252 |
||
253 |
if (_cm->verbose_medium()) { |
|
14073
6012178c0c19
7127708: G1: change task num types from int to uint in concurrent mark
johnc
parents:
13919
diff
changeset
|
254 |
gclog_or_tty->print_cr("[%u] task queue overflow, " |
9994 | 255 |
"moving entries to the global stack", |
14073
6012178c0c19
7127708: G1: change task num types from int to uint in concurrent mark
johnc
parents:
13919
diff
changeset
|
256 |
_worker_id); |
9994 | 257 |
} |
258 |
move_entries_to_global_stack(); |
|
259 |
||
260 |
// this should succeed since, even if we overflow the global |
|
261 |
// stack, we should have definitely removed some entries from the |
|
262 |
// local queue. So, there must be space on it. |
|
263 |
bool success = _task_queue->push(obj); |
|
264 |
assert(success, "invariant"); |
|
265 |
} |
|
266 |
||
29472 | 267 |
statsOnly( size_t tmp_size = (size_t)_task_queue->size(); |
9999
5f0b78217054
7055073: G1: code cleanup in the concurrentMark.* files
tonyp
parents:
9994
diff
changeset
|
268 |
if (tmp_size > _local_max_size) { |
9994 | 269 |
_local_max_size = tmp_size; |
9999
5f0b78217054
7055073: G1: code cleanup in the concurrentMark.* files
tonyp
parents:
9994
diff
changeset
|
270 |
} |
9994 | 271 |
++_local_pushes ); |
272 |
} |
|
273 |
||
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
274 |
inline bool CMTask::is_below_finger(oop obj, HeapWord* global_finger) const { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
275 |
// If obj is above the global finger, then the mark bitmap scan |
30162 | 276 |
// 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
|
277 |
// a current region and obj is between the local finger and the |
30162 | 278 |
// end of the current region, then no push is needed. The tradeoff |
279 |
// of checking both vs only checking the global finger is that the |
|
280 |
// local check will be more accurate and so result in fewer pushes, |
|
281 |
// but may also be a little slower. |
|
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
282 |
HeapWord* objAddr = (HeapWord*)obj; |
30162 | 283 |
if (_finger != NULL) { |
284 |
// We have a current region. |
|
9994 | 285 |
|
30162 | 286 |
// Finger and region values are all NULL or all non-NULL. We |
287 |
// use _finger to check since we immediately use its value. |
|
288 |
assert(_curr_region != NULL, "invariant"); |
|
289 |
assert(_region_limit != NULL, "invariant"); |
|
290 |
assert(_region_limit <= global_finger, "invariant"); |
|
291 |
||
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
292 |
// True if obj is less than the local finger, or is between |
30162 | 293 |
// the region limit and the global finger. |
294 |
if (objAddr < _finger) { |
|
295 |
return true; |
|
296 |
} else if (objAddr < _region_limit) { |
|
297 |
return false; |
|
298 |
} // Else check global finger. |
|
299 |
} |
|
300 |
// Check global finger. |
|
301 |
return objAddr < global_finger; |
|
302 |
} |
|
9994 | 303 |
|
32350
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
304 |
template<bool scan> |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
305 |
inline void CMTask::process_grey_object(oop obj) { |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
306 |
assert(scan || obj->is_typeArray(), "Skipping scan of grey non-typeArray"); |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
307 |
assert(_nextMarkBitMap->isMarked((HeapWord*) obj), "invariant"); |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
308 |
|
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
309 |
if (_cm->verbose_high()) { |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
310 |
gclog_or_tty->print_cr("[%u] processing grey object " PTR_FORMAT, |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
311 |
_worker_id, p2i((void*) obj)); |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
312 |
} |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
313 |
|
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
314 |
size_t obj_size = obj->size(); |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
315 |
_words_scanned += obj_size; |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
316 |
|
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
317 |
if (scan) { |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
318 |
obj->oop_iterate(_cm_oop_closure); |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
319 |
} |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
320 |
statsOnly( ++_objs_scanned ); |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
321 |
check_limits(); |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
322 |
} |
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
323 |
|
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
324 |
|
fec57eb3f599
8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents:
31994
diff
changeset
|
325 |
|
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
326 |
inline void CMTask::make_reference_grey(oop obj, HeapRegion* hr) { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
327 |
if (_cm->par_mark_and_count(obj, hr, _marked_bytes_array, _card_bm)) { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
328 |
|
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
329 |
if (_cm->verbose_high()) { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
330 |
gclog_or_tty->print_cr("[%u] marked object " PTR_FORMAT, |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
331 |
_worker_id, p2i(obj)); |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
332 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
333 |
|
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
334 |
// No OrderAccess:store_load() is needed. It is implicit in the |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
335 |
// CAS done in CMBitMap::parMark() call in the routine above. |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
336 |
HeapWord* global_finger = _cm->finger(); |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
337 |
|
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
338 |
// 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
|
339 |
// 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
|
340 |
// scan has already examined. Mark bitmap scanning |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
341 |
// maintains progress "fingers" for determining that. |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
342 |
// |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
343 |
// Notice that the global finger might be moving forward |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
344 |
// 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
|
345 |
// 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
|
346 |
// 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
|
347 |
// past this object. In this case, the object will probably |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
348 |
// 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
|
349 |
// 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
|
350 |
// correctness problems. |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
351 |
if (is_below_finger(obj, global_finger)) { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
352 |
if (obj->is_typeArray()) { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
353 |
// Immediately process arrays of primitive types, rather |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
354 |
// than pushing on the mark stack. This keeps us from |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
355 |
// adding humongous objects to the mark stack that might |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
356 |
// be reclaimed before the entry is processed - see |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
357 |
// selection of candidates for eager reclaim of humongous |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
358 |
// objects. The cost of the additional type test is |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
359 |
// mitigated by avoiding a trip through the mark stack, |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
360 |
// by only doing a bookkeeping update and avoiding the |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
361 |
// actual scan of the object - a typeArray contains no |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
362 |
// references, and the metadata is built-in. |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
363 |
process_grey_object<false>(obj); |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
364 |
} else { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
365 |
if (_cm->verbose_high()) { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
366 |
gclog_or_tty->print_cr("[%u] below a finger (local: " PTR_FORMAT |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
367 |
", global: " PTR_FORMAT ") pushing " |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
368 |
PTR_FORMAT " on mark stack", |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
369 |
_worker_id, p2i(_finger), |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
370 |
p2i(global_finger), p2i(obj)); |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
371 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
372 |
push(obj); |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
373 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
374 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
375 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
376 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
377 |
|
9994 | 378 |
inline void CMTask::deal_with_reference(oop obj) { |
379 |
if (_cm->verbose_high()) { |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
30764
diff
changeset
|
380 |
gclog_or_tty->print_cr("[%u] we're dealing with reference = " PTR_FORMAT, |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22551
diff
changeset
|
381 |
_worker_id, p2i((void*) obj)); |
9994 | 382 |
} |
383 |
||
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
384 |
increment_refs_reached(); |
9994 | 385 |
|
386 |
HeapWord* objAddr = (HeapWord*) obj; |
|
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
387 |
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
|
388 |
if (_g1h->is_in_g1_reserved(objAddr)) { |
9994 | 389 |
assert(obj != NULL, "null check is implicit"); |
390 |
if (!_nextMarkBitMap->isMarked(objAddr)) { |
|
391 |
// Only get the containing region if the object is not marked on the |
|
392 |
// bitmap (otherwise, it's a waste of time since we won't do |
|
393 |
// anything with it). |
|
394 |
HeapRegion* hr = _g1h->heap_region_containing_raw(obj); |
|
395 |
if (!hr->obj_allocated_since_next_marking(obj)) { |
|
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30566
diff
changeset
|
396 |
make_reference_grey(obj, hr); |
9994 | 397 |
} |
398 |
} |
|
399 |
} |
|
400 |
} |
|
401 |
||
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
402 |
inline void ConcurrentMark::markPrev(oop p) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
403 |
assert(!_prevMarkBitMap->isMarked((HeapWord*) p), "sanity"); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
404 |
// 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
|
405 |
// the cast. |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
406 |
((CMBitMap*)_prevMarkBitMap)->mark((HeapWord*) p); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
407 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
408 |
|
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
409 |
inline void ConcurrentMark::grayRoot(oop obj, size_t word_size, |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
410 |
uint worker_id, HeapRegion* hr) { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
411 |
assert(obj != NULL, "pre-condition"); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
412 |
HeapWord* addr = (HeapWord*) obj; |
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
413 |
if (hr == NULL) { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
414 |
hr = _g1h->heap_region_containing_raw(addr); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
415 |
} else { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
416 |
assert(hr->is_in(addr), "pre-condition"); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
417 |
} |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
418 |
assert(hr != NULL, "sanity"); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
419 |
// 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
|
420 |
// 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
|
421 |
assert(!hr->is_continues_humongous(), "sanity"); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
422 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
423 |
// We cannot assert that word_size == obj->size() given that obj |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
424 |
// might not be in a consistent state (another thread might be in |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
425 |
// the process of copying it). So the best thing we can do is to |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
426 |
// assert that word_size is under an upper bound which is its |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
427 |
// containing region's capacity. |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
428 |
assert(word_size * HeapWordSize <= hr->capacity(), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
429 |
"size: " SIZE_FORMAT " capacity: " SIZE_FORMAT " " HR_FORMAT, |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
430 |
word_size * HeapWordSize, hr->capacity(), |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32350
diff
changeset
|
431 |
HR_FORMAT_PARAMS(hr)); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
432 |
|
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
433 |
if (addr < hr->next_top_at_mark_start()) { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
434 |
if (!_nextMarkBitMap->isMarked(addr)) { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
435 |
par_mark_and_count(obj, word_size, hr, worker_id); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11583
diff
changeset
|
436 |
} |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
437 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
438 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
9999
diff
changeset
|
439 |
|
30764 | 440 |
#endif // SHARE_VM_GC_G1_CONCURRENTMARK_INLINE_HPP |