8202740: runtime/8176717/TestInheritFD.java fails with java.lang.RuntimeException: could not match: VM RESULT => RETAINS FD
authorlkorinth
Thu, 14 Jun 2018 11:13:30 +0200
changeset 50563 1372f66e0a17
parent 50562 3903ab54107e
child 50564 ef7c4c77d9fa
8202740: runtime/8176717/TestInheritFD.java fails with java.lang.RuntimeException: could not match: VM RESULT => RETAINS FD Reviewed-by: rehn, dholmes
test/hotspot/jtreg/runtime/8176717/TestInheritFD.java
--- a/test/hotspot/jtreg/runtime/8176717/TestInheritFD.java	Thu Jun 14 11:46:53 2018 +0200
+++ b/test/hotspot/jtreg/runtime/8176717/TestInheritFD.java	Thu Jun 14 11:13:30 2018 +0200
@@ -1,14 +1,20 @@
 import static java.io.File.createTempFile;
 import static java.lang.Long.parseLong;
 import static java.lang.System.getProperty;
-import static java.lang.management.ManagementFactory.getOperatingSystemMXBean;
 import static java.nio.file.Files.readAllBytes;
+import static java.util.Arrays.stream;
+import static java.util.stream.Collectors.joining;
+import static java.util.stream.Collectors.toList;
 import static jdk.test.lib.process.ProcessTools.createJavaProcessBuilder;
 
+import java.io.BufferedReader;
 import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.FileOutputStream;
 import java.io.IOException;
-
-import com.sun.management.UnixOperatingSystemMXBean;
+import java.io.InputStreamReader;
+import java.util.Collection;
+import java.util.stream.Stream;
 
 /*
  * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved.
@@ -57,8 +63,7 @@
  * The third VM communicates the success to rename the file by printing "CLOSED
  * FD". The first VM checks that the string was printed by the third VM.
  *
- * On unix like systems, UnixOperatingSystemMXBean is used to check open file
- * descriptors.
+ * On unix like systems "lsof" or "pfiles" is used.
  */
 
 public class TestInheritFD {
@@ -66,10 +71,11 @@
     public static final String LEAKS_FD = "VM RESULT => LEAKS FD";
     public static final String RETAINS_FD = "VM RESULT => RETAINS FD";
     public static final String EXIT = "VM RESULT => VM EXIT";
+    public static final String LOG_SUFFIX = ".strangelogsuffixthatcanbecheckedfor";
 
     // first VM
     public static void main(String[] args) throws Exception {
-        String logPath = createTempFile("logging", ".log").getName();
+        String logPath = createTempFile("logging", LOG_SUFFIX).getName();
         File commFile = createTempFile("communication", ".txt");
 
         ProcessBuilder pb = createJavaProcessBuilder(
@@ -103,46 +109,86 @@
                 "-Dtest.jdk=" + getProperty("test.jdk"),
                 VMShouldNotInheritFileDescriptors.class.getName(),
                 args[0],
-                "" + ProcessHandle.current().pid(),
-                "" + (supportsUnixMXBean()?+unixNrFD():-1));
+                "" + ProcessHandle.current().pid());
             pb.inheritIO(); // in future, redirect information from third VM to first VM
             pb.start();
+
+            if (getProperty("os.name").toLowerCase().contains("win") == false) {
+                System.out.println("(Second VM) Open file descriptors:\n" + outputContainingFilenames().stream().collect(joining("\n")));
+            }
         }
     }
 
     static class VMShouldNotInheritFileDescriptors {
         // third VM
         public static void main(String[] args) throws InterruptedException {
-            File logFile = new File(args[0]);
-            long parentPid = parseLong(args[1]);
-            long parentFDCount = parseLong(args[2]);
+            try {
+                File logFile = new File(args[0]);
+                long parentPid = parseLong(args[1]);
+                fakeLeakyJVM(false); // for debugging of test case
 
-            if(supportsUnixMXBean()){
-                long thisFDCount = unixNrFD();
-                System.out.println("This VM FD-count (" + thisFDCount + ") should be strictly less than parent VM FD-count (" + parentFDCount + ") as log file should have been closed");
-                System.out.println(thisFDCount<parentFDCount?RETAINS_FD:LEAKS_FD);
-            } else if (getProperty("os.name").toLowerCase().contains("win")) {
-                windows(logFile, parentPid);
-            } else {
-                System.out.println(LEAKS_FD); // default fail on unknown configuration
+                if (getProperty("os.name").toLowerCase().contains("win")) {
+                    windows(logFile, parentPid);
+                } else {
+                    Collection<String> output = outputContainingFilenames();
+                    System.out.println("(Third VM) Open file descriptors:\n" + output.stream().collect(joining("\n")));
+                    System.out.println(findOpenLogFile(output) ? LEAKS_FD : RETAINS_FD);
+                }
+            } catch (Exception e) {
+                System.out.println(e.toString());
+            } finally {
+                System.out.println(EXIT);
             }
-            System.out.println(EXIT);
+        }
+    }
+
+    // for debugging of test case
+    @SuppressWarnings("resource")
+    static void fakeLeakyJVM(boolean fake) {
+        if (fake) {
+            try {
+                new FileOutputStream("fakeLeakyJVM" + LOG_SUFFIX, false);
+            } catch (FileNotFoundException e) {
+            }
         }
     }
 
-    static boolean supportsUnixMXBean() {
-        return getOperatingSystemMXBean() instanceof UnixOperatingSystemMXBean;
+    static Stream<String> run(String... args){
+        try {
+            return new BufferedReader(new InputStreamReader(new ProcessBuilder(args).start().getInputStream())).lines();
+        } catch (IOException e) {
+            throw new RuntimeException(e);
+        }
     }
 
-    static long unixNrFD() {
-        UnixOperatingSystemMXBean osBean = (UnixOperatingSystemMXBean) getOperatingSystemMXBean();
-        return osBean.getOpenFileDescriptorCount();
+    static Collection<String> outputContainingFilenames() {
+        long pid = ProcessHandle.current().pid();
+        String[] command = stream(new String[][]{
+                {"/usr/bin/lsof", "-p"},
+                {"/usr/sbin/lsof", "-p"},
+                {"/bin/lsof", "-p"},
+                {"/sbin/lsof", "-p"},
+                {"/usr/local/bin/lsof", "-p"},
+                {"/usr/bin/pfiles", "-F"}}) // Solaris
+            .filter(args -> new File(args[0]).exists())
+            .findFirst()
+            .orElseThrow(() -> new RuntimeException("could not find lsof-like command"));
+
+        System.out.println("using command: " + command[0] + " " + command[1]);
+        return run(command[0], command[1], "" + pid).collect(toList());
+    }
+
+    static boolean findOpenLogFile(Collection<String> fileNames) {
+        return fileNames.stream()
+            .filter(fileName -> fileName.contains(LOG_SUFFIX))
+            .findAny()
+            .isPresent();
     }
 
     static void windows(File f, long parentPid) throws InterruptedException {
         System.out.println("waiting for pid: " + parentPid);
         ProcessHandle.of(parentPid).ifPresent(handle -> handle.onExit().join());
         System.out.println("trying to rename file to the same name: " + f);
-        System.out.println(f.renameTo(f)?RETAINS_FD:LEAKS_FD); // this parts communicates a closed file descriptor by printing "CLOSED FD"
+        System.out.println(f.renameTo(f) ? RETAINS_FD : LEAKS_FD); // this parts communicates a closed file descriptor by printing "VM RESULT => RETAINS FD"
     }
 }
\ No newline at end of file