> 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:

  glean main thread from ClassPrepareEvent

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/24867/files
  - new: https://git.openjdk.org/jdk/pull/24867/files/c467d8c9..060c8d3d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=24867&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=24867&range=00-01

  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/24867.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24867/head:pull/24867

PR: https://git.openjdk.org/jdk/pull/24867

Reply via email to