8372851: Modify java/io/File/GetXSpace.java to print path on failure of native call

Reviewed-by: jpai, naoto

Author: Brian Burkhalter

test/jdk/java/io/File/GetXSpace.java

@@ -106,15 +106,15 @@ private static class Space {
         Space(String name) {
             this.name = name;
             long[] sizes = new long[4];
-            if (Platform.isWindows() & isCDDrive(name)) {
+            if (Platform.isWindows() && isCDDrive(name)) {
                 try {
                     getCDDriveSpace(name, sizes);
                 } catch (IOException e) {
                     e.printStackTrace();
                     throw new RuntimeException("can't get CDDrive sizes");
                 }
             } else {
-                if (getSpace0(name, sizes))
+                if (getSpace(name, sizes))
                     System.err.println("WARNING: total space is estimated");
             }
             this.size = sizes[0];
@@ -184,7 +184,7 @@ private static void compare(Space s) {
 
         out.format("%s (%d):%n", s.name(), s.size());
         String fmt = "  %-4s total = %12d free = %12d usable = %12d%n";
-        String method = Platform.isWindows() & isCDDrive(s.name()) ? "getCDDriveSpace" : "getSpace0";
+        String method = Platform.isWindows() && isCDDrive(s.name()) ? "getCDDriveSpace" : "getSpace";
         out.format(fmt, method, s.total(), s.free(), s.available());
         out.format(fmt, "getXSpace", ts, fs, us);
 
@@ -336,7 +336,7 @@ private static int testFile(Path dir) {
     private static int testVolumes() {
         out.println("--- Testing volumes");
         // Find all of the partitions on the machine and verify that the sizes
-        // returned by File::getXSpace are equivalent to those from getSpace0 or getCDDriveSpace
+        // returned by File::getXSpace are equivalent to those from getSpace or getCDDriveSpace
         ArrayList<String> l;
         try {
             l = paths();
@@ -412,6 +412,19 @@ public static void main(String[] args) throws Exception {
 
     private static native boolean isCDDrive(String root);
 
+    private static boolean getSpace(String root, long[] space) {
+        try {
+            return getSpace0(root, space);
+        } catch (RuntimeException e) {
+            File f = new File(root);
+            boolean exists = f.exists();
+            boolean readable = f.canRead();
+            System.err.printf("getSpace0 failed for %s (%s, %s)%n",
+                              root, exists, readable);
+            throw e;
+        }
+    }
+
     private static void getCDDriveSpace(String root, long[] sizes)
         throws IOException {
         String[] cmd = new String[] {"df", "-k", "-P", root};