RFR: 8355569: Some nsk/jdi tests can glean the "main" thread by using the ClassPrepareEvent for the debuggee main class [v3]
Serguei Spitsyn
sspitsyn at openjdk.org
Tue Apr 29 22:41:51 UTC 2025
On Mon, 28 Apr 2025 19:13:07 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:
>> As part of the work for [JDK-8353955](https://bugs.openjdk.org/browse/JDK-8353955) I am reducing the number of tests that need to be run with includevirtualhreads=y due to using JDI to lookup threads in the debuggee. There are many tests that lookup the "main" thread. They can instead glean the "main" thread from the ClassPrepareEvent for the debuggee main class. Some tests already wait for this ClassPrepareEvent, and can take advantage of it. Most do not, but can be made to do so. The easiest way to do this for many of the tests is to wait for the event in Debuggee.prepareDebuggee() after having called Debuggee.bindToDebuggee(). For tests that don't call Debuggee.prepareDebuggee(), waiting for the ClassPrepareEvent was added after the bind. This doesn't seem to have any ill affect on the tests.
>>
>> Tested by running all nsk/jdi tests, including all tier2, tier3, and tier5 nsk/jdi testing.
>
> Chris Plummer has updated the pull request incrementally with one additional commit since the last revision:
>
> found one more test to fix
This looks okay. I've not found problems so far. Will do one more pass.
-------------
PR Review: https://git.openjdk.org/jdk/pull/24867#pullrequestreview-2805224656
More information about the serviceability-dev
mailing list