Mercurial > hg > truffle
annotate src/share/vm/code/vtableStubs.cpp @ 14649:f6301b007a16
6498581: ThreadInterruptTest3 produces wrong output on Windows
Summary: There is race condition between os::interrupt and os::is_interrupted on Windows. In JVM_Sleep(Thread.sleep), check if thread gets interrupted, it may see interrupted but not really interrupted so cause spurious waking up (early return from sleep). Fix by checking if interrupt event really gets set thus prevent false return. For intrinsic of _isInterrupted, on Windows, go fastpath only on bit not set.
Reviewed-by: acorn, kvn
Contributed-by: david.holmes@oracle.com, yumin.qi@oracle.com
| author | minqi |
|---|---|
| date | Wed, 26 Feb 2014 15:20:41 -0800 |
| parents | 9341a9963d36 |
| children | 4ca6dc0799b6 78bbf4d43a14 |
| rev | line source |
|---|---|
| 0 | 1 /* |
|
12146
9758d9f36299
8021954: VM SIGSEGV during classloading on MacOS; hs_err_pid file produced
coleenp
parents:
10161
diff
changeset
|
2 * Copyright (c) 1997, 2013, Oracle and/or its affiliates. All rights reserved. |
| 0 | 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
| 4 * | |
| 5 * This code is free software; you can redistribute it and/or modify it | |
| 6 * under the terms of the GNU General Public License version 2 only, as | |
| 7 * published by the Free Software Foundation. | |
| 8 * | |
| 9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
| 10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
| 11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
| 12 * version 2 for more details (a copy is included in the LICENSE file that | |
| 13 * accompanied this code). | |
| 14 * | |
| 15 * You should have received a copy of the GNU General Public License version | |
| 16 * 2 along with this work; if not, write to the Free Software Foundation, | |
| 17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
| 18 * | |
|
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1490
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1490
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:
1490
diff
changeset
|
21 * questions. |
| 0 | 22 * |
| 23 */ | |
| 24 | |
| 1972 | 25 #include "precompiled.hpp" |
| 26 #include "code/vtableStubs.hpp" | |
| 27 #include "compiler/disassembler.hpp" | |
| 28 #include "memory/allocation.inline.hpp" | |
| 29 #include "memory/resourceArea.hpp" | |
| 30 #include "oops/instanceKlass.hpp" | |
| 31 #include "oops/klassVtable.hpp" | |
| 32 #include "oops/oop.inline.hpp" | |
| 33 #include "prims/forte.hpp" | |
| 34 #include "prims/jvmtiExport.hpp" | |
| 35 #include "runtime/handles.inline.hpp" | |
| 36 #include "runtime/mutexLocker.hpp" | |
| 37 #include "runtime/sharedRuntime.hpp" | |
| 38 #ifdef COMPILER2 | |
| 39 #include "opto/matcher.hpp" | |
| 40 #endif | |
| 0 | 41 |
| 42 // ----------------------------------------------------------------------------------------- | |
| 43 // Implementation of VtableStub | |
| 44 | |
| 45 address VtableStub::_chunk = NULL; | |
| 46 address VtableStub::_chunk_end = NULL; | |
| 47 VMReg VtableStub::_receiver_location = VMRegImpl::Bad(); | |
| 48 | |
| 49 | |
|
12146
9758d9f36299
8021954: VM SIGSEGV during classloading on MacOS; hs_err_pid file produced
coleenp
parents:
10161
diff
changeset
|
50 void* VtableStub::operator new(size_t size, int code_size) throw() { |
| 0 | 51 assert(size == sizeof(VtableStub), "mismatched size"); |
| 52 // compute real VtableStub size (rounded to nearest word) | |
| 53 const int real_size = round_to(code_size + sizeof(VtableStub), wordSize); | |
| 54 // malloc them in chunks to minimize header overhead | |
| 55 const int chunk_factor = 32; | |
| 56 if (_chunk == NULL || _chunk + real_size > _chunk_end) { | |
| 57 const int bytes = chunk_factor * real_size + pd_code_alignment(); | |
|
14330
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
58 |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
59 // There is a dependency on the name of the blob in src/share/vm/prims/jvmtiCodeBlobEvents.cpp |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
60 // If changing the name, update the other file accordingly. |
| 0 | 61 BufferBlob* blob = BufferBlob::create("vtable chunks", bytes); |
|
1490
f03d0a26bf83
6888954: argument formatting for assert() and friends
jcoomes
parents:
844
diff
changeset
|
62 if (blob == NULL) { |
| 12294 | 63 return NULL; |
|
1490
f03d0a26bf83
6888954: argument formatting for assert() and friends
jcoomes
parents:
844
diff
changeset
|
64 } |
| 1748 | 65 _chunk = blob->content_begin(); |
| 0 | 66 _chunk_end = _chunk + bytes; |
| 67 Forte::register_stub("vtable stub", _chunk, _chunk_end); | |
| 68 align_chunk(); | |
| 69 } | |
| 70 assert(_chunk + real_size <= _chunk_end, "bad allocation"); | |
| 71 void* res = _chunk; | |
| 72 _chunk += real_size; | |
| 73 align_chunk(); | |
| 74 return res; | |
| 75 } | |
| 76 | |
| 77 | |
|
1681
126ea7725993
6953477: Increase portability and flexibility of building Hotspot
bobv
parents:
1644
diff
changeset
|
78 void VtableStub::print_on(outputStream* st) const { |
|
126ea7725993
6953477: Increase portability and flexibility of building Hotspot
bobv
parents:
1644
diff
changeset
|
79 st->print("vtable stub (index = %d, receiver_location = %d, code = [" INTPTR_FORMAT ", " INTPTR_FORMAT "[)", |
| 0 | 80 index(), receiver_location(), code_begin(), code_end()); |
| 81 } | |
| 82 | |
| 83 | |
| 84 // ----------------------------------------------------------------------------------------- | |
| 85 // Implementation of VtableStubs | |
| 86 // | |
| 87 // For each hash value there's a linked list of vtable stubs (with that | |
| 88 // hash value). Each list is anchored in a little hash _table, indexed | |
| 89 // by that hash value. | |
| 90 | |
| 91 VtableStub* VtableStubs::_table[VtableStubs::N]; | |
| 92 int VtableStubs::_number_of_vtable_stubs = 0; | |
| 93 | |
| 94 | |
| 95 void VtableStubs::initialize() { | |
| 96 VtableStub::_receiver_location = SharedRuntime::name_for_receiver(); | |
| 97 { | |
| 98 MutexLocker ml(VtableStubs_lock); | |
| 99 assert(_number_of_vtable_stubs == 0, "potential performance bug: VtableStubs initialized more than once"); | |
| 100 assert(is_power_of_2(N), "N must be a power of 2"); | |
| 101 for (int i = 0; i < N; i++) { | |
| 102 _table[i] = NULL; | |
| 103 } | |
| 104 } | |
| 105 } | |
| 106 | |
| 107 | |
|
12264
b2e698d2276c
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
12146
diff
changeset
|
108 address VtableStubs::find_stub(bool is_vtable_stub, int vtable_index) { |
| 0 | 109 assert(vtable_index >= 0, "must be positive"); |
| 110 | |
| 111 VtableStub* s = ShareVtableStubs ? lookup(is_vtable_stub, vtable_index) : NULL; | |
| 112 if (s == NULL) { | |
| 113 if (is_vtable_stub) { | |
| 114 s = create_vtable_stub(vtable_index); | |
| 115 } else { | |
| 116 s = create_itable_stub(vtable_index); | |
| 117 } | |
| 12294 | 118 |
| 119 // Creation of vtable or itable can fail if there is not enough free space in the code cache. | |
| 120 if (s == NULL) { | |
| 121 return NULL; | |
| 122 } | |
| 123 | |
| 0 | 124 enter(is_vtable_stub, vtable_index, s); |
| 125 if (PrintAdapterHandlers) { | |
| 126 tty->print_cr("Decoding VtableStub %s[%d]@%d", | |
| 127 is_vtable_stub? "vtbl": "itbl", vtable_index, VtableStub::receiver_location()); | |
| 128 Disassembler::decode(s->code_begin(), s->code_end()); | |
| 129 } | |
|
14330
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
130 // Notify JVMTI about this stub. The event will be recorded by the enclosing |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
131 // JvmtiDynamicCodeEventCollector and posted when this thread has released |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
132 // all locks. |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
133 if (JvmtiExport::should_post_dynamic_code_generated()) { |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
134 JvmtiExport::post_dynamic_code_generated_while_holding_locks(is_vtable_stub? "vtable stub": "itable stub", |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
135 s->code_begin(), s->code_end()); |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
136 } |
| 0 | 137 } |
| 138 return s->entry_point(); | |
| 139 } | |
| 140 | |
| 141 | |
| 142 inline uint VtableStubs::hash(bool is_vtable_stub, int vtable_index){ | |
| 143 // Assumption: receiver_location < 4 in most cases. | |
| 144 int hash = ((vtable_index << 2) ^ VtableStub::receiver_location()->value()) + vtable_index; | |
| 145 return (is_vtable_stub ? ~hash : hash) & mask; | |
| 146 } | |
| 147 | |
| 148 | |
| 149 VtableStub* VtableStubs::lookup(bool is_vtable_stub, int vtable_index) { | |
| 150 MutexLocker ml(VtableStubs_lock); | |
| 151 unsigned hash = VtableStubs::hash(is_vtable_stub, vtable_index); | |
| 152 VtableStub* s = _table[hash]; | |
| 153 while( s && !s->matches(is_vtable_stub, vtable_index)) s = s->next(); | |
| 154 return s; | |
| 155 } | |
| 156 | |
| 157 | |
| 158 void VtableStubs::enter(bool is_vtable_stub, int vtable_index, VtableStub* s) { | |
| 159 MutexLocker ml(VtableStubs_lock); | |
| 160 assert(s->matches(is_vtable_stub, vtable_index), "bad vtable stub"); | |
| 161 unsigned int h = VtableStubs::hash(is_vtable_stub, vtable_index); | |
| 162 // enter s at the beginning of the corresponding list | |
| 163 s->set_next(_table[h]); | |
| 164 _table[h] = s; | |
| 165 _number_of_vtable_stubs++; | |
| 166 } | |
| 167 | |
| 168 | |
| 169 bool VtableStubs::is_entry_point(address pc) { | |
| 170 MutexLocker ml(VtableStubs_lock); | |
| 171 VtableStub* stub = (VtableStub*)(pc - VtableStub::entry_offset()); | |
| 172 uint hash = VtableStubs::hash(stub->is_vtable_stub(), stub->index()); | |
| 173 VtableStub* s; | |
| 174 for (s = _table[hash]; s != NULL && s != stub; s = s->next()) {} | |
| 175 return s == stub; | |
| 176 } | |
| 177 | |
| 178 | |
| 179 bool VtableStubs::contains(address pc) { | |
| 180 // simple solution for now - we may want to use | |
| 181 // a faster way if this function is called often | |
| 182 return stub_containing(pc) != NULL; | |
| 183 } | |
| 184 | |
| 185 | |
| 186 VtableStub* VtableStubs::stub_containing(address pc) { | |
| 187 // Note: No locking needed since any change to the data structure | |
| 188 // happens with an atomic store into it (we don't care about | |
| 189 // consistency with the _number_of_vtable_stubs counter). | |
| 190 for (int i = 0; i < N; i++) { | |
| 191 for (VtableStub* s = _table[i]; s != NULL; s = s->next()) { | |
| 192 if (s->contains(pc)) return s; | |
| 193 } | |
| 194 } | |
| 195 return NULL; | |
| 196 } | |
| 197 | |
| 198 void vtableStubs_init() { | |
| 199 VtableStubs::initialize(); | |
| 200 } | |
| 201 | |
|
14330
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
202 void VtableStubs::vtable_stub_do(void f(VtableStub*)) { |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
203 for (int i = 0; i < N; i++) { |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
204 for (VtableStub* s = _table[i]; s != NULL; s = s->next()) { |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
205 f(s); |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
206 } |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
207 } |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
208 } |
|
9341a9963d36
8025841: JVMTI: "vtable stub" dynamic code notification is misplaced
sspitsyn
parents:
12294
diff
changeset
|
209 |
| 0 | 210 |
| 211 //----------------------------------------------------------------------------------------------------- | |
| 212 // Non-product code | |
| 213 #ifndef PRODUCT | |
| 214 | |
| 215 extern "C" void bad_compiled_vtable_index(JavaThread* thread, oop receiver, int index) { | |
| 216 ResourceMark rm; | |
| 217 HandleMark hm; | |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
1972
diff
changeset
|
218 Klass* klass = receiver->klass(); |
|
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
1972
diff
changeset
|
219 InstanceKlass* ik = InstanceKlass::cast(klass); |
| 0 | 220 klassVtable* vt = ik->vtable(); |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
1972
diff
changeset
|
221 ik->print(); |
|
1490
f03d0a26bf83
6888954: argument formatting for assert() and friends
jcoomes
parents:
844
diff
changeset
|
222 fatal(err_msg("bad compiled vtable dispatch: receiver " INTPTR_FORMAT ", " |
|
f03d0a26bf83
6888954: argument formatting for assert() and friends
jcoomes
parents:
844
diff
changeset
|
223 "index %d (vtable length %d)", |
|
f03d0a26bf83
6888954: argument formatting for assert() and friends
jcoomes
parents:
844
diff
changeset
|
224 (address)receiver, index, vt->length())); |
| 0 | 225 } |
| 226 | |
| 227 #endif // Product |