RFR: 8262271: SA: Add new stress test that tests getting the stack trace of an active thread
Chris Plummer
cjplummer at openjdk.java.net
Tue Mar 9 06:24:07 UTC 2021
On Tue, 9 Mar 2021 05:32:28 GMT, Yasumasa Suenaga <ysuenaga at openjdk.org> wrote:
>> This a new test that runs jstack on a process that is busy doing things. It runs jstack 4 times as the process is starting up.
>
> test/jdk/sun/tools/jhsdb/JStackStressTest.java line 91:
>
>> 89: } else {
>> 90: System.out.println("Jshell not alive");
>> 91: }
>
> Shouldn't we use `Process::waitFor` at this point to ensure the jshell is terminated? This test might not have enough time to shutdown since issuing `/exit`.
> `destroy()` should be called when timeout is happened at `waitFor()`.
Yeah, I'll do that. Not sure why I didn't do it that way in the first place. I think I copied this from somewhere, but now I can't find where.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2720
More information about the serviceability-dev
mailing list