RFR: 8262881: port JVM/DI tests from JDK-4413752 to JVM/TI [v4]

Daniel D.Daugherty dcubed at openjdk.java.net
Thu Apr 1 19:46:39 UTC 2021


On Thu, 1 Apr 2021 19:39:07 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> @sspitsyn - Thanks for the re-review. I'll take care of the unused variables
>> and I'll do an audit of all three tests and look for more.
>
> Changes for the next version of the tests:
> 
> - @robehn CR changes:
>     -  changed the JVM/TI function wrappers to be much simpler and just return the JVM/TI return code to the Java code caller; all error checking is now on the Java side of the test.
>     - dropped the 'id' parameter; deleted many native support functions.
> 
> @robehn - I kept the catch of UnsatisfiedLinkError because what I'm doing there is printing a nice error message and then rethrowing the same exception; it makes it easier to debug the build process for the test.
> @robehn - I moved the argument parsing code to the main() method; while the default configuration of the test doesn't use command line arguments, I have stress wrappers for these tests that use the command line args.
> 
> @lyndseyBeil - I renamed the remaining native methods to `camelCase()` style.
> 
> @sspitsyn - I've removed the unused variables from the three tests.
> 
> @robehn, @lyndseyBeil and @sspitsyn - thanks for your reviews! New commit coming shortly.

The v03 version was tested with Mach5 Tier[134567] testing. The three new tests passed
in all configurations in all of those tiers.

I also used the latest version of the tests to reproduce the failure mode that I'm
hunting in "JDK-8264393 JDK-8258284 introduced dangling TLH race" so these
tests still help reproduce that bug.

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

PR: https://git.openjdk.java.net/jdk/pull/2899


More information about the serviceability-dev mailing list