author | stefank |
Mon, 25 Nov 2019 12:31:39 +0100 | |
changeset 59249 | 29b0d0b61615 |
parent 59247 | 56bf71d64d51 |
child 59250 | a6deb69743d4 |
permissions | -rw-r--r-- |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1 |
/* |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52421
diff
changeset
|
2 |
* Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. |
48787
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" |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
34 |
#include "runtime/interfaceSupport.inline.hpp" |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
35 |
#include "runtime/mutex.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
36 |
#include "runtime/mutexLocker.hpp" |
50429
83aec1d357d4
8204301: Make OrderAccess functions available to hpp rather than inline.hpp files
coleenp
parents:
50332
diff
changeset
|
37 |
#include "runtime/orderAccess.hpp" |
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
38 |
#include "runtime/os.hpp" |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
39 |
#include "runtime/safepoint.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
40 |
#include "runtime/stubRoutines.hpp" |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
41 |
#include "runtime/thread.hpp" |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
42 |
#include "utilities/align.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
43 |
#include "utilities/count_trailing_zeros.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
44 |
#include "utilities/debug.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
45 |
#include "utilities/globalDefinitions.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
46 |
#include "utilities/macros.hpp" |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
47 |
#include "utilities/ostream.hpp" |
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() : _prev(NULL), _next(NULL) {} |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
50 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
51 |
OopStorage::AllocationListEntry::~AllocationListEntry() { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
52 |
assert(_prev == NULL, "deleting attached block"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
53 |
assert(_next == NULL, "deleting attached block"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
54 |
} |
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() : _head(NULL), _tail(NULL) {} |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
57 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
58 |
OopStorage::AllocationList::~AllocationList() { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
59 |
// ~OopStorage() empties its lists before destroying them. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
60 |
assert(_head == NULL, "deleting non-empty block list"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
61 |
assert(_tail == NULL, "deleting non-empty block list"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
62 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
63 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
64 |
void OopStorage::AllocationList::push_front(const Block& block) { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
65 |
const Block* old = _head; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
66 |
if (old == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
67 |
assert(_tail == NULL, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
68 |
_head = _tail = █ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
69 |
} else { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
70 |
block.allocation_list_entry()._next = old; |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
71 |
old->allocation_list_entry()._prev = █ |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
72 |
_head = █ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
73 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
74 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
75 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
76 |
void OopStorage::AllocationList::push_back(const Block& block) { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
77 |
const Block* old = _tail; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
78 |
if (old == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
79 |
assert(_head == NULL, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
80 |
_head = _tail = █ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
81 |
} else { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
82 |
old->allocation_list_entry()._next = █ |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
83 |
block.allocation_list_entry()._prev = old; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
84 |
_tail = █ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
85 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
86 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
87 |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
88 |
void OopStorage::AllocationList::unlink(const Block& block) { |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
89 |
const AllocationListEntry& block_entry = block.allocation_list_entry(); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
90 |
const Block* prev_blk = block_entry._prev; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
91 |
const Block* next_blk = block_entry._next; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
92 |
block_entry._prev = NULL; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
93 |
block_entry._next = NULL; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
94 |
if ((prev_blk == NULL) && (next_blk == NULL)) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
95 |
assert(_head == &block, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
96 |
assert(_tail == &block, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
97 |
_head = _tail = NULL; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
98 |
} else if (prev_blk == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
99 |
assert(_head == &block, "invariant"); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
100 |
next_blk->allocation_list_entry()._prev = NULL; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
101 |
_head = next_blk; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
102 |
} else if (next_blk == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
103 |
assert(_tail == &block, "invariant"); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
104 |
prev_blk->allocation_list_entry()._next = NULL; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
105 |
_tail = prev_blk; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
106 |
} else { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
107 |
next_blk->allocation_list_entry()._prev = prev_blk; |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
108 |
prev_blk->allocation_list_entry()._next = next_blk; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
109 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
110 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
111 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
112 |
OopStorage::ActiveArray::ActiveArray(size_t size) : |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
113 |
_size(size), |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
114 |
_block_count(0), |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
115 |
_refcount(0) |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
116 |
{} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
117 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
118 |
OopStorage::ActiveArray::~ActiveArray() { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
119 |
assert(_refcount == 0, "precondition"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
120 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
121 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
122 |
OopStorage::ActiveArray* OopStorage::ActiveArray::create(size_t size, AllocFailType alloc_fail) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
123 |
size_t size_in_bytes = blocks_offset() + sizeof(Block*) * size; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
124 |
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
|
125 |
if (mem == NULL) return NULL; |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
126 |
return new (mem) ActiveArray(size); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
127 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
128 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
129 |
void OopStorage::ActiveArray::destroy(ActiveArray* ba) { |
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
130 |
ba->~ActiveArray(); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
131 |
FREE_C_HEAP_ARRAY(char, ba); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
132 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
133 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
134 |
size_t OopStorage::ActiveArray::size() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
135 |
return _size; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
136 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
137 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
138 |
size_t OopStorage::ActiveArray::block_count() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
139 |
return _block_count; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
140 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
141 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
142 |
size_t OopStorage::ActiveArray::block_count_acquire() const { |
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
143 |
return Atomic::load_acquire(&_block_count); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
144 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
145 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
146 |
void OopStorage::ActiveArray::increment_refcount() const { |
59249
29b0d0b61615
8234737: Harmonize parameter order in Atomic - add
stefank
parents:
59247
diff
changeset
|
147 |
int new_value = Atomic::add(&_refcount, 1); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
148 |
assert(new_value >= 1, "negative refcount %d", new_value - 1); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
149 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
150 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
151 |
bool OopStorage::ActiveArray::decrement_refcount() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
152 |
int new_value = Atomic::sub(1, &_refcount); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
153 |
assert(new_value >= 0, "negative refcount %d", new_value); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
154 |
return new_value == 0; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
155 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
156 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
157 |
bool OopStorage::ActiveArray::push(Block* block) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
158 |
size_t index = _block_count; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
159 |
if (index < _size) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
160 |
block->set_active_index(index); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
161 |
*block_ptr(index) = block; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
162 |
// Use a release_store to ensure all the setup is complete before |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
163 |
// making the block visible. |
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
164 |
Atomic::release_store(&_block_count, index + 1); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
165 |
return true; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
166 |
} else { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
167 |
return false; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
168 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
169 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
170 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
171 |
void OopStorage::ActiveArray::remove(Block* block) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
172 |
assert(_block_count > 0, "array is empty"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
173 |
size_t index = block->active_index(); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
174 |
assert(*block_ptr(index) == block, "block not present"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
175 |
size_t last_index = _block_count - 1; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
176 |
Block* last_block = *block_ptr(last_index); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
177 |
last_block->set_active_index(index); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
178 |
*block_ptr(index) = last_block; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
179 |
_block_count = last_index; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
180 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
181 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
182 |
void OopStorage::ActiveArray::copy_from(const ActiveArray* from) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
183 |
assert(_block_count == 0, "array must be empty"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
184 |
size_t count = from->_block_count; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
185 |
assert(count <= _size, "precondition"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
186 |
Block* const* from_ptr = from->block_ptr(0); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
187 |
Block** to_ptr = block_ptr(0); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
188 |
for (size_t i = 0; i < count; ++i) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
189 |
Block* block = *from_ptr++; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
190 |
assert(block->active_index() == i, "invariant"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
191 |
*to_ptr++ = block; |
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 |
_block_count = count; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
194 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
195 |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
196 |
// Blocks start with an array of BitsPerWord oop entries. That array |
48886 | 197 |
// is divided into conceptual BytesPerWord sections of BitsPerByte |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
198 |
// entries. Blocks are allocated aligned on section boundaries, for |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
199 |
// 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
|
200 |
// 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
|
201 |
// 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
|
202 |
// work in block_for_ptr(). |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
203 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
204 |
const unsigned section_size = BitsPerByte; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
205 |
const unsigned section_count = BytesPerWord; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
206 |
const unsigned block_alignment = sizeof(oop) * section_size; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
207 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
208 |
OopStorage::Block::Block(const OopStorage* owner, void* memory) : |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
209 |
_data(), |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
210 |
_allocated_bitmask(0), |
57550
278795ad438a
8228631: Fix inconsistent OopStorage::Block owner usage
kbarrett
parents:
55592
diff
changeset
|
211 |
_owner_address(reinterpret_cast<intptr_t>(owner)), |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
212 |
_memory(memory), |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
213 |
_active_index(0), |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
214 |
_allocation_list_entry(), |
48886 | 215 |
_deferred_updates_next(NULL), |
216 |
_release_refcount(0) |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
217 |
{ |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
218 |
STATIC_ASSERT(_data_pos == 0); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
219 |
STATIC_ASSERT(section_size * section_count == ARRAY_SIZE(_data)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
220 |
assert(offset_of(Block, _data) == _data_pos, "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
221 |
assert(owner != NULL, "NULL owner"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
222 |
assert(is_aligned(this, block_alignment), "misaligned block"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
223 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
224 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
225 |
OopStorage::Block::~Block() { |
48886 | 226 |
assert(_release_refcount == 0, "deleting block while releasing"); |
227 |
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
|
228 |
// 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
|
229 |
// might help catch bugs. Volatile to prevent dead-store elimination. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
230 |
const_cast<uintx volatile&>(_allocated_bitmask) = 0; |
57550
278795ad438a
8228631: Fix inconsistent OopStorage::Block owner usage
kbarrett
parents:
55592
diff
changeset
|
231 |
const_cast<intptr_t volatile&>(_owner_address) = 0; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
232 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
233 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
234 |
size_t OopStorage::Block::allocation_size() { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
235 |
// _data must be first member, so aligning Block aligns _data. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
236 |
STATIC_ASSERT(_data_pos == 0); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
237 |
return sizeof(Block) + block_alignment - sizeof(void*); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
238 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
239 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
240 |
size_t OopStorage::Block::allocation_alignment_shift() { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
241 |
return exact_log2(block_alignment); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
242 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
243 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
244 |
inline bool is_full_bitmask(uintx bitmask) { return ~bitmask == 0; } |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
245 |
inline bool is_empty_bitmask(uintx bitmask) { return bitmask == 0; } |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
246 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
247 |
bool OopStorage::Block::is_full() const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
248 |
return is_full_bitmask(allocated_bitmask()); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
249 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
250 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
251 |
bool OopStorage::Block::is_empty() const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
252 |
return is_empty_bitmask(allocated_bitmask()); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
253 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
254 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
255 |
uintx OopStorage::Block::bitmask_for_entry(const oop* ptr) const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
256 |
return bitmask_for_index(get_index(ptr)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
257 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
258 |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
259 |
// An empty block is not yet deletable if either: |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
260 |
// (1) There is a release() operation currently operating on it. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
261 |
// (2) It is in the deferred updates list. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
262 |
// For interaction with release(), these must follow the empty check, |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
263 |
// and the order of these checks is important. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
264 |
bool OopStorage::Block::is_safe_to_delete() const { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
265 |
assert(is_empty(), "precondition"); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
266 |
OrderAccess::loadload(); |
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
267 |
return (Atomic::load_acquire(&_release_refcount) == 0) && |
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
268 |
(Atomic::load_acquire(&_deferred_updates_next) == NULL); |
48886 | 269 |
} |
270 |
||
271 |
OopStorage::Block* OopStorage::Block::deferred_updates_next() const { |
|
272 |
return _deferred_updates_next; |
|
273 |
} |
|
274 |
||
275 |
void OopStorage::Block::set_deferred_updates_next(Block* block) { |
|
276 |
_deferred_updates_next = block; |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
277 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
278 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
279 |
bool OopStorage::Block::contains(const oop* ptr) const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
280 |
const oop* base = get_pointer(0); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
281 |
return (base <= ptr) && (ptr < (base + ARRAY_SIZE(_data))); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
282 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
283 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
284 |
size_t OopStorage::Block::active_index() const { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
285 |
return _active_index; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
286 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
287 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
288 |
void OopStorage::Block::set_active_index(size_t index) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
289 |
_active_index = index; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
290 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
291 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
292 |
size_t OopStorage::Block::active_index_safe(const Block* block) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
293 |
STATIC_ASSERT(sizeof(intptr_t) == sizeof(block->_active_index)); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
294 |
assert(CanUseSafeFetchN(), "precondition"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
295 |
return SafeFetchN((intptr_t*)&block->_active_index, 0); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
296 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
297 |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
298 |
unsigned OopStorage::Block::get_index(const oop* ptr) const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
299 |
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
|
300 |
return static_cast<unsigned>(ptr - get_pointer(0)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
301 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
302 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
303 |
oop* OopStorage::Block::allocate() { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
304 |
// 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
|
305 |
uintx allocated = allocated_bitmask(); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
306 |
while (true) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
307 |
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
|
308 |
unsigned index = count_trailing_zeros(~allocated); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
309 |
uintx new_value = allocated | bitmask_for_index(index); |
48886 | 310 |
uintx fetched = Atomic::cmpxchg(new_value, &_allocated_bitmask, allocated); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
311 |
if (fetched == allocated) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
312 |
return get_pointer(index); // CAS succeeded; return entry for index. |
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 |
allocated = fetched; // CAS failed; retry with latest value. |
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 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
317 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
318 |
OopStorage::Block* OopStorage::Block::new_block(const OopStorage* owner) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
319 |
// _data must be first member: aligning block => aligning _data. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
320 |
STATIC_ASSERT(_data_pos == 0); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
321 |
size_t size_needed = allocation_size(); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
322 |
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
|
323 |
if (memory == NULL) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
324 |
return NULL; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
325 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
326 |
void* block_mem = align_up(memory, block_alignment); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
327 |
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
|
328 |
"allocated insufficient space for aligned block"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
329 |
return ::new (block_mem) Block(owner, memory); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
330 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
331 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
332 |
void OopStorage::Block::delete_block(const Block& block) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
333 |
void* memory = block._memory; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
334 |
block.Block::~Block(); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
335 |
FREE_C_HEAP_ARRAY(char, memory); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
336 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
337 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
338 |
// 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
|
339 |
// 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
|
340 |
// 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
|
341 |
// require additional validation of the result. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
342 |
OopStorage::Block* |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
343 |
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
|
344 |
assert(CanUseSafeFetchN(), "precondition"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
345 |
STATIC_ASSERT(_data_pos == 0); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
346 |
// 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
|
347 |
// Blocks are allocated section-aligned, so get the containing section. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
348 |
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
|
349 |
// 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
|
350 |
// so the block starts section_count-1 sections earlier. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
351 |
oop* section = section_start - (section_size * (section_count - 1)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
352 |
// Walk up through the potential block start positions, looking for |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
353 |
// 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
|
354 |
// 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
|
355 |
// (possibly NULL), which can never match the owner. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
356 |
intptr_t owner_addr = reinterpret_cast<intptr_t>(owner); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
357 |
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
|
358 |
Block* candidate = reinterpret_cast<Block*>(section); |
57550
278795ad438a
8228631: Fix inconsistent OopStorage::Block owner usage
kbarrett
parents:
55592
diff
changeset
|
359 |
if (SafeFetchN(&candidate->_owner_address, 0) == owner_addr) { |
48787
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 |
// |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
376 |
// allocate(), and delete_empty_blocks() 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 |
// |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
389 |
// release() is performed lock-free. (Note: This means it can't notify the |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
390 |
// service thread of pending cleanup work. It must be lock-free because |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
391 |
// it is called in all kinds of contexts where even quite low ranked locks |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
392 |
// may be held.) release() first looks up the block for |
48886 | 393 |
// the entry, using address alignment to find the enclosing block (thereby |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
394 |
// avoiding iteration over the _active_array). Once the block has been |
48886 | 395 |
// 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
|
396 |
// 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
|
397 |
// |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
398 |
// (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
|
399 |
// 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
|
400 |
// update fails, the applicable case may change for the retry. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
401 |
// |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
402 |
// (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
|
403 |
// locking the _allocation_mutex. To keep the release() operation lock-free, |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
404 |
// rather than updating the _allocation_list itself, it instead performs a |
48886 | 405 |
// lock-free push of the block onto the _deferred_updates list. Entries on |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
406 |
// that list are processed by allocate() and delete_empty_blocks(), while |
48886 | 407 |
// they already hold the necessary lock. That processing makes the block's |
408 |
// 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
|
409 |
// 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
|
410 |
// full. The block is moved to the end of the _allocation_list if the bitmask |
48886 | 411 |
// is empty, for ease of empty block deletion processing. |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
412 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
413 |
oop* OopStorage::allocate() { |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52421
diff
changeset
|
414 |
MutexLocker ml(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
48886 | 415 |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
416 |
Block* block = block_for_allocation(); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
417 |
if (block == NULL) return NULL; // Block allocation failed. |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
418 |
assert(!block->is_full(), "invariant"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
419 |
if (block->is_empty()) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
420 |
// Transitioning from empty to not empty. |
57551
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
421 |
log_trace(oopstorage, blocks)("%s: block not empty " PTR_FORMAT, name(), p2i(block)); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
422 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
423 |
oop* result = block->allocate(); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
424 |
assert(result != NULL, "allocation failed"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
425 |
assert(!block->is_empty(), "postcondition"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
426 |
Atomic::inc(&_allocation_count); // release updates outside lock. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
427 |
if (block->is_full()) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
428 |
// Transitioning from not full to full. |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
429 |
// Remove full blocks from consideration by future allocates. |
57551
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
430 |
log_trace(oopstorage, blocks)("%s: block full " PTR_FORMAT, name(), p2i(block)); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
431 |
_allocation_list.unlink(*block); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
432 |
} |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
433 |
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
|
434 |
return result; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
435 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
436 |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
437 |
bool OopStorage::try_add_block() { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
438 |
assert_lock_strong(_allocation_mutex); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
439 |
Block* block; |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
440 |
{ |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52421
diff
changeset
|
441 |
MutexUnlocker ul(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
442 |
block = Block::new_block(this); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
443 |
} |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
444 |
if (block == NULL) return false; |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
445 |
|
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
446 |
// Add new block to the _active_array, growing if needed. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
447 |
if (!_active_array->push(block)) { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
448 |
if (expand_active_array()) { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
449 |
guarantee(_active_array->push(block), "push failed after expansion"); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
450 |
} else { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
451 |
log_debug(oopstorage, blocks)("%s: failed active array expand", name()); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
452 |
Block::delete_block(*block); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
453 |
return false; |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
454 |
} |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
455 |
} |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
456 |
// Add to end of _allocation_list. The mutex release allowed other |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
457 |
// threads to add blocks to the _allocation_list. We prefer to |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
458 |
// allocate from non-empty blocks, to allow empty blocks to be |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
459 |
// deleted. But we don't bother notifying about the empty block |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
460 |
// because we're (probably) about to allocate an entry from it. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
461 |
_allocation_list.push_back(*block); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
462 |
log_debug(oopstorage, blocks)("%s: new block " PTR_FORMAT, name(), p2i(block)); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
463 |
return true; |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
464 |
} |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
465 |
|
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
466 |
OopStorage::Block* OopStorage::block_for_allocation() { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
467 |
assert_lock_strong(_allocation_mutex); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
468 |
while (true) { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
469 |
// Use the first block in _allocation_list for the allocation. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
470 |
Block* block = _allocation_list.head(); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
471 |
if (block != NULL) { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
472 |
return block; |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
473 |
} else if (reduce_deferred_updates()) { |
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
474 |
// Might have added a block to the _allocation_list, so retry. |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
475 |
} else if (try_add_block()) { |
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
476 |
// Successfully added a new block to the list, so retry. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
477 |
assert(_allocation_list.chead() != NULL, "invariant"); |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
478 |
} else if (_allocation_list.chead() != NULL) { |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
479 |
// Trying to add a block failed, but some other thread added to the |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
480 |
// list while we'd dropped the lock over the new block allocation. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
481 |
} else if (!reduce_deferred_updates()) { // Once more before failure. |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
482 |
// Attempt to add a block failed, no other thread added a block, |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
483 |
// and no deferred updated added a block, then allocation failed. |
57551
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
484 |
log_info(oopstorage, blocks)("%s: failed block allocation", name()); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
485 |
return NULL; |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
486 |
} |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
487 |
} |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
488 |
} |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
489 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
490 |
// Create a new, larger, active array with the same content as the |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
491 |
// current array, and then replace, relinquishing the old array. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
492 |
// Return true if the array was successfully expanded, false to |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
493 |
// indicate allocation failure. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
494 |
bool OopStorage::expand_active_array() { |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
495 |
assert_lock_strong(_allocation_mutex); |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
496 |
ActiveArray* old_array = _active_array; |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
497 |
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
|
498 |
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
|
499 |
name(), new_size); |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
500 |
ActiveArray* new_array = ActiveArray::create(new_size, AllocFailStrategy::RETURN_NULL); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
501 |
if (new_array == NULL) return false; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
502 |
new_array->copy_from(old_array); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
503 |
replace_active_array(new_array); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
504 |
relinquish_block_array(old_array); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
505 |
return true; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
506 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
507 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
508 |
// Make new_array the _active_array. Increments new_array's refcount |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
509 |
// to account for the new reference. The assignment is atomic wrto |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
510 |
// obtain_active_array; once this function returns, it is safe for the |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
511 |
// caller to relinquish the old array. |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
512 |
void OopStorage::replace_active_array(ActiveArray* new_array) { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
513 |
// 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
|
514 |
// Update new_array refcount to account for the new reference. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
515 |
new_array->increment_refcount(); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
516 |
// Install new_array, ensuring its initialization is complete first. |
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
517 |
Atomic::release_store(&_active_array, new_array); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
518 |
// 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
|
519 |
// 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
|
520 |
// 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
|
521 |
// when inserting a StringTable entry. |
eb8d5aeabab3
8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents:
50954
diff
changeset
|
522 |
_protect_active.synchronize(); |
50332
d0d933d61610
8204179: [BACKOUT] OopStorage should use GlobalCounter
kbarrett
parents:
50300
diff
changeset
|
523 |
// All obtain critical sections that could see the old array have |
d0d933d61610
8204179: [BACKOUT] OopStorage should use GlobalCounter
kbarrett
parents:
50300
diff
changeset
|
524 |
// completed, having incremented the refcount of the old array. The |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
525 |
// caller can now safely relinquish the old array. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
526 |
} |
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 |
// Atomically (wrto replace_active_array) get the active array and |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
529 |
// increment its refcount. This provides safe access to the array, |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
530 |
// even if an allocate operation expands and replaces the value of |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
531 |
// _active_array. The caller must relinquish the array when done |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
532 |
// using it. |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
533 |
OopStorage::ActiveArray* OopStorage::obtain_active_array() const { |
51511
eb8d5aeabab3
8209850: Allow NamedThreads to use GlobalCounter critical sections
kbarrett
parents:
50954
diff
changeset
|
534 |
SingleWriterSynchronizer::CriticalSection cs(&_protect_active); |
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
535 |
ActiveArray* result = Atomic::load_acquire(&_active_array); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
536 |
result->increment_refcount(); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
537 |
return result; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
538 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
539 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
540 |
// Decrement refcount of array and destroy if refcount is zero. |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
541 |
void OopStorage::relinquish_block_array(ActiveArray* array) const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
542 |
if (array->decrement_refcount()) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
543 |
assert(array != _active_array, "invariant"); |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
544 |
ActiveArray::destroy(array); |
49977
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 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
548 |
class OopStorage::WithActiveArray : public StackObj { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
549 |
const OopStorage* _storage; |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
550 |
ActiveArray* _active_array; |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
551 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
552 |
public: |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
553 |
WithActiveArray(const OopStorage* storage) : |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
554 |
_storage(storage), |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
555 |
_active_array(storage->obtain_active_array()) |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
556 |
{} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
557 |
|
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
558 |
~WithActiveArray() { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
559 |
_storage->relinquish_block_array(_active_array); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
560 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
561 |
|
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
562 |
ActiveArray& active_array() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
563 |
return *_active_array; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
564 |
} |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
565 |
}; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
566 |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
567 |
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
|
568 |
assert(ptr != NULL, "precondition"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
569 |
return Block::block_for_ptr(this, ptr); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
570 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
571 |
|
48886 | 572 |
static void log_release_transitions(uintx releasing, |
573 |
uintx old_allocated, |
|
574 |
const OopStorage* owner, |
|
575 |
const void* block) { |
|
57551
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
576 |
LogTarget(Trace, oopstorage, blocks) lt; |
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
577 |
if (lt.is_enabled()) { |
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
578 |
LogStream ls(lt); |
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
579 |
if (is_full_bitmask(old_allocated)) { |
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
580 |
ls.print_cr("%s: block not full " PTR_FORMAT, owner->name(), p2i(block)); |
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
581 |
} |
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
582 |
if (releasing == old_allocated) { |
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
583 |
ls.print_cr("%s: block empty " PTR_FORMAT, owner->name(), p2i(block)); |
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
584 |
} |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
585 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
586 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
587 |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
588 |
void OopStorage::Block::release_entries(uintx releasing, OopStorage* owner) { |
48886 | 589 |
assert(releasing != 0, "preconditon"); |
590 |
// Prevent empty block deletion when transitioning to empty. |
|
591 |
Atomic::inc(&_release_refcount); |
|
592 |
||
593 |
// Atomically update allocated bitmask. |
|
594 |
uintx old_allocated = _allocated_bitmask; |
|
595 |
while (true) { |
|
596 |
assert((releasing & ~old_allocated) == 0, "releasing unallocated entries"); |
|
597 |
uintx new_value = old_allocated ^ releasing; |
|
598 |
uintx fetched = Atomic::cmpxchg(new_value, &_allocated_bitmask, old_allocated); |
|
599 |
if (fetched == old_allocated) break; // Successful update. |
|
600 |
old_allocated = fetched; // Retry with updated bitmask. |
|
601 |
} |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
602 |
|
48886 | 603 |
// Now that the bitmask has been updated, if we have a state transition |
604 |
// (updated bitmask is empty or old bitmask was full), atomically push |
|
605 |
// this block onto the deferred updates list. Some future call to |
|
606 |
// reduce_deferred_updates will make any needed changes related to this |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
607 |
// block and _allocation_list. This deferral avoids _allocation_list |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
608 |
// updates and the associated locking here. |
48886 | 609 |
if ((releasing == old_allocated) || is_full_bitmask(old_allocated)) { |
610 |
// Log transitions. Both transitions are possible in a single update. |
|
57551
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
611 |
log_release_transitions(releasing, old_allocated, owner, this); |
48886 | 612 |
// Attempt to claim responsibility for adding this block to the deferred |
613 |
// list, by setting the link to non-NULL by self-looping. If this fails, |
|
614 |
// then someone else has made such a claim and the deferred update has not |
|
615 |
// yet been processed and will include our change, so we don't need to do |
|
616 |
// anything further. |
|
617 |
if (Atomic::replace_if_null(this, &_deferred_updates_next)) { |
|
618 |
// Successfully claimed. Push, with self-loop for end-of-list. |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
619 |
Block* head = owner->_deferred_updates; |
48886 | 620 |
while (true) { |
621 |
_deferred_updates_next = (head == NULL) ? this : head; |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
622 |
Block* fetched = Atomic::cmpxchg(this, &owner->_deferred_updates, head); |
48886 | 623 |
if (fetched == head) break; // Successful update. |
624 |
head = fetched; // Retry with updated head. |
|
625 |
} |
|
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
626 |
// Only request cleanup for to-empty transitions, not for from-full. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
627 |
// There isn't any rush to process from-full transitions. Allocation |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
628 |
// will reduce deferrals before allocating new blocks, so may process |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
629 |
// some. And the service thread will drain the entire deferred list |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
630 |
// if there are any pending to-empty transitions. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
631 |
if (releasing == old_allocated) { |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
632 |
owner->record_needs_cleanup(); |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
633 |
} |
57551
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
634 |
log_trace(oopstorage, blocks)("%s: deferred update " PTR_FORMAT, |
57550
278795ad438a
8228631: Fix inconsistent OopStorage::Block owner usage
kbarrett
parents:
55592
diff
changeset
|
635 |
owner->name(), p2i(this)); |
48886 | 636 |
} |
637 |
} |
|
638 |
// Release hold on empty block deletion. |
|
639 |
Atomic::dec(&_release_refcount); |
|
640 |
} |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
641 |
|
48886 | 642 |
// Process one available deferred update. Returns true if one was processed. |
643 |
bool OopStorage::reduce_deferred_updates() { |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
644 |
assert_lock_strong(_allocation_mutex); |
48886 | 645 |
// Atomically pop a block off the list, if any available. |
646 |
// No ABA issue because this is only called by one thread at a time. |
|
647 |
// The atomicity is wrto pushes by release(). |
|
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
648 |
Block* block = Atomic::load_acquire(&_deferred_updates); |
48886 | 649 |
while (true) { |
650 |
if (block == NULL) return false; |
|
651 |
// Try atomic pop of block from list. |
|
652 |
Block* tail = block->deferred_updates_next(); |
|
653 |
if (block == tail) tail = NULL; // Handle self-loop end marker. |
|
654 |
Block* fetched = Atomic::cmpxchg(tail, &_deferred_updates, block); |
|
655 |
if (fetched == block) break; // Update successful. |
|
656 |
block = fetched; // Retry with updated block. |
|
657 |
} |
|
658 |
block->set_deferred_updates_next(NULL); // Clear tail after updating head. |
|
659 |
// Ensure bitmask read after pop is complete, including clearing tail, for |
|
660 |
// ordering with release(). Without this, we may be processing a stale |
|
661 |
// bitmask state here while blocking a release() operation from recording |
|
662 |
// the deferred update needed for its bitmask change. |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
663 |
OrderAccess::fence(); |
48886 | 664 |
// Process popped block. |
665 |
uintx allocated = block->allocated_bitmask(); |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
666 |
|
48886 | 667 |
// Make membership in list consistent with bitmask state. |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
668 |
if ((_allocation_list.ctail() != NULL) && |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
669 |
((_allocation_list.ctail() == block) || |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
670 |
(_allocation_list.next(*block) != NULL))) { |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
671 |
// Block is in the _allocation_list. |
48886 | 672 |
assert(!is_full_bitmask(allocated), "invariant"); |
673 |
} else if (!is_full_bitmask(allocated)) { |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
674 |
// Block is not in the _allocation_list, but now should be. |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
675 |
_allocation_list.push_front(*block); |
48886 | 676 |
} // Else block is full and not in list, which is correct. |
677 |
||
678 |
// Move empty block to end of list, for possible deletion. |
|
679 |
if (is_empty_bitmask(allocated)) { |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
680 |
_allocation_list.unlink(*block); |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
681 |
_allocation_list.push_back(*block); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
682 |
} |
48886 | 683 |
|
57551
02cffb476ab0
8226796: Reduce debug level logging for oopstorage+blocks
kbarrett
parents:
57550
diff
changeset
|
684 |
log_trace(oopstorage, blocks)("%s: processed deferred update " PTR_FORMAT, |
48886 | 685 |
name(), p2i(block)); |
686 |
return true; // Processed one pending update. |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
687 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
688 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
689 |
inline void check_release_entry(const oop* entry) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
690 |
assert(entry != NULL, "Releasing NULL"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
691 |
assert(*entry == NULL, "Releasing uncleared entry: " PTR_FORMAT, p2i(entry)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
692 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
693 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
694 |
void OopStorage::release(const oop* ptr) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
695 |
check_release_entry(ptr); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
696 |
Block* block = find_block_or_null(ptr); |
48886 | 697 |
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
|
698 |
log_trace(oopstorage, ref)("%s: released " PTR_FORMAT, name(), p2i(ptr)); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
699 |
block->release_entries(block->bitmask_for_entry(ptr), this); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
700 |
Atomic::dec(&_allocation_count); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
701 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
702 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
703 |
void OopStorage::release(const oop* const* ptrs, size_t size) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
704 |
size_t i = 0; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
705 |
while (i < size) { |
48808
2b0b7f222800
8195690: JNI GetObjectRefType doesn't handle NULL
kbarrett
parents:
48787
diff
changeset
|
706 |
check_release_entry(ptrs[i]); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
707 |
Block* block = find_block_or_null(ptrs[i]); |
48886 | 708 |
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
|
709 |
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
|
710 |
size_t count = 0; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
711 |
uintx releasing = 0; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
712 |
for ( ; i < size; ++i) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
713 |
const oop* entry = ptrs[i]; |
48886 | 714 |
check_release_entry(entry); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
715 |
// 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
|
716 |
if (!block->contains(entry)) break; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
717 |
// Add entry to releasing bitmap. |
51752
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
718 |
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
|
719 |
uintx entry_bitmask = block->bitmask_for_entry(entry); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
720 |
assert((releasing & entry_bitmask) == 0, |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
721 |
"Duplicate entry: " PTR_FORMAT, p2i(entry)); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
722 |
releasing |= entry_bitmask; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
723 |
++count; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
724 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
725 |
// Release the contiguous entries that are in block. |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
726 |
block->release_entries(releasing, this); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
727 |
Atomic::sub(count, &_allocation_count); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
728 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
729 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
730 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
731 |
const size_t initial_active_array_size = 8; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
732 |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
733 |
OopStorage::OopStorage(const char* name, |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
734 |
Mutex* allocation_mutex, |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
735 |
Mutex* active_mutex) : |
55592
fd1a4c4b7616
8226793: Replace OopStorage dup_name with os::strdup
kbarrett
parents:
55569
diff
changeset
|
736 |
_name(os::strdup(name)), |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
737 |
_active_array(ActiveArray::create(initial_active_array_size)), |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
738 |
_allocation_list(), |
48886 | 739 |
_deferred_updates(NULL), |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
740 |
_allocation_mutex(allocation_mutex), |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
741 |
_active_mutex(active_mutex), |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
742 |
_allocation_count(0), |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
743 |
_concurrent_iteration_count(0), |
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
744 |
_needs_cleanup(false) |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
745 |
{ |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
746 |
_active_array->increment_refcount(); |
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
747 |
assert(_active_mutex->rank() < _allocation_mutex->rank(), |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
748 |
"%s: active_mutex must have lower rank than allocation_mutex", _name); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
749 |
assert(Service_lock->rank() < _active_mutex->rank(), |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
750 |
"%s: active_mutex must have higher rank than Service_lock", _name); |
54663
f03d5a093093
8074355: make MutexLocker smarter about non-JavaThreads
coleenp
parents:
54623
diff
changeset
|
751 |
assert(_active_mutex->_safepoint_check_required == Mutex::_safepoint_check_never, |
f03d5a093093
8074355: make MutexLocker smarter about non-JavaThreads
coleenp
parents:
54623
diff
changeset
|
752 |
"%s: active mutex requires never safepoint check", _name); |
f03d5a093093
8074355: make MutexLocker smarter about non-JavaThreads
coleenp
parents:
54623
diff
changeset
|
753 |
assert(_allocation_mutex->_safepoint_check_required == Mutex::_safepoint_check_never, |
f03d5a093093
8074355: make MutexLocker smarter about non-JavaThreads
coleenp
parents:
54623
diff
changeset
|
754 |
"%s: allocation mutex requires never safepoint check", _name); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
755 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
756 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
757 |
void OopStorage::delete_empty_block(const Block& block) { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
758 |
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
|
759 |
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
|
760 |
Block::delete_block(block); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
761 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
762 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
763 |
OopStorage::~OopStorage() { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
764 |
Block* block; |
48886 | 765 |
while ((block = _deferred_updates) != NULL) { |
766 |
_deferred_updates = block->deferred_updates_next(); |
|
767 |
block->set_deferred_updates_next(NULL); |
|
768 |
} |
|
50954
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
769 |
while ((block = _allocation_list.head()) != NULL) { |
f85092465b0c
8204834: Fix confusing "allocate" naming in OopStorage
kbarrett
parents:
50513
diff
changeset
|
770 |
_allocation_list.unlink(*block); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
771 |
} |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
772 |
bool unreferenced = _active_array->decrement_refcount(); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
773 |
assert(unreferenced, "deleting storage while _active_array is referenced"); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
774 |
for (size_t i = _active_array->block_count(); 0 < i; ) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
775 |
block = _active_array->at(--i); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
776 |
Block::delete_block(*block); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
777 |
} |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
778 |
ActiveArray::destroy(_active_array); |
55592
fd1a4c4b7616
8226793: Replace OopStorage dup_name with os::strdup
kbarrett
parents:
55569
diff
changeset
|
779 |
os::free(const_cast<char*>(_name)); |
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 |
|
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
782 |
// Managing service thread notifications. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
783 |
// |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
784 |
// We don't want cleanup work to linger indefinitely, but we also don't want |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
785 |
// to run the service thread too often. We're also very limited in what we |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
786 |
// can do in a release operation, where cleanup work is created. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
787 |
// |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
788 |
// When a release operation changes a block's state to empty, it records the |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
789 |
// need for cleanup in both the associated storage object and in the global |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
790 |
// request state. A safepoint cleanup task notifies the service thread when |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
791 |
// there may be cleanup work for any storage object, based on the global |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
792 |
// request state. But that notification is deferred if the service thread |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
793 |
// has run recently, and we also avoid duplicate notifications. The service |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
794 |
// thread updates the timestamp and resets the state flags on every iteration. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
795 |
|
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
796 |
// Global cleanup request state. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
797 |
static volatile bool needs_cleanup_requested = false; |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
798 |
|
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
799 |
// Flag for avoiding duplicate notifications. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
800 |
static bool needs_cleanup_triggered = false; |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
801 |
|
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
802 |
// Time after which a notification can be made. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
803 |
static jlong cleanup_trigger_permit_time = 0; |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
804 |
|
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
805 |
// Minimum time since last service thread check before notification is |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
806 |
// permitted. The value of 500ms was an arbitrary choice; frequent, but not |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
807 |
// too frequent. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
808 |
const jlong cleanup_trigger_defer_period = 500 * NANOSECS_PER_MILLISEC; |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
809 |
|
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
810 |
void OopStorage::trigger_cleanup_if_needed() { |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
811 |
MonitorLocker ml(Service_lock, Monitor::_no_safepoint_check_flag); |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
812 |
if (Atomic::load(&needs_cleanup_requested) && |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
813 |
!needs_cleanup_triggered && |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
814 |
(os::javaTimeNanos() > cleanup_trigger_permit_time)) { |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
815 |
needs_cleanup_triggered = true; |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
816 |
ml.notify_all(); |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
817 |
} |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
818 |
} |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
819 |
|
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
820 |
bool OopStorage::has_cleanup_work_and_reset() { |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
821 |
assert_lock_strong(Service_lock); |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
822 |
cleanup_trigger_permit_time = |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
823 |
os::javaTimeNanos() + cleanup_trigger_defer_period; |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
824 |
needs_cleanup_triggered = false; |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
825 |
// Set the request flag false and return its old value. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
826 |
// Needs to be atomic to avoid dropping a concurrent request. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
827 |
// Can't use Atomic::xchg, which may not support bool. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
828 |
return Atomic::cmpxchg(false, &needs_cleanup_requested, true); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
829 |
} |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
830 |
|
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
831 |
// Record that cleanup is needed, without notifying the Service thread. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
832 |
// Used by release(), where we can't lock even Service_lock. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
833 |
void OopStorage::record_needs_cleanup() { |
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
834 |
// Set local flag first, else service thread could wake up and miss |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
835 |
// the request. This order may instead (rarely) unnecessarily notify. |
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
836 |
Atomic::release_store(&_needs_cleanup, true); |
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
837 |
Atomic::release_store_fence(&needs_cleanup_requested, true); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
838 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
839 |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
840 |
bool OopStorage::delete_empty_blocks() { |
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
841 |
// Service thread might have oopstorage work, but not for this object. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
842 |
// Check for deferred updates even though that's not a service thread |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
843 |
// trigger; since we're here, we might as well process them. |
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
844 |
if (!Atomic::load_acquire(&_needs_cleanup) && |
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
845 |
(Atomic::load_acquire(&_deferred_updates) == NULL)) { |
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
846 |
return false; |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
847 |
} |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
848 |
|
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52421
diff
changeset
|
849 |
MutexLocker ml(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
850 |
|
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
851 |
// Clear the request before processing. |
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
852 |
Atomic::release_store_fence(&_needs_cleanup, false); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
853 |
|
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
854 |
// Other threads could be adding to the empty block count or the |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
855 |
// deferred update list while we're working. Set an upper bound on |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
856 |
// how many updates we'll process and blocks we'll try to release, |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
857 |
// so other threads can't cause an unbounded stay in this function. |
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
858 |
// We add a bit of slop because the reduce_deferred_updates clause |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
859 |
// can cause blocks to be double counted. If there are few blocks |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
860 |
// and many of them are deferred and empty, we might hit the limit |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
861 |
// and spin the caller without doing very much work. Otherwise, |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
862 |
// we don't normally hit the limit anyway, instead running out of |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
863 |
// work to do. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
864 |
size_t limit = block_count() + 10; |
48886 | 865 |
|
866 |
for (size_t i = 0; i < limit; ++i) { |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
867 |
// Process deferred updates, which might make empty blocks available. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
868 |
// Continue checking once deletion starts, since additional updates |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
869 |
// might become available while we're working. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
870 |
if (reduce_deferred_updates()) { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
871 |
// Be safepoint-polite while looping. |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52421
diff
changeset
|
872 |
MutexUnlocker ul(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
873 |
ThreadBlockInVM tbiv(JavaThread::current()); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
874 |
} else { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
875 |
Block* block = _allocation_list.tail(); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
876 |
if ((block == NULL) || !block->is_empty()) { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
877 |
return false; |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
878 |
} else if (!block->is_safe_to_delete()) { |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
879 |
// Look for other work while waiting for block to be deletable. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
880 |
break; |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
881 |
} |
48886 | 882 |
|
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
883 |
// Try to delete the block. First, try to remove from _active_array. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
884 |
{ |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52421
diff
changeset
|
885 |
MutexLocker aml(_active_mutex, Mutex::_no_safepoint_check_flag); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
886 |
// Don't interfere with an active concurrent iteration. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
887 |
// Instead, give up immediately. There is more work to do, |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
888 |
// but don't re-notify, to avoid useless spinning of the |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
889 |
// service thread. Instead, iteration completion notifies. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
890 |
if (_concurrent_iteration_count > 0) return true; |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
891 |
_active_array->remove(block); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
892 |
} |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
893 |
// Remove block from _allocation_list and delete it. |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
894 |
_allocation_list.unlink(*block); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
895 |
// Be safepoint-polite while deleting and looping. |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52421
diff
changeset
|
896 |
MutexUnlocker ul(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
897 |
delete_empty_block(*block); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
898 |
ThreadBlockInVM tbiv(JavaThread::current()); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
899 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
900 |
} |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
901 |
// Exceeded work limit or can't delete last block. This will |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
902 |
// cause the service thread to loop, giving other subtasks an |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
903 |
// opportunity to run too. There's no need for a notification, |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
904 |
// because we are part of the service thread (unless gtesting). |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
905 |
record_needs_cleanup(); |
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
906 |
return true; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
907 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
908 |
|
48886 | 909 |
OopStorage::EntryStatus OopStorage::allocation_status(const oop* ptr) const { |
910 |
const Block* block = find_block_or_null(ptr); |
|
911 |
if (block != NULL) { |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
912 |
// Prevent block deletion and _active_array modification. |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52421
diff
changeset
|
913 |
MutexLocker ml(_allocation_mutex, Mutex::_no_safepoint_check_flag); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
914 |
// Block could be a false positive, so get index carefully. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
915 |
size_t index = Block::active_index_safe(block); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
916 |
if ((index < _active_array->block_count()) && |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
917 |
(block == _active_array->at(index)) && |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
918 |
block->contains(ptr)) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
919 |
if ((block->allocated_bitmask() & block->bitmask_for_entry(ptr)) != 0) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
920 |
return ALLOCATED_ENTRY; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
921 |
} else { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
922 |
return UNALLOCATED_ENTRY; |
48886 | 923 |
} |
924 |
} |
|
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
925 |
} |
48886 | 926 |
return INVALID_ENTRY; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
927 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
928 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
929 |
size_t OopStorage::allocation_count() const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
930 |
return _allocation_count; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
931 |
} |
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 |
size_t OopStorage::block_count() const { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
934 |
WithActiveArray wab(this); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
935 |
// Count access is racy, but don't care. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
936 |
return wab.active_array().block_count(); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
937 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
938 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
939 |
size_t OopStorage::total_memory_usage() const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
940 |
size_t total_size = sizeof(OopStorage); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
941 |
total_size += strlen(name()) + 1; |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
942 |
total_size += sizeof(ActiveArray); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
943 |
WithActiveArray wab(this); |
50209
2fdce199fcb9
8202863: Rename OopStorage inner collection classes
kbarrett
parents:
49977
diff
changeset
|
944 |
const ActiveArray& blocks = wab.active_array(); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
945 |
// Count access is racy, but don't care. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
946 |
total_size += blocks.block_count() * Block::allocation_size(); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
947 |
total_size += blocks.size() * sizeof(Block*); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
948 |
return total_size; |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
949 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
950 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
951 |
// Parallel iteration support |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
952 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
953 |
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
|
954 |
uint configured = concurrent ? ConcGCThreads : ParallelGCThreads; |
188a87cbfac3
8203843: BasicParState::default_estimated_thread_count(false) can return 0 in gtest
kbarrett
parents:
50209
diff
changeset
|
955 |
return MAX2(1u, configured); // Never estimate zero threads. |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
956 |
} |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
957 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
958 |
OopStorage::BasicParState::BasicParState(const OopStorage* storage, |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
959 |
uint estimated_thread_count, |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
960 |
bool concurrent) : |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
961 |
_storage(storage), |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
962 |
_active_array(_storage->obtain_active_array()), |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
963 |
_block_count(0), // initialized properly below |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
964 |
_next_block(0), |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
965 |
_estimated_thread_count(estimated_thread_count), |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
966 |
_concurrent(concurrent) |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
967 |
{ |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
968 |
assert(estimated_thread_count > 0, "estimated thread count must be positive"); |
52037
d2a6c3cbc110
8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents:
51752
diff
changeset
|
969 |
update_concurrent_iteration_count(1); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
970 |
// Get the block count *after* iteration state updated, so concurrent |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
971 |
// empty block deletion is suppressed and can't reduce the count. But |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
972 |
// 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
|
973 |
// was fully initialized; see ActiveArray::push. |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
974 |
_block_count = _active_array->block_count_acquire(); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
975 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
976 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
977 |
OopStorage::BasicParState::~BasicParState() { |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
978 |
_storage->relinquish_block_array(_active_array); |
52037
d2a6c3cbc110
8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents:
51752
diff
changeset
|
979 |
update_concurrent_iteration_count(-1); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
980 |
if (_concurrent) { |
55569
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
981 |
// We may have deferred some cleanup work. |
8e3a0ebf3497
8226366: Excessive ServiceThread wakeups for OopStorage cleanup
kbarrett
parents:
54663
diff
changeset
|
982 |
const_cast<OopStorage*>(_storage)->record_needs_cleanup(); |
52421
3021c1ad958b
8210986: Add OopStorage cleanup to ServiceThread
kbarrett
parents:
52037
diff
changeset
|
983 |
} |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
984 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
985 |
|
52037
d2a6c3cbc110
8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents:
51752
diff
changeset
|
986 |
void OopStorage::BasicParState::update_concurrent_iteration_count(int value) { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
987 |
if (_concurrent) { |
54623
1126f0607c70
8222811: Consolidate MutexLockerEx and MutexLocker
coleenp
parents:
52421
diff
changeset
|
988 |
MutexLocker ml(_storage->_active_mutex, Mutex::_no_safepoint_check_flag); |
52037
d2a6c3cbc110
8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents:
51752
diff
changeset
|
989 |
_storage->_concurrent_iteration_count += value; |
d2a6c3cbc110
8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents:
51752
diff
changeset
|
990 |
assert(_storage->_concurrent_iteration_count >= 0, "invariant"); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
991 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
992 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
993 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
994 |
bool OopStorage::BasicParState::claim_next_segment(IterationData* data) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
995 |
data->_processed += data->_segment_end - data->_segment_start; |
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
57551
diff
changeset
|
996 |
size_t start = Atomic::load_acquire(&_next_block); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
997 |
if (start >= _block_count) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
998 |
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
|
999 |
} |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1000 |
// 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
|
1001 |
// avoid deciding there are many available and selecting a large |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1002 |
// quantity, get delayed, and then end up claiming most or all of |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1003 |
// the remaining largish amount of work, leaving nothing for other |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1004 |
// threads to do. But too small a step can lead to contention |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1005 |
// over _next_block, esp. when the work per block is small. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1006 |
size_t max_step = 10; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1007 |
size_t remaining = _block_count - start; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1008 |
size_t step = MIN2(max_step, 1 + (remaining / _estimated_thread_count)); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1009 |
// Atomic::add with possible overshoot. This can perform better |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1010 |
// than a CAS loop on some platforms when there is contention. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1011 |
// We can cope with the uncertainty by recomputing start/end from |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1012 |
// the result of the add, and dealing with potential overshoot. |
59249
29b0d0b61615
8234737: Harmonize parameter order in Atomic - add
stefank
parents:
59247
diff
changeset
|
1013 |
size_t end = Atomic::add(&_next_block, step); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1014 |
// _next_block may have changed, so recompute start from result of add. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1015 |
start = end - step; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1016 |
// _next_block may have changed so much that end has overshot. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1017 |
end = MIN2(end, _block_count); |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1018 |
// _next_block may have changed so much that even start has overshot. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1019 |
if (start < _block_count) { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1020 |
// Record claimed segment for iteration. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1021 |
data->_segment_start = start; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1022 |
data->_segment_end = end; |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1023 |
return true; // Success. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1024 |
} else { |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1025 |
// No more blocks to claim. |
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1026 |
return finish_iteration(data); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1027 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1028 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1029 |
|
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1030 |
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
|
1031 |
log_info(oopstorage, blocks, stats) |
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
1032 |
("Parallel iteration on %s: blocks = " SIZE_FORMAT |
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
1033 |
", processed = " SIZE_FORMAT " (%2.f%%)", |
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
1034 |
_storage->name(), _block_count, data->_processed, |
43323ced5e40
8210724: Change the verbosity threshold of logging for [oopstorage,ref]
jcbeyler
parents:
51511
diff
changeset
|
1035 |
percent_of(data->_processed, _block_count)); |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1036 |
return false; |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1037 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1038 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1039 |
const char* OopStorage::name() const { return _name; } |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1040 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1041 |
#ifndef PRODUCT |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1042 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1043 |
void OopStorage::print_on(outputStream* st) const { |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1044 |
size_t allocations = _allocation_count; |
49977
9f758f0bb058
8200557: OopStorage parallel iteration scales poorly
kbarrett
parents:
49711
diff
changeset
|
1045 |
size_t blocks = _active_array->block_count(); |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1046 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1047 |
double data_size = section_size * section_count; |
48886 | 1048 |
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
|
1049 |
|
48886 | 1050 |
st->print("%s: " SIZE_FORMAT " entries in " SIZE_FORMAT " blocks (%.F%%), " SIZE_FORMAT " bytes", |
1051 |
name(), allocations, blocks, alloc_percentage, total_memory_usage()); |
|
52037
d2a6c3cbc110
8211718: Supporting multiple concurrent OopStorage iterators
eosterlund
parents:
51752
diff
changeset
|
1052 |
if (_concurrent_iteration_count > 0) { |
48787
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1053 |
st->print(", concurrent iteration active"); |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1054 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1055 |
} |
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1056 |
|
7638bf98a312
8194312: Support parallel and concurrent JNI global handle processing
kbarrett
parents:
diff
changeset
|
1057 |
#endif // !PRODUCT |