annotate src/share/vm/utilities/hashtable.cpp @ 1721:413ad0331a0c

6977924: Changes for 6975078 produce build error with certain gcc versions Summary: The changes introduced for 6975078 assign badHeapOopVal to the _allocation field in the ResourceObj class. In 32 bit linux builds with certain versions of gcc this assignment will be flagged as an error while compiling allocation.cpp. In 32 bit builds the constant value badHeapOopVal (which is cast to an intptr_t) is negative. The _allocation field is typed as an unsigned intptr_t and gcc catches this as an error. Reviewed-by: jcoomes, ysr, phh
author johnc
date Wed, 18 Aug 2010 10:59:06 -0700
parents c18cbe5936b8
children f95d63e2154a
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 470
diff changeset
2 * Copyright (c) 2003, 2008, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 470
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 470
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 470
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
a61af66fc99e Initial load
duke
parents:
diff changeset
25 # include "incls/_precompiled.incl"
a61af66fc99e Initial load
duke
parents:
diff changeset
26 # include "incls/_hashtable.cpp.incl"
a61af66fc99e Initial load
duke
parents:
diff changeset
27
a61af66fc99e Initial load
duke
parents:
diff changeset
28 HS_DTRACE_PROBE_DECL4(hs_private, hashtable__new_entry,
a61af66fc99e Initial load
duke
parents:
diff changeset
29 void*, unsigned int, oop, void*);
a61af66fc99e Initial load
duke
parents:
diff changeset
30
a61af66fc99e Initial load
duke
parents:
diff changeset
31 // This is a generic hashtable, designed to be used for the symbol
a61af66fc99e Initial load
duke
parents:
diff changeset
32 // and string tables.
a61af66fc99e Initial load
duke
parents:
diff changeset
33 //
a61af66fc99e Initial load
duke
parents:
diff changeset
34 // It is implemented as an open hash table with a fixed number of buckets.
a61af66fc99e Initial load
duke
parents:
diff changeset
35 //
a61af66fc99e Initial load
duke
parents:
diff changeset
36 // %note:
a61af66fc99e Initial load
duke
parents:
diff changeset
37 // - HashtableEntrys are allocated in blocks to reduce the space overhead.
a61af66fc99e Initial load
duke
parents:
diff changeset
38
a61af66fc99e Initial load
duke
parents:
diff changeset
39 BasicHashtableEntry* BasicHashtable::new_entry(unsigned int hashValue) {
a61af66fc99e Initial load
duke
parents:
diff changeset
40 BasicHashtableEntry* entry;
a61af66fc99e Initial load
duke
parents:
diff changeset
41
a61af66fc99e Initial load
duke
parents:
diff changeset
42 if (_free_list) {
a61af66fc99e Initial load
duke
parents:
diff changeset
43 entry = _free_list;
a61af66fc99e Initial load
duke
parents:
diff changeset
44 _free_list = _free_list->next();
a61af66fc99e Initial load
duke
parents:
diff changeset
45 } else {
432
275a3b7ff0d6 6770949: minor tweaks before 6655638
jrose
parents: 0
diff changeset
46 if (_first_free_entry + _entry_size >= _end_block) {
275a3b7ff0d6 6770949: minor tweaks before 6655638
jrose
parents: 0
diff changeset
47 int block_size = MIN2(512, MAX2((int)_table_size / 2, (int)_number_of_entries));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
48 int len = _entry_size * block_size;
432
275a3b7ff0d6 6770949: minor tweaks before 6655638
jrose
parents: 0
diff changeset
49 len = 1 << log2_intptr(len); // round down to power of 2
275a3b7ff0d6 6770949: minor tweaks before 6655638
jrose
parents: 0
diff changeset
50 assert(len >= _entry_size, "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
51 _first_free_entry = NEW_C_HEAP_ARRAY(char, len);
a61af66fc99e Initial load
duke
parents:
diff changeset
52 _end_block = _first_free_entry + len;
a61af66fc99e Initial load
duke
parents:
diff changeset
53 }
a61af66fc99e Initial load
duke
parents:
diff changeset
54 entry = (BasicHashtableEntry*)_first_free_entry;
a61af66fc99e Initial load
duke
parents:
diff changeset
55 _first_free_entry += _entry_size;
a61af66fc99e Initial load
duke
parents:
diff changeset
56 }
a61af66fc99e Initial load
duke
parents:
diff changeset
57
432
275a3b7ff0d6 6770949: minor tweaks before 6655638
jrose
parents: 0
diff changeset
58 assert(_entry_size % HeapWordSize == 0, "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
59 entry->set_hash(hashValue);
a61af66fc99e Initial load
duke
parents:
diff changeset
60 return entry;
a61af66fc99e Initial load
duke
parents:
diff changeset
61 }
a61af66fc99e Initial load
duke
parents:
diff changeset
62
a61af66fc99e Initial load
duke
parents:
diff changeset
63
a61af66fc99e Initial load
duke
parents:
diff changeset
64 HashtableEntry* Hashtable::new_entry(unsigned int hashValue, oop obj) {
a61af66fc99e Initial load
duke
parents:
diff changeset
65 HashtableEntry* entry;
a61af66fc99e Initial load
duke
parents:
diff changeset
66
a61af66fc99e Initial load
duke
parents:
diff changeset
67 entry = (HashtableEntry*)BasicHashtable::new_entry(hashValue);
a61af66fc99e Initial load
duke
parents:
diff changeset
68 entry->set_literal(obj); // clears literal string field
a61af66fc99e Initial load
duke
parents:
diff changeset
69 HS_DTRACE_PROBE4(hs_private, hashtable__new_entry,
a61af66fc99e Initial load
duke
parents:
diff changeset
70 this, hashValue, obj, entry);
a61af66fc99e Initial load
duke
parents:
diff changeset
71 return entry;
a61af66fc99e Initial load
duke
parents:
diff changeset
72 }
a61af66fc99e Initial load
duke
parents:
diff changeset
73
a61af66fc99e Initial load
duke
parents:
diff changeset
74
a61af66fc99e Initial load
duke
parents:
diff changeset
75 // GC support
a61af66fc99e Initial load
duke
parents:
diff changeset
76
a61af66fc99e Initial load
duke
parents:
diff changeset
77 void Hashtable::unlink(BoolObjectClosure* is_alive) {
a61af66fc99e Initial load
duke
parents:
diff changeset
78 // Readers of the table are unlocked, so we should only be removing
a61af66fc99e Initial load
duke
parents:
diff changeset
79 // entries at a safepoint.
a61af66fc99e Initial load
duke
parents:
diff changeset
80 assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
a61af66fc99e Initial load
duke
parents:
diff changeset
81 for (int i = 0; i < table_size(); ++i) {
a61af66fc99e Initial load
duke
parents:
diff changeset
82 for (HashtableEntry** p = bucket_addr(i); *p != NULL; ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
83 HashtableEntry* entry = *p;
a61af66fc99e Initial load
duke
parents:
diff changeset
84 if (entry->is_shared()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
85 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
86 }
a61af66fc99e Initial load
duke
parents:
diff changeset
87 assert(entry->literal() != NULL, "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
88 if (is_alive->do_object_b(entry->literal())) {
a61af66fc99e Initial load
duke
parents:
diff changeset
89 p = entry->next_addr();
a61af66fc99e Initial load
duke
parents:
diff changeset
90 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
91 *p = entry->next();
a61af66fc99e Initial load
duke
parents:
diff changeset
92 free_entry(entry);
a61af66fc99e Initial load
duke
parents:
diff changeset
93 }
a61af66fc99e Initial load
duke
parents:
diff changeset
94 }
a61af66fc99e Initial load
duke
parents:
diff changeset
95 }
a61af66fc99e Initial load
duke
parents:
diff changeset
96 }
a61af66fc99e Initial load
duke
parents:
diff changeset
97
a61af66fc99e Initial load
duke
parents:
diff changeset
98
a61af66fc99e Initial load
duke
parents:
diff changeset
99 void Hashtable::oops_do(OopClosure* f) {
a61af66fc99e Initial load
duke
parents:
diff changeset
100 for (int i = 0; i < table_size(); ++i) {
a61af66fc99e Initial load
duke
parents:
diff changeset
101 HashtableEntry** p = bucket_addr(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
102 HashtableEntry* entry = bucket(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
103 while (entry != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
104 f->do_oop(entry->literal_addr());
a61af66fc99e Initial load
duke
parents:
diff changeset
105
a61af66fc99e Initial load
duke
parents:
diff changeset
106 // Did the closure remove the literal from the table?
a61af66fc99e Initial load
duke
parents:
diff changeset
107 if (entry->literal() == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
108 assert(!entry->is_shared(), "immutable hashtable entry?");
a61af66fc99e Initial load
duke
parents:
diff changeset
109 *p = entry->next();
a61af66fc99e Initial load
duke
parents:
diff changeset
110 free_entry(entry);
a61af66fc99e Initial load
duke
parents:
diff changeset
111 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
112 p = entry->next_addr();
a61af66fc99e Initial load
duke
parents:
diff changeset
113 }
a61af66fc99e Initial load
duke
parents:
diff changeset
114 entry = (HashtableEntry*)HashtableEntry::make_ptr(*p);
a61af66fc99e Initial load
duke
parents:
diff changeset
115 }
a61af66fc99e Initial load
duke
parents:
diff changeset
116 }
a61af66fc99e Initial load
duke
parents:
diff changeset
117 }
a61af66fc99e Initial load
duke
parents:
diff changeset
118
a61af66fc99e Initial load
duke
parents:
diff changeset
119
a61af66fc99e Initial load
duke
parents:
diff changeset
120 // Reverse the order of elements in the hash buckets.
a61af66fc99e Initial load
duke
parents:
diff changeset
121
a61af66fc99e Initial load
duke
parents:
diff changeset
122 void BasicHashtable::reverse() {
a61af66fc99e Initial load
duke
parents:
diff changeset
123
a61af66fc99e Initial load
duke
parents:
diff changeset
124 for (int i = 0; i < _table_size; ++i) {
a61af66fc99e Initial load
duke
parents:
diff changeset
125 BasicHashtableEntry* new_list = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
126 BasicHashtableEntry* p = bucket(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
127 while (p != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
128 BasicHashtableEntry* next = p->next();
a61af66fc99e Initial load
duke
parents:
diff changeset
129 p->set_next(new_list);
a61af66fc99e Initial load
duke
parents:
diff changeset
130 new_list = p;
a61af66fc99e Initial load
duke
parents:
diff changeset
131 p = next;
a61af66fc99e Initial load
duke
parents:
diff changeset
132 }
a61af66fc99e Initial load
duke
parents:
diff changeset
133 *bucket_addr(i) = new_list;
a61af66fc99e Initial load
duke
parents:
diff changeset
134 }
a61af66fc99e Initial load
duke
parents:
diff changeset
135 }
a61af66fc99e Initial load
duke
parents:
diff changeset
136
a61af66fc99e Initial load
duke
parents:
diff changeset
137
a61af66fc99e Initial load
duke
parents:
diff changeset
138 // Copy the table to the shared space.
a61af66fc99e Initial load
duke
parents:
diff changeset
139
a61af66fc99e Initial load
duke
parents:
diff changeset
140 void BasicHashtable::copy_table(char** top, char* end) {
a61af66fc99e Initial load
duke
parents:
diff changeset
141
a61af66fc99e Initial load
duke
parents:
diff changeset
142 // Dump the hash table entries.
a61af66fc99e Initial load
duke
parents:
diff changeset
143
a61af66fc99e Initial load
duke
parents:
diff changeset
144 intptr_t *plen = (intptr_t*)(*top);
a61af66fc99e Initial load
duke
parents:
diff changeset
145 *top += sizeof(*plen);
a61af66fc99e Initial load
duke
parents:
diff changeset
146
a61af66fc99e Initial load
duke
parents:
diff changeset
147 int i;
a61af66fc99e Initial load
duke
parents:
diff changeset
148 for (i = 0; i < _table_size; ++i) {
a61af66fc99e Initial load
duke
parents:
diff changeset
149 for (BasicHashtableEntry** p = _buckets[i].entry_addr();
a61af66fc99e Initial load
duke
parents:
diff changeset
150 *p != NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
151 p = (*p)->next_addr()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
152 if (*top + entry_size() > end) {
a61af66fc99e Initial load
duke
parents:
diff changeset
153 warning("\nThe shared miscellaneous data space is not large "
a61af66fc99e Initial load
duke
parents:
diff changeset
154 "enough to \npreload requested classes. Use "
a61af66fc99e Initial load
duke
parents:
diff changeset
155 "-XX:SharedMiscDataSize= to increase \nthe initial "
a61af66fc99e Initial load
duke
parents:
diff changeset
156 "size of the miscellaneous data space.\n");
a61af66fc99e Initial load
duke
parents:
diff changeset
157 exit(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
158 }
a61af66fc99e Initial load
duke
parents:
diff changeset
159 *p = (BasicHashtableEntry*)memcpy(*top, *p, entry_size());
a61af66fc99e Initial load
duke
parents:
diff changeset
160 *top += entry_size();
a61af66fc99e Initial load
duke
parents:
diff changeset
161 }
a61af66fc99e Initial load
duke
parents:
diff changeset
162 }
a61af66fc99e Initial load
duke
parents:
diff changeset
163 *plen = (char*)(*top) - (char*)plen - sizeof(*plen);
a61af66fc99e Initial load
duke
parents:
diff changeset
164
a61af66fc99e Initial load
duke
parents:
diff changeset
165 // Set the shared bit.
a61af66fc99e Initial load
duke
parents:
diff changeset
166
a61af66fc99e Initial load
duke
parents:
diff changeset
167 for (i = 0; i < _table_size; ++i) {
a61af66fc99e Initial load
duke
parents:
diff changeset
168 for (BasicHashtableEntry* p = bucket(i); p != NULL; p = p->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
169 p->set_shared();
a61af66fc99e Initial load
duke
parents:
diff changeset
170 }
a61af66fc99e Initial load
duke
parents:
diff changeset
171 }
a61af66fc99e Initial load
duke
parents:
diff changeset
172 }
a61af66fc99e Initial load
duke
parents:
diff changeset
173
a61af66fc99e Initial load
duke
parents:
diff changeset
174
a61af66fc99e Initial load
duke
parents:
diff changeset
175
a61af66fc99e Initial load
duke
parents:
diff changeset
176 // Reverse the order of elements in the hash buckets.
a61af66fc99e Initial load
duke
parents:
diff changeset
177
a61af66fc99e Initial load
duke
parents:
diff changeset
178 void Hashtable::reverse(void* boundary) {
a61af66fc99e Initial load
duke
parents:
diff changeset
179
a61af66fc99e Initial load
duke
parents:
diff changeset
180 for (int i = 0; i < table_size(); ++i) {
a61af66fc99e Initial load
duke
parents:
diff changeset
181 HashtableEntry* high_list = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
182 HashtableEntry* low_list = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
183 HashtableEntry* last_low_entry = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
184 HashtableEntry* p = bucket(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
185 while (p != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
186 HashtableEntry* next = p->next();
a61af66fc99e Initial load
duke
parents:
diff changeset
187 if ((void*)p->literal() >= boundary) {
a61af66fc99e Initial load
duke
parents:
diff changeset
188 p->set_next(high_list);
a61af66fc99e Initial load
duke
parents:
diff changeset
189 high_list = p;
a61af66fc99e Initial load
duke
parents:
diff changeset
190 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
191 p->set_next(low_list);
a61af66fc99e Initial load
duke
parents:
diff changeset
192 low_list = p;
a61af66fc99e Initial load
duke
parents:
diff changeset
193 if (last_low_entry == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
194 last_low_entry = p;
a61af66fc99e Initial load
duke
parents:
diff changeset
195 }
a61af66fc99e Initial load
duke
parents:
diff changeset
196 }
a61af66fc99e Initial load
duke
parents:
diff changeset
197 p = next;
a61af66fc99e Initial load
duke
parents:
diff changeset
198 }
a61af66fc99e Initial load
duke
parents:
diff changeset
199 if (low_list != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
200 *bucket_addr(i) = low_list;
a61af66fc99e Initial load
duke
parents:
diff changeset
201 last_low_entry->set_next(high_list);
a61af66fc99e Initial load
duke
parents:
diff changeset
202 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
203 *bucket_addr(i) = high_list;
a61af66fc99e Initial load
duke
parents:
diff changeset
204 }
a61af66fc99e Initial load
duke
parents:
diff changeset
205 }
a61af66fc99e Initial load
duke
parents:
diff changeset
206 }
a61af66fc99e Initial load
duke
parents:
diff changeset
207
a61af66fc99e Initial load
duke
parents:
diff changeset
208
a61af66fc99e Initial load
duke
parents:
diff changeset
209 // Dump the hash table buckets.
a61af66fc99e Initial load
duke
parents:
diff changeset
210
a61af66fc99e Initial load
duke
parents:
diff changeset
211 void BasicHashtable::copy_buckets(char** top, char* end) {
a61af66fc99e Initial load
duke
parents:
diff changeset
212 intptr_t len = _table_size * sizeof(HashtableBucket);
a61af66fc99e Initial load
duke
parents:
diff changeset
213 *(intptr_t*)(*top) = len;
a61af66fc99e Initial load
duke
parents:
diff changeset
214 *top += sizeof(intptr_t);
a61af66fc99e Initial load
duke
parents:
diff changeset
215
a61af66fc99e Initial load
duke
parents:
diff changeset
216 *(intptr_t*)(*top) = _number_of_entries;
a61af66fc99e Initial load
duke
parents:
diff changeset
217 *top += sizeof(intptr_t);
a61af66fc99e Initial load
duke
parents:
diff changeset
218
a61af66fc99e Initial load
duke
parents:
diff changeset
219 if (*top + len > end) {
a61af66fc99e Initial load
duke
parents:
diff changeset
220 warning("\nThe shared miscellaneous data space is not large "
a61af66fc99e Initial load
duke
parents:
diff changeset
221 "enough to \npreload requested classes. Use "
a61af66fc99e Initial load
duke
parents:
diff changeset
222 "-XX:SharedMiscDataSize= to increase \nthe initial "
a61af66fc99e Initial load
duke
parents:
diff changeset
223 "size of the miscellaneous data space.\n");
a61af66fc99e Initial load
duke
parents:
diff changeset
224 exit(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
225 }
a61af66fc99e Initial load
duke
parents:
diff changeset
226 _buckets = (HashtableBucket*)memcpy(*top, _buckets, len);
a61af66fc99e Initial load
duke
parents:
diff changeset
227 *top += len;
a61af66fc99e Initial load
duke
parents:
diff changeset
228 }
a61af66fc99e Initial load
duke
parents:
diff changeset
229
a61af66fc99e Initial load
duke
parents:
diff changeset
230
a61af66fc99e Initial load
duke
parents:
diff changeset
231 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
232
a61af66fc99e Initial load
duke
parents:
diff changeset
233 void Hashtable::print() {
a61af66fc99e Initial load
duke
parents:
diff changeset
234 ResourceMark rm;
a61af66fc99e Initial load
duke
parents:
diff changeset
235
a61af66fc99e Initial load
duke
parents:
diff changeset
236 for (int i = 0; i < table_size(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
237 HashtableEntry* entry = bucket(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
238 while(entry != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
239 tty->print("%d : ", i);
a61af66fc99e Initial load
duke
parents:
diff changeset
240 entry->literal()->print();
a61af66fc99e Initial load
duke
parents:
diff changeset
241 tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
242 entry = entry->next();
a61af66fc99e Initial load
duke
parents:
diff changeset
243 }
a61af66fc99e Initial load
duke
parents:
diff changeset
244 }
a61af66fc99e Initial load
duke
parents:
diff changeset
245 }
a61af66fc99e Initial load
duke
parents:
diff changeset
246
a61af66fc99e Initial load
duke
parents:
diff changeset
247
a61af66fc99e Initial load
duke
parents:
diff changeset
248 void BasicHashtable::verify() {
a61af66fc99e Initial load
duke
parents:
diff changeset
249 int count = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
250 for (int i = 0; i < table_size(); i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
251 for (BasicHashtableEntry* p = bucket(i); p != NULL; p = p->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
252 ++count;
a61af66fc99e Initial load
duke
parents:
diff changeset
253 }
a61af66fc99e Initial load
duke
parents:
diff changeset
254 }
a61af66fc99e Initial load
duke
parents:
diff changeset
255 assert(count == number_of_entries(), "number of hashtable entries incorrect");
a61af66fc99e Initial load
duke
parents:
diff changeset
256 }
a61af66fc99e Initial load
duke
parents:
diff changeset
257
a61af66fc99e Initial load
duke
parents:
diff changeset
258
a61af66fc99e Initial load
duke
parents:
diff changeset
259 #endif // PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
260
a61af66fc99e Initial load
duke
parents:
diff changeset
261
a61af66fc99e Initial load
duke
parents:
diff changeset
262 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
263
a61af66fc99e Initial load
duke
parents:
diff changeset
264 void BasicHashtable::verify_lookup_length(double load) {
a61af66fc99e Initial load
duke
parents:
diff changeset
265 if ((double)_lookup_length / (double)_lookup_count > load * 2.0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
266 warning("Performance bug: SystemDictionary lookup_count=%d "
a61af66fc99e Initial load
duke
parents:
diff changeset
267 "lookup_length=%d average=%lf load=%f",
a61af66fc99e Initial load
duke
parents:
diff changeset
268 _lookup_count, _lookup_length,
a61af66fc99e Initial load
duke
parents:
diff changeset
269 (double) _lookup_length / _lookup_count, load);
a61af66fc99e Initial load
duke
parents:
diff changeset
270 }
a61af66fc99e Initial load
duke
parents:
diff changeset
271 }
a61af66fc99e Initial load
duke
parents:
diff changeset
272
a61af66fc99e Initial load
duke
parents:
diff changeset
273 #endif