On Tue, 26 Sep 2023 13:49:02 GMT, Varada M <d...@openjdk.org> wrote:
> Similar issue [JDK-8303549](https://bugs.openjdk.org/browse/JDK-8303549)
> where AttachCurrentThread is failing on AIX due t stack size issue.
> Test cases:
> runtime/jni/terminatedThread/TestTerminatedThread.java
> vmTestbase/nsk/jvmti/scenarios/jni_interception/JI05/ji05t001/TestDescription.java
>
> vmTestbase/nsk/jvmti/scenarios/jni_interception/JI06/ji06t001/TestDescription.java
>
> vmTestbase/nsk/jvmti/SetJNIFunctionTable/setjniftab001/TestDescription.java
>
> Reported Issue : [JDK-8219652](https://bugs.openjdk.org/browse/JDK-8219652)
test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/jni_interception/JI06/ji06t001/ji06t001.cpp
line 52:
> 50: NSK_DISPLAY0("Detaching thread ...\n"); \
> 51: return 0; \
> 52: exit(-1); \
The indent of the `return statement` is wrong. It's at the same level as the
`exit()`, which means the `exit()` is never executed.
This code now ignores the `status` argument, so it doesn't seem to be correct.
I'm not sure why any of the changes here were necessary.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15924#discussion_r1337457128
