Mercurial > hg > truffle
annotate src/cpu/x86/vm/bytecodeInterpreter_x86.hpp @ 17716:cdb71841f4bc
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 | f95d63e2154a |
| children | 018b357638aa |
| rev | line source |
|---|---|
| 0 | 1 /* |
| 1972 | 2 * Copyright (c) 2002, 2010, 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:
0
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
0
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:
0
diff
changeset
|
21 * questions. |
| 0 | 22 * |
| 23 */ | |
| 24 | |
| 1972 | 25 #ifndef CPU_X86_VM_BYTECODEINTERPRETER_X86_HPP |
| 26 #define CPU_X86_VM_BYTECODEINTERPRETER_X86_HPP | |
| 27 | |
| 0 | 28 // Platform specific for C++ based Interpreter |
| 29 | |
| 30 private: | |
| 31 | |
| 32 interpreterState _self_link; /* Previous interpreter state */ /* sometimes points to self??? */ | |
| 33 address _result_handler; /* temp for saving native result handler */ | |
| 34 intptr_t* _sender_sp; /* sender's sp before stack (locals) extension */ | |
| 35 | |
| 36 address _extra_junk1; /* temp to save on recompiles */ | |
| 37 address _extra_junk2; /* temp to save on recompiles */ | |
| 38 address _extra_junk3; /* temp to save on recompiles */ | |
| 39 // address dummy_for_native2; /* a native frame result handler would be here... */ | |
| 40 // address dummy_for_native1; /* native result type stored here in a interpreter native frame */ | |
| 41 address _extra_junk4; /* temp to save on recompiles */ | |
| 42 address _extra_junk5; /* temp to save on recompiles */ | |
| 43 address _extra_junk6; /* temp to save on recompiles */ | |
| 44 public: | |
| 45 // we have an interpreter frame... | |
| 46 inline intptr_t* sender_sp() { | |
| 47 return _sender_sp; | |
| 48 } | |
| 49 | |
| 50 // The interpreter always has the frame anchor fully setup so we don't | |
| 51 // have to do anything going to vm from the interpreter. On return | |
| 52 // we do have to clear the flags in case they we're modified to | |
| 53 // maintain the stack walking invariants. | |
| 54 // | |
| 55 #define SET_LAST_JAVA_FRAME() | |
| 56 | |
| 57 #define RESET_LAST_JAVA_FRAME() | |
| 58 | |
| 59 /* | |
| 60 * Macros for accessing the stack. | |
| 61 */ | |
| 62 #undef STACK_INT | |
| 63 #undef STACK_FLOAT | |
| 64 #undef STACK_ADDR | |
| 65 #undef STACK_OBJECT | |
| 66 #undef STACK_DOUBLE | |
| 67 #undef STACK_LONG | |
| 68 | |
| 69 // JavaStack Implementation | |
| 70 | |
| 71 #define GET_STACK_SLOT(offset) (*((intptr_t*) &topOfStack[-(offset)])) | |
| 72 #define STACK_SLOT(offset) ((address) &topOfStack[-(offset)]) | |
| 73 #define STACK_ADDR(offset) (*((address *) &topOfStack[-(offset)])) | |
| 74 #define STACK_INT(offset) (*((jint*) &topOfStack[-(offset)])) | |
| 75 #define STACK_FLOAT(offset) (*((jfloat *) &topOfStack[-(offset)])) | |
| 76 #define STACK_OBJECT(offset) (*((oop *) &topOfStack [-(offset)])) | |
| 77 #define STACK_DOUBLE(offset) (((VMJavaVal64*) &topOfStack[-(offset)])->d) | |
| 78 #define STACK_LONG(offset) (((VMJavaVal64 *) &topOfStack[-(offset)])->l) | |
| 79 | |
| 80 #define SET_STACK_SLOT(value, offset) (*(intptr_t*)&topOfStack[-(offset)] = *(intptr_t*)(value)) | |
| 81 #define SET_STACK_ADDR(value, offset) (*((address *)&topOfStack[-(offset)]) = (value)) | |
| 82 #define SET_STACK_INT(value, offset) (*((jint *)&topOfStack[-(offset)]) = (value)) | |
| 83 #define SET_STACK_FLOAT(value, offset) (*((jfloat *)&topOfStack[-(offset)]) = (value)) | |
| 84 #define SET_STACK_OBJECT(value, offset) (*((oop *)&topOfStack[-(offset)]) = (value)) | |
| 85 #define SET_STACK_DOUBLE(value, offset) (((VMJavaVal64*)&topOfStack[-(offset)])->d = (value)) | |
| 86 #define SET_STACK_DOUBLE_FROM_ADDR(addr, offset) (((VMJavaVal64*)&topOfStack[-(offset)])->d = \ | |
| 87 ((VMJavaVal64*)(addr))->d) | |
| 88 #define SET_STACK_LONG(value, offset) (((VMJavaVal64*)&topOfStack[-(offset)])->l = (value)) | |
| 89 #define SET_STACK_LONG_FROM_ADDR(addr, offset) (((VMJavaVal64*)&topOfStack[-(offset)])->l = \ | |
| 90 ((VMJavaVal64*)(addr))->l) | |
| 91 // JavaLocals implementation | |
| 92 | |
| 93 #define LOCALS_SLOT(offset) ((intptr_t*)&locals[-(offset)]) | |
| 94 #define LOCALS_ADDR(offset) ((address)locals[-(offset)]) | |
| 95 #define LOCALS_INT(offset) ((jint)(locals[-(offset)])) | |
| 96 #define LOCALS_FLOAT(offset) (*((jfloat*)&locals[-(offset)])) | |
| 97 #define LOCALS_OBJECT(offset) ((oop)locals[-(offset)]) | |
| 98 #define LOCALS_DOUBLE(offset) (((VMJavaVal64*)&locals[-((offset) + 1)])->d) | |
| 99 #define LOCALS_LONG(offset) (((VMJavaVal64*)&locals[-((offset) + 1)])->l) | |
| 100 #define LOCALS_LONG_AT(offset) (((address)&locals[-((offset) + 1)])) | |
| 101 #define LOCALS_DOUBLE_AT(offset) (((address)&locals[-((offset) + 1)])) | |
| 102 | |
| 103 #define SET_LOCALS_SLOT(value, offset) (*(intptr_t*)&locals[-(offset)] = *(intptr_t *)(value)) | |
| 104 #define SET_LOCALS_ADDR(value, offset) (*((address *)&locals[-(offset)]) = (value)) | |
| 105 #define SET_LOCALS_INT(value, offset) (*((jint *)&locals[-(offset)]) = (value)) | |
| 106 #define SET_LOCALS_FLOAT(value, offset) (*((jfloat *)&locals[-(offset)]) = (value)) | |
| 107 #define SET_LOCALS_OBJECT(value, offset) (*((oop *)&locals[-(offset)]) = (value)) | |
| 108 #define SET_LOCALS_DOUBLE(value, offset) (((VMJavaVal64*)&locals[-((offset)+1)])->d = (value)) | |
| 109 #define SET_LOCALS_LONG(value, offset) (((VMJavaVal64*)&locals[-((offset)+1)])->l = (value)) | |
| 110 #define SET_LOCALS_DOUBLE_FROM_ADDR(addr, offset) (((VMJavaVal64*)&locals[-((offset)+1)])->d = \ | |
| 111 ((VMJavaVal64*)(addr))->d) | |
| 112 #define SET_LOCALS_LONG_FROM_ADDR(addr, offset) (((VMJavaVal64*)&locals[-((offset)+1)])->l = \ | |
| 113 ((VMJavaVal64*)(addr))->l) | |
| 1972 | 114 |
| 115 #endif // CPU_X86_VM_BYTECODEINTERPRETER_X86_HPP |