[Misc] Backport JDK-8242314 to fix test/hotspot/jtreg/vmTestbase/* failure

Summary: Backport JDK-8242314 to fix test/hotspot/jtreg/vmTestbase/* failure

Testing: test/hotspot/jtreg/vmTestbase/*

Reviewers: MaxXSoft, D-D-H

Backport-of: 6ff66db

Issue: #983

Author: yaqsun

test/hotspot/jtreg/vmTestbase/ExecDriver.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2017, 2019, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2017, 2020, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -21,16 +21,16 @@
  * questions.
  */
 
-import jdk.test.lib.Platform;
-import jdk.test.lib.Utils;
-
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
 import java.nio.file.Path;
 import java.nio.file.Paths;
+import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collections;
+import java.util.List;
 
 /**
  * Starts a new process to execute a command.
@@ -48,6 +48,10 @@
  * {@code --java}, i.e. 0 or 95 means pass.
  */
 public class ExecDriver {
+    // copied from jdk.test.lib.Utils.TEST_CLASS_PATH
+    private static final String TEST_CLASS_PATH = System.getProperty("test.class.path", ".");
+    // copied from jdk.test.lib.Utils.TEST_CLASS_PATH
+    private static final String TEST_JDK = System.getProperty("test.jdk");
     public static void main(String[] args) throws IOException, InterruptedException {
         boolean java = false;
         boolean launcher = false;
@@ -68,7 +72,7 @@ public static void main(String[] args) throws IOException, InterruptedException
                 }
                 args[0] = javaBin();
                 args[1] = "-cp";
-                args[2] = Utils.TEST_CLASS_PATH;
+                args[2] = TEST_CLASS_PATH;
                 System.arraycopy(oldArgs, 1, args, count, oldArgs.length - 1);
                 java = true;
                 break;
@@ -84,7 +88,7 @@ public static void main(String[] args) throws IOException, InterruptedException
         // adding 'test.vm.opts' and 'test.java.opts'
         if (java) {
             String[] oldArgs = args;
-            String[] testJavaOpts = Utils.getTestJavaOpts();
+            String[] testJavaOpts = getTestJavaOpts();
             if (testJavaOpts.length > 0) {
                 args = new String[args.length + testJavaOpts.length];
                 // bin/java goes before options
@@ -101,10 +105,11 @@ public static void main(String[] args) throws IOException, InterruptedException
         ProcessBuilder pb = new ProcessBuilder(args);
         // adding jvm.so to library path
         if (launcher) {
-            Path dir = Paths.get(Utils.TEST_JDK);
+            Path dir = Paths.get(TEST_JDK);
             String value;
-            String name = Platform.sharedLibraryPathVariableName();
-            if (Platform.isWindows()) {
+            String name = sharedLibraryPathVariableName();
+            // if (jdk.test.lib.Platform.isWindows()) {
+            if (System.getProperty("os.name").toLowerCase().startsWith("win")) {
                 value = dir.resolve("bin")
                            .resolve(variant())
                            .toAbsolutePath()
@@ -125,7 +130,7 @@ public static void main(String[] args) throws IOException, InterruptedException
                       .merge(name, value, (x, y) -> y + File.pathSeparator + x));
             System.out.println("  with CLASSPATH = " +
                     pb.environment()
-                      .put("CLASSPATH", Utils.TEST_CLASS_PATH));
+                      .put("CLASSPATH", TEST_CLASS_PATH));
         }
         Process p = pb.start();
         // inheritIO does not work as expected for @run driver
@@ -138,19 +143,52 @@ public static void main(String[] args) throws IOException, InterruptedException
         }
     }
 
+    // copied from jdk.test.lib.Platform::sharedLibraryPathVariableName
+    private static String sharedLibraryPathVariableName() {
+        String osName = System.getProperty("os.name").toLowerCase();
+        if (osName.startsWith("win")) {
+            return "PATH";
+        } else if (osName.startsWith("mac")) {
+            return "DYLD_LIBRARY_PATH";
+        } else if (osName.startsWith("aix")) {
+            return "LIBPATH";
+        } else {
+            return "LD_LIBRARY_PATH";
+        }
+    }
+
+    // copied from jdk.test.lib.Utils::getTestJavaOpts()
+    private static String[] getTestJavaOpts() {
+        List<String> opts = new ArrayList<String>();
+        {
+            String v = System.getProperty("test.vm.opts", "").trim();
+            if (!v.isEmpty()) {
+                Collections.addAll(opts, v.split("\\s+"));
+            }
+        }
+        {
+            String v = System.getProperty("test.java.opts", "").trim();
+            if (!v.isEmpty()) {
+                Collections.addAll(opts, v.split("\\s+"));
+            }
+        }
+        return opts.toArray(new String[0]);
+    }
+
+    // copied jdk.test.lib.Platform::variant
     private static String variant() {
-        if (Platform.isServer()) {
+        String vmName = System.getProperty("java.vm.name");
+        if (vmName.endsWith(" Server VM")) {
             return "server";
-        } else if (Platform.isClient()) {
+        } else if (vmName.endsWith(" Client VM")) {
             return "client";
-        } else if (Platform.isMinimal()) {
+        } else if (vmName.endsWith(" Minimal VM")) {
             return "minimal";
         } else {
             throw new Error("TESTBUG: unsuppported vm variant");
         }
     }
 
-
     private static void copy(InputStream is, OutputStream os) {
         byte[] buffer = new byte[1024];
         int n;
@@ -165,7 +203,7 @@ private static void copy(InputStream is, OutputStream os) {
     }
 
     private static String javaBin() {
-        return Paths.get(Utils.TEST_JDK)
+        return Paths.get(TEST_JDK)
                     .resolve("bin")
                     .resolve("java")
                     .toAbsolutePath()

test/hotspot/jtreg/vmTestbase/nsk/share/gc/NonbranchyTree.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2003, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2003, 2020, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -27,6 +27,7 @@
 import java.util.*;
 
 import nsk.share.test.ExecutionController;
+import nsk.share.test.LocalRandom;
 
 /**
  * <tt>NonbranchyTree</tt> defines a tree structure. Each node of the tree
@@ -38,7 +39,6 @@ public class NonbranchyTree {
     /** Minimal size of each node (in bytes) */
     public final static int MIN_NODE_SIZE = 20;
     private Node root;
-    private Random random;
     private int numberOfNodes;
     private float branchiness;
     private int size;
@@ -61,21 +61,16 @@ public class NonbranchyTree {
      *
      */
     public NonbranchyTree(int numberOfNodes, float branchiness, int size) {
-        this(numberOfNodes, branchiness, size, new Random(System.currentTimeMillis()), null);
+        this(numberOfNodes, branchiness, size, null);
         initTree();
     }
 
     public NonbranchyTree(int numberOfNodes, float branchiness, int size, ExecutionController controller) {
-        this(numberOfNodes, branchiness, size, new Random(System.currentTimeMillis()), controller);
-        initTree();
-    }
-
-    private NonbranchyTree(int numberOfNodes, float branchiness, int size, Random random, ExecutionController controller) {
         this.numberOfNodes = numberOfNodes;
         this.branchiness = branchiness;
         this.size = size;
-        this.random = random;
         this.controller = controller;
+        initTree();
     }
 
     private void initTree() {
@@ -94,6 +89,8 @@ private void initTree() {
             throw new IllegalArgumentException("Illegal size of nodes: "
                                              + size + ", must be at least 1.");
         }
+        // ensure that LocalRandom is loaded and has enough memory
+        LocalRandom.nextBoolean();
         root = createTree(numberOfNodes, size);
     }
 
@@ -119,7 +116,7 @@ private Node createTree(int numberOfNodes, int size) {
                 // Create a few nodes
                 if (makeRightNode()) {
                     // The node will have two sons
-                    int leftNodes = 1 + random.nextInt(numberOfNodes - 2);
+                    int leftNodes = 1 + LocalRandom.nextInt(numberOfNodes - 2);
                     int rightNodes = numberOfNodes - 1 - leftNodes;
 
                     node.left = createTree(leftNodes, size);
@@ -142,7 +139,7 @@ private Node createTree(int numberOfNodes, int size) {
 
     // Define the "branchiness" of the tree
     private boolean makeRightNode() {
-        return (random.nextFloat() < branchiness);
+        return (LocalRandom.nextFloat() < branchiness);
     }
 
     /**

test/hotspot/jtreg/vmTestbase/nsk/share/jdi/SerialExecutionDebugger.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2006, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2006, 2020, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -27,6 +27,7 @@
 import nsk.share.jpda.AbstractDebuggeeTest;
 import java.io.*;
 import java.util.*;
+import jdk.test.lib.Utils;
 
 /*
  * This class serial executes several JDI tests based on nsk.share.jdi.TestDebuggerType2 in single VM
@@ -192,7 +193,7 @@ else if (arguments[i].equalsIgnoreCase("iterations") && (i < (arguments.length -
             if (testWorkDir == null)
                 throw new TestBug("Debugger requires -testWorkDir parameter");
 
-            Collections.shuffle(result);
+            Collections.shuffle(result, Utils.getRandomInstance());
 
             // save resulted tests sequence in file (to simplify reproducing)
             try {

test/hotspot/jtreg/vmTestbase/nsk/share/runner/RunParams.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2007, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2007, 2020, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -26,13 +26,13 @@
 import nsk.share.log.Log;
 import nsk.share.test.StressOptions;
 import java.io.PrintStream;
+import jdk.test.lib.Utils;
 
 public class RunParams {
         private StressOptions stressOptions;
         private long sleepTime = 500;
         private long iterations = 0;
         private int numberOfThreads;
-        private long seed = System.currentTimeMillis();
         private boolean runGCThread = false;
         private boolean runFinThread = false;
         private boolean runMemDiagThread = false;
@@ -126,11 +126,9 @@ public final void setNumberOfThreads(int numberOfThreads) {
         }
 
         public final long getSeed() {
-                return seed;
-        }
-
-        public final void setSeed(long seed) {
-                this.seed = seed;
+                // ensure that seed got printed out
+                Utils.getRandomInstance();
+                return Utils.SEED;
         }
 
         public final boolean isRunGCThread() {
@@ -212,8 +210,6 @@ else if (args[i].equals("-Df1"))
                                 runFinDiagThread = true;
                         else if (args[i].equals("-Df"))
                                 runFinDiagThread = true;
-                        else if (args[i].equals("-s"))
-                                seed = Long.parseLong(args[++i]);
                         else if (args[i].equals("-t"))
                                 numberOfThreads = Integer.parseInt(args[++i]);
                         else if (args[i].equals("-it"))
@@ -233,7 +229,6 @@ public void printConfig(PrintStream out) {
                 out.println("Sleep time: " + sleepTime);
                 out.println("Iterations: " + iterations);
                 out.println("Number of threads: " + numberOfThreads);
-                out.println("Seed: " + seed);
                 out.println("Run GC thread: " + runGCThread);
                 out.println("Run mem diag thread: " + runMemDiagThread);
                 out.println("Run forever: " + runForever);
@@ -244,7 +239,6 @@ public void logConfig(Log log) {
                 log.debug("Sleep time: " + sleepTime);
                 log.debug("Iterations: " + iterations);
                 log.debug("Number of threads: " + numberOfThreads);
-                log.debug("Seed: " + seed);
                 log.debug("Run GC thread: " + runGCThread);
                 log.debug("Run mem diag thread: " + runMemDiagThread);
                 log.debug("Run forever: " + runForever);