RFR: 8288532: additional review changes for JDK-8286830 [v2]
Daniel D. Daugherty
dcubed at openjdk.org
Fri Jun 17 21:28:01 UTC 2022
On Fri, 17 Jun 2022 19:32:46 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:
>> Update a couple of tests that were modified by JDK-8286830. Now the tests properly
>> honor any specified time limit by splitting the time between both sub-tests.
>> Also pick up a sanity check based on JDK-8288497 that I've been using in recent
>> stress testing.
>
> Daniel D. Daugherty has updated the pull request incrementally with one additional commit since the last revision:
>
> Check in AsyncExceptionHandshake() dtr can assume the caller is a JavaThread.
I forgot that install_async_exception() can delete the AsyncExceptionHandshake which calls the AsyncExceptionHandshake dtr:
```--------------- T H R E A D ---------------
Current thread (0x0000ffff001bab70): VMThread "VM Thread" [stack: 0x0000fffee8110000,0x0000fffee8310000] [id=661857] _threads_hazard_ptr=0x0000fffc8021f770, _nested_threads_hazard_ptr_cnt=0
Stack: [0x0000fffee8110000,0x0000fffee8310000], sp=0x0000fffee830e030, free space=2040k
Native frames: (J=compiled Java code, j=interpreted, Vv=VM code, C=native code)
V [libjvm.so+0x1909960] AsyncExceptionHandshake::~AsyncExceptionHandshake()+0x230
V [libjvm.so+0x1908a7c] JavaThread::install_async_exception(AsyncExceptionHandshake*)+0x4c
V [libjvm.so+0x190ace8] InstallAsyncExceptionHandshake::do_thread(Thread*)+0x38
V [libjvm.so+0xe67d20] HandshakeOperation::do_handshake(JavaThread*)+0x13c
V [libjvm.so+0xe68cec] HandshakeState::try_process(HandshakeOperation*)+0x23c
V [libjvm.so+0xe6b384] VM_HandshakeAllThreads::doit()+0x1f4
V [libjvm.so+0x19f2788] VM_Operation::evaluate()+0x168
V [libjvm.so+0x1a1c9d4] VMThread::evaluate_operation(VM_Operation*)+0x154
V [libjvm.so+0x1a1e1bc] VMThread::inner_execute(VM_Operation*)+0x45c
V [libjvm.so+0x1a1e374] VMThread::loop()+0xc4
V [libjvm.so+0x1a1e4a0] VMThread::run()+0xcc
V [libjvm.so+0x1907bb8] Thread::call_run()+0xf8
V [libjvm.so+0x16033a4] thread_native_entry(Thread*)+0x104
C [libpthread.so.0+0x78f8] start_thread+0x188
Gotta back out last change... and restart all the testing... sigh...
-------------
PR: https://git.openjdk.org/jdk19/pull/32
More information about the hotspot-runtime-dev
mailing list