RFR 8143628: Fork sun.misc.Unsafe and jdk.internal.misc.Unsafe native method tables

[Paul Sandoz]

Hi Christian,

> On 1 Dec 2015, at 20:19, Christian Tornqvist <christian.tornqvist at oracle.com> wrote:
> 
> Hi Paul,
> 
> Tests in hotspot/test/runtime needs to be jtreg tests.

They are jtreg tests. They are require to be run (re: “launched") with jtreg see:

http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8143628-unsafe-native-hotspot/webrev/test/runtime/Unsafe/JdkInternalMiscUnsafeAccessTestBoolean.java.html <http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8143628-unsafe-native-hotspot/webrev/test/runtime/Unsafe/JdkInternalMiscUnsafeAccessTestBoolean.java.html>
  24 /*
  25  * @test
  26  * @bug 8143628
  27  * @summary Test unsafe access for boolean
  28  * @modules java.base/jdk.internal.misc
  29  * @run testng/othervm -Diters=100   -Xint                   JdkInternalMiscUnsafeAccessTestBoolean
  30  * @run testng/othervm -Diters=20000 -XX:TieredStopAtLevel=1 JdkInternalMiscUnsafeAccessTestBoolean
  31  * @run testng/othervm -Diters=20000 -XX:-TieredCompilation  JdkInternalMiscUnsafeAccessTestBoolean
  32  * @run testng/othervm -Diters=20000                         JdkInternalMiscUnsafeAccessTestBoolean
  33  */

That’s the point i was making with:

   jtreg is to testng as launcher is to library

Note the use of the "@modules java.base/jdk.internal.misc”. That’s gonna be important later on.


> Looking at your tests, I can't see a reason why they can't easily be modified to be jtreg tests instead?

That’s not the point. There is a principle here about what test libraries one can or cannot use with the test in a particular area of a particular repo. At the moment i am not hearing any consistent and solid technical argument as to why testng cannot be used for HotSpot runtime tests.

Paul.