test/jdk/java/io/RandomAccessFile/UnreferencedRAFClosesFd.java
author tschatzl
Wed, 24 Jul 2019 11:49:39 +0200
changeset 57508 28ab01c06755
parent 50727 081b132c4dc0
permissions -rw-r--r--
8228388: Add information about dirty/skipped card for Merge HCC in G1 log Summary: Collect and print informatio about the number of processed cards during the Merge HCC phase to improve log output. Reviewed-by: kbarrett, sangheki
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
48224
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
     1
/*
48534
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
     2
 * Copyright (c) 2007, 2018, Oracle and/or its affiliates. All rights reserved.
48224
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
     4
 *
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
     7
 * published by the Free Software Foundation.
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
     8
 *
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    13
 * accompanied this code).
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    14
 *
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    18
 *
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    21
 * questions.
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    22
 */
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    23
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    24
import java.io.File;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    25
import java.io.FileDescriptor;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    26
import java.io.FileNotFoundException;
50727
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
    27
import java.io.IOException;
48224
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    28
import java.io.RandomAccessFile;
48534
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
    29
import java.lang.management.ManagementFactory;
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
    30
import java.lang.management.OperatingSystemMXBean;
48224
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    31
import java.lang.ref.Cleaner;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    32
import java.lang.ref.Reference;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    33
import java.lang.ref.ReferenceQueue;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    34
import java.lang.ref.WeakReference;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    35
import java.lang.reflect.Field;
50727
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
    36
import java.nio.file.Path;
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
    37
import java.util.ArrayDeque;
48224
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    38
import java.util.HashSet;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    39
48534
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
    40
import com.sun.management.UnixOperatingSystemMXBean;
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
    41
50727
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
    42
import jdk.test.lib.util.FileUtils;
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
    43
48224
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    44
/**
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    45
 * @test
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    46
 * @bug 8080225
50727
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
    47
 * @library /test/lib
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
    48
 * @build jdk.test.lib.util.FileUtils UnreferencedRAFClosesFd
48224
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    49
 * @modules java.base/java.io:open
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    50
 * @summary Test to ensure that an unclosed and unreferenced RandomAccessFile closes the fd
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    51
 * @run main/othervm UnreferencedRAFClosesFd
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    52
 */
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    53
public class UnreferencedRAFClosesFd {
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    54
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    55
    static final String FILE_NAME = "empty.txt";
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    56
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    57
    /* standalone interface */
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    58
    public static void main(String argv[]) throws Exception {
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    59
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    60
        File inFile= new File(System.getProperty("test.dir", "."), FILE_NAME);
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    61
        inFile.createNewFile();
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    62
        inFile.deleteOnExit();
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    63
50727
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
    64
        FileUtils.listFileDescriptors(System.out);
48534
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
    65
        long fdCount0 = getFdCount();
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
    66
48224
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    67
        String name = inFile.getPath();
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    68
        RandomAccessFile raf;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    69
        try {
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    70
            // raf is explicitly *not* closed to allow cleaner to work
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    71
            raf = new RandomAccessFile(name, "rw");
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    72
        } catch (FileNotFoundException e) {
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    73
            System.out.println("Unexpected exception " + e);
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    74
            throw(e);
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    75
        }
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    76
        FileDescriptor fd = raf.getFD();
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    77
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    78
        Field fdField = FileDescriptor.class.getDeclaredField("cleanup");
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    79
        fdField.setAccessible(true);
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    80
        Cleaner.Cleanable cleanup = (Cleaner.Cleanable)fdField.get(fd);
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    81
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    82
        // Prepare to wait for FOS, FD, Cleanup to be reclaimed
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    83
        ReferenceQueue<Object> queue = new ReferenceQueue<>();
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    84
        HashSet<Reference<?>> pending = new HashSet<>(3);
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    85
        pending.add(new WeakReference<>(cleanup, queue));
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    86
        pending.add(new WeakReference<>(raf, queue));
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    87
        pending.add(new WeakReference<>(fd, queue));
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    88
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    89
        Reference<?> r;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    90
        while (((r = queue.remove(10L)) != null)
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    91
                || !pending.isEmpty()) {
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    92
            System.out.printf("r: %s, pending: %d%n", r, pending.size());
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    93
            if (r != null) {
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    94
                pending.remove(r);
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    95
            } else {
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    96
                cleanup = null;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    97
                raf = null;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    98
                fd = null;
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
    99
                System.gc();  // attempt to reclaim the RAF, cleanup, and fd
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
   100
            }
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
   101
        }
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
   102
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
   103
        // Keep these variables in scope as gc roots
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
   104
        Reference.reachabilityFence(cleanup);
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
   105
        Reference.reachabilityFence(fd);
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
   106
        Reference.reachabilityFence(raf);
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
   107
        Reference.reachabilityFence(pending);
48534
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   108
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   109
        // Check the final count of open file descriptors
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   110
        long fdCount = getFdCount();
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   111
        if (fdCount != fdCount0) {
50727
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
   112
            System.out.printf("initial count of open file descriptors: %d%n", fdCount0);
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
   113
            System.out.printf("final count of open file descriptors: %d%n", fdCount);
081b132c4dc0 8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents: 48534
diff changeset
   114
            FileUtils.listFileDescriptors(System.out);
48534
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   115
        }
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   116
    }
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   117
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   118
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   119
    // Get the count of open file descriptors, or -1 if not available
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   120
    private static long getFdCount() {
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   121
        OperatingSystemMXBean mxBean = ManagementFactory.getOperatingSystemMXBean();
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   122
        return  (mxBean instanceof UnixOperatingSystemMXBean)
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   123
                ? ((UnixOperatingSystemMXBean) mxBean).getOpenFileDescriptorCount()
12d9ff9e0a4b 8194929: Unreferenced FileDescriptors not closed
rriggs
parents: 48224
diff changeset
   124
                : -1L;
48224
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
   125
    }
be0df5ab3093 8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
diff changeset
   126
}