author | kbarrett |
Wed, 04 Nov 2015 13:09:57 -0500 | |
changeset 33761 | 329db4b51480 |
parent 33105 | 294e48b4f704 |
child 33762 | 3d1dd03dfd3f |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
30255
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
2 |
* Copyright (c) 2001, 2015, Oracle and/or its affiliates. All rights reserved. |
1374 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5082
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5082
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5082
diff
changeset
|
21 |
* questions. |
1374 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
30764 | 26 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
27 |
#include "gc/g1/satbQueue.hpp" |
|
28 |
#include "gc/shared/collectedHeap.hpp" |
|
7397 | 29 |
#include "memory/allocation.inline.hpp" |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
30 |
#include "oops/oop.inline.hpp" |
7397 | 31 |
#include "runtime/mutexLocker.hpp" |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
32 |
#include "runtime/safepoint.hpp" |
7397 | 33 |
#include "runtime/thread.hpp" |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
7920
diff
changeset
|
34 |
#include "runtime/vmThread.hpp" |
1374 | 35 |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
36 |
ObjPtrQueue::ObjPtrQueue(SATBMarkQueueSet* qset, bool permanent) : |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
37 |
// SATB queues are only active during marking cycles. We create |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
38 |
// them with their active field set to false. If a thread is |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
39 |
// created during a cycle and its SATB queue needs to be activated |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
40 |
// before the thread starts running, we'll need to set its active |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
41 |
// field to true. This is done in JavaThread::initialize_queues(). |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
42 |
PtrQueue(qset, permanent, false /* active */) |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
43 |
{ } |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
44 |
|
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
45 |
void ObjPtrQueue::flush() { |
30255
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
46 |
// Filter now to possibly save work later. If filtering empties the |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
47 |
// buffer then flush_impl can deallocate the buffer. |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
48 |
filter(); |
28507 | 49 |
flush_impl(); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
50 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
51 |
|
30255
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
52 |
// Return true if a SATB buffer entry refers to an object that |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
53 |
// requires marking. |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
54 |
// |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
55 |
// The entry must point into the G1 heap. In particular, it must not |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
56 |
// be a NULL pointer. NULL pointers are pre-filtered and never |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
57 |
// inserted into a SATB buffer. |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
58 |
// |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
59 |
// An entry that is below the NTAMS pointer for the containing heap |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
60 |
// region requires marking. Such an entry must point to a valid object. |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
61 |
// |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
62 |
// An entry that is at least the NTAMS pointer for the containing heap |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
63 |
// region might be any of the following, none of which should be marked. |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
64 |
// |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
65 |
// * A reference to an object allocated since marking started. |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
66 |
// According to SATB, such objects are implicitly kept live and do |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
67 |
// not need to be dealt with via SATB buffer processing. |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
68 |
// |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
69 |
// * A reference to a young generation object. Young objects are |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
70 |
// handled separately and are not marked by concurrent marking. |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
71 |
// |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
72 |
// * A stale reference to a young generation object. If a young |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
73 |
// generation object reference is recorded and not filtered out |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
74 |
// before being moved by a young collection, the reference becomes |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
75 |
// stale. |
7920 | 76 |
// |
30255
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
77 |
// * A stale reference to an eagerly reclaimed humongous object. If a |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
78 |
// humongous object is recorded and then reclaimed, the reference |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
79 |
// becomes stale. |
7920 | 80 |
// |
30255
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
81 |
// The stale reference cases are implicitly handled by the NTAMS |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
82 |
// comparison. Because of the possibility of stale references, buffer |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
83 |
// processing must be somewhat circumspect and not assume entries |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
84 |
// in an unfiltered buffer refer to valid objects. |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
85 |
|
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
86 |
inline bool requires_marking(const void* entry, G1CollectedHeap* heap) { |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
87 |
// Includes rejection of NULL pointers. |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
88 |
assert(heap->is_in_reserved(entry), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
89 |
"Non-heap pointer in SATB buffer: " PTR_FORMAT, p2i(entry)); |
30255
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
90 |
|
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
91 |
HeapRegion* region = heap->heap_region_containing_raw(entry); |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
92 |
assert(region != NULL, "No region for " PTR_FORMAT, p2i(entry)); |
30255
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
93 |
if (entry >= region->next_top_at_mark_start()) { |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
94 |
return false; |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
95 |
} |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
96 |
|
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
97 |
assert(((oop)entry)->is_oop(true /* ignore mark word */), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
31592
diff
changeset
|
98 |
"Invalid oop in SATB buffer: " PTR_FORMAT, p2i(entry)); |
30255
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
99 |
|
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
100 |
return true; |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
101 |
} |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
102 |
|
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
103 |
// This method removes entries from a SATB buffer that will not be |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
104 |
// useful to the concurrent marking threads. Entries are retained if |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
105 |
// they require marking and are not already marked. Retained entries |
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
106 |
// are compacted toward the top of the buffer. |
7920 | 107 |
|
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
108 |
void ObjPtrQueue::filter() { |
7920 | 109 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
110 |
void** buf = _buf; |
|
111 |
||
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
112 |
if (buf == NULL) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
113 |
// nothing to do |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
114 |
return; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
115 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
116 |
|
7920 | 117 |
// Used for sanity checking at the end of the loop. |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
118 |
DEBUG_ONLY(size_t entries = 0; size_t retained = 0;) |
7920 | 119 |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
120 |
assert(_index <= _sz, "invariant"); |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
121 |
void** limit = &buf[byte_index_to_index(_index)]; |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
122 |
void** src = &buf[byte_index_to_index(_sz)]; |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
123 |
void** dst = src; |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
124 |
|
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
125 |
while (limit < src) { |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
126 |
DEBUG_ONLY(entries += 1;) |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
127 |
--src; |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
128 |
void* entry = *src; |
7920 | 129 |
// NULL the entry so that unused parts of the buffer contain NULLs |
130 |
// at the end. If we are going to retain it we will copy it to its |
|
131 |
// final place. If we have retained all entries we have visited so |
|
132 |
// far, we'll just end up copying it to the same place. |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
133 |
*src = NULL; |
7920 | 134 |
|
30255
f43e306ec51e
8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents:
30154
diff
changeset
|
135 |
if (requires_marking(entry, g1h) && !g1h->isMarkedNext((oop)entry)) { |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
136 |
--dst; |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
137 |
assert(*dst == NULL, "filtering destination should be clear"); |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
138 |
*dst = entry; |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
139 |
DEBUG_ONLY(retained += 1;); |
7920 | 140 |
} |
141 |
} |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
142 |
size_t new_index = pointer_delta(dst, buf, 1); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
143 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
144 |
#ifdef ASSERT |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
145 |
size_t entries_calc = (_sz - _index) / sizeof(void*); |
7920 | 146 |
assert(entries == entries_calc, "the number of entries we counted " |
147 |
"should match the number of entries we calculated"); |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
148 |
size_t retained_calc = (_sz - new_index) / sizeof(void*); |
7920 | 149 |
assert(retained == retained_calc, "the number of retained entries we counted " |
150 |
"should match the number of retained entries we calculated"); |
|
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
151 |
#endif // ASSERT |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
152 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
153 |
_index = new_index; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
154 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
155 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
156 |
// This method will first apply the above filtering to the buffer. If |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
157 |
// post-filtering a large enough chunk of the buffer has been cleared |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
158 |
// we can re-use the buffer (instead of enqueueing it) and we can just |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
159 |
// allow the mutator to carry on executing using the same buffer |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
160 |
// instead of replacing it. |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
161 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
162 |
bool ObjPtrQueue::should_enqueue_buffer() { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
163 |
assert(_lock == NULL || _lock->owned_by_self(), |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
164 |
"we should have taken the lock before calling this"); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
165 |
|
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
166 |
// If G1SATBBufferEnqueueingThresholdPercent == 0 we could skip filtering. |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
167 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
168 |
// This method should only be called if there is a non-NULL buffer |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
169 |
// that is full. |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
170 |
assert(_index == 0, "pre-condition"); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
171 |
assert(_buf != NULL, "pre-condition"); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
172 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
173 |
filter(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
174 |
|
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
175 |
size_t percent_used = ((_sz - _index) * 100) / _sz; |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
176 |
bool should_enqueue = percent_used > G1SATBBufferEnqueueingThresholdPercent; |
7920 | 177 |
return should_enqueue; |
178 |
} |
|
179 |
||
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
180 |
void ObjPtrQueue::apply_closure_and_empty(SATBBufferClosure* cl) { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
181 |
assert(SafepointSynchronize::is_at_safepoint(), |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
182 |
"SATB queues must only be processed at safepoints"); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
183 |
if (_buf != NULL) { |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
184 |
assert(_index % sizeof(void*) == 0, "invariant"); |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
185 |
assert(_sz % sizeof(void*) == 0, "invariant"); |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
186 |
assert(_index <= _sz, "invariant"); |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
187 |
cl->do_buffer(_buf + byte_index_to_index(_index), |
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
188 |
byte_index_to_index(_sz - _index)); |
1374 | 189 |
_index = _sz; |
190 |
} |
|
191 |
} |
|
192 |
||
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
193 |
#ifndef PRODUCT |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
194 |
// Helpful for debugging |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
195 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
196 |
void ObjPtrQueue::print(const char* name) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
197 |
print(name, _buf, _index, _sz); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
198 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
199 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
200 |
void ObjPtrQueue::print(const char* name, |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
201 |
void** buf, size_t index, size_t sz) { |
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
30764
diff
changeset
|
202 |
gclog_or_tty->print_cr(" SATB BUFFER [%s] buf: " PTR_FORMAT " " |
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
30764
diff
changeset
|
203 |
"index: " SIZE_FORMAT " sz: " SIZE_FORMAT, |
29796
7a04e5c250d1
8076054: g1: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
28507
diff
changeset
|
204 |
name, p2i(buf), index, sz); |
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
205 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
206 |
#endif // PRODUCT |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
207 |
|
1374 | 208 |
#ifdef _MSC_VER // the use of 'this' below gets a warning, make it go away |
209 |
#pragma warning( disable:4355 ) // 'this' : used in base member initializer list |
|
210 |
#endif // _MSC_VER |
|
211 |
||
212 |
SATBMarkQueueSet::SATBMarkQueueSet() : |
|
30268
d28a9aa10804
8078021: SATB apply_closure_to_completed_buffer should have closure argument
kbarrett
parents:
30255
diff
changeset
|
213 |
PtrQueueSet(), |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
214 |
_shared_satb_queue(this, true /* permanent */) { } |
1374 | 215 |
|
216 |
void SATBMarkQueueSet::initialize(Monitor* cbl_mon, Mutex* fl_lock, |
|
4481 | 217 |
int process_completed_threshold, |
1374 | 218 |
Mutex* lock) { |
4481 | 219 |
PtrQueueSet::initialize(cbl_mon, fl_lock, process_completed_threshold, -1); |
1374 | 220 |
_shared_satb_queue.set_lock(lock); |
221 |
} |
|
222 |
||
223 |
void SATBMarkQueueSet::handle_zero_index_for_thread(JavaThread* t) { |
|
224 |
t->satb_mark_queue().handle_zero_index(); |
|
225 |
} |
|
226 |
||
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
227 |
#ifdef ASSERT |
22497 | 228 |
void SATBMarkQueueSet::dump_active_states(bool expected_active) { |
229 |
gclog_or_tty->print_cr("Expected SATB active state: %s", |
|
230 |
expected_active ? "ACTIVE" : "INACTIVE"); |
|
231 |
gclog_or_tty->print_cr("Actual SATB active states:"); |
|
232 |
gclog_or_tty->print_cr(" Queue set: %s", is_active() ? "ACTIVE" : "INACTIVE"); |
|
233 |
for (JavaThread* t = Threads::first(); t; t = t->next()) { |
|
234 |
gclog_or_tty->print_cr(" Thread \"%s\" queue: %s", t->name(), |
|
235 |
t->satb_mark_queue().is_active() ? "ACTIVE" : "INACTIVE"); |
|
236 |
} |
|
237 |
gclog_or_tty->print_cr(" Shared queue: %s", |
|
238 |
shared_satb_queue()->is_active() ? "ACTIVE" : "INACTIVE"); |
|
239 |
} |
|
240 |
||
241 |
void SATBMarkQueueSet::verify_active_states(bool expected_active) { |
|
242 |
// Verify queue set state |
|
243 |
if (is_active() != expected_active) { |
|
244 |
dump_active_states(expected_active); |
|
245 |
guarantee(false, "SATB queue set has an unexpected active state"); |
|
246 |
} |
|
247 |
||
248 |
// Verify thread queue states |
|
249 |
for (JavaThread* t = Threads::first(); t; t = t->next()) { |
|
250 |
if (t->satb_mark_queue().is_active() != expected_active) { |
|
251 |
dump_active_states(expected_active); |
|
252 |
guarantee(false, "Thread SATB queue has an unexpected active state"); |
|
253 |
} |
|
254 |
} |
|
255 |
||
256 |
// Verify shared queue state |
|
257 |
if (shared_satb_queue()->is_active() != expected_active) { |
|
258 |
dump_active_states(expected_active); |
|
259 |
guarantee(false, "Shared SATB queue has an unexpected active state"); |
|
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
260 |
} |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
261 |
} |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
262 |
#endif // ASSERT |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
263 |
|
22497 | 264 |
void SATBMarkQueueSet::set_active_all_threads(bool active, bool expected_active) { |
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
265 |
assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint."); |
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
266 |
#ifdef ASSERT |
22497 | 267 |
verify_active_states(expected_active); |
5082
19e725a3d2eb
6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents:
4481
diff
changeset
|
268 |
#endif // ASSERT |
22497 | 269 |
_all_active = active; |
270 |
for (JavaThread* t = Threads::first(); t; t = t->next()) { |
|
271 |
t->satb_mark_queue().set_active(active); |
|
1374 | 272 |
} |
22497 | 273 |
shared_satb_queue()->set_active(active); |
1374 | 274 |
} |
275 |
||
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
276 |
void SATBMarkQueueSet::filter_thread_buffers() { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
277 |
for(JavaThread* t = Threads::first(); t; t = t->next()) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
278 |
t->satb_mark_queue().filter(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
279 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
280 |
shared_satb_queue()->filter(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
281 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
282 |
|
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
283 |
bool SATBMarkQueueSet::apply_closure_to_completed_buffer(SATBBufferClosure* cl) { |
4481 | 284 |
BufferNode* nd = NULL; |
1374 | 285 |
{ |
286 |
MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag); |
|
287 |
if (_completed_buffers_head != NULL) { |
|
288 |
nd = _completed_buffers_head; |
|
4481 | 289 |
_completed_buffers_head = nd->next(); |
1374 | 290 |
if (_completed_buffers_head == NULL) _completed_buffers_tail = NULL; |
291 |
_n_completed_buffers--; |
|
292 |
if (_n_completed_buffers == 0) _process_completed = false; |
|
293 |
} |
|
294 |
} |
|
295 |
if (nd != NULL) { |
|
4481 | 296 |
void **buf = BufferNode::make_buffer_from_node(nd); |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
297 |
// Skip over NULL entries at beginning (e.g. push end) of buffer. |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
298 |
// Filtering can result in non-full completed buffers; see |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
299 |
// should_enqueue_buffer. |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
300 |
assert(_sz % sizeof(void*) == 0, "invariant"); |
33761
329db4b51480
6899049: G1: Clean up code in ptrQueue.[ch]pp and ptrQueue.inline.hpp
kbarrett
parents:
33105
diff
changeset
|
301 |
size_t limit = ObjPtrQueue::byte_index_to_index(_sz); |
30577
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
302 |
for (size_t i = 0; i < limit; ++i) { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
303 |
if (buf[i] != NULL) { |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
304 |
// Found the end of the block of NULLs; process the remainder. |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
305 |
cl->do_buffer(buf + i, limit - i); |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
306 |
break; |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
307 |
} |
de9fa7ccc1bc
8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents:
30279
diff
changeset
|
308 |
} |
4481 | 309 |
deallocate_buffer(buf); |
1374 | 310 |
return true; |
311 |
} else { |
|
312 |
return false; |
|
313 |
} |
|
314 |
} |
|
315 |
||
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
316 |
#ifndef PRODUCT |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
317 |
// Helpful for debugging |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
318 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
319 |
#define SATB_PRINTER_BUFFER_SIZE 256 |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
320 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
321 |
void SATBMarkQueueSet::print_all(const char* msg) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
322 |
char buffer[SATB_PRINTER_BUFFER_SIZE]; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
323 |
assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint."); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
324 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
325 |
gclog_or_tty->cr(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
326 |
gclog_or_tty->print_cr("SATB BUFFERS [%s]", msg); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
327 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
328 |
BufferNode* nd = _completed_buffers_head; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
329 |
int i = 0; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
330 |
while (nd != NULL) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
331 |
void** buf = BufferNode::make_buffer_from_node(nd); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
332 |
jio_snprintf(buffer, SATB_PRINTER_BUFFER_SIZE, "Enqueued: %d", i); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
333 |
ObjPtrQueue::print(buffer, buf, 0, _sz); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
334 |
nd = nd->next(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
335 |
i += 1; |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
336 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
337 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
338 |
for (JavaThread* t = Threads::first(); t; t = t->next()) { |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
339 |
jio_snprintf(buffer, SATB_PRINTER_BUFFER_SIZE, "Thread: %s", t->name()); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
340 |
t->satb_mark_queue().print(buffer); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
341 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
342 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
343 |
shared_satb_queue()->print("Shared"); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
344 |
|
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
345 |
gclog_or_tty->cr(); |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
346 |
} |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
347 |
#endif // PRODUCT |
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
348 |
|
1374 | 349 |
void SATBMarkQueueSet::abandon_partial_marking() { |
4481 | 350 |
BufferNode* buffers_to_delete = NULL; |
1374 | 351 |
{ |
352 |
MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag); |
|
353 |
while (_completed_buffers_head != NULL) { |
|
4481 | 354 |
BufferNode* nd = _completed_buffers_head; |
355 |
_completed_buffers_head = nd->next(); |
|
356 |
nd->set_next(buffers_to_delete); |
|
1374 | 357 |
buffers_to_delete = nd; |
358 |
} |
|
359 |
_completed_buffers_tail = NULL; |
|
360 |
_n_completed_buffers = 0; |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
361 |
DEBUG_ONLY(assert_completed_buffer_list_len_correct_locked()); |
1374 | 362 |
} |
363 |
while (buffers_to_delete != NULL) { |
|
4481 | 364 |
BufferNode* nd = buffers_to_delete; |
365 |
buffers_to_delete = nd->next(); |
|
366 |
deallocate_buffer(BufferNode::make_buffer_from_node(nd)); |
|
1374 | 367 |
} |
368 |
assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint."); |
|
369 |
// So we can safely manipulate these queues. |
|
370 |
for (JavaThread* t = Threads::first(); t; t = t->next()) { |
|
371 |
t->satb_mark_queue().reset(); |
|
372 |
} |
|
11455
a6ab3d8b9a4c
6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents:
10670
diff
changeset
|
373 |
shared_satb_queue()->reset(); |
1374 | 374 |
} |