author | zgu |
Tue, 29 Jan 2019 08:39:04 -0500 | |
changeset 53545 | 2c38991dd9b0 |
parent 53544 | aed190ed8549 |
child 53586 | 5bc1634bc0ca |
permissions | -rw-r--r-- |
1 | 1 |
/* |
53244
9807daeb47c4
8216167: Update include guards to reflect correct directories
coleenp
parents:
52905
diff
changeset
|
2 |
* Copyright (c) 2001, 2019, Oracle and/or its affiliates. All rights reserved. |
1 | 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:
5076
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5076
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:
5076
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
53244
9807daeb47c4
8216167: Update include guards to reflect correct directories
coleenp
parents:
52905
diff
changeset
|
25 |
#ifndef SHARE_GC_SHARED_TASKQUEUE_HPP |
9807daeb47c4
8216167: Update include guards to reflect correct directories
coleenp
parents:
52905
diff
changeset
|
26 |
#define SHARE_GC_SHARED_TASKQUEUE_HPP |
7397 | 27 |
|
28 |
#include "memory/allocation.hpp" |
|
51292
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
29 |
#include "memory/padded.hpp" |
48157 | 30 |
#include "oops/oopsHierarchy.hpp" |
31 |
#include "utilities/ostream.hpp" |
|
7397 | 32 |
#include "utilities/stack.hpp" |
33 |
||
6067 | 34 |
// Simple TaskQueue stats that are collected by default in debug builds. |
35 |
||
36 |
#if !defined(TASKQUEUE_STATS) && defined(ASSERT) |
|
37 |
#define TASKQUEUE_STATS 1 |
|
38 |
#elif !defined(TASKQUEUE_STATS) |
|
39 |
#define TASKQUEUE_STATS 0 |
|
40 |
#endif |
|
41 |
||
42 |
#if TASKQUEUE_STATS |
|
43 |
#define TASKQUEUE_STATS_ONLY(code) code |
|
44 |
#else |
|
45 |
#define TASKQUEUE_STATS_ONLY(code) |
|
46 |
#endif // TASKQUEUE_STATS |
|
47 |
||
48 |
#if TASKQUEUE_STATS |
|
49 |
class TaskQueueStats { |
|
50 |
public: |
|
51 |
enum StatId { |
|
52 |
push, // number of taskqueue pushes |
|
53 |
pop, // number of taskqueue pops |
|
54 |
pop_slow, // subset of taskqueue pops that were done slow-path |
|
55 |
steal_attempt, // number of taskqueue steal attempts |
|
56 |
steal, // number of taskqueue steals |
|
57 |
overflow, // number of overflow pushes |
|
58 |
overflow_max_len, // max length of overflow stack |
|
59 |
last_stat_id |
|
60 |
}; |
|
61 |
||
62 |
public: |
|
63 |
inline TaskQueueStats() { reset(); } |
|
64 |
||
50953
0fad17c646c9
8206453: Taskqueue stats should count real steal attempts, not calls to GenericTaskQueueSet::steal
tschatzl
parents:
49911
diff
changeset
|
65 |
inline void record_push() { ++_stats[push]; } |
0fad17c646c9
8206453: Taskqueue stats should count real steal attempts, not calls to GenericTaskQueueSet::steal
tschatzl
parents:
49911
diff
changeset
|
66 |
inline void record_pop() { ++_stats[pop]; } |
0fad17c646c9
8206453: Taskqueue stats should count real steal attempts, not calls to GenericTaskQueueSet::steal
tschatzl
parents:
49911
diff
changeset
|
67 |
inline void record_pop_slow() { record_pop(); ++_stats[pop_slow]; } |
0fad17c646c9
8206453: Taskqueue stats should count real steal attempts, not calls to GenericTaskQueueSet::steal
tschatzl
parents:
49911
diff
changeset
|
68 |
inline void record_steal_attempt() { ++_stats[steal_attempt]; } |
0fad17c646c9
8206453: Taskqueue stats should count real steal attempts, not calls to GenericTaskQueueSet::steal
tschatzl
parents:
49911
diff
changeset
|
69 |
inline void record_steal() { ++_stats[steal]; } |
6067 | 70 |
inline void record_overflow(size_t new_length); |
71 |
||
6251 | 72 |
TaskQueueStats & operator +=(const TaskQueueStats & addend); |
73 |
||
6067 | 74 |
inline size_t get(StatId id) const { return _stats[id]; } |
75 |
inline const size_t* get() const { return _stats; } |
|
76 |
||
77 |
inline void reset(); |
|
78 |
||
6251 | 79 |
// Print the specified line of the header (does not include a line separator). |
6067 | 80 |
static void print_header(unsigned int line, outputStream* const stream = tty, |
81 |
unsigned int width = 10); |
|
6251 | 82 |
// Print the statistics (does not include a line separator). |
6067 | 83 |
void print(outputStream* const stream = tty, unsigned int width = 10) const; |
84 |
||
6251 | 85 |
DEBUG_ONLY(void verify() const;) |
86 |
||
6067 | 87 |
private: |
88 |
size_t _stats[last_stat_id]; |
|
89 |
static const char * const _names[last_stat_id]; |
|
90 |
}; |
|
91 |
||
92 |
void TaskQueueStats::record_overflow(size_t new_len) { |
|
93 |
++_stats[overflow]; |
|
94 |
if (new_len > _stats[overflow_max_len]) _stats[overflow_max_len] = new_len; |
|
95 |
} |
|
96 |
||
97 |
void TaskQueueStats::reset() { |
|
98 |
memset(_stats, 0, sizeof(_stats)); |
|
99 |
} |
|
100 |
#endif // TASKQUEUE_STATS |
|
101 |
||
19346
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
102 |
// TaskQueueSuper collects functionality common to all GenericTaskQueue instances. |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
103 |
|
13195 | 104 |
template <unsigned int N, MEMFLAGS F> |
105 |
class TaskQueueSuper: public CHeapObj<F> { |
|
1 | 106 |
protected: |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
107 |
// Internal type for indexing the queue; also used for the tag. |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
108 |
typedef NOT_LP64(uint16_t) LP64_ONLY(uint32_t) idx_t; |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
109 |
|
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
110 |
// The first free element after the last one pushed (mod N). |
2005
42075507972b
6787254: Work queue capacity can be increased substantially on some platforms
ysr
parents:
1407
diff
changeset
|
111 |
volatile uint _bottom; |
1 | 112 |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
113 |
enum { MOD_N_MASK = N - 1 }; |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
114 |
|
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
115 |
class Age { |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
116 |
public: |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
117 |
Age(size_t data = 0) { _data = data; } |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
118 |
Age(const Age& age) { _data = age._data; } |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
119 |
Age(idx_t top, idx_t tag) { _fields._top = top; _fields._tag = tag; } |
1 | 120 |
|
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
121 |
Age get() const volatile { return _data; } |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
122 |
void set(Age age) volatile { _data = age._data; } |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
123 |
|
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
124 |
idx_t top() const volatile { return _fields._top; } |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
125 |
idx_t tag() const volatile { return _fields._tag; } |
1 | 126 |
|
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
127 |
// Increment top; if it wraps, increment tag also. |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
128 |
void increment() { |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
129 |
_fields._top = increment_index(_fields._top); |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
130 |
if (_fields._top == 0) ++_fields._tag; |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
131 |
} |
1 | 132 |
|
30566
18eb9aa972d0
8076177: Remove usage of stack.inline.hpp functions from taskqueue.hpp
stefank
parents:
27880
diff
changeset
|
133 |
Age cmpxchg(const Age new_age, const Age old_age) volatile; |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
134 |
|
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
135 |
bool operator ==(const Age& other) const { return _data == other._data; } |
1 | 136 |
|
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
137 |
private: |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
138 |
struct fields { |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
139 |
idx_t _top; |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
140 |
idx_t _tag; |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
141 |
}; |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
142 |
union { |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
143 |
size_t _data; |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
144 |
fields _fields; |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
145 |
}; |
1 | 146 |
}; |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
147 |
|
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
148 |
volatile Age _age; |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
149 |
|
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
150 |
// These both operate mod N. |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
151 |
static uint increment_index(uint ind) { |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
152 |
return (ind + 1) & MOD_N_MASK; |
1 | 153 |
} |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
154 |
static uint decrement_index(uint ind) { |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
155 |
return (ind - 1) & MOD_N_MASK; |
1 | 156 |
} |
157 |
||
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
158 |
// Returns a number in the range [0..N). If the result is "N-1", it should be |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
159 |
// interpreted as 0. |
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
160 |
uint dirty_size(uint bot, uint top) const { |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
161 |
return (bot - top) & MOD_N_MASK; |
1 | 162 |
} |
163 |
||
164 |
// Returns the size corresponding to the given "bot" and "top". |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
165 |
uint size(uint bot, uint top) const { |
2005
42075507972b
6787254: Work queue capacity can be increased substantially on some platforms
ysr
parents:
1407
diff
changeset
|
166 |
uint sz = dirty_size(bot, top); |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
167 |
// Has the queue "wrapped", so that bottom is less than top? There's a |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
168 |
// complicated special case here. A pair of threads could perform pop_local |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
169 |
// and pop_global operations concurrently, starting from a state in which |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
170 |
// _bottom == _top+1. The pop_local could succeed in decrementing _bottom, |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
171 |
// and the pop_global in incrementing _top (in which case the pop_global |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
172 |
// will be awarded the contested queue element.) The resulting state must |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
173 |
// be interpreted as an empty queue. (We only need to worry about one such |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
174 |
// event: only the queue owner performs pop_local's, and several concurrent |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
175 |
// threads attempting to perform the pop_global will all perform the same |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
176 |
// CAS, and only one can succeed.) Any stealing thread that reads after |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
177 |
// either the increment or decrement will see an empty queue, and will not |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
178 |
// join the competitors. The "sz == -1 || sz == N-1" state will not be |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
179 |
// modified by concurrent queues, so the owner thread can reset the state to |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
180 |
// _bottom == top so subsequent pushes will be performed normally. |
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
181 |
return (sz == N - 1) ? 0 : sz; |
1 | 182 |
} |
183 |
||
184 |
public: |
|
185 |
TaskQueueSuper() : _bottom(0), _age() {} |
|
186 |
||
5918 | 187 |
// Return true if the TaskQueue contains/does not contain any tasks. |
188 |
bool peek() const { return _bottom != _age.top(); } |
|
189 |
bool is_empty() const { return size() == 0; } |
|
1 | 190 |
|
191 |
// Return an estimate of the number of elements in the queue. |
|
192 |
// The "careful" version admits the possibility of pop_local/pop_global |
|
193 |
// races. |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
194 |
uint size() const { |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
195 |
return size(_bottom, _age.top()); |
1 | 196 |
} |
197 |
||
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
198 |
uint dirty_size() const { |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
199 |
return dirty_size(_bottom, _age.top()); |
1 | 200 |
} |
201 |
||
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
202 |
void set_empty() { |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
203 |
_bottom = 0; |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
204 |
_age.set(0); |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
205 |
} |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
206 |
|
1 | 207 |
// Maximum number of elements allowed in the queue. This is two less |
208 |
// than the actual queue size, for somewhat complicated reasons. |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
209 |
uint max_elems() const { return N - 2; } |
5035 | 210 |
|
211 |
// Total size of queue. |
|
212 |
static const uint total_size() { return N; } |
|
6067 | 213 |
|
214 |
TASKQUEUE_STATS_ONLY(TaskQueueStats stats;) |
|
1 | 215 |
}; |
216 |
||
19346
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
217 |
// |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
218 |
// GenericTaskQueue implements an ABP, Aurora-Blumofe-Plaxton, double- |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
219 |
// ended-queue (deque), intended for use in work stealing. Queue operations |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
220 |
// are non-blocking. |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
221 |
// |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
222 |
// A queue owner thread performs push() and pop_local() operations on one end |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
223 |
// of the queue, while other threads may steal work using the pop_global() |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
224 |
// method. |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
225 |
// |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
226 |
// The main difference to the original algorithm is that this |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
227 |
// implementation allows wrap-around at the end of its allocated |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
228 |
// storage, which is an array. |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
229 |
// |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
230 |
// The original paper is: |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
231 |
// |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
232 |
// Arora, N. S., Blumofe, R. D., and Plaxton, C. G. |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
233 |
// Thread scheduling for multiprogrammed multiprocessors. |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
234 |
// Theory of Computing Systems 34, 2 (2001), 115-144. |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
235 |
// |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
236 |
// The following paper provides an correctness proof and an |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
237 |
// implementation for weakly ordered memory models including (pseudo-) |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
238 |
// code containing memory barriers for a Chase-Lev deque. Chase-Lev is |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
239 |
// similar to ABP, with the main difference that it allows resizing of the |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
240 |
// underlying storage: |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
241 |
// |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
242 |
// Le, N. M., Pop, A., Cohen A., and Nardell, F. Z. |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
243 |
// Correct and efficient work-stealing for weak memory models |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
244 |
// Proceedings of the 18th ACM SIGPLAN symposium on Principles and |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
245 |
// practice of parallel programming (PPoPP 2013), 69-80 |
7a5310d39603
8022784: TaskQueue misses minimal documentation and references for analysis
tschatzl
parents:
19153
diff
changeset
|
246 |
// |
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
247 |
|
13195 | 248 |
template <class E, MEMFLAGS F, unsigned int N = TASKQUEUE_SIZE> |
249 |
class GenericTaskQueue: public TaskQueueSuper<N, F> { |
|
250 |
protected: |
|
251 |
typedef typename TaskQueueSuper<N, F>::Age Age; |
|
252 |
typedef typename TaskQueueSuper<N, F>::idx_t idx_t; |
|
253 |
||
254 |
using TaskQueueSuper<N, F>::_bottom; |
|
255 |
using TaskQueueSuper<N, F>::_age; |
|
256 |
using TaskQueueSuper<N, F>::increment_index; |
|
257 |
using TaskQueueSuper<N, F>::decrement_index; |
|
258 |
using TaskQueueSuper<N, F>::dirty_size; |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
259 |
|
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
260 |
public: |
13195 | 261 |
using TaskQueueSuper<N, F>::max_elems; |
262 |
using TaskQueueSuper<N, F>::size; |
|
263 |
||
264 |
#if TASKQUEUE_STATS |
|
265 |
using TaskQueueSuper<N, F>::stats; |
|
266 |
#endif |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
267 |
|
1 | 268 |
private: |
269 |
// Slow paths for push, pop_local. (pop_global has no fast path.) |
|
2005
42075507972b
6787254: Work queue capacity can be increased substantially on some platforms
ysr
parents:
1407
diff
changeset
|
270 |
bool push_slow(E t, uint dirty_n_elems); |
42075507972b
6787254: Work queue capacity can be increased substantially on some platforms
ysr
parents:
1407
diff
changeset
|
271 |
bool pop_local_slow(uint localBot, Age oldAge); |
1 | 272 |
|
273 |
public: |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
274 |
typedef E element_type; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
275 |
|
1 | 276 |
// Initializes the queue to empty. |
277 |
GenericTaskQueue(); |
|
278 |
||
279 |
void initialize(); |
|
280 |
||
5918 | 281 |
// Push the task "t" on the queue. Returns "false" iff the queue is full. |
1 | 282 |
inline bool push(E t); |
283 |
||
5918 | 284 |
// Attempts to claim a task from the "local" end of the queue (the most |
49911
358be4680d12
6672778: G1 should trim task queues more aggressively during evacuation pauses
tschatzl
parents:
48157
diff
changeset
|
285 |
// recently pushed) as long as the number of entries exceeds the threshold. |
358be4680d12
6672778: G1 should trim task queues more aggressively during evacuation pauses
tschatzl
parents:
48157
diff
changeset
|
286 |
// If successful, returns true and sets t to the task; otherwise, returns false |
358be4680d12
6672778: G1 should trim task queues more aggressively during evacuation pauses
tschatzl
parents:
48157
diff
changeset
|
287 |
// (the queue is empty or the number of elements below the threshold). |
358be4680d12
6672778: G1 should trim task queues more aggressively during evacuation pauses
tschatzl
parents:
48157
diff
changeset
|
288 |
inline bool pop_local(volatile E& t, uint threshold = 0); |
1 | 289 |
|
5918 | 290 |
// Like pop_local(), but uses the "global" end of the queue (the least |
291 |
// recently pushed). |
|
20282
7f9cbdf89af2
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
19346
diff
changeset
|
292 |
bool pop_global(volatile E& t); |
1 | 293 |
|
294 |
// Delete any resource associated with the queue. |
|
295 |
~GenericTaskQueue(); |
|
296 |
||
31994
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
30878
diff
changeset
|
297 |
// Apply fn to each element in the task queue. The queue must not |
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
30878
diff
changeset
|
298 |
// be modified while iterating. |
3721b7aa3a0d
8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents:
30878
diff
changeset
|
299 |
template<typename Fn> void iterate(Fn fn); |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
300 |
|
1 | 301 |
private: |
51292
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
302 |
DEFINE_PAD_MINUS_SIZE(0, DEFAULT_CACHE_LINE_SIZE, 0); |
1 | 303 |
// Element array. |
304 |
volatile E* _elems; |
|
51292
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
305 |
|
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
306 |
DEFINE_PAD_MINUS_SIZE(1, DEFAULT_CACHE_LINE_SIZE, sizeof(E*)); |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
307 |
// Queue owner local variables. Not to be accessed by other threads. |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
308 |
|
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
309 |
static const uint InvalidQueueId = uint(-1); |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
310 |
uint _last_stolen_queue_id; // The id of the queue we last stole from |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
311 |
|
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
312 |
int _seed; // Current random seed used for selecting a random queue during stealing. |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
313 |
|
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
314 |
DEFINE_PAD_MINUS_SIZE(2, DEFAULT_CACHE_LINE_SIZE, sizeof(uint) + sizeof(int)); |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
315 |
public: |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
316 |
int next_random_queue_id(); |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
317 |
|
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
318 |
void set_last_stolen_queue_id(uint id) { _last_stolen_queue_id = id; } |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
319 |
uint last_stolen_queue_id() const { return _last_stolen_queue_id; } |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
320 |
bool is_last_stolen_queue_id_valid() const { return _last_stolen_queue_id != InvalidQueueId; } |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
321 |
void invalidate_last_stolen_queue_id() { _last_stolen_queue_id = InvalidQueueId; } |
1 | 322 |
}; |
323 |
||
13195 | 324 |
template<class E, MEMFLAGS F, unsigned int N> |
51292
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
325 |
GenericTaskQueue<E, F, N>::GenericTaskQueue() : _last_stolen_queue_id(InvalidQueueId), _seed(17 /* random number */) { |
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
326 |
assert(sizeof(Age) == sizeof(size_t), "Depends on this."); |
1 | 327 |
} |
328 |
||
5918 | 329 |
// OverflowTaskQueue is a TaskQueue that also includes an overflow stack for |
330 |
// elements that do not fit in the TaskQueue. |
|
331 |
// |
|
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
6759
diff
changeset
|
332 |
// This class hides two methods from super classes: |
5918 | 333 |
// |
334 |
// push() - push onto the task queue or, if that fails, onto the overflow stack |
|
335 |
// is_empty() - return true if both the TaskQueue and overflow stack are empty |
|
336 |
// |
|
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
6759
diff
changeset
|
337 |
// Note that size() is not hidden--it returns the number of elements in the |
5918 | 338 |
// TaskQueue, and does not include the size of the overflow stack. This |
339 |
// simplifies replacement of GenericTaskQueues with OverflowTaskQueues. |
|
13195 | 340 |
template<class E, MEMFLAGS F, unsigned int N = TASKQUEUE_SIZE> |
341 |
class OverflowTaskQueue: public GenericTaskQueue<E, F, N> |
|
5918 | 342 |
{ |
343 |
public: |
|
13195 | 344 |
typedef Stack<E, F> overflow_t; |
345 |
typedef GenericTaskQueue<E, F, N> taskqueue_t; |
|
5918 | 346 |
|
6067 | 347 |
TASKQUEUE_STATS_ONLY(using taskqueue_t::stats;) |
348 |
||
5918 | 349 |
// Push task t onto the queue or onto the overflow stack. Return true. |
350 |
inline bool push(E t); |
|
39407
2e75eb109278
8152438: Threads may do significant work out of the non-shared overflow buffer
tschatzl
parents:
37057
diff
changeset
|
351 |
// Try to push task t onto the queue only. Returns true if successful, false otherwise. |
2e75eb109278
8152438: Threads may do significant work out of the non-shared overflow buffer
tschatzl
parents:
37057
diff
changeset
|
352 |
inline bool try_push_to_taskqueue(E t); |
5918 | 353 |
|
354 |
// Attempt to pop from the overflow stack; return true if anything was popped. |
|
355 |
inline bool pop_overflow(E& t); |
|
356 |
||
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
6759
diff
changeset
|
357 |
inline overflow_t* overflow_stack() { return &_overflow_stack; } |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
6759
diff
changeset
|
358 |
|
5918 | 359 |
inline bool taskqueue_empty() const { return taskqueue_t::is_empty(); } |
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
6759
diff
changeset
|
360 |
inline bool overflow_empty() const { return _overflow_stack.is_empty(); } |
5918 | 361 |
inline bool is_empty() const { |
362 |
return taskqueue_empty() && overflow_empty(); |
|
363 |
} |
|
364 |
||
365 |
private: |
|
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
6759
diff
changeset
|
366 |
overflow_t _overflow_stack; |
5918 | 367 |
}; |
368 |
||
13195 | 369 |
class TaskQueueSetSuper { |
1 | 370 |
public: |
371 |
// Returns "true" if some TaskQueue in the set contains a task. |
|
372 |
virtual bool peek() = 0; |
|
52122
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
373 |
// Tasks in queue |
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
374 |
virtual uint tasks() const = 0; |
1 | 375 |
}; |
376 |
||
13195 | 377 |
template <MEMFLAGS F> class TaskQueueSetSuperImpl: public CHeapObj<F>, public TaskQueueSetSuper { |
378 |
}; |
|
379 |
||
380 |
template<class T, MEMFLAGS F> |
|
381 |
class GenericTaskQueueSet: public TaskQueueSetSuperImpl<F> { |
|
50953
0fad17c646c9
8206453: Taskqueue stats should count real steal attempts, not calls to GenericTaskQueueSet::steal
tschatzl
parents:
49911
diff
changeset
|
382 |
public: |
0fad17c646c9
8206453: Taskqueue stats should count real steal attempts, not calls to GenericTaskQueueSet::steal
tschatzl
parents:
49911
diff
changeset
|
383 |
typedef typename T::element_type E; |
0fad17c646c9
8206453: Taskqueue stats should count real steal attempts, not calls to GenericTaskQueueSet::steal
tschatzl
parents:
49911
diff
changeset
|
384 |
|
1 | 385 |
private: |
2005
42075507972b
6787254: Work queue capacity can be increased substantially on some platforms
ysr
parents:
1407
diff
changeset
|
386 |
uint _n; |
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
387 |
T** _queues; |
1 | 388 |
|
51292
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
389 |
bool steal_best_of_2(uint queue_num, E& t); |
50953
0fad17c646c9
8206453: Taskqueue stats should count real steal attempts, not calls to GenericTaskQueueSet::steal
tschatzl
parents:
49911
diff
changeset
|
390 |
|
1 | 391 |
public: |
51292
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
392 |
GenericTaskQueueSet(uint n); |
47885
5caa1d5f74c1
8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents:
47216
diff
changeset
|
393 |
~GenericTaskQueueSet(); |
1 | 394 |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
395 |
void register_queue(uint i, T* q); |
1 | 396 |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
397 |
T* queue(uint n); |
1 | 398 |
|
51292
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
399 |
// Try to steal a task from some other queue than queue_num. It may perform several attempts at doing so. |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
400 |
// Returns if stealing succeeds, and sets "t" to the stolen task. |
0538a5cdb474
8205921: Optimizing best-of-2 work stealing queue selection
zgu
parents:
50953
diff
changeset
|
401 |
bool steal(uint queue_num, E& t); |
1 | 402 |
|
403 |
bool peek(); |
|
52122
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
404 |
uint tasks() const; |
30878
f1702744b3a4
8080877: Don't use workers()->total_workers() when walking G1CollectedHeap::_task_queues
stefank
parents:
30764
diff
changeset
|
405 |
|
f1702744b3a4
8080877: Don't use workers()->total_workers() when walking G1CollectedHeap::_task_queues
stefank
parents:
30764
diff
changeset
|
406 |
uint size() const { return _n; } |
1 | 407 |
}; |
408 |
||
13195 | 409 |
template<class T, MEMFLAGS F> void |
410 |
GenericTaskQueueSet<T, F>::register_queue(uint i, T* q) { |
|
2005
42075507972b
6787254: Work queue capacity can be increased substantially on some platforms
ysr
parents:
1407
diff
changeset
|
411 |
assert(i < _n, "index out of range."); |
1 | 412 |
_queues[i] = q; |
413 |
} |
|
414 |
||
13195 | 415 |
template<class T, MEMFLAGS F> T* |
416 |
GenericTaskQueueSet<T, F>::queue(uint i) { |
|
1 | 417 |
return _queues[i]; |
418 |
} |
|
419 |
||
13195 | 420 |
template<class T, MEMFLAGS F> |
421 |
bool GenericTaskQueueSet<T, F>::peek() { |
|
1 | 422 |
// Try all the queues. |
2005
42075507972b
6787254: Work queue capacity can be increased substantially on some platforms
ysr
parents:
1407
diff
changeset
|
423 |
for (uint j = 0; j < _n; j++) { |
1 | 424 |
if (_queues[j]->peek()) |
425 |
return true; |
|
426 |
} |
|
427 |
return false; |
|
428 |
} |
|
429 |
||
52122
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
430 |
template<class T, MEMFLAGS F> |
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
431 |
uint GenericTaskQueueSet<T, F>::tasks() const { |
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
432 |
uint n = 0; |
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
433 |
for (uint j = 0; j < _n; j++) { |
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
434 |
n += _queues[j]->size(); |
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
435 |
} |
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
436 |
return n; |
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
437 |
} |
cb20bf10cfbd
8212074: Add method to peek the remaining tasks in task queues
zgu
parents:
51292
diff
changeset
|
438 |
|
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
439 |
// When to terminate from the termination protocol. |
13195 | 440 |
class TerminatorTerminator: public CHeapObj<mtInternal> { |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
441 |
public: |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
442 |
virtual bool should_exit_termination() = 0; |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
443 |
}; |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
444 |
|
1 | 445 |
// A class to aid in the termination of a set of parallel tasks using |
446 |
// TaskQueueSet's for work stealing. |
|
447 |
||
2010
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
448 |
#undef TRACESPINNING |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
449 |
|
52905
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
450 |
class ParallelTaskTerminator: public CHeapObj<mtGC> { |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
451 |
protected: |
30585 | 452 |
uint _n_threads; |
1 | 453 |
TaskQueueSetSuper* _queue_set; |
53544
aed190ed8549
8217785: Padding ParallelTaskTerminator::_offered_termination variable
zgu
parents:
53244
diff
changeset
|
454 |
|
aed190ed8549
8217785: Padding ParallelTaskTerminator::_offered_termination variable
zgu
parents:
53244
diff
changeset
|
455 |
DEFINE_PAD_MINUS_SIZE(0, DEFAULT_CACHE_LINE_SIZE, 0); |
48140
ae61a5eb9da4
8188877: Improper synchronization in offer_termination
drwhite
parents:
47885
diff
changeset
|
456 |
volatile uint _offered_termination; |
53544
aed190ed8549
8217785: Padding ParallelTaskTerminator::_offered_termination variable
zgu
parents:
53244
diff
changeset
|
457 |
DEFINE_PAD_MINUS_SIZE(1, DEFAULT_CACHE_LINE_SIZE, sizeof(volatile uint)); |
1 | 458 |
|
2010
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
459 |
#ifdef TRACESPINNING |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
460 |
static uint _total_yields; |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
461 |
static uint _total_spins; |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
462 |
static uint _total_peeks; |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
463 |
#endif |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
464 |
|
1 | 465 |
bool peek_in_queue_set(); |
466 |
protected: |
|
467 |
virtual void yield(); |
|
468 |
void sleep(uint millis); |
|
469 |
||
53545
2c38991dd9b0
8215047: Task terminators do not complete termination in consistent state
zgu
parents:
53544
diff
changeset
|
470 |
// Called when exiting termination is requested. |
2c38991dd9b0
8215047: Task terminators do not complete termination in consistent state
zgu
parents:
53544
diff
changeset
|
471 |
// When the request is made, terminator may have already terminated |
2c38991dd9b0
8215047: Task terminators do not complete termination in consistent state
zgu
parents:
53544
diff
changeset
|
472 |
// (e.g. all threads are arrived and offered termination). In this case, |
2c38991dd9b0
8215047: Task terminators do not complete termination in consistent state
zgu
parents:
53544
diff
changeset
|
473 |
// it should ignore the request and complete the termination. |
2c38991dd9b0
8215047: Task terminators do not complete termination in consistent state
zgu
parents:
53544
diff
changeset
|
474 |
// Return true if termination is completed. Otherwise, return false. |
2c38991dd9b0
8215047: Task terminators do not complete termination in consistent state
zgu
parents:
53544
diff
changeset
|
475 |
bool complete_or_exit_termination(); |
1 | 476 |
public: |
477 |
||
478 |
// "n_threads" is the number of threads to be terminated. "queue_set" is a |
|
479 |
// queue sets of work queues of other threads. |
|
30585 | 480 |
ParallelTaskTerminator(uint n_threads, TaskQueueSetSuper* queue_set); |
53545
2c38991dd9b0
8215047: Task terminators do not complete termination in consistent state
zgu
parents:
53544
diff
changeset
|
481 |
virtual ~ParallelTaskTerminator(); |
1 | 482 |
|
483 |
// The current thread has no work, and is ready to terminate if everyone |
|
484 |
// else is. If returns "true", all threads are terminated. If returns |
|
485 |
// "false", available work has been observed in one of the task queues, |
|
486 |
// so the global task is not complete. |
|
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
487 |
bool offer_termination() { |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
488 |
return offer_termination(NULL); |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
489 |
} |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
490 |
|
3607
fa62293fa65c
6821693: 64-bit TaskQueue capacity still too small
jcoomes
parents:
3262
diff
changeset
|
491 |
// As above, but it also terminates if the should_exit_termination() |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
492 |
// method of the terminator parameter returns true. If terminator is |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
493 |
// NULL, then it is ignored. |
52905
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
494 |
virtual bool offer_termination(TerminatorTerminator* terminator); |
1 | 495 |
|
496 |
// Reset the terminator, so that it may be reused again. |
|
497 |
// The caller is responsible for ensuring that this is done |
|
498 |
// in an MT-safe manner, once the previous round of use of |
|
499 |
// the terminator is finished. |
|
500 |
void reset_for_reuse(); |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
6251
diff
changeset
|
501 |
// Same as above but the number of parallel threads is set to the |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
6251
diff
changeset
|
502 |
// given number. |
30585 | 503 |
void reset_for_reuse(uint n_threads); |
1 | 504 |
|
2010
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
505 |
#ifdef TRACESPINNING |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
506 |
static uint total_yields() { return _total_yields; } |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
507 |
static uint total_spins() { return _total_spins; } |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
508 |
static uint total_peeks() { return _total_peeks; } |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
509 |
static void print_termination_counts(); |
c13462bbad17
6690928: Use spinning in combination with yields for workstealing termination.
jmasa
parents:
2005
diff
changeset
|
510 |
#endif |
1 | 511 |
}; |
512 |
||
52905
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
513 |
#ifdef _MSC_VER |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
514 |
#pragma warning(push) |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
515 |
// warning C4521: multiple copy constructors specified |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
516 |
#pragma warning(disable:4521) |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
517 |
// warning C4522: multiple assignment operators specified |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
518 |
#pragma warning(disable:4522) |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
519 |
#endif |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
520 |
|
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
521 |
class TaskTerminator : public StackObj { |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
522 |
private: |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
523 |
ParallelTaskTerminator* _terminator; |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
524 |
|
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
525 |
// Disable following copy constructors and assignment operator |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
526 |
TaskTerminator(TaskTerminator& o) { } |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
527 |
TaskTerminator(const TaskTerminator& o) { } |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
528 |
TaskTerminator& operator=(TaskTerminator& o) { return *this; } |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
529 |
public: |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
530 |
TaskTerminator(uint n_threads, TaskQueueSetSuper* queue_set); |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
531 |
~TaskTerminator(); |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
532 |
|
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
533 |
// Move assignment |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
534 |
TaskTerminator& operator=(const TaskTerminator& o); |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
535 |
|
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
536 |
ParallelTaskTerminator* terminator() const { |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
537 |
return _terminator; |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
538 |
} |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
539 |
}; |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
540 |
#ifdef _MSC_VER |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
541 |
#pragma warning(pop) |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
542 |
#endif |
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
543 |
|
bec57b4a6d69
8204947: Port ShenandoahTaskTerminator to mainline and make it default
zgu
parents:
52122
diff
changeset
|
544 |
|
13195 | 545 |
typedef GenericTaskQueue<oop, mtGC> OopTaskQueue; |
546 |
typedef GenericTaskQueueSet<OopTaskQueue, mtGC> OopTaskQueueSet; |
|
1 | 547 |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
548 |
#ifdef _MSC_VER |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
549 |
#pragma warning(push) |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
550 |
// warning C4522: multiple assignment operators specified |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
551 |
#pragma warning(disable:4522) |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
552 |
#endif |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
553 |
|
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
554 |
// This is a container class for either an oop* or a narrowOop*. |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
555 |
// Both are pushed onto a task queue and the consumer will test is_narrow() |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
556 |
// to determine which should be processed. |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
557 |
class StarTask { |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
558 |
void* _holder; // either union oop* or narrowOop* |
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
559 |
|
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
560 |
enum { COMPRESSED_OOP_MASK = 1 }; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
561 |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
562 |
public: |
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2105
diff
changeset
|
563 |
StarTask(narrowOop* p) { |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2105
diff
changeset
|
564 |
assert(((uintptr_t)p & COMPRESSED_OOP_MASK) == 0, "Information loss!"); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2105
diff
changeset
|
565 |
_holder = (void *)((uintptr_t)p | COMPRESSED_OOP_MASK); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2105
diff
changeset
|
566 |
} |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2105
diff
changeset
|
567 |
StarTask(oop* p) { |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2105
diff
changeset
|
568 |
assert(((uintptr_t)p & COMPRESSED_OOP_MASK) == 0, "Information loss!"); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2105
diff
changeset
|
569 |
_holder = (void*)p; |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
2105
diff
changeset
|
570 |
} |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
571 |
StarTask() { _holder = NULL; } |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
572 |
operator oop*() { return (oop*)_holder; } |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
573 |
operator narrowOop*() { |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
574 |
return (narrowOop*)((uintptr_t)_holder & ~COMPRESSED_OOP_MASK); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
575 |
} |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
576 |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
577 |
StarTask& operator=(const StarTask& t) { |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
578 |
_holder = t._holder; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
579 |
return *this; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
580 |
} |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
581 |
volatile StarTask& operator=(const volatile StarTask& t) volatile { |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
582 |
_holder = t._holder; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
583 |
return *this; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
584 |
} |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
585 |
|
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
586 |
bool is_narrow() const { |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
587 |
return (((uintptr_t)_holder & COMPRESSED_OOP_MASK) != 0); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
588 |
} |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
589 |
}; |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
590 |
|
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
591 |
class ObjArrayTask |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
592 |
{ |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
593 |
public: |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
594 |
ObjArrayTask(oop o = NULL, int idx = 0): _obj(o), _index(idx) { } |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
595 |
ObjArrayTask(oop o, size_t idx): _obj(o), _index(int(idx)) { |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
596 |
assert(idx <= size_t(max_jint), "too big"); |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
597 |
} |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
598 |
ObjArrayTask(const ObjArrayTask& t): _obj(t._obj), _index(t._index) { } |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
599 |
|
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
600 |
ObjArrayTask& operator =(const ObjArrayTask& t) { |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
601 |
_obj = t._obj; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
602 |
_index = t._index; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
603 |
return *this; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
604 |
} |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
605 |
volatile ObjArrayTask& |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
606 |
operator =(const volatile ObjArrayTask& t) volatile { |
20282
7f9cbdf89af2
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
19346
diff
changeset
|
607 |
(void)const_cast<oop&>(_obj = t._obj); |
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
608 |
_index = t._index; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
609 |
return *this; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
610 |
} |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
611 |
|
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
612 |
inline oop obj() const { return _obj; } |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
613 |
inline int index() const { return _index; } |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
614 |
|
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
615 |
DEBUG_ONLY(bool is_valid() const); // Tasks to be pushed/popped must be valid. |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
616 |
|
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
617 |
private: |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
618 |
oop _obj; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
619 |
int _index; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
620 |
}; |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
621 |
|
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
622 |
#ifdef _MSC_VER |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
623 |
#pragma warning(pop) |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
624 |
#endif |
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
5035
diff
changeset
|
625 |
|
46764
39432ed123bb
8185884: NMT: taskqueues are miscategorized to mtClass
zgu
parents:
39407
diff
changeset
|
626 |
typedef OverflowTaskQueue<StarTask, mtGC> OopStarTaskQueue; |
39432ed123bb
8185884: NMT: taskqueues are miscategorized to mtClass
zgu
parents:
39407
diff
changeset
|
627 |
typedef GenericTaskQueueSet<OopStarTaskQueue, mtGC> OopStarTaskQueueSet; |
1 | 628 |
|
46764
39432ed123bb
8185884: NMT: taskqueues are miscategorized to mtClass
zgu
parents:
39407
diff
changeset
|
629 |
typedef OverflowTaskQueue<size_t, mtGC> RegionTaskQueue; |
39432ed123bb
8185884: NMT: taskqueues are miscategorized to mtClass
zgu
parents:
39407
diff
changeset
|
630 |
typedef GenericTaskQueueSet<RegionTaskQueue, mtGC> RegionTaskQueueSet; |
7397 | 631 |
|
53244
9807daeb47c4
8216167: Update include guards to reflect correct directories
coleenp
parents:
52905
diff
changeset
|
632 |
#endif // SHARE_GC_SHARED_TASKQUEUE_HPP |