author | jcbeyler |
Fri, 14 Sep 2018 08:48:36 -0700 | |
changeset 51752 | 43323ced5e40 |
parent 51511 | eb8d5aeabab3 |
child 52037 | d2a6c3cbc110 |
permissions | -rw-r--r-- |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1 |
/* |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
2 |
* Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
4 |
* |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
8 |
* |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
13 |
* accompanied this code). |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
14 |
* |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
15 |
* You should have received a copy of the GNU General Public License version |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
18 |
* |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
21 |
* questions. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
22 |
* |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
23 |
*/ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
24 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
25 |
#include "precompiled.hpp" |
48816
3495d6050efe
8194898: Move OopStorage inline definitions to an .inline.hpp
kbarrett
parents:
48808
diff
changeset
|
26 |
#include "gc/shared/oopStorage.inline.hpp" |
3495d6050efe
8194898: Move OopStorage inline definitions to an .inline.hpp
kbarrett
parents:
48808
diff
changeset
|
27 |
#include "gc/shared/oopStorageParState.inline.hpp" |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
28 |
#include "logging/log.hpp" |
48886 | 29 |
#include "logging/logStream.hpp" |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
30 |
#include "memory/allocation.inline.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
31 |
#include "runtime/atomic.hpp" |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
32 |
#include "runtime/globals.hpp" |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
33 |
#include "runtime/handles.inline.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
34 |
#include "runtime/mutex.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
35 |
#include "runtime/mutexLocker.hpp" |
50429
83aec1d357d4
8204301: Make OrderAccess functions available to hpp rather than inline.hpp files
coleenp
parents:
50332
diff
changeset
|
36 |
#include "runtime/orderAccess.hpp" |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
37 |
#include "runtime/safepoint.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
38 |
#include "runtime/stubRoutines.hpp" |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
39 |
#include "runtime/thread.hpp" |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
40 |
#include "utilities/align.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
41 |
#include "utilities/count_trailing_zeros.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
42 |
#include "utilities/debug.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
43 |
#include "utilities/globalDefinitions.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
44 |
#include "utilities/macros.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
45 |
#include "utilities/ostream.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
46 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
47 |
OopStorage::AllocationListEntry::AllocationListEntry() : _prev(NULL), _next(NULL) {} |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
48 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
49 |
OopStorage::AllocationListEntry::~AllocationListEntry() { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
50 |
assert(_prev == NULL, "deleting attached block"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
51 |
assert(_next == NULL, "deleting attached block"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
52 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
53 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
54 |
OopStorage::AllocationList::AllocationList() : _head(NULL), _tail(NULL) {} |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
55 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
56 |
OopStorage::AllocationList::~AllocationList() { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
57 |
// ~OopStorage() empties its lists before destroying them. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
58 |
assert(_head == NULL, "deleting non-empty block list"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
59 |
assert(_tail == NULL, "deleting non-empty block list"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
60 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
61 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
62 |
void OopStorage::AllocationList::push_front(const Block& block) { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
63 |
const Block* old = _head; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
64 |
if (old == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
65 |
assert(_tail == NULL, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
66 |
_head = _tail = █ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
67 |
} else { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
68 |
block.allocation_list_entry()._next = old; |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
69 |
old->allocation_list_entry()._prev = █ |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
70 |
_head = █ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
71 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
72 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
73 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
74 |
void OopStorage::AllocationList::push_back(const Block& block) { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
75 |
const Block* old = _tail; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
76 |
if (old == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
77 |
assert(_head == NULL, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
78 |
_head = _tail = █ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
79 |
} else { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
80 |
old->allocation_list_entry()._next = █ |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
81 |
block.allocation_list_entry()._prev = old; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
82 |
_tail = █ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
83 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
84 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
85 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
86 |
void OopStorage::AllocationList::unlink(const Block& block) { |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
87 |
const AllocationListEntry& block_entry = block.allocation_list_entry(); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
88 |
const Block* prev_blk = block_entry._prev; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
89 |
const Block* next_blk = block_entry._next; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
90 |
block_entry._prev = NULL; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
91 |
block_entry._next = NULL; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
92 |
if ((prev_blk == NULL) && (next_blk == NULL)) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
93 |
assert(_head == &block, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
94 |
assert(_tail == &block, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
95 |
_head = _tail = NULL; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
96 |
} else if (prev_blk == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
97 |
assert(_head == &block, "invariant"); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
98 |
next_blk->allocation_list_entry()._prev = NULL; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
99 |
_head = next_blk; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
100 |
} else if (next_blk == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
101 |
assert(_tail == &block, "invariant"); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
102 |
prev_blk->allocation_list_entry()._next = NULL; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
103 |
_tail = prev_blk; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
104 |
} else { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
105 |
next_blk->allocation_list_entry()._prev = prev_blk; |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
106 |
prev_blk->allocation_list_entry()._next = next_blk; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
107 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
108 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
109 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
110 |
OopStorage::ActiveArray::ActiveArray(size_t size) : |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
111 |
_size(size), |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
112 |
_block_count(0), |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
113 |
_refcount(0) |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
114 |
{} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
115 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
116 |
OopStorage::ActiveArray::~ActiveArray() { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
117 |
assert(_refcount == 0, "precondition"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
118 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
119 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
120 |
OopStorage::ActiveArray* OopStorage::ActiveArray::create(size_t size, AllocFailType alloc_fail) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
121 |
size_t size_in_bytes = blocks_offset() + sizeof(Block*) * size; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
122 |
void* mem = NEW_C_HEAP_ARRAY3(char, size_in_bytes, mtGC, CURRENT_PC, alloc_fail); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
123 |
if (mem == NULL) return NULL; |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
124 |
return new (mem) ActiveArray(size); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
125 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
126 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
127 |
void OopStorage::ActiveArray::destroy(ActiveArray* ba) { |
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
128 |
ba->~ActiveArray(); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
129 |
FREE_C_HEAP_ARRAY(char, ba); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
130 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
131 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
132 |
size_t OopStorage::ActiveArray::size() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
133 |
return _size; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
134 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
135 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
136 |
size_t OopStorage::ActiveArray::block_count() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
137 |
return _block_count; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
138 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
139 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
140 |
size_t OopStorage::ActiveArray::block_count_acquire() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
141 |
return OrderAccess::load_acquire(&_block_count); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
142 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
143 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
144 |
void OopStorage::ActiveArray::increment_refcount() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
145 |
int new_value = Atomic::add(1, &_refcount); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
146 |
assert(new_value >= 1, "negative refcount %d", new_value - 1); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
147 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
148 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
149 |
bool OopStorage::ActiveArray::decrement_refcount() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
150 |
int new_value = Atomic::sub(1, &_refcount); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
151 |
assert(new_value >= 0, "negative refcount %d", new_value); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
152 |
return new_value == 0; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
153 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
154 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
155 |
bool OopStorage::ActiveArray::push(Block* block) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
156 |
size_t index = _block_count; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
157 |
if (index < _size) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
158 |
block->set_active_index(index); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
159 |
*block_ptr(index) = block; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
160 |
// Use a release_store to ensure all the setup is complete before |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
161 |
// making the block visible. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
162 |
OrderAccess::release_store(&_block_count, index + 1); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
163 |
return true; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
164 |
} else { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
165 |
return false; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
166 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
167 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
168 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
169 |
void OopStorage::ActiveArray::remove(Block* block) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
170 |
assert(_block_count > 0, "array is empty"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
171 |
size_t index = block->active_index(); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
172 |
assert(*block_ptr(index) == block, "block not present"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
173 |
size_t last_index = _block_count - 1; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
174 |
Block* last_block = *block_ptr(last_index); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
175 |
last_block->set_active_index(index); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
176 |
*block_ptr(index) = last_block; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
177 |
_block_count = last_index; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
178 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
179 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
180 |
void OopStorage::ActiveArray::copy_from(const ActiveArray* from) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
181 |
assert(_block_count == 0, "array must be empty"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
182 |
size_t count = from->_block_count; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
183 |
assert(count <= _size, "precondition"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
184 |
Block* const* from_ptr = from->block_ptr(0); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
185 |
Block** to_ptr = block_ptr(0); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
186 |
for (size_t i = 0; i < count; ++i) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
187 |
Block* block = *from_ptr++; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
188 |
assert(block->active_index() == i, "invariant"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
189 |
*to_ptr++ = block; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
190 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
191 |
_block_count = count; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
192 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
193 |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
194 |
// Blocks start with an array of BitsPerWord oop entries. That array |
48886 | 195 |
// is divided into conceptual BytesPerWord sections of BitsPerByte |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
196 |
// entries. Blocks are allocated aligned on section boundaries, for |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
197 |
// the convenience of mapping from an entry to the containing block; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
198 |
// see block_for_ptr(). Aligning on section boundary rather than on |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
199 |
// the full _data wastes a lot less space, but makes for a bit more |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
200 |
// work in block_for_ptr(). |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
201 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
202 |
const unsigned section_size = BitsPerByte; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
203 |
const unsigned section_count = BytesPerWord; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
204 |
const unsigned block_alignment = sizeof(oop) * section_size; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
205 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
206 |
OopStorage::Block::Block(const OopStorage* owner, void* memory) : |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
207 |
_data(), |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
208 |
_allocated_bitmask(0), |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
209 |
_owner(owner), |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
210 |
_memory(memory), |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
211 |
_active_index(0), |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
212 |
_allocation_list_entry(), |
48886 | 213 |
_deferred_updates_next(NULL), |
214 |
_release_refcount(0) |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
215 |
{ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
216 |
STATIC_ASSERT(_data_pos == 0); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
217 |
STATIC_ASSERT(section_size * section_count == ARRAY_SIZE(_data)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
218 |
assert(offset_of(Block, _data) == _data_pos, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
219 |
assert(owner != NULL, "NULL owner"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
220 |
assert(is_aligned(this, block_alignment), "misaligned block"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
221 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
222 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
223 |
OopStorage::Block::~Block() { |
48886 | 224 |
assert(_release_refcount == 0, "deleting block while releasing"); |
225 |
assert(_deferred_updates_next == NULL, "deleting block with deferred update"); |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
226 |
// Clear fields used by block_for_ptr and entry validation, which |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
227 |
// might help catch bugs. Volatile to prevent dead-store elimination. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
228 |
const_cast<uintx volatile&>(_allocated_bitmask) = 0; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
229 |
const_cast<OopStorage* volatile&>(_owner) = NULL; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
230 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
231 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
232 |
size_t OopStorage::Block::allocation_size() { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
233 |
// _data must be first member, so aligning Block aligns _data. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
234 |
STATIC_ASSERT(_data_pos == 0); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
235 |
return sizeof(Block) + block_alignment - sizeof(void*); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
236 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
237 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
238 |
size_t OopStorage::Block::allocation_alignment_shift() { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
239 |
return exact_log2(block_alignment); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
240 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
241 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
242 |
inline bool is_full_bitmask(uintx bitmask) { return ~bitmask == 0; } |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
243 |
inline bool is_empty_bitmask(uintx bitmask) { return bitmask == 0; } |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
244 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
245 |
bool OopStorage::Block::is_full() const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
246 |
return is_full_bitmask(allocated_bitmask()); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
247 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
248 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
249 |
bool OopStorage::Block::is_empty() const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
250 |
return is_empty_bitmask(allocated_bitmask()); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
251 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
252 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
253 |
uintx OopStorage::Block::bitmask_for_entry(const oop* ptr) const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
254 |
return bitmask_for_index(get_index(ptr)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
255 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
256 |
|
48886 | 257 |
// A block is deletable if |
258 |
// (1) It is empty. |
|
259 |
// (2) There is not a release() operation currently operating on it. |
|
260 |
// (3) It is not in the deferred updates list. |
|
261 |
// The order of tests is important for proper interaction between release() |
|
262 |
// and concurrent deletion. |
|
263 |
bool OopStorage::Block::is_deletable() const { |
|
264 |
return (OrderAccess::load_acquire(&_allocated_bitmask) == 0) && |
|
265 |
(OrderAccess::load_acquire(&_release_refcount) == 0) && |
|
266 |
(OrderAccess::load_acquire(&_deferred_updates_next) == NULL); |
|
267 |
} |
|
268 |
||
269 |
OopStorage::Block* OopStorage::Block::deferred_updates_next() const { |
|
270 |
return _deferred_updates_next; |
|
271 |
} |
|
272 |
||
273 |
void OopStorage::Block::set_deferred_updates_next(Block* block) { |
|
274 |
_deferred_updates_next = block; |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
275 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
276 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
277 |
bool OopStorage::Block::contains(const oop* ptr) const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
278 |
const oop* base = get_pointer(0); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
279 |
return (base <= ptr) && (ptr < (base + ARRAY_SIZE(_data))); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
280 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
281 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
282 |
size_t OopStorage::Block::active_index() const { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
283 |
return _active_index; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
284 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
285 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
286 |
void OopStorage::Block::set_active_index(size_t index) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
287 |
_active_index = index; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
288 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
289 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
290 |
size_t OopStorage::Block::active_index_safe(const Block* block) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
291 |
STATIC_ASSERT(sizeof(intptr_t) == sizeof(block->_active_index)); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
292 |
assert(CanUseSafeFetchN(), "precondition"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
293 |
return SafeFetchN((intptr_t*)&block->_active_index, 0); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
294 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
295 |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
296 |
unsigned OopStorage::Block::get_index(const oop* ptr) const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
297 |
assert(contains(ptr), PTR_FORMAT " not in block " PTR_FORMAT, p2i(ptr), p2i(this)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
298 |
return static_cast<unsigned>(ptr - get_pointer(0)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
299 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
300 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
301 |
oop* OopStorage::Block::allocate() { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
302 |
// Use CAS loop because release may change bitmask outside of lock. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
303 |
uintx allocated = allocated_bitmask(); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
304 |
while (true) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
305 |
assert(!is_full_bitmask(allocated), "attempt to allocate from full block"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
306 |
unsigned index = count_trailing_zeros(~allocated); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
307 |
uintx new_value = allocated | bitmask_for_index(index); |
48886 | 308 |
uintx fetched = Atomic::cmpxchg(new_value, &_allocated_bitmask, allocated); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
309 |
if (fetched == allocated) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
310 |
return get_pointer(index); // CAS succeeded; return entry for index. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
311 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
312 |
allocated = fetched; // CAS failed; retry with latest value. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
313 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
314 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
315 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
316 |
OopStorage::Block* OopStorage::Block::new_block(const OopStorage* owner) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
317 |
// _data must be first member: aligning block => aligning _data. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
318 |
STATIC_ASSERT(_data_pos == 0); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
319 |
size_t size_needed = allocation_size(); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
320 |
void* memory = NEW_C_HEAP_ARRAY_RETURN_NULL(char, size_needed, mtGC); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
321 |
if (memory == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
322 |
return NULL; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
323 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
324 |
void* block_mem = align_up(memory, block_alignment); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
325 |
assert(sizeof(Block) + pointer_delta(block_mem, memory, 1) <= size_needed, |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
326 |
"allocated insufficient space for aligned block"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
327 |
return ::new (block_mem) Block(owner, memory); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
328 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
329 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
330 |
void OopStorage::Block::delete_block(const Block& block) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
331 |
void* memory = block._memory; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
332 |
block.Block::~Block(); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
333 |
FREE_C_HEAP_ARRAY(char, memory); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
334 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
335 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
336 |
// This can return a false positive if ptr is not contained by some |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
337 |
// block. For some uses, it is a precondition that ptr is valid, |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
338 |
// e.g. contained in some block in owner's _active_array. Other uses |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
339 |
// require additional validation of the result. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
340 |
OopStorage::Block* |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
341 |
OopStorage::Block::block_for_ptr(const OopStorage* owner, const oop* ptr) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
342 |
assert(CanUseSafeFetchN(), "precondition"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
343 |
STATIC_ASSERT(_data_pos == 0); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
344 |
// Const-ness of ptr is not related to const-ness of containing block. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
345 |
// Blocks are allocated section-aligned, so get the containing section. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
346 |
oop* section_start = align_down(const_cast<oop*>(ptr), block_alignment); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
347 |
// Start with a guess that the containing section is the last section, |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
348 |
// so the block starts section_count-1 sections earlier. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
349 |
oop* section = section_start - (section_size * (section_count - 1)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
350 |
// Walk up through the potential block start positions, looking for |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
351 |
// the owner in the expected location. If we're below the actual block |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
352 |
// start position, the value at the owner position will be some oop |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
353 |
// (possibly NULL), which can never match the owner. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
354 |
intptr_t owner_addr = reinterpret_cast<intptr_t>(owner); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
355 |
for (unsigned i = 0; i < section_count; ++i, section += section_size) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
356 |
Block* candidate = reinterpret_cast<Block*>(section); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
357 |
intptr_t* candidate_owner_addr |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
358 |
= reinterpret_cast<intptr_t*>(&candidate->_owner); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
359 |
if (SafeFetchN(candidate_owner_addr, 0) == owner_addr) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
360 |
return candidate; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
361 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
362 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
363 |
return NULL; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
364 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
365 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
366 |
////////////////////////////////////////////////////////////////////////////// |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
367 |
// Allocation |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
368 |
// |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
369 |
// Allocation involves the _allocation_list, which contains a subset of the |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
370 |
// blocks owned by a storage object. This is a doubly-linked list, linked |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
371 |
// through dedicated fields in the blocks. Full blocks are removed from this |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
372 |
// list, though they are still present in the _active_array. Empty blocks are |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
373 |
// kept at the end of the _allocation_list, to make it easy for empty block |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
374 |
// deletion to find them. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
375 |
// |
48886 | 376 |
// allocate(), and delete_empty_blocks_concurrent() lock the |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
377 |
// _allocation_mutex while performing any list and array modifications. |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
378 |
// |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
379 |
// allocate() and release() update a block's _allocated_bitmask using CAS |
48886 | 380 |
// loops. This prevents loss of updates even though release() performs |
381 |
// its updates without any locking. |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
382 |
// |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
383 |
// allocate() obtains the entry from the first block in the _allocation_list, |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
384 |
// and updates that block's _allocated_bitmask to indicate the entry is in |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
385 |
// use. If this makes the block full (all entries in use), the block is |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
386 |
// removed from the _allocation_list so it won't be considered by future |
48886 | 387 |
// allocations until some entries in it are released. |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
388 |
// |
48886 | 389 |
// release() is performed lock-free. release() first looks up the block for |
390 |
// the entry, using address alignment to find the enclosing block (thereby |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
391 |
// avoiding iteration over the _active_array). Once the block has been |
48886 | 392 |
// determined, its _allocated_bitmask needs to be updated, and its position in |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
393 |
// the _allocation_list may need to be updated. There are two cases: |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
394 |
// |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
395 |
// (a) If the block is neither full nor would become empty with the release of |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
396 |
// the entry, only its _allocated_bitmask needs to be updated. But if the CAS |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
397 |
// update fails, the applicable case may change for the retry. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
398 |
// |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
399 |
// (b) Otherwise, the _allocation_list also needs to be modified. This requires |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
400 |
// locking the _allocation_mutex. To keep the release() operation lock-free, |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
401 |
// rather than updating the _allocation_list itself, it instead performs a |
48886 | 402 |
// lock-free push of the block onto the _deferred_updates list. Entries on |
403 |
// that list are processed by allocate() and delete_empty_blocks_XXX(), while |
|
404 |
// they already hold the necessary lock. That processing makes the block's |
|
405 |
// list state consistent with its current _allocated_bitmask. The block is |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
406 |
// added to the _allocation_list if not already present and the bitmask is not |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
407 |
// full. The block is moved to the end of the _allocation_list if the bitmask |
48886 | 408 |
// is empty, for ease of empty block deletion processing. |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
409 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
410 |
oop* OopStorage::allocate() { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
411 |
MutexLockerEx ml(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
48886 | 412 |
// Do some deferred update processing every time we allocate. |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
413 |
// Continue processing deferred updates if _allocation_list is empty, |
48886 | 414 |
// in the hope that we'll get a block from that, rather than |
415 |
// allocating a new block. |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
416 |
while (reduce_deferred_updates() && (_allocation_list.head() == NULL)) {} |
48886 | 417 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
418 |
// Use the first block in _allocation_list for the allocation. |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
419 |
Block* block = _allocation_list.head(); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
420 |
if (block == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
421 |
// No available blocks; make a new one, and add to storage. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
422 |
{ |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
423 |
MutexUnlockerEx mul(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
424 |
block = Block::new_block(this); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
425 |
} |
48886 | 426 |
if (block == NULL) { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
427 |
while (_allocation_list.head() == NULL) { |
48886 | 428 |
if (!reduce_deferred_updates()) { |
429 |
// Failed to make new block, no other thread made a block |
|
430 |
// available while the mutex was released, and didn't get |
|
431 |
// one from a deferred update either, so return failure. |
|
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
432 |
log_debug(oopstorage, blocks)("%s: failed block allocation", name()); |
48886 | 433 |
return NULL; |
434 |
} |
|
435 |
} |
|
436 |
} else { |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
437 |
// Add new block to storage. |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
438 |
log_debug(oopstorage, blocks)("%s: new block " PTR_FORMAT, name(), p2i(block)); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
439 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
440 |
// Add new block to the _active_array, growing if needed. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
441 |
if (!_active_array->push(block)) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
442 |
if (expand_active_array()) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
443 |
guarantee(_active_array->push(block), "push failed after expansion"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
444 |
} else { |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
445 |
log_debug(oopstorage, blocks)("%s: failed active array expand", name()); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
446 |
Block::delete_block(*block); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
447 |
return NULL; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
448 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
449 |
} |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
450 |
// Add to end of _allocation_list. The mutex release allowed |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
451 |
// other threads to add blocks to the _allocation_list. We prefer |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
452 |
// to allocate from non-empty blocks, to allow empty blocks to |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
453 |
// be deleted. |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
454 |
_allocation_list.push_back(*block); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
455 |
} |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
456 |
block = _allocation_list.head(); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
457 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
458 |
// Allocate from first block. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
459 |
assert(block != NULL, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
460 |
assert(!block->is_full(), "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
461 |
if (block->is_empty()) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
462 |
// Transitioning from empty to not empty. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
463 |
log_debug(oopstorage, blocks)("%s: block not empty " PTR_FORMAT, name(), p2i(block)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
464 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
465 |
oop* result = block->allocate(); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
466 |
assert(result != NULL, "allocation failed"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
467 |
assert(!block->is_empty(), "postcondition"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
468 |
Atomic::inc(&_allocation_count); // release updates outside lock. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
469 |
if (block->is_full()) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
470 |
// Transitioning from not full to full. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
471 |
// Remove full blocks from consideration by future allocates. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
472 |
log_debug(oopstorage, blocks)("%s: block full " PTR_FORMAT, name(), p2i(block)); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
473 |
_allocation_list.unlink(*block); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
474 |
} |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
475 |
log_trace(oopstorage, ref)("%s: allocated " PTR_FORMAT, name(), p2i(result)); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
476 |
return result; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
477 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
478 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
479 |
// Create a new, larger, active array with the same content as the |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
480 |
// current array, and then replace, relinquishing the old array. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
481 |
// Return true if the array was successfully expanded, false to |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
482 |
// indicate allocation failure. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
483 |
bool OopStorage::expand_active_array() { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
484 |
assert_lock_strong(_allocation_mutex); |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
485 |
ActiveArray* old_array = _active_array; |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
486 |
size_t new_size = 2 * old_array->size(); |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
487 |
log_debug(oopstorage, blocks)("%s: expand active array " SIZE_FORMAT, |
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
488 |
name(), new_size); |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
489 |
ActiveArray* new_array = ActiveArray::create(new_size, AllocFailStrategy::RETURN_NULL); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
490 |
if (new_array == NULL) return false; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
491 |
new_array->copy_from(old_array); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
492 |
replace_active_array(new_array); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
493 |
relinquish_block_array(old_array); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
494 |
return true; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
495 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
496 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
497 |
// Make new_array the _active_array. Increments new_array's refcount |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
498 |
// to account for the new reference. The assignment is atomic wrto |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
499 |
// obtain_active_array; once this function returns, it is safe for the |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
500 |
// caller to relinquish the old array. |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
501 |
void OopStorage::replace_active_array(ActiveArray* new_array) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
502 |
// Caller has the old array that is the current value of _active_array. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
503 |
// Update new_array refcount to account for the new reference. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
504 |
new_array->increment_refcount(); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
505 |
// Install new_array, ensuring its initialization is complete first. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
506 |
OrderAccess::release_store(&_active_array, new_array); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
507 |
// Wait for any readers that could read the old array from _active_array. |
51511
eb8d5aeabab3
8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents:
50954
diff
changeset
|
508 |
// Can't use GlobalCounter here, because this is called from allocate(), |
eb8d5aeabab3
8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents:
50954
diff
changeset
|
509 |
// which may be called in the scope of a GlobalCounter critical section |
eb8d5aeabab3
8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents:
50954
diff
changeset
|
510 |
// when inserting a StringTable entry. |
eb8d5aeabab3
8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents:
50954
diff
changeset
|
511 |
_protect_active.synchronize(); |
50332
d0d933d61610
8204179: [BACKOUT] OopStorage should use GlobalCounter
kbarrett
parents:
50300
diff
changeset
|
512 |
// All obtain critical sections that could see the old array have |
d0d933d61610
8204179: [BACKOUT] OopStorage should use GlobalCounter
kbarrett
parents:
50300
diff
changeset
|
513 |
// completed, having incremented the refcount of the old array. The |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
514 |
// caller can now safely relinquish the old array. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
515 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
516 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
517 |
// Atomically (wrto replace_active_array) get the active array and |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
518 |
// increment its refcount. This provides safe access to the array, |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
519 |
// even if an allocate operation expands and replaces the value of |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
520 |
// _active_array. The caller must relinquish the array when done |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
521 |
// using it. |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
522 |
OopStorage::ActiveArray* OopStorage::obtain_active_array() const { |
51511
eb8d5aeabab3
8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents:
50954
diff
changeset
|
523 |
SingleWriterSynchronizer::CriticalSection cs(&_protect_active); |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
524 |
ActiveArray* result = OrderAccess::load_acquire(&_active_array); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
525 |
result->increment_refcount(); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
526 |
return result; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
527 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
528 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
529 |
// Decrement refcount of array and destroy if refcount is zero. |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
530 |
void OopStorage::relinquish_block_array(ActiveArray* array) const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
531 |
if (array->decrement_refcount()) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
532 |
assert(array != _active_array, "invariant"); |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
533 |
ActiveArray::destroy(array); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
534 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
535 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
536 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
537 |
class OopStorage::WithActiveArray : public StackObj { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
538 |
const OopStorage* _storage; |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
539 |
ActiveArray* _active_array; |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
540 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
541 |
public: |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
542 |
WithActiveArray(const OopStorage* storage) : |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
543 |
_storage(storage), |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
544 |
_active_array(storage->obtain_active_array()) |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
545 |
{} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
546 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
547 |
~WithActiveArray() { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
548 |
_storage->relinquish_block_array(_active_array); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
549 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
550 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
551 |
ActiveArray& active_array() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
552 |
return *_active_array; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
553 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
554 |
}; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
555 |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
556 |
OopStorage::Block* OopStorage::find_block_or_null(const oop* ptr) const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
557 |
assert(ptr != NULL, "precondition"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
558 |
return Block::block_for_ptr(this, ptr); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
559 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
560 |
|
48886 | 561 |
static void log_release_transitions(uintx releasing, |
562 |
uintx old_allocated, |
|
563 |
const OopStorage* owner, |
|
564 |
const void* block) { |
|
565 |
Log(oopstorage, blocks) log; |
|
566 |
LogStream ls(log.debug()); |
|
567 |
if (is_full_bitmask(old_allocated)) { |
|
568 |
ls.print_cr("%s: block not full " PTR_FORMAT, owner->name(), p2i(block)); |
|
569 |
} |
|
570 |
if (releasing == old_allocated) { |
|
571 |
ls.print_cr("%s: block empty " PTR_FORMAT, owner->name(), p2i(block)); |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
572 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
573 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
574 |
|
48886 | 575 |
void OopStorage::Block::release_entries(uintx releasing, Block* volatile* deferred_list) { |
576 |
assert(releasing != 0, "preconditon"); |
|
577 |
// Prevent empty block deletion when transitioning to empty. |
|
578 |
Atomic::inc(&_release_refcount); |
|
579 |
||
580 |
// Atomically update allocated bitmask. |
|
581 |
uintx old_allocated = _allocated_bitmask; |
|
582 |
while (true) { |
|
583 |
assert((releasing & ~old_allocated) == 0, "releasing unallocated entries"); |
|
584 |
uintx new_value = old_allocated ^ releasing; |
|
585 |
uintx fetched = Atomic::cmpxchg(new_value, &_allocated_bitmask, old_allocated); |
|
586 |
if (fetched == old_allocated) break; // Successful update. |
|
587 |
old_allocated = fetched; // Retry with updated bitmask. |
|
588 |
} |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
589 |
|
48886 | 590 |
// Now that the bitmask has been updated, if we have a state transition |
591 |
// (updated bitmask is empty or old bitmask was full), atomically push |
|
592 |
// this block onto the deferred updates list. Some future call to |
|
593 |
// reduce_deferred_updates will make any needed changes related to this |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
594 |
// block and _allocation_list. This deferral avoids list updates and the |
48886 | 595 |
// associated locking here. |
596 |
if ((releasing == old_allocated) || is_full_bitmask(old_allocated)) { |
|
597 |
// Log transitions. Both transitions are possible in a single update. |
|
598 |
if (log_is_enabled(Debug, oopstorage, blocks)) { |
|
599 |
log_release_transitions(releasing, old_allocated, _owner, this); |
|
600 |
} |
|
601 |
// Attempt to claim responsibility for adding this block to the deferred |
|
602 |
// list, by setting the link to non-NULL by self-looping. If this fails, |
|
603 |
// then someone else has made such a claim and the deferred update has not |
|
604 |
// yet been processed and will include our change, so we don't need to do |
|
605 |
// anything further. |
|
606 |
if (Atomic::replace_if_null(this, &_deferred_updates_next)) { |
|
607 |
// Successfully claimed. Push, with self-loop for end-of-list. |
|
608 |
Block* head = *deferred_list; |
|
609 |
while (true) { |
|
610 |
_deferred_updates_next = (head == NULL) ? this : head; |
|
611 |
Block* fetched = Atomic::cmpxchg(this, deferred_list, head); |
|
612 |
if (fetched == head) break; // Successful update. |
|
613 |
head = fetched; // Retry with updated head. |
|
614 |
} |
|
615 |
log_debug(oopstorage, blocks)("%s: deferred update " PTR_FORMAT, |
|
616 |
_owner->name(), p2i(this)); |
|
617 |
} |
|
618 |
} |
|
619 |
// Release hold on empty block deletion. |
|
620 |
Atomic::dec(&_release_refcount); |
|
621 |
} |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
622 |
|
48886 | 623 |
// Process one available deferred update. Returns true if one was processed. |
624 |
bool OopStorage::reduce_deferred_updates() { |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
625 |
assert_locked_or_safepoint(_allocation_mutex); |
48886 | 626 |
// Atomically pop a block off the list, if any available. |
627 |
// No ABA issue because this is only called by one thread at a time. |
|
628 |
// The atomicity is wrto pushes by release(). |
|
629 |
Block* block = OrderAccess::load_acquire(&_deferred_updates); |
|
630 |
while (true) { |
|
631 |
if (block == NULL) return false; |
|
632 |
// Try atomic pop of block from list. |
|
633 |
Block* tail = block->deferred_updates_next(); |
|
634 |
if (block == tail) tail = NULL; // Handle self-loop end marker. |
|
635 |
Block* fetched = Atomic::cmpxchg(tail, &_deferred_updates, block); |
|
636 |
if (fetched == block) break; // Update successful. |
|
637 |
block = fetched; // Retry with updated block. |
|
638 |
} |
|
639 |
block->set_deferred_updates_next(NULL); // Clear tail after updating head. |
|
640 |
// Ensure bitmask read after pop is complete, including clearing tail, for |
|
641 |
// ordering with release(). Without this, we may be processing a stale |
|
642 |
// bitmask state here while blocking a release() operation from recording |
|
643 |
// the deferred update needed for its bitmask change. |
|
644 |
OrderAccess::storeload(); |
|
645 |
// Process popped block. |
|
646 |
uintx allocated = block->allocated_bitmask(); |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
647 |
|
48886 | 648 |
// Make membership in list consistent with bitmask state. |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
649 |
if ((_allocation_list.ctail() != NULL) && |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
650 |
((_allocation_list.ctail() == block) || |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
651 |
(_allocation_list.next(*block) != NULL))) { |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
652 |
// Block is in the _allocation_list. |
48886 | 653 |
assert(!is_full_bitmask(allocated), "invariant"); |
654 |
} else if (!is_full_bitmask(allocated)) { |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
655 |
// Block is not in the _allocation_list, but now should be. |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
656 |
_allocation_list.push_front(*block); |
48886 | 657 |
} // Else block is full and not in list, which is correct. |
658 |
||
659 |
// Move empty block to end of list, for possible deletion. |
|
660 |
if (is_empty_bitmask(allocated)) { |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
661 |
_allocation_list.unlink(*block); |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
662 |
_allocation_list.push_back(*block); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
663 |
} |
48886 | 664 |
|
665 |
log_debug(oopstorage, blocks)("%s: processed deferred update " PTR_FORMAT, |
|
666 |
name(), p2i(block)); |
|
667 |
return true; // Processed one pending update. |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
668 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
669 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
670 |
inline void check_release_entry(const oop* entry) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
671 |
assert(entry != NULL, "Releasing NULL"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
672 |
assert(*entry == NULL, "Releasing uncleared entry: " PTR_FORMAT, p2i(entry)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
673 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
674 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
675 |
void OopStorage::release(const oop* ptr) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
676 |
check_release_entry(ptr); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
677 |
Block* block = find_block_or_null(ptr); |
48886 | 678 |
assert(block != NULL, "%s: invalid release " PTR_FORMAT, name(), p2i(ptr)); |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
679 |
log_trace(oopstorage, ref)("%s: released " PTR_FORMAT, name(), p2i(ptr)); |
48886 | 680 |
block->release_entries(block->bitmask_for_entry(ptr), &_deferred_updates); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
681 |
Atomic::dec(&_allocation_count); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
682 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
683 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
684 |
void OopStorage::release(const oop* const* ptrs, size_t size) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
685 |
size_t i = 0; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
686 |
while (i < size) { |
48808
2b0b7f222800
8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents:
48787
diff
changeset
|
687 |
check_release_entry(ptrs[i]); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
688 |
Block* block = find_block_or_null(ptrs[i]); |
48886 | 689 |
assert(block != NULL, "%s: invalid release " PTR_FORMAT, name(), p2i(ptrs[i])); |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
690 |
log_trace(oopstorage, ref)("%s: released " PTR_FORMAT, name(), p2i(ptrs[i])); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
691 |
size_t count = 0; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
692 |
uintx releasing = 0; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
693 |
for ( ; i < size; ++i) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
694 |
const oop* entry = ptrs[i]; |
48886 | 695 |
check_release_entry(entry); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
696 |
// If entry not in block, finish block and resume outer loop with entry. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
697 |
if (!block->contains(entry)) break; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
698 |
// Add entry to releasing bitmap. |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
699 |
log_trace(oopstorage, ref)("%s: released " PTR_FORMAT, name(), p2i(entry)); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
700 |
uintx entry_bitmask = block->bitmask_for_entry(entry); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
701 |
assert((releasing & entry_bitmask) == 0, |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
702 |
"Duplicate entry: " PTR_FORMAT, p2i(entry)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
703 |
releasing |= entry_bitmask; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
704 |
++count; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
705 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
706 |
// Release the contiguous entries that are in block. |
48886 | 707 |
block->release_entries(releasing, &_deferred_updates); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
708 |
Atomic::sub(count, &_allocation_count); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
709 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
710 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
711 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
712 |
const char* dup_name(const char* name) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
713 |
char* dup = NEW_C_HEAP_ARRAY(char, strlen(name) + 1, mtGC); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
714 |
strcpy(dup, name); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
715 |
return dup; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
716 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
717 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
718 |
const size_t initial_active_array_size = 8; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
719 |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
720 |
OopStorage::OopStorage(const char* name, |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
721 |
Mutex* allocation_mutex, |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
722 |
Mutex* active_mutex) : |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
723 |
_name(dup_name(name)), |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
724 |
_active_array(ActiveArray::create(initial_active_array_size)), |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
725 |
_allocation_list(), |
48886 | 726 |
_deferred_updates(NULL), |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
727 |
_allocation_mutex(allocation_mutex), |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
728 |
_active_mutex(active_mutex), |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
729 |
_allocation_count(0), |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
730 |
_concurrent_iteration_active(false) |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
731 |
{ |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
732 |
_active_array->increment_refcount(); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
733 |
assert(_active_mutex->rank() < _allocation_mutex->rank(), |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
734 |
"%s: active_mutex must have lower rank than allocation_mutex", _name); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
735 |
assert(_active_mutex->_safepoint_check_required != Mutex::_safepoint_check_always, |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
736 |
"%s: active mutex requires safepoint check", _name); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
737 |
assert(_allocation_mutex->_safepoint_check_required != Mutex::_safepoint_check_always, |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
738 |
"%s: allocation mutex requires safepoint check", _name); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
739 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
740 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
741 |
void OopStorage::delete_empty_block(const Block& block) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
742 |
assert(block.is_empty(), "discarding non-empty block"); |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
743 |
log_debug(oopstorage, blocks)("%s: delete empty block " PTR_FORMAT, name(), p2i(&block)); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
744 |
Block::delete_block(block); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
745 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
746 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
747 |
OopStorage::~OopStorage() { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
748 |
Block* block; |
48886 | 749 |
while ((block = _deferred_updates) != NULL) { |
750 |
_deferred_updates = block->deferred_updates_next(); |
|
751 |
block->set_deferred_updates_next(NULL); |
|
752 |
} |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
753 |
while ((block = _allocation_list.head()) != NULL) { |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
754 |
_allocation_list.unlink(*block); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
755 |
} |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
756 |
bool unreferenced = _active_array->decrement_refcount(); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
757 |
assert(unreferenced, "deleting storage while _active_array is referenced"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
758 |
for (size_t i = _active_array->block_count(); 0 < i; ) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
759 |
block = _active_array->at(--i); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
760 |
Block::delete_block(*block); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
761 |
} |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
762 |
ActiveArray::destroy(_active_array); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
763 |
FREE_C_HEAP_ARRAY(char, _name); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
764 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
765 |
|
48886 | 766 |
void OopStorage::delete_empty_blocks_safepoint() { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
767 |
assert_at_safepoint(); |
48886 | 768 |
// Process any pending release updates, which may make more empty |
769 |
// blocks available for deletion. |
|
770 |
while (reduce_deferred_updates()) {} |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
771 |
// Don't interfere with a concurrent iteration. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
772 |
if (_concurrent_iteration_active) return; |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
773 |
// Delete empty (and otherwise deletable) blocks from end of _allocation_list. |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
774 |
for (Block* block = _allocation_list.tail(); |
48886 | 775 |
(block != NULL) && block->is_deletable(); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
776 |
block = _allocation_list.tail()) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
777 |
_active_array->remove(block); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
778 |
_allocation_list.unlink(*block); |
48886 | 779 |
delete_empty_block(*block); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
780 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
781 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
782 |
|
48886 | 783 |
void OopStorage::delete_empty_blocks_concurrent() { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
784 |
MutexLockerEx ml(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
785 |
// Other threads could be adding to the empty block count while we |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
786 |
// release the mutex across the block deletions. Set an upper bound |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
787 |
// on how many blocks we'll try to release, so other threads can't |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
788 |
// cause an unbounded stay in this function. |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
789 |
size_t limit = block_count(); |
48886 | 790 |
|
791 |
for (size_t i = 0; i < limit; ++i) { |
|
792 |
// Additional updates might become available while we dropped the |
|
793 |
// lock. But limit number processed to limit lock duration. |
|
794 |
reduce_deferred_updates(); |
|
795 |
||
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
796 |
Block* block = _allocation_list.tail(); |
48886 | 797 |
if ((block == NULL) || !block->is_deletable()) { |
798 |
// No block to delete, so done. There could be more pending |
|
799 |
// deferred updates that could give us more work to do; deal with |
|
800 |
// that in some later call, to limit lock duration here. |
|
801 |
return; |
|
802 |
} |
|
803 |
||
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
804 |
{ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
805 |
MutexLockerEx aml(_active_mutex, Mutex::_no_safepoint_check_flag); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
806 |
// Don't interfere with a concurrent iteration. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
807 |
if (_concurrent_iteration_active) return; |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
808 |
_active_array->remove(block); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
809 |
} |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
810 |
// Remove block from _allocation_list and delete it. |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
811 |
_allocation_list.unlink(*block); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
812 |
// Release mutex while deleting block. |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
813 |
MutexUnlockerEx ul(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
814 |
delete_empty_block(*block); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
815 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
816 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
817 |
|
48886 | 818 |
OopStorage::EntryStatus OopStorage::allocation_status(const oop* ptr) const { |
819 |
const Block* block = find_block_or_null(ptr); |
|
820 |
if (block != NULL) { |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
821 |
// Prevent block deletion and _active_array modification. |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
822 |
MutexLockerEx ml(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
823 |
// Block could be a false positive, so get index carefully. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
824 |
size_t index = Block::active_index_safe(block); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
825 |
if ((index < _active_array->block_count()) && |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
826 |
(block == _active_array->at(index)) && |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
827 |
block->contains(ptr)) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
828 |
if ((block->allocated_bitmask() & block->bitmask_for_entry(ptr)) != 0) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
829 |
return ALLOCATED_ENTRY; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
830 |
} else { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
831 |
return UNALLOCATED_ENTRY; |
48886 | 832 |
} |
833 |
} |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
834 |
} |
48886 | 835 |
return INVALID_ENTRY; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
836 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
837 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
838 |
size_t OopStorage::allocation_count() const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
839 |
return _allocation_count; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
840 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
841 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
842 |
size_t OopStorage::block_count() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
843 |
WithActiveArray wab(this); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
844 |
// Count access is racy, but don't care. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
845 |
return wab.active_array().block_count(); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
846 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
847 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
848 |
size_t OopStorage::total_memory_usage() const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
849 |
size_t total_size = sizeof(OopStorage); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
850 |
total_size += strlen(name()) + 1; |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
851 |
total_size += sizeof(ActiveArray); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
852 |
WithActiveArray wab(this); |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
853 |
const ActiveArray& blocks = wab.active_array(); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
854 |
// Count access is racy, but don't care. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
855 |
total_size += blocks.block_count() * Block::allocation_size(); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
856 |
total_size += blocks.size() * sizeof(Block*); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
857 |
return total_size; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
858 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
859 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
860 |
// Parallel iteration support |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
861 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
862 |
uint OopStorage::BasicParState::default_estimated_thread_count(bool concurrent) { |
50298
188a87cbfac3
8203843: BasicParState::default_estimated_thread_count(false) can return 0 in gtest
kbarrett
parents:
50209
diff
changeset
|
863 |
uint configured = concurrent ? ConcGCThreads : ParallelGCThreads; |
188a87cbfac3
8203843: BasicParState::default_estimated_thread_count(false) can return 0 in gtest
kbarrett
parents:
50209
diff
changeset
|
864 |
return MAX2(1u, configured); // Never estimate zero threads. |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
865 |
} |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
866 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
867 |
OopStorage::BasicParState::BasicParState(const OopStorage* storage, |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
868 |
uint estimated_thread_count, |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
869 |
bool concurrent) : |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
870 |
_storage(storage), |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
871 |
_active_array(_storage->obtain_active_array()), |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
872 |
_block_count(0), // initialized properly below |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
873 |
_next_block(0), |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
874 |
_estimated_thread_count(estimated_thread_count), |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
875 |
_concurrent(concurrent) |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
876 |
{ |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
877 |
assert(estimated_thread_count > 0, "estimated thread count must be positive"); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
878 |
update_iteration_state(true); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
879 |
// Get the block count *after* iteration state updated, so concurrent |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
880 |
// empty block deletion is suppressed and can't reduce the count. But |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
881 |
// ensure the count we use was written after the block with that count |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
882 |
// was fully initialized; see ActiveArray::push. |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
883 |
_block_count = _active_array->block_count_acquire(); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
884 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
885 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
886 |
OopStorage::BasicParState::~BasicParState() { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
887 |
_storage->relinquish_block_array(_active_array); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
888 |
update_iteration_state(false); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
889 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
890 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
891 |
void OopStorage::BasicParState::update_iteration_state(bool value) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
892 |
if (_concurrent) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
893 |
MutexLockerEx ml(_storage->_active_mutex, Mutex::_no_safepoint_check_flag); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
894 |
assert(_storage->_concurrent_iteration_active != value, "precondition"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
895 |
_storage->_concurrent_iteration_active = value; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
896 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
897 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
898 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
899 |
bool OopStorage::BasicParState::claim_next_segment(IterationData* data) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
900 |
data->_processed += data->_segment_end - data->_segment_start; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
901 |
size_t start = OrderAccess::load_acquire(&_next_block); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
902 |
if (start >= _block_count) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
903 |
return finish_iteration(data); // No more blocks available. |
49333
489f1dd40582
8196876: OopStorage::assert_at_safepoint clashes with assert_at_safepoint macros in g1CollectedHeap.hpp
kbarrett
parents:
48886
diff
changeset
|
904 |
} |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
905 |
// Try to claim several at a time, but not *too* many. We want to |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
906 |
// avoid deciding there are many available and selecting a large |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
907 |
// quantity, get delayed, and then end up claiming most or all of |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
908 |
// the remaining largish amount of work, leaving nothing for other |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
909 |
// threads to do. But too small a step can lead to contention |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
910 |
// over _next_block, esp. when the work per block is small. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
911 |
size_t max_step = 10; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
912 |
size_t remaining = _block_count - start; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
913 |
size_t step = MIN2(max_step, 1 + (remaining / _estimated_thread_count)); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
914 |
// Atomic::add with possible overshoot. This can perform better |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
915 |
// than a CAS loop on some platforms when there is contention. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
916 |
// We can cope with the uncertainty by recomputing start/end from |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
917 |
// the result of the add, and dealing with potential overshoot. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
918 |
size_t end = Atomic::add(step, &_next_block); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
919 |
// _next_block may have changed, so recompute start from result of add. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
920 |
start = end - step; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
921 |
// _next_block may have changed so much that end has overshot. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
922 |
end = MIN2(end, _block_count); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
923 |
// _next_block may have changed so much that even start has overshot. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
924 |
if (start < _block_count) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
925 |
// Record claimed segment for iteration. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
926 |
data->_segment_start = start; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
927 |
data->_segment_end = end; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
928 |
return true; // Success. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
929 |
} else { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
930 |
// No more blocks to claim. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
931 |
return finish_iteration(data); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
932 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
933 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
934 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
935 |
bool OopStorage::BasicParState::finish_iteration(const IterationData* data) const { |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
936 |
log_info(oopstorage, blocks, stats) |
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
937 |
("Parallel iteration on %s: blocks = " SIZE_FORMAT |
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
938 |
", processed = " SIZE_FORMAT " (%2.f%%)", |
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
939 |
_storage->name(), _block_count, data->_processed, |
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
940 |
percent_of(data->_processed, _block_count)); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
941 |
return false; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
942 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
943 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
944 |
const char* OopStorage::name() const { return _name; } |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
945 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
946 |
#ifndef PRODUCT |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
947 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
948 |
void OopStorage::print_on(outputStream* st) const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
949 |
size_t allocations = _allocation_count; |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
950 |
size_t blocks = _active_array->block_count(); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
951 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
952 |
double data_size = section_size * section_count; |
48886 | 953 |
double alloc_percentage = percent_of((double)allocations, blocks * data_size); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
954 |
|
48886 | 955 |
st->print("%s: " SIZE_FORMAT " entries in " SIZE_FORMAT " blocks (%.F%%), " SIZE_FORMAT " bytes", |
956 |
name(), allocations, blocks, alloc_percentage, total_memory_usage()); |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
957 |
if (_concurrent_iteration_active) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
958 |
st->print(", concurrent iteration active"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
959 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
960 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
961 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
962 |
#endif // !PRODUCT |