annotate src/share/vm/classfile/symbolTable.cpp @ 6162:e9140bf80b4a

7158800: Improve storage of symbol tables Summary: Use an alternate version of hashing algorithm for symbol string tables and after a certain bucket size to improve performance Reviewed-by: pbk, kamg, dlong, kvn, fparain
author coleenp
date Wed, 13 Jun 2012 19:52:59 -0400
parents fc9d8850ab8b
children 246d977b51f2
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
2 * Copyright (c) 1997, 2012, 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: 665
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 665
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: 665
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
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
25 #include "precompiled.hpp"
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
26 #include "classfile/altHashing.hpp"
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
27 #include "classfile/javaClasses.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
28 #include "classfile/symbolTable.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
29 #include "classfile/systemDictionary.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
30 #include "gc_interface/collectedHeap.inline.hpp"
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
31 #include "memory/allocation.inline.hpp"
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
32 #include "memory/filemap.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
33 #include "memory/gcLocker.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
34 #include "oops/oop.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
35 #include "oops/oop.inline2.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
36 #include "runtime/mutexLocker.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
37 #include "utilities/hashtable.inline.hpp"
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
38 #include "utilities/numberSeq.hpp"
0
a61af66fc99e Initial load
duke
parents:
diff changeset
39
a61af66fc99e Initial load
duke
parents:
diff changeset
40 // --------------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
41
a61af66fc99e Initial load
duke
parents:
diff changeset
42 SymbolTable* SymbolTable::_the_table = NULL;
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
43 // Static arena for symbols that are not deallocated
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
44 Arena* SymbolTable::_arena = NULL;
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
45 bool SymbolTable::_needs_rehashing = false;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
46 jint SymbolTable::_seed = 0;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
47
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
48 Symbol* SymbolTable::allocate_symbol(const u1* name, int len, bool c_heap, TRAPS) {
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
49 // Don't allow symbols to be created which cannot fit in a Symbol*.
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
50 if (len > Symbol::max_length()) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
51 THROW_MSG_0(vmSymbols::java_lang_InternalError(),
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
52 "name is too long to represent");
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
53 }
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
54 Symbol* sym;
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
55 // Allocate symbols in the C heap when dumping shared spaces in case there
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
56 // are temporary symbols we can remove.
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
57 if (c_heap || DumpSharedSpaces) {
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
58 // refcount starts as 1
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
59 sym = new (len, THREAD) Symbol(name, len, 1);
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
60 } else {
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
61 sym = new (len, arena(), THREAD) Symbol(name, len, -1);
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
62 }
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
63 assert(sym != NULL, "new should call vm_exit_out_of_memory if C_HEAP is exhausted");
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
64 return sym;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
65 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
66
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
67 void SymbolTable::initialize_symbols(int arena_alloc_size) {
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
68 // Initialize the arena for global symbols, size passed in depends on CDS.
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
69 if (arena_alloc_size == 0) {
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
70 _arena = new Arena();
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
71 } else {
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
72 _arena = new Arena(arena_alloc_size);
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
73 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
74 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
75
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
76 // Call function for all symbols in the symbol table.
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
77 void SymbolTable::symbols_do(SymbolClosure *cl) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
78 const int n = the_table()->table_size();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
79 for (int i = 0; i < n; i++) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
80 for (HashtableEntry<Symbol*>* p = the_table()->bucket(i);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
81 p != NULL;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
82 p = p->next()) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
83 cl->do_symbol(p->literal_addr());
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
84 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
85 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
86 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
87
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
88 int SymbolTable::symbols_removed = 0;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
89 int SymbolTable::symbols_counted = 0;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
90
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
91 // Remove unreferenced symbols from the symbol table
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
92 // This is done late during GC.
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
93 void SymbolTable::unlink() {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
94 int removed = 0;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
95 int total = 0;
2336
df1347358fe6 7024584: Symbol printouts shouldnt be under PrintGCDetails
coleenp
parents: 2177
diff changeset
96 size_t memory_total = 0;
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
97 for (int i = 0; i < the_table()->table_size(); ++i) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
98 for (HashtableEntry<Symbol*>** p = the_table()->bucket_addr(i); *p != NULL; ) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
99 HashtableEntry<Symbol*>* entry = *p;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
100 if (entry->is_shared()) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
101 break;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
102 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
103 Symbol* s = entry->literal();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
104 memory_total += s->object_size();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
105 total++;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
106 assert(s != NULL, "just checking");
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
107 // If reference count is zero, remove.
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
108 if (s->refcount() == 0) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
109 delete s;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
110 removed++;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
111 *p = entry->next();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
112 the_table()->free_entry(entry);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
113 } else {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
114 p = entry->next_addr();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
115 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
116 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
117 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
118 symbols_removed += removed;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
119 symbols_counted += total;
2336
df1347358fe6 7024584: Symbol printouts shouldnt be under PrintGCDetails
coleenp
parents: 2177
diff changeset
120 // Exclude printing for normal PrintGCDetails because people parse
df1347358fe6 7024584: Symbol printouts shouldnt be under PrintGCDetails
coleenp
parents: 2177
diff changeset
121 // this output.
df1347358fe6 7024584: Symbol printouts shouldnt be under PrintGCDetails
coleenp
parents: 2177
diff changeset
122 if (PrintGCDetails && Verbose && WizardMode) {
df1347358fe6 7024584: Symbol printouts shouldnt be under PrintGCDetails
coleenp
parents: 2177
diff changeset
123 gclog_or_tty->print(" [Symbols=%d size=" SIZE_FORMAT "K] ", total,
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
124 (memory_total*HeapWordSize)/1024);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
125 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
126 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
127
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
128 unsigned int SymbolTable::new_hash(Symbol* sym) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
129 ResourceMark rm;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
130 // Use alternate hashing algorithm on this symbol.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
131 return AltHashing::murmur3_32(seed(), (const jbyte*)sym->as_C_string(), sym->utf8_length());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
132 }
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
133
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
134 // Create a new table and using alternate hash code, populate the new table
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
135 // with the existing strings. Set flag to use the alternate hash code afterwards.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
136 void SymbolTable::rehash_table() {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
137 assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
138 assert(!DumpSharedSpaces, "this should never happen with -Xshare:dump");
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
139 // Create a new symbol table
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
140 SymbolTable* new_table = new SymbolTable();
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
141
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
142 // Initialize the global seed for hashing.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
143 _seed = AltHashing::compute_seed();
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
144 assert(seed() != 0, "shouldn't be zero");
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
145
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
146 the_table()->move_to(new_table);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
147
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
148 // Delete the table and buckets (entries are reused in new table).
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
149 delete _the_table;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
150 // Don't check if we need rehashing until the table gets unbalanced again.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
151 // Then rehash with a new global seed.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
152 _needs_rehashing = false;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
153 _the_table = new_table;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
154 }
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
155
0
a61af66fc99e Initial load
duke
parents:
diff changeset
156 // Lookup a symbol in a bucket.
a61af66fc99e Initial load
duke
parents:
diff changeset
157
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
158 Symbol* SymbolTable::lookup(int index, const char* name,
0
a61af66fc99e Initial load
duke
parents:
diff changeset
159 int len, unsigned int hash) {
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
160 int count = 0;
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
161 for (HashtableEntry<Symbol*>* e = bucket(index); e != NULL; e = e->next()) {
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
162 count++; // count all entries in this bucket, not just ones with same hash
0
a61af66fc99e Initial load
duke
parents:
diff changeset
163 if (e->hash() == hash) {
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
164 Symbol* sym = e->literal();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
165 if (sym->equals(name, len)) {
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
166 // something is referencing this symbol now.
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
167 sym->increment_refcount();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
168 return sym;
a61af66fc99e Initial load
duke
parents:
diff changeset
169 }
a61af66fc99e Initial load
duke
parents:
diff changeset
170 }
a61af66fc99e Initial load
duke
parents:
diff changeset
171 }
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
172 // If the bucket size is too deep check if this hash code is insufficient.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
173 if (count >= BasicHashtable::rehash_count && !needs_rehashing()) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
174 _needs_rehashing = check_rehash_table(count);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
175 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
176 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
177 }
a61af66fc99e Initial load
duke
parents:
diff changeset
178
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
179 // Pick hashing algorithm, but return value already given if not using a new
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
180 // hash algorithm.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
181 unsigned int SymbolTable::hash_symbol(const char* s, int len, unsigned int hashValue) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
182 return use_alternate_hashcode() ?
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
183 AltHashing::murmur3_32(seed(), (const jbyte*)s, len) :
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
184 (hashValue != 0 ? hashValue : java_lang_String::to_hash(s, len));
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
185 }
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
186
0
a61af66fc99e Initial load
duke
parents:
diff changeset
187
a61af66fc99e Initial load
duke
parents:
diff changeset
188 // We take care not to be blocking while holding the
a61af66fc99e Initial load
duke
parents:
diff changeset
189 // SymbolTable_lock. Otherwise, the system might deadlock, since the
a61af66fc99e Initial load
duke
parents:
diff changeset
190 // symboltable is used during compilation (VM_thread) The lock free
a61af66fc99e Initial load
duke
parents:
diff changeset
191 // synchronization is simplified by the fact that we do not delete
a61af66fc99e Initial load
duke
parents:
diff changeset
192 // entries in the symbol table during normal execution (only during
a61af66fc99e Initial load
duke
parents:
diff changeset
193 // safepoints).
a61af66fc99e Initial load
duke
parents:
diff changeset
194
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
195 Symbol* SymbolTable::lookup(const char* name, int len, TRAPS) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
196 unsigned int hashValue = hash_symbol(name, len);
a61af66fc99e Initial load
duke
parents:
diff changeset
197 int index = the_table()->hash_to_index(hashValue);
a61af66fc99e Initial load
duke
parents:
diff changeset
198
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
199 Symbol* s = the_table()->lookup(index, name, len, hashValue);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
200
a61af66fc99e Initial load
duke
parents:
diff changeset
201 // Found
a61af66fc99e Initial load
duke
parents:
diff changeset
202 if (s != NULL) return s;
a61af66fc99e Initial load
duke
parents:
diff changeset
203
a61af66fc99e Initial load
duke
parents:
diff changeset
204 // Otherwise, add to symbol to table
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
205 return the_table()->basic_add(index, (u1*)name, len, hashValue, true, CHECK_NULL);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
206 }
a61af66fc99e Initial load
duke
parents:
diff changeset
207
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
208 Symbol* SymbolTable::lookup(const Symbol* sym, int begin, int end, TRAPS) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
209 char* buffer;
a61af66fc99e Initial load
duke
parents:
diff changeset
210 int index, len;
a61af66fc99e Initial load
duke
parents:
diff changeset
211 unsigned int hashValue;
a61af66fc99e Initial load
duke
parents:
diff changeset
212 char* name;
a61af66fc99e Initial load
duke
parents:
diff changeset
213 {
a61af66fc99e Initial load
duke
parents:
diff changeset
214 debug_only(No_Safepoint_Verifier nsv;)
a61af66fc99e Initial load
duke
parents:
diff changeset
215
a61af66fc99e Initial load
duke
parents:
diff changeset
216 name = (char*)sym->base() + begin;
a61af66fc99e Initial load
duke
parents:
diff changeset
217 len = end - begin;
a61af66fc99e Initial load
duke
parents:
diff changeset
218 hashValue = hash_symbol(name, len);
a61af66fc99e Initial load
duke
parents:
diff changeset
219 index = the_table()->hash_to_index(hashValue);
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
220 Symbol* s = the_table()->lookup(index, name, len, hashValue);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
221
a61af66fc99e Initial load
duke
parents:
diff changeset
222 // Found
a61af66fc99e Initial load
duke
parents:
diff changeset
223 if (s != NULL) return s;
a61af66fc99e Initial load
duke
parents:
diff changeset
224 }
a61af66fc99e Initial load
duke
parents:
diff changeset
225
a61af66fc99e Initial load
duke
parents:
diff changeset
226 // Otherwise, add to symbol to table. Copy to a C string first.
a61af66fc99e Initial load
duke
parents:
diff changeset
227 char stack_buf[128];
a61af66fc99e Initial load
duke
parents:
diff changeset
228 ResourceMark rm(THREAD);
a61af66fc99e Initial load
duke
parents:
diff changeset
229 if (len <= 128) {
a61af66fc99e Initial load
duke
parents:
diff changeset
230 buffer = stack_buf;
a61af66fc99e Initial load
duke
parents:
diff changeset
231 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
232 buffer = NEW_RESOURCE_ARRAY_IN_THREAD(THREAD, char, len);
a61af66fc99e Initial load
duke
parents:
diff changeset
233 }
a61af66fc99e Initial load
duke
parents:
diff changeset
234 for (int i=0; i<len; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
235 buffer[i] = name[i];
a61af66fc99e Initial load
duke
parents:
diff changeset
236 }
a61af66fc99e Initial load
duke
parents:
diff changeset
237 // Make sure there is no safepoint in the code above since name can't move.
a61af66fc99e Initial load
duke
parents:
diff changeset
238 // We can't include the code in No_Safepoint_Verifier because of the
a61af66fc99e Initial load
duke
parents:
diff changeset
239 // ResourceMark.
a61af66fc99e Initial load
duke
parents:
diff changeset
240
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
241 return the_table()->basic_add(index, (u1*)buffer, len, hashValue, true, CHECK_NULL);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
242 }
a61af66fc99e Initial load
duke
parents:
diff changeset
243
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
244 Symbol* SymbolTable::lookup_only(const char* name, int len,
0
a61af66fc99e Initial load
duke
parents:
diff changeset
245 unsigned int& hash) {
a61af66fc99e Initial load
duke
parents:
diff changeset
246 hash = hash_symbol(name, len);
a61af66fc99e Initial load
duke
parents:
diff changeset
247 int index = the_table()->hash_to_index(hash);
a61af66fc99e Initial load
duke
parents:
diff changeset
248
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
249 Symbol* s = the_table()->lookup(index, name, len, hash);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
250 return s;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
251 }
a61af66fc99e Initial load
duke
parents:
diff changeset
252
4800
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
253 // Look up the address of the literal in the SymbolTable for this Symbol*
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
254 // Do not create any new symbols
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
255 // Do not increment the reference count to keep this alive
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
256 Symbol** SymbolTable::lookup_symbol_addr(Symbol* sym){
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
257 unsigned int hash = hash_symbol((char*)sym->bytes(), sym->utf8_length());
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
258 int index = the_table()->hash_to_index(hash);
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
259
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
260 for (HashtableEntry<Symbol*>* e = the_table()->bucket(index); e != NULL; e = e->next()) {
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
261 if (e->hash() == hash) {
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
262 Symbol* literal_sym = e->literal();
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
263 if (sym == literal_sym) {
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
264 return e->literal_addr();
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
265 }
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
266 }
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
267 }
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
268 return NULL;
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
269 }
94ec88ca68e2 7115199: Add event tracing hooks and Java Flight Recorder infrastructure
phh
parents: 2418
diff changeset
270
665
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
271 // Suggestion: Push unicode-based lookup all the way into the hashing
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
272 // and probing logic, so there is no need for convert_to_utf8 until
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
273 // an actual new Symbol* is created.
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
274 Symbol* SymbolTable::lookup_unicode(const jchar* name, int utf16_length, TRAPS) {
665
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
275 int utf8_length = UNICODE::utf8_length((jchar*) name, utf16_length);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
276 char stack_buf[128];
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
277 if (utf8_length < (int) sizeof(stack_buf)) {
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
278 char* chars = stack_buf;
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
279 UNICODE::convert_to_utf8(name, utf16_length, chars);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
280 return lookup(chars, utf8_length, THREAD);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
281 } else {
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
282 ResourceMark rm(THREAD);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
283 char* chars = NEW_RESOURCE_ARRAY(char, utf8_length + 1);;
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
284 UNICODE::convert_to_utf8(name, utf16_length, chars);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
285 return lookup(chars, utf8_length, THREAD);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
286 }
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
287 }
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
288
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
289 Symbol* SymbolTable::lookup_only_unicode(const jchar* name, int utf16_length,
665
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
290 unsigned int& hash) {
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
291 int utf8_length = UNICODE::utf8_length((jchar*) name, utf16_length);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
292 char stack_buf[128];
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
293 if (utf8_length < (int) sizeof(stack_buf)) {
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
294 char* chars = stack_buf;
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
295 UNICODE::convert_to_utf8(name, utf16_length, chars);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
296 return lookup_only(chars, utf8_length, hash);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
297 } else {
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
298 ResourceMark rm;
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
299 char* chars = NEW_RESOURCE_ARRAY(char, utf8_length + 1);;
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
300 UNICODE::convert_to_utf8(name, utf16_length, chars);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
301 return lookup_only(chars, utf8_length, hash);
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
302 }
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
303 }
c89f86385056 6814659: separable cleanups and subroutines for 6655638
jrose
parents: 605
diff changeset
304
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
305 void SymbolTable::add(Handle class_loader, constantPoolHandle cp,
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
306 int names_count,
0
a61af66fc99e Initial load
duke
parents:
diff changeset
307 const char** names, int* lengths, int* cp_indices,
a61af66fc99e Initial load
duke
parents:
diff changeset
308 unsigned int* hashValues, TRAPS) {
a61af66fc99e Initial load
duke
parents:
diff changeset
309 SymbolTable* table = the_table();
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
310 bool added = table->basic_add(class_loader, cp, names_count, names, lengths,
0
a61af66fc99e Initial load
duke
parents:
diff changeset
311 cp_indices, hashValues, CHECK);
a61af66fc99e Initial load
duke
parents:
diff changeset
312 if (!added) {
a61af66fc99e Initial load
duke
parents:
diff changeset
313 // do it the hard way
a61af66fc99e Initial load
duke
parents:
diff changeset
314 for (int i=0; i<names_count; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
315 int index = table->hash_to_index(hashValues[i]);
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
316 bool c_heap = class_loader() != NULL;
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
317 Symbol* sym = table->basic_add(index, (u1*)names[i], lengths[i], hashValues[i], c_heap, CHECK);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
318 cp->symbol_at_put(cp_indices[i], sym);
a61af66fc99e Initial load
duke
parents:
diff changeset
319 }
a61af66fc99e Initial load
duke
parents:
diff changeset
320 }
a61af66fc99e Initial load
duke
parents:
diff changeset
321 }
a61af66fc99e Initial load
duke
parents:
diff changeset
322
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
323 Symbol* SymbolTable::new_permanent_symbol(const char* name, TRAPS) {
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
324 unsigned int hash;
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
325 Symbol* result = SymbolTable::lookup_only((char*)name, (int)strlen(name), hash);
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
326 if (result != NULL) {
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
327 return result;
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
328 }
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
329 SymbolTable* table = the_table();
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
330 int index = table->hash_to_index(hash);
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
331 return table->basic_add(index, (u1*)name, (int)strlen(name), hash, false, THREAD);
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
332 }
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
333
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
334 Symbol* SymbolTable::basic_add(int index, u1 *name, int len,
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
335 unsigned int hashValue_arg, bool c_heap, TRAPS) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
336 assert(!Universe::heap()->is_in_reserved(name) || GC_locker::is_active(),
a61af66fc99e Initial load
duke
parents:
diff changeset
337 "proposed name of symbol must be stable");
a61af66fc99e Initial load
duke
parents:
diff changeset
338
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
339 // Grab SymbolTable_lock first.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
340 MutexLocker ml(SymbolTable_lock, THREAD);
a61af66fc99e Initial load
duke
parents:
diff changeset
341
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
342 // Check if the symbol table has been rehashed, if so, need to recalculate
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
343 // the hash value.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
344 unsigned int hashValue = hash_symbol((const char*)name, len, hashValue_arg);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
345
0
a61af66fc99e Initial load
duke
parents:
diff changeset
346 // Since look-up was done lock-free, we need to check if another
a61af66fc99e Initial load
duke
parents:
diff changeset
347 // thread beat us in the race to insert the symbol.
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
348 Symbol* test = lookup(index, (char*)name, len, hashValue);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
349 if (test != NULL) {
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
350 // A race occurred and another thread introduced the symbol.
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
351 assert(test->refcount() != 0, "lookup should have incremented the count");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
352 return test;
a61af66fc99e Initial load
duke
parents:
diff changeset
353 }
a61af66fc99e Initial load
duke
parents:
diff changeset
354
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
355 // Create a new symbol.
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
356 Symbol* sym = allocate_symbol(name, len, c_heap, CHECK_NULL);
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
357 assert(sym->equals((char*)name, len), "symbol must be properly initialized");
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
358
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
359 HashtableEntry<Symbol*>* entry = new_entry(hashValue, sym);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
360 add_entry(index, entry);
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
361 return sym;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
362 }
a61af66fc99e Initial load
duke
parents:
diff changeset
363
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
364 // This version of basic_add adds symbols in batch from the constant pool
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
365 // parsing.
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
366 bool SymbolTable::basic_add(Handle class_loader, constantPoolHandle cp,
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
367 int names_count,
0
a61af66fc99e Initial load
duke
parents:
diff changeset
368 const char** names, int* lengths,
a61af66fc99e Initial load
duke
parents:
diff changeset
369 int* cp_indices, unsigned int* hashValues,
a61af66fc99e Initial load
duke
parents:
diff changeset
370 TRAPS) {
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
371
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
372 // Check symbol names are not too long. If any are too long, don't add any.
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
373 for (int i = 0; i< names_count; i++) {
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
374 if (lengths[i] > Symbol::max_length()) {
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
375 THROW_MSG_0(vmSymbols::java_lang_InternalError(),
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
376 "name is too long to represent");
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
377 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
378 }
a61af66fc99e Initial load
duke
parents:
diff changeset
379
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
380 // Hold SymbolTable_lock through the symbol creation
0
a61af66fc99e Initial load
duke
parents:
diff changeset
381 MutexLocker ml(SymbolTable_lock, THREAD);
a61af66fc99e Initial load
duke
parents:
diff changeset
382
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
383 for (int i=0; i<names_count; i++) {
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
384 // Check if the symbol table has been rehashed, if so, need to recalculate
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
385 // the hash value.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
386 unsigned int hashValue = hash_symbol(names[i], lengths[i], hashValues[i]);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
387 // Since look-up was done lock-free, we need to check if another
a61af66fc99e Initial load
duke
parents:
diff changeset
388 // thread beat us in the race to insert the symbol.
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
389 int index = hash_to_index(hashValue);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
390 Symbol* test = lookup(index, names[i], lengths[i], hashValue);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
391 if (test != NULL) {
605
98cb887364d3 6810672: Comment typos
twisti
parents: 0
diff changeset
392 // A race occurred and another thread introduced the symbol, this one
0
a61af66fc99e Initial load
duke
parents:
diff changeset
393 // will be dropped and collected. Use test instead.
a61af66fc99e Initial load
duke
parents:
diff changeset
394 cp->symbol_at_put(cp_indices[i], test);
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
395 assert(test->refcount() != 0, "lookup should have incremented the count");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
396 } else {
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
397 // Create a new symbol. The null class loader is never unloaded so these
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
398 // are allocated specially in a permanent arena.
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
399 bool c_heap = class_loader() != NULL;
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
400 Symbol* sym = allocate_symbol((const u1*)names[i], lengths[i], c_heap, CHECK_(false));
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
401 assert(sym->equals(names[i], lengths[i]), "symbol must be properly initialized"); // why wouldn't it be???
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
402 HashtableEntry<Symbol*>* entry = new_entry(hashValue, sym);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
403 add_entry(index, entry);
a61af66fc99e Initial load
duke
parents:
diff changeset
404 cp->symbol_at_put(cp_indices[i], sym);
a61af66fc99e Initial load
duke
parents:
diff changeset
405 }
a61af66fc99e Initial load
duke
parents:
diff changeset
406 }
a61af66fc99e Initial load
duke
parents:
diff changeset
407 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
408 }
a61af66fc99e Initial load
duke
parents:
diff changeset
409
a61af66fc99e Initial load
duke
parents:
diff changeset
410
a61af66fc99e Initial load
duke
parents:
diff changeset
411 void SymbolTable::verify() {
a61af66fc99e Initial load
duke
parents:
diff changeset
412 for (int i = 0; i < the_table()->table_size(); ++i) {
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
413 HashtableEntry<Symbol*>* p = the_table()->bucket(i);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
414 for ( ; p != NULL; p = p->next()) {
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
415 Symbol* s = (Symbol*)(p->literal());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
416 guarantee(s != NULL, "symbol is NULL");
a61af66fc99e Initial load
duke
parents:
diff changeset
417 unsigned int h = hash_symbol((char*)s->bytes(), s->utf8_length());
a61af66fc99e Initial load
duke
parents:
diff changeset
418 guarantee(p->hash() == h, "broken hash in symbol table entry");
a61af66fc99e Initial load
duke
parents:
diff changeset
419 guarantee(the_table()->hash_to_index(h) == i,
a61af66fc99e Initial load
duke
parents:
diff changeset
420 "wrong index in symbol table");
a61af66fc99e Initial load
duke
parents:
diff changeset
421 }
a61af66fc99e Initial load
duke
parents:
diff changeset
422 }
a61af66fc99e Initial load
duke
parents:
diff changeset
423 }
a61af66fc99e Initial load
duke
parents:
diff changeset
424
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
425 void SymbolTable::dump(outputStream* st) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
426 NumberSeq summary;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
427 for (int i = 0; i < the_table()->table_size(); ++i) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
428 int count = 0;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
429 for (HashtableEntry<Symbol*>* e = the_table()->bucket(i);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
430 e != NULL; e = e->next()) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
431 count++;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
432 }
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
433 summary.add((double)count);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
434 }
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
435 st->print_cr("SymbolTable statistics:");
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
436 st->print_cr("Number of buckets : %7d", summary.num());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
437 st->print_cr("Average bucket size : %7.0f", summary.avg());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
438 st->print_cr("Variance of bucket size : %7.0f", summary.variance());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
439 st->print_cr("Std. dev. of bucket size: %7.0f", summary.sd());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
440 st->print_cr("Maximum bucket size : %7.0f", summary.maximum());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
441 }
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
442
0
a61af66fc99e Initial load
duke
parents:
diff changeset
443
a61af66fc99e Initial load
duke
parents:
diff changeset
444 //---------------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
445 // Non-product code
a61af66fc99e Initial load
duke
parents:
diff changeset
446
a61af66fc99e Initial load
duke
parents:
diff changeset
447 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
448
a61af66fc99e Initial load
duke
parents:
diff changeset
449 void SymbolTable::print_histogram() {
a61af66fc99e Initial load
duke
parents:
diff changeset
450 MutexLocker ml(SymbolTable_lock);
a61af66fc99e Initial load
duke
parents:
diff changeset
451 const int results_length = 100;
a61af66fc99e Initial load
duke
parents:
diff changeset
452 int results[results_length];
a61af66fc99e Initial load
duke
parents:
diff changeset
453 int i,j;
a61af66fc99e Initial load
duke
parents:
diff changeset
454
a61af66fc99e Initial load
duke
parents:
diff changeset
455 // initialize results to zero
a61af66fc99e Initial load
duke
parents:
diff changeset
456 for (j = 0; j < results_length; j++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
457 results[j] = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
458 }
a61af66fc99e Initial load
duke
parents:
diff changeset
459
a61af66fc99e Initial load
duke
parents:
diff changeset
460 int total = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
461 int max_symbols = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
462 int out_of_range = 0;
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
463 int memory_total = 0;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
464 int count = 0;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
465 for (i = 0; i < the_table()->table_size(); i++) {
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
466 HashtableEntry<Symbol*>* p = the_table()->bucket(i);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
467 for ( ; p != NULL; p = p->next()) {
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
468 memory_total += p->literal()->object_size();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
469 count++;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
470 int counter = p->literal()->utf8_length();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
471 total += counter;
a61af66fc99e Initial load
duke
parents:
diff changeset
472 if (counter < results_length) {
a61af66fc99e Initial load
duke
parents:
diff changeset
473 results[counter]++;
a61af66fc99e Initial load
duke
parents:
diff changeset
474 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
475 out_of_range++;
a61af66fc99e Initial load
duke
parents:
diff changeset
476 }
a61af66fc99e Initial load
duke
parents:
diff changeset
477 max_symbols = MAX2(max_symbols, counter);
a61af66fc99e Initial load
duke
parents:
diff changeset
478 }
a61af66fc99e Initial load
duke
parents:
diff changeset
479 }
a61af66fc99e Initial load
duke
parents:
diff changeset
480 tty->print_cr("Symbol Table:");
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
481 tty->print_cr("Total number of symbols %5d", count);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
482 tty->print_cr("Total size in memory %5dK",
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
483 (memory_total*HeapWordSize)/1024);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
484 tty->print_cr("Total counted %5d", symbols_counted);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
485 tty->print_cr("Total removed %5d", symbols_removed);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
486 if (symbols_counted > 0) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
487 tty->print_cr("Percent removed %3.2f",
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
488 ((float)symbols_removed/(float)symbols_counted)* 100);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
489 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
490 tty->print_cr("Reference counts %5d", Symbol::_total_count);
5979
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
491 tty->print_cr("Symbol arena size %5d used %5d",
fc9d8850ab8b 7150058: Allocate symbols from null boot loader to an arena for NMT
coleenp
parents: 4800
diff changeset
492 arena()->size_in_bytes(), arena()->used());
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
493 tty->print_cr("Histogram of symbol length:");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
494 tty->print_cr("%8s %5d", "Total ", total);
a61af66fc99e Initial load
duke
parents:
diff changeset
495 tty->print_cr("%8s %5d", "Maximum", max_symbols);
a61af66fc99e Initial load
duke
parents:
diff changeset
496 tty->print_cr("%8s %3.2f", "Average",
a61af66fc99e Initial load
duke
parents:
diff changeset
497 ((float) total / (float) the_table()->table_size()));
a61af66fc99e Initial load
duke
parents:
diff changeset
498 tty->print_cr("%s", "Histogram:");
a61af66fc99e Initial load
duke
parents:
diff changeset
499 tty->print_cr(" %s %29s", "Length", "Number chains that length");
a61af66fc99e Initial load
duke
parents:
diff changeset
500 for (i = 0; i < results_length; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
501 if (results[i] > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
502 tty->print_cr("%6d %10d", i, results[i]);
a61af66fc99e Initial load
duke
parents:
diff changeset
503 }
a61af66fc99e Initial load
duke
parents:
diff changeset
504 }
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
505 if (Verbose) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
506 int line_length = 70;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
507 tty->print_cr("%s %30s", " Length", "Number chains that length");
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
508 for (i = 0; i < results_length; i++) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
509 if (results[i] > 0) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
510 tty->print("%4d", i);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
511 for (j = 0; (j < results[i]) && (j < line_length); j++) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
512 tty->print("%1s", "*");
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
513 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
514 if (j == line_length) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
515 tty->print("%1s", "+");
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
516 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
517 tty->cr();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
518 }
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
519 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
520 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
521 tty->print_cr(" %s %d: %d\n", "Number chains longer than",
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
522 results_length, out_of_range);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
523 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
524
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
525 void SymbolTable::print() {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
526 for (int i = 0; i < the_table()->table_size(); ++i) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
527 HashtableEntry<Symbol*>** p = the_table()->bucket_addr(i);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
528 HashtableEntry<Symbol*>* entry = the_table()->bucket(i);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
529 if (entry != NULL) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
530 while (entry != NULL) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
531 tty->print(PTR_FORMAT " ", entry->literal());
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
532 entry->literal()->print();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
533 tty->print(" %d", entry->literal()->refcount());
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
534 p = entry->next_addr();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
535 entry = (HashtableEntry<Symbol*>*)HashtableEntry<Symbol*>::make_ptr(*p);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
536 }
a61af66fc99e Initial load
duke
parents:
diff changeset
537 tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
538 }
a61af66fc99e Initial load
duke
parents:
diff changeset
539 }
a61af66fc99e Initial load
duke
parents:
diff changeset
540 }
a61af66fc99e Initial load
duke
parents:
diff changeset
541 #endif // PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
542
a61af66fc99e Initial load
duke
parents:
diff changeset
543 // --------------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
544
a61af66fc99e Initial load
duke
parents:
diff changeset
545 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
546 class StableMemoryChecker : public StackObj {
a61af66fc99e Initial load
duke
parents:
diff changeset
547 enum { _bufsize = wordSize*4 };
a61af66fc99e Initial load
duke
parents:
diff changeset
548
a61af66fc99e Initial load
duke
parents:
diff changeset
549 address _region;
a61af66fc99e Initial load
duke
parents:
diff changeset
550 jint _size;
a61af66fc99e Initial load
duke
parents:
diff changeset
551 u1 _save_buf[_bufsize];
a61af66fc99e Initial load
duke
parents:
diff changeset
552
a61af66fc99e Initial load
duke
parents:
diff changeset
553 int sample(u1* save_buf) {
a61af66fc99e Initial load
duke
parents:
diff changeset
554 if (_size <= _bufsize) {
a61af66fc99e Initial load
duke
parents:
diff changeset
555 memcpy(save_buf, _region, _size);
a61af66fc99e Initial load
duke
parents:
diff changeset
556 return _size;
a61af66fc99e Initial load
duke
parents:
diff changeset
557 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
558 // copy head and tail
a61af66fc99e Initial load
duke
parents:
diff changeset
559 memcpy(&save_buf[0], _region, _bufsize/2);
a61af66fc99e Initial load
duke
parents:
diff changeset
560 memcpy(&save_buf[_bufsize/2], _region + _size - _bufsize/2, _bufsize/2);
a61af66fc99e Initial load
duke
parents:
diff changeset
561 return (_bufsize/2)*2;
a61af66fc99e Initial load
duke
parents:
diff changeset
562 }
a61af66fc99e Initial load
duke
parents:
diff changeset
563 }
a61af66fc99e Initial load
duke
parents:
diff changeset
564
a61af66fc99e Initial load
duke
parents:
diff changeset
565 public:
a61af66fc99e Initial load
duke
parents:
diff changeset
566 StableMemoryChecker(const void* region, jint size) {
a61af66fc99e Initial load
duke
parents:
diff changeset
567 _region = (address) region;
a61af66fc99e Initial load
duke
parents:
diff changeset
568 _size = size;
a61af66fc99e Initial load
duke
parents:
diff changeset
569 sample(_save_buf);
a61af66fc99e Initial load
duke
parents:
diff changeset
570 }
a61af66fc99e Initial load
duke
parents:
diff changeset
571
a61af66fc99e Initial load
duke
parents:
diff changeset
572 bool verify() {
a61af66fc99e Initial load
duke
parents:
diff changeset
573 u1 check_buf[sizeof(_save_buf)];
a61af66fc99e Initial load
duke
parents:
diff changeset
574 int check_size = sample(check_buf);
a61af66fc99e Initial load
duke
parents:
diff changeset
575 return (0 == memcmp(_save_buf, check_buf, check_size));
a61af66fc99e Initial load
duke
parents:
diff changeset
576 }
a61af66fc99e Initial load
duke
parents:
diff changeset
577
a61af66fc99e Initial load
duke
parents:
diff changeset
578 void set_region(const void* region) { _region = (address) region; }
a61af66fc99e Initial load
duke
parents:
diff changeset
579 };
a61af66fc99e Initial load
duke
parents:
diff changeset
580 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
581
a61af66fc99e Initial load
duke
parents:
diff changeset
582
a61af66fc99e Initial load
duke
parents:
diff changeset
583 // --------------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
584 StringTable* StringTable::_the_table = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
585
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
586 bool StringTable::_needs_rehashing = false;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
587 jint StringTable::_seed = 0;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
588
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
589 // Pick hashing algorithm
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
590 unsigned int StringTable::hash_string(const jchar* s, int len, unsigned int hashValue) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
591 return use_alternate_hashcode() ? AltHashing::murmur3_32(seed(), s, len) :
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
592 (hashValue != 0 ? hashValue : java_lang_String::to_hash(s, len));
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
593 }
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
594
0
a61af66fc99e Initial load
duke
parents:
diff changeset
595 oop StringTable::lookup(int index, jchar* name,
a61af66fc99e Initial load
duke
parents:
diff changeset
596 int len, unsigned int hash) {
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
597 int count = 0;
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
598 for (HashtableEntry<oop>* l = bucket(index); l != NULL; l = l->next()) {
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
599 count++;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
600 if (l->hash() == hash) {
a61af66fc99e Initial load
duke
parents:
diff changeset
601 if (java_lang_String::equals(l->literal(), name, len)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
602 return l->literal();
a61af66fc99e Initial load
duke
parents:
diff changeset
603 }
a61af66fc99e Initial load
duke
parents:
diff changeset
604 }
a61af66fc99e Initial load
duke
parents:
diff changeset
605 }
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
606 // If the bucket size is too deep check if this hash code is insufficient.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
607 if (count >= BasicHashtable::rehash_count && !needs_rehashing()) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
608 _needs_rehashing = check_rehash_table(count);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
609 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
610 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
611 }
a61af66fc99e Initial load
duke
parents:
diff changeset
612
a61af66fc99e Initial load
duke
parents:
diff changeset
613
a61af66fc99e Initial load
duke
parents:
diff changeset
614 oop StringTable::basic_add(int index, Handle string_or_null, jchar* name,
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
615 int len, unsigned int hashValue_arg, TRAPS) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
616 debug_only(StableMemoryChecker smc(name, len * sizeof(name[0])));
a61af66fc99e Initial load
duke
parents:
diff changeset
617 assert(!Universe::heap()->is_in_reserved(name) || GC_locker::is_active(),
a61af66fc99e Initial load
duke
parents:
diff changeset
618 "proposed name of symbol must be stable");
a61af66fc99e Initial load
duke
parents:
diff changeset
619
a61af66fc99e Initial load
duke
parents:
diff changeset
620 Handle string;
a61af66fc99e Initial load
duke
parents:
diff changeset
621 // try to reuse the string if possible
2379
b099aaf51bf8 6962931: move interned strings out of the perm gen
jcoomes
parents: 2336
diff changeset
622 if (!string_or_null.is_null() && (!JavaObjectsInPerm || string_or_null()->is_perm())) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
623 string = string_or_null;
a61af66fc99e Initial load
duke
parents:
diff changeset
624 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
625 string = java_lang_String::create_tenured_from_unicode(name, len, CHECK_NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
626 }
a61af66fc99e Initial load
duke
parents:
diff changeset
627
a61af66fc99e Initial load
duke
parents:
diff changeset
628 // Allocation must be done before grapping the SymbolTable_lock lock
a61af66fc99e Initial load
duke
parents:
diff changeset
629 MutexLocker ml(StringTable_lock, THREAD);
a61af66fc99e Initial load
duke
parents:
diff changeset
630
a61af66fc99e Initial load
duke
parents:
diff changeset
631 assert(java_lang_String::equals(string(), name, len),
a61af66fc99e Initial load
duke
parents:
diff changeset
632 "string must be properly initialized");
a61af66fc99e Initial load
duke
parents:
diff changeset
633
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
634 // Check if the symbol table has been rehashed, if so, need to recalculate
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
635 // the hash value before second lookup.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
636 unsigned int hashValue = hash_string(name, len, hashValue_arg);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
637
0
a61af66fc99e Initial load
duke
parents:
diff changeset
638 // Since look-up was done lock-free, we need to check if another
a61af66fc99e Initial load
duke
parents:
diff changeset
639 // thread beat us in the race to insert the symbol.
a61af66fc99e Initial load
duke
parents:
diff changeset
640
a61af66fc99e Initial load
duke
parents:
diff changeset
641 oop test = lookup(index, name, len, hashValue); // calls lookup(u1*, int)
a61af66fc99e Initial load
duke
parents:
diff changeset
642 if (test != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
643 // Entry already added
a61af66fc99e Initial load
duke
parents:
diff changeset
644 return test;
a61af66fc99e Initial load
duke
parents:
diff changeset
645 }
a61af66fc99e Initial load
duke
parents:
diff changeset
646
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
647 HashtableEntry<oop>* entry = new_entry(hashValue, string());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
648 add_entry(index, entry);
a61af66fc99e Initial load
duke
parents:
diff changeset
649 return string();
a61af66fc99e Initial load
duke
parents:
diff changeset
650 }
a61af66fc99e Initial load
duke
parents:
diff changeset
651
a61af66fc99e Initial load
duke
parents:
diff changeset
652
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
653 oop StringTable::lookup(Symbol* symbol) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
654 ResourceMark rm;
a61af66fc99e Initial load
duke
parents:
diff changeset
655 int length;
a61af66fc99e Initial load
duke
parents:
diff changeset
656 jchar* chars = symbol->as_unicode(length);
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
657 unsigned int hashValue = hash_string(chars, length);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
658 int index = the_table()->hash_to_index(hashValue);
a61af66fc99e Initial load
duke
parents:
diff changeset
659 return the_table()->lookup(index, chars, length, hashValue);
a61af66fc99e Initial load
duke
parents:
diff changeset
660 }
a61af66fc99e Initial load
duke
parents:
diff changeset
661
a61af66fc99e Initial load
duke
parents:
diff changeset
662
a61af66fc99e Initial load
duke
parents:
diff changeset
663 oop StringTable::intern(Handle string_or_null, jchar* name,
a61af66fc99e Initial load
duke
parents:
diff changeset
664 int len, TRAPS) {
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
665 unsigned int hashValue = hash_string(name, len);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
666 int index = the_table()->hash_to_index(hashValue);
a61af66fc99e Initial load
duke
parents:
diff changeset
667 oop string = the_table()->lookup(index, name, len, hashValue);
a61af66fc99e Initial load
duke
parents:
diff changeset
668
a61af66fc99e Initial load
duke
parents:
diff changeset
669 // Found
a61af66fc99e Initial load
duke
parents:
diff changeset
670 if (string != NULL) return string;
a61af66fc99e Initial load
duke
parents:
diff changeset
671
a61af66fc99e Initial load
duke
parents:
diff changeset
672 // Otherwise, add to symbol to table
a61af66fc99e Initial load
duke
parents:
diff changeset
673 return the_table()->basic_add(index, string_or_null, name, len,
a61af66fc99e Initial load
duke
parents:
diff changeset
674 hashValue, CHECK_NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
675 }
a61af66fc99e Initial load
duke
parents:
diff changeset
676
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
677 oop StringTable::intern(Symbol* symbol, TRAPS) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
678 if (symbol == NULL) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
679 ResourceMark rm(THREAD);
a61af66fc99e Initial load
duke
parents:
diff changeset
680 int length;
a61af66fc99e Initial load
duke
parents:
diff changeset
681 jchar* chars = symbol->as_unicode(length);
a61af66fc99e Initial load
duke
parents:
diff changeset
682 Handle string;
a61af66fc99e Initial load
duke
parents:
diff changeset
683 oop result = intern(string, chars, length, CHECK_NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
684 return result;
a61af66fc99e Initial load
duke
parents:
diff changeset
685 }
a61af66fc99e Initial load
duke
parents:
diff changeset
686
a61af66fc99e Initial load
duke
parents:
diff changeset
687
a61af66fc99e Initial load
duke
parents:
diff changeset
688 oop StringTable::intern(oop string, TRAPS)
a61af66fc99e Initial load
duke
parents:
diff changeset
689 {
a61af66fc99e Initial load
duke
parents:
diff changeset
690 if (string == NULL) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
691 ResourceMark rm(THREAD);
a61af66fc99e Initial load
duke
parents:
diff changeset
692 int length;
a61af66fc99e Initial load
duke
parents:
diff changeset
693 Handle h_string (THREAD, string);
a61af66fc99e Initial load
duke
parents:
diff changeset
694 jchar* chars = java_lang_String::as_unicode_string(string, length);
a61af66fc99e Initial load
duke
parents:
diff changeset
695 oop result = intern(h_string, chars, length, CHECK_NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
696 return result;
a61af66fc99e Initial load
duke
parents:
diff changeset
697 }
a61af66fc99e Initial load
duke
parents:
diff changeset
698
a61af66fc99e Initial load
duke
parents:
diff changeset
699
a61af66fc99e Initial load
duke
parents:
diff changeset
700 oop StringTable::intern(const char* utf8_string, TRAPS) {
a61af66fc99e Initial load
duke
parents:
diff changeset
701 if (utf8_string == NULL) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
702 ResourceMark rm(THREAD);
a61af66fc99e Initial load
duke
parents:
diff changeset
703 int length = UTF8::unicode_length(utf8_string);
a61af66fc99e Initial load
duke
parents:
diff changeset
704 jchar* chars = NEW_RESOURCE_ARRAY(jchar, length);
a61af66fc99e Initial load
duke
parents:
diff changeset
705 UTF8::convert_to_unicode(utf8_string, chars, length);
a61af66fc99e Initial load
duke
parents:
diff changeset
706 Handle string;
a61af66fc99e Initial load
duke
parents:
diff changeset
707 oop result = intern(string, chars, length, CHECK_NULL);
a61af66fc99e Initial load
duke
parents:
diff changeset
708 return result;
a61af66fc99e Initial load
duke
parents:
diff changeset
709 }
a61af66fc99e Initial load
duke
parents:
diff changeset
710
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
711 void StringTable::unlink(BoolObjectClosure* is_alive) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
712 // Readers of the table are unlocked, so we should only be removing
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
713 // entries at a safepoint.
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
714 assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
715 for (int i = 0; i < the_table()->table_size(); ++i) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
716 for (HashtableEntry<oop>** p = the_table()->bucket_addr(i); *p != NULL; ) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
717 HashtableEntry<oop>* entry = *p;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
718 if (entry->is_shared()) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
719 break;
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
720 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
721 assert(entry->literal() != NULL, "just checking");
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
722 if (is_alive->do_object_b(entry->literal())) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
723 p = entry->next_addr();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
724 } else {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
725 *p = entry->next();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
726 the_table()->free_entry(entry);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
727 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
728 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
729 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
730 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
731
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
732 void StringTable::oops_do(OopClosure* f) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
733 for (int i = 0; i < the_table()->table_size(); ++i) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
734 HashtableEntry<oop>** p = the_table()->bucket_addr(i);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
735 HashtableEntry<oop>* entry = the_table()->bucket(i);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
736 while (entry != NULL) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
737 f->do_oop((oop*)entry->literal_addr());
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
738
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
739 // Did the closure remove the literal from the table?
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
740 if (entry->literal() == NULL) {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
741 assert(!entry->is_shared(), "immutable hashtable entry?");
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
742 *p = entry->next();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
743 the_table()->free_entry(entry);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
744 } else {
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
745 p = entry->next_addr();
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
746 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
747 entry = (HashtableEntry<oop>*)HashtableEntry<oop>::make_ptr(*p);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
748 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
749 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
750 }
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
751
0
a61af66fc99e Initial load
duke
parents:
diff changeset
752 void StringTable::verify() {
a61af66fc99e Initial load
duke
parents:
diff changeset
753 for (int i = 0; i < the_table()->table_size(); ++i) {
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
754 HashtableEntry<oop>* p = the_table()->bucket(i);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
755 for ( ; p != NULL; p = p->next()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
756 oop s = p->literal();
a61af66fc99e Initial load
duke
parents:
diff changeset
757 guarantee(s != NULL, "interned string is NULL");
2379
b099aaf51bf8 6962931: move interned strings out of the perm gen
jcoomes
parents: 2336
diff changeset
758 guarantee(s->is_perm() || !JavaObjectsInPerm, "interned string not in permspace");
2418
352622fd140a 7032129: Native memory usage grow unexpectedly for vm/oom/*InternedString tests
never
parents: 2379
diff changeset
759 unsigned int h = java_lang_String::hash_string(s);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
760 guarantee(p->hash() == h, "broken hash in string table entry");
a61af66fc99e Initial load
duke
parents:
diff changeset
761 guarantee(the_table()->hash_to_index(h) == i,
a61af66fc99e Initial load
duke
parents:
diff changeset
762 "wrong index in string table");
a61af66fc99e Initial load
duke
parents:
diff changeset
763 }
a61af66fc99e Initial load
duke
parents:
diff changeset
764 }
a61af66fc99e Initial load
duke
parents:
diff changeset
765 }
6162
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
766
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
767 void StringTable::dump(outputStream* st) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
768 NumberSeq summary;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
769 for (int i = 0; i < the_table()->table_size(); ++i) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
770 HashtableEntry<oop>* p = the_table()->bucket(i);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
771 int count = 0;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
772 for ( ; p != NULL; p = p->next()) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
773 count++;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
774 }
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
775 summary.add((double)count);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
776 }
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
777 st->print_cr("StringTable statistics:");
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
778 st->print_cr("Number of buckets : %7d", summary.num());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
779 st->print_cr("Average bucket size : %7.0f", summary.avg());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
780 st->print_cr("Variance of bucket size : %7.0f", summary.variance());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
781 st->print_cr("Std. dev. of bucket size: %7.0f", summary.sd());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
782 st->print_cr("Maximum bucket size : %7.0f", summary.maximum());
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
783 }
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
784
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
785
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
786 unsigned int StringTable::new_hash(oop string) {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
787 ResourceMark rm;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
788 int length;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
789 jchar* chars = java_lang_String::as_unicode_string(string, length);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
790 // Use alternate hashing algorithm on the string
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
791 return AltHashing::murmur3_32(seed(), chars, length);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
792 }
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
793
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
794 // Create a new table and using alternate hash code, populate the new table
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
795 // with the existing strings. Set flag to use the alternate hash code afterwards.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
796 void StringTable::rehash_table() {
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
797 assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint");
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
798 assert(!DumpSharedSpaces, "this should never happen with -Xshare:dump");
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
799 StringTable* new_table = new StringTable();
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
800
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
801 // Initialize new global seed for hashing.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
802 _seed = AltHashing::compute_seed();
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
803 assert(seed() != 0, "shouldn't be zero");
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
804
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
805 // Rehash the table
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
806 the_table()->move_to(new_table);
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
807
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
808 // Delete the table and buckets (entries are reused in new table).
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
809 delete _the_table;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
810 // Don't check if we need rehashing until the table gets unbalanced again.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
811 // Then rehash with a new global seed.
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
812 _needs_rehashing = false;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
813 _the_table = new_table;
e9140bf80b4a 7158800: Improve storage of symbol tables
coleenp
parents: 5979
diff changeset
814 }