annotate src/share/vm/gc_implementation/g1/satbQueue.cpp @ 23027:2e5e058881f4

8075466: SATB queue pre-filter verify found reclaimed humongous object Summary: Removed pre-filter verify, and made filtering more careful. Reviewed-by: brutisso, tschatzl
author kbarrett
date Wed, 15 Apr 2015 16:37:57 -0400
parents c2844108a708
children b5d14ef905b5
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
1 /*
23027
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
2 * Copyright (c) 2001, 2015, Oracle and/or its affiliates. All rights reserved.
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
4 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
7 * published by the Free Software Foundation.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
8 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
13 * accompanied this code).
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
14 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1317
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1317
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1317
diff changeset
21 * questions.
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
22 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
23 */
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
24
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
25 #include "precompiled.hpp"
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
26 #include "gc_implementation/g1/g1CollectedHeap.inline.hpp"
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
27 #include "gc_implementation/g1/satbQueue.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
28 #include "memory/allocation.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
29 #include "memory/sharedHeap.hpp"
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6197
diff changeset
30 #include "oops/oop.inline.hpp"
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
31 #include "runtime/mutexLocker.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
32 #include "runtime/thread.hpp"
3979
4dfb2df418f2 6484982: G1: process references during evacuation pauses
johnc
parents: 2149
diff changeset
33 #include "runtime/vmThread.hpp"
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
34
17937
78bbf4d43a14 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 17844
diff changeset
35 PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC
78bbf4d43a14 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 17844
diff changeset
36
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
37 void ObjPtrQueue::flush() {
23027
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
38 // Filter now to possibly save work later. If filtering empties the
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
39 // buffer then flush_impl can deallocate the buffer.
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
40 filter();
20731
42c091d63c72 8048949: Requeue queue implementation
tschatzl
parents: 17937
diff changeset
41 flush_impl();
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
42 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
43
23027
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
44 // Return true if a SATB buffer entry refers to an object that
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
45 // requires marking.
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
46 //
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
47 // The entry must point into the G1 heap. In particular, it must not
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
48 // be a NULL pointer. NULL pointers are pre-filtered and never
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
49 // inserted into a SATB buffer.
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
50 //
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
51 // An entry that is below the NTAMS pointer for the containing heap
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
52 // region requires marking. Such an entry must point to a valid object.
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
53 //
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
54 // An entry that is at least the NTAMS pointer for the containing heap
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
55 // region might be any of the following, none of which should be marked.
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
56 //
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
57 // * A reference to an object allocated since marking started.
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
58 // According to SATB, such objects are implicitly kept live and do
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
59 // not need to be dealt with via SATB buffer processing.
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
60 //
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
61 // * A reference to a young generation object. Young objects are
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
62 // handled separately and are not marked by concurrent marking.
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
63 //
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
64 // * A stale reference to a young generation object. If a young
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
65 // generation object reference is recorded and not filtered out
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
66 // before being moved by a young collection, the reference becomes
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
67 // stale.
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
68 //
23027
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
69 // * A stale reference to an eagerly reclaimed humongous object. If a
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
70 // humongous object is recorded and then reclaimed, the reference
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
71 // becomes stale.
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
72 //
23027
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
73 // The stale reference cases are implicitly handled by the NTAMS
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
74 // comparison. Because of the possibility of stale references, buffer
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
75 // processing must be somewhat circumspect and not assume entries
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
76 // in an unfiltered buffer refer to valid objects.
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
77
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
78 inline bool requires_marking(const void* entry, G1CollectedHeap* heap) {
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
79 // Includes rejection of NULL pointers.
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
80 assert(heap->is_in_reserved(entry),
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
81 err_msg("Non-heap pointer in SATB buffer: " PTR_FORMAT, p2i(entry)));
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
82
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
83 HeapRegion* region = heap->heap_region_containing_raw(entry);
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
84 assert(region != NULL, err_msg("No region for " PTR_FORMAT, p2i(entry)));
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
85 if (entry >= region->next_top_at_mark_start()) {
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
86 return false;
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
87 }
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
88
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
89 assert(((oop)entry)->is_oop(true /* ignore mark word */),
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
90 err_msg("Invalid oop in SATB buffer: " PTR_FORMAT, p2i(entry)));
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
91
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
92 return true;
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
93 }
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
94
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
95 // This method removes entries from a SATB buffer that will not be
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
96 // useful to the concurrent marking threads. Entries are retained if
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
97 // they require marking and are not already marked. Retained entries
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
98 // are compacted toward the top of the buffer.
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
99
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
100 void ObjPtrQueue::filter() {
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
101 G1CollectedHeap* g1h = G1CollectedHeap::heap();
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
102 void** buf = _buf;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
103 size_t sz = _sz;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
104
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
105 if (buf == NULL) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
106 // nothing to do
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
107 return;
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
108 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
109
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
110 // Used for sanity checking at the end of the loop.
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
111 debug_only(size_t entries = 0; size_t retained = 0;)
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
112
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
113 size_t i = sz;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
114 size_t new_index = sz;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
115
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
116 while (i > _index) {
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
117 assert(i > 0, "we should have at least one more entry to process");
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
118 i -= oopSize;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
119 debug_only(entries += 1;)
23027
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
120 void** p = &buf[byte_index_to_index((int) i)];
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
121 void* entry = *p;
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
122 // NULL the entry so that unused parts of the buffer contain NULLs
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
123 // at the end. If we are going to retain it we will copy it to its
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
124 // final place. If we have retained all entries we have visited so
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
125 // far, we'll just end up copying it to the same place.
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
126 *p = NULL;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
127
23027
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
128 if (requires_marking(entry, g1h) && !g1h->isMarkedNext((oop)entry)) {
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
129 assert(new_index > 0, "we should not have already filled up the buffer");
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
130 new_index -= oopSize;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
131 assert(new_index >= i,
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
132 "new_index should never be below i, as we alwaysr compact 'up'");
23027
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
133 void** new_p = &buf[byte_index_to_index((int) new_index)];
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
134 assert(new_p >= p, "the destination location should never be below "
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
135 "the source as we always compact 'up'");
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
136 assert(*new_p == NULL,
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
137 "we should have already cleared the destination location");
23027
2e5e058881f4 8075466: SATB queue pre-filter verify found reclaimed humongous object
kbarrett
parents: 20762
diff changeset
138 *new_p = entry;
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
139 debug_only(retained += 1;)
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
140 }
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
141 }
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
142
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
143 #ifdef ASSERT
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
144 size_t entries_calc = (sz - _index) / oopSize;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
145 assert(entries == entries_calc, "the number of entries we counted "
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
146 "should match the number of entries we calculated");
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
147 size_t retained_calc = (sz - new_index) / oopSize;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
148 assert(retained == retained_calc, "the number of retained entries we counted "
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
149 "should match the number of retained entries we calculated");
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
150 #endif // ASSERT
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
151
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
152 _index = new_index;
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
153 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
154
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
155 // This method will first apply the above filtering to the buffer. If
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
156 // post-filtering a large enough chunk of the buffer has been cleared
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
157 // we can re-use the buffer (instead of enqueueing it) and we can just
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
158 // allow the mutator to carry on executing using the same buffer
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
159 // instead of replacing it.
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
160
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
161 bool ObjPtrQueue::should_enqueue_buffer() {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
162 assert(_lock == NULL || _lock->owned_by_self(),
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
163 "we should have taken the lock before calling this");
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
164
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
165 // Even if G1SATBBufferEnqueueingThresholdPercent == 0 we have to
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
166 // filter the buffer given that this will remove any references into
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
167 // the CSet as we currently assume that no such refs will appear in
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
168 // enqueued buffers.
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
169
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
170 // This method should only be called if there is a non-NULL buffer
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
171 // that is full.
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
172 assert(_index == 0, "pre-condition");
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
173 assert(_buf != NULL, "pre-condition");
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
174
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
175 filter();
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
176
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
177 size_t sz = _sz;
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
178 size_t all_entries = sz / oopSize;
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
179 size_t retained_entries = (sz - _index) / oopSize;
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
180 size_t perc = retained_entries * 100 / all_entries;
2149
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
181 bool should_enqueue = perc > (size_t) G1SATBBufferEnqueueingThresholdPercent;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
182 return should_enqueue;
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
183 }
7e37af9d69ef 7011379: G1: overly long concurrent marking cycles
tonyp
parents: 1972
diff changeset
184
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
185 void ObjPtrQueue::apply_closure(ObjectClosure* cl) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
186 if (_buf != NULL) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
187 apply_closure_to_buffer(cl, _buf, _index, _sz);
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
188 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
189 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
190
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
191 void ObjPtrQueue::apply_closure_and_empty(ObjectClosure* cl) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
192 if (_buf != NULL) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
193 apply_closure_to_buffer(cl, _buf, _index, _sz);
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
194 _index = _sz;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
195 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
196 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
197
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
198 void ObjPtrQueue::apply_closure_to_buffer(ObjectClosure* cl,
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
199 void** buf, size_t index, size_t sz) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
200 if (cl == NULL) return;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
201 for (size_t i = index; i < sz; i += oopSize) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
202 oop obj = (oop)buf[byte_index_to_index((int)i)];
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
203 // There can be NULL entries because of destructors.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
204 if (obj != NULL) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
205 cl->do_object(obj);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
206 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
207 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
208 }
845
df6caf649ff7 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 342
diff changeset
209
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
210 #ifndef PRODUCT
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
211 // Helpful for debugging
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
212
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
213 void ObjPtrQueue::print(const char* name) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
214 print(name, _buf, _index, _sz);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
215 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
216
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
217 void ObjPtrQueue::print(const char* name,
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
218 void** buf, size_t index, size_t sz) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
219 gclog_or_tty->print_cr(" SATB BUFFER [%s] buf: "PTR_FORMAT" "
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
220 "index: "SIZE_FORMAT" sz: "SIZE_FORMAT,
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
221 name, buf, index, sz);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
222 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
223 #endif // PRODUCT
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
224
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
225 #ifdef _MSC_VER // the use of 'this' below gets a warning, make it go away
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
226 #pragma warning( disable:4355 ) // 'this' : used in base member initializer list
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
227 #endif // _MSC_VER
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
228
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
229 SATBMarkQueueSet::SATBMarkQueueSet() :
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
230 PtrQueueSet(), _closure(NULL), _par_closures(NULL),
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
231 _shared_satb_queue(this, true /*perm*/) { }
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
232
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
233 void SATBMarkQueueSet::initialize(Monitor* cbl_mon, Mutex* fl_lock,
1111
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
234 int process_completed_threshold,
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
235 Mutex* lock) {
1111
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
236 PtrQueueSet::initialize(cbl_mon, fl_lock, process_completed_threshold, -1);
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
237 _shared_satb_queue.set_lock(lock);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
238 if (ParallelGCThreads > 0) {
6197
d2a62e0f25eb 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 4787
diff changeset
239 _par_closures = NEW_C_HEAP_ARRAY(ObjectClosure*, ParallelGCThreads, mtGC);
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
240 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
241 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
242
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
243 void SATBMarkQueueSet::handle_zero_index_for_thread(JavaThread* t) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
244 t->satb_mark_queue().handle_zero_index();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
245 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
246
1317
d4197f8d516a 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 1111
diff changeset
247 #ifdef ASSERT
17747
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
248 void SATBMarkQueueSet::dump_active_states(bool expected_active) {
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
249 gclog_or_tty->print_cr("Expected SATB active state: %s",
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
250 expected_active ? "ACTIVE" : "INACTIVE");
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
251 gclog_or_tty->print_cr("Actual SATB active states:");
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
252 gclog_or_tty->print_cr(" Queue set: %s", is_active() ? "ACTIVE" : "INACTIVE");
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
253 for (JavaThread* t = Threads::first(); t; t = t->next()) {
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
254 gclog_or_tty->print_cr(" Thread \"%s\" queue: %s", t->name(),
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
255 t->satb_mark_queue().is_active() ? "ACTIVE" : "INACTIVE");
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
256 }
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
257 gclog_or_tty->print_cr(" Shared queue: %s",
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
258 shared_satb_queue()->is_active() ? "ACTIVE" : "INACTIVE");
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
259 }
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
260
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
261 void SATBMarkQueueSet::verify_active_states(bool expected_active) {
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
262 // Verify queue set state
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
263 if (is_active() != expected_active) {
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
264 dump_active_states(expected_active);
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
265 guarantee(false, "SATB queue set has an unexpected active state");
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
266 }
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
267
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
268 // Verify thread queue states
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
269 for (JavaThread* t = Threads::first(); t; t = t->next()) {
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
270 if (t->satb_mark_queue().is_active() != expected_active) {
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
271 dump_active_states(expected_active);
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
272 guarantee(false, "Thread SATB queue has an unexpected active state");
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
273 }
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
274 }
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
275
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
276 // Verify shared queue state
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
277 if (shared_satb_queue()->is_active() != expected_active) {
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
278 dump_active_states(expected_active);
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
279 guarantee(false, "Shared SATB queue has an unexpected active state");
1317
d4197f8d516a 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 1111
diff changeset
280 }
d4197f8d516a 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 1111
diff changeset
281 }
d4197f8d516a 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 1111
diff changeset
282 #endif // ASSERT
d4197f8d516a 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 1111
diff changeset
283
17747
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
284 void SATBMarkQueueSet::set_active_all_threads(bool active, bool expected_active) {
1317
d4197f8d516a 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 1111
diff changeset
285 assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint.");
d4197f8d516a 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 1111
diff changeset
286 #ifdef ASSERT
17747
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
287 verify_active_states(expected_active);
1317
d4197f8d516a 6935821: G1: threads created during marking do not active their SATB queues
tonyp
parents: 1111
diff changeset
288 #endif // ASSERT
17747
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
289 _all_active = active;
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
290 for (JavaThread* t = Threads::first(); t; t = t->next()) {
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
291 t->satb_mark_queue().set_active(active);
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
292 }
17747
f99e331f6ef6 8029162: G1: Shared SATB queue never enabled
pliden
parents: 6725
diff changeset
293 shared_satb_queue()->set_active(active);
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
294 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
295
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
296 void SATBMarkQueueSet::filter_thread_buffers() {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
297 for(JavaThread* t = Threads::first(); t; t = t->next()) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
298 t->satb_mark_queue().filter();
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
299 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
300 shared_satb_queue()->filter();
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
301 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
302
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
303 void SATBMarkQueueSet::set_closure(ObjectClosure* closure) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
304 _closure = closure;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
305 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
306
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
307 void SATBMarkQueueSet::set_par_closure(int i, ObjectClosure* par_closure) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
308 assert(ParallelGCThreads > 0 && _par_closures != NULL, "Precondition");
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
309 _par_closures[i] = par_closure;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
310 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
311
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
312 bool SATBMarkQueueSet::apply_closure_to_completed_buffer_work(bool par,
17844
8847586c9037 8016302: Change type of the number of GC workers to unsigned int (2)
vkempik
parents: 17747
diff changeset
313 uint worker) {
1111
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
314 BufferNode* nd = NULL;
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
315 {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
316 MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
317 if (_completed_buffers_head != NULL) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
318 nd = _completed_buffers_head;
1111
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
319 _completed_buffers_head = nd->next();
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
320 if (_completed_buffers_head == NULL) _completed_buffers_tail = NULL;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
321 _n_completed_buffers--;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
322 if (_n_completed_buffers == 0) _process_completed = false;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
323 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
324 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
325 ObjectClosure* cl = (par ? _par_closures[worker] : _closure);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
326 if (nd != NULL) {
1111
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
327 void **buf = BufferNode::make_buffer_from_node(nd);
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
328 ObjPtrQueue::apply_closure_to_buffer(cl, buf, 0, _sz);
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
329 deallocate_buffer(buf);
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
330 return true;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
331 } else {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
332 return false;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
333 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
334 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
335
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
336 void SATBMarkQueueSet::iterate_completed_buffers_read_only(ObjectClosure* cl) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
337 assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint.");
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
338 assert(cl != NULL, "pre-condition");
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
339
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
340 BufferNode* nd = _completed_buffers_head;
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
341 while (nd != NULL) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
342 void** buf = BufferNode::make_buffer_from_node(nd);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
343 ObjPtrQueue::apply_closure_to_buffer(cl, buf, 0, _sz);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
344 nd = nd->next();
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
345 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
346 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
347
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
348 void SATBMarkQueueSet::iterate_thread_buffers_read_only(ObjectClosure* cl) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
349 assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint.");
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
350 assert(cl != NULL, "pre-condition");
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
351
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
352 for (JavaThread* t = Threads::first(); t; t = t->next()) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
353 t->satb_mark_queue().apply_closure(cl);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
354 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
355 shared_satb_queue()->apply_closure(cl);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
356 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
357
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
358 #ifndef PRODUCT
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
359 // Helpful for debugging
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
360
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
361 #define SATB_PRINTER_BUFFER_SIZE 256
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
362
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
363 void SATBMarkQueueSet::print_all(const char* msg) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
364 char buffer[SATB_PRINTER_BUFFER_SIZE];
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
365 assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint.");
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
366
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
367 gclog_or_tty->cr();
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
368 gclog_or_tty->print_cr("SATB BUFFERS [%s]", msg);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
369
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
370 BufferNode* nd = _completed_buffers_head;
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
371 int i = 0;
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
372 while (nd != NULL) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
373 void** buf = BufferNode::make_buffer_from_node(nd);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
374 jio_snprintf(buffer, SATB_PRINTER_BUFFER_SIZE, "Enqueued: %d", i);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
375 ObjPtrQueue::print(buffer, buf, 0, _sz);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
376 nd = nd->next();
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
377 i += 1;
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
378 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
379
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
380 for (JavaThread* t = Threads::first(); t; t = t->next()) {
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
381 jio_snprintf(buffer, SATB_PRINTER_BUFFER_SIZE, "Thread: %s", t->name());
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
382 t->satb_mark_queue().print(buffer);
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
383 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
384
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
385 shared_satb_queue()->print("Shared");
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
386
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
387 gclog_or_tty->cr();
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
388 }
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
389 #endif // PRODUCT
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
390
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
391 void SATBMarkQueueSet::abandon_partial_marking() {
1111
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
392 BufferNode* buffers_to_delete = NULL;
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
393 {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
394 MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
395 while (_completed_buffers_head != NULL) {
1111
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
396 BufferNode* nd = _completed_buffers_head;
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
397 _completed_buffers_head = nd->next();
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
398 nd->set_next(buffers_to_delete);
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
399 buffers_to_delete = nd;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
400 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
401 _completed_buffers_tail = NULL;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
402 _n_completed_buffers = 0;
845
df6caf649ff7 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 342
diff changeset
403 DEBUG_ONLY(assert_completed_buffer_list_len_correct_locked());
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
404 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
405 while (buffers_to_delete != NULL) {
1111
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
406 BufferNode* nd = buffers_to_delete;
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
407 buffers_to_delete = nd->next();
44f61c24ddab 6862387: tune concurrent refinement further
iveresov
parents: 845
diff changeset
408 deallocate_buffer(BufferNode::make_buffer_from_node(nd));
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
409 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
410 assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint.");
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
411 // So we can safely manipulate these queues.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
412 for (JavaThread* t = Threads::first(); t; t = t->next()) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
413 t->satb_mark_queue().reset();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
414 }
4787
2ace1c4ee8da 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 3979
diff changeset
415 shared_satb_queue()->reset();
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
416 }