author | tschatzl |
Wed, 24 Jul 2019 11:49:39 +0200 | |
changeset 57508 | 28ab01c06755 |
parent 52149 | 0edbbc64393c |
permissions | -rw-r--r-- |
2 | 1 |
/* |
48534 | 2 |
* Copyright (c) 2007, 2018, Oracle and/or its affiliates. All rights reserved. |
2 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5506 | 19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
20 |
* or visit www.oracle.com if you need additional information or have any |
|
21 |
* questions. |
|
2 | 22 |
*/ |
23 |
||
24 |
/** |
|
25 |
* |
|
26 |
* @test |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
27 |
* @modules java.base/java.io:open |
50727
081b132c4dc0
8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents:
48941
diff
changeset
|
28 |
* @library /test/lib |
081b132c4dc0
8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents:
48941
diff
changeset
|
29 |
* @build jdk.test.lib.util.FileUtils UnreferencedFISClosesFd |
2 | 30 |
* @bug 6524062 |
31 |
* @summary Test to ensure that FIS.finalize() invokes the close() method as per |
|
32 |
* the specification. |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
33 |
* @run main/othervm UnreferencedFISClosesFd |
2 | 34 |
*/ |
35 |
import java.io.File; |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
36 |
import java.io.FileDescriptor; |
2 | 37 |
import java.io.FileInputStream; |
38 |
import java.io.FileNotFoundException; |
|
39 |
import java.io.IOException; |
|
48534 | 40 |
import java.lang.management.ManagementFactory; |
41 |
import java.lang.management.OperatingSystemMXBean; |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
42 |
import java.lang.ref.Reference; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
43 |
import java.lang.ref.ReferenceQueue; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
44 |
import java.lang.ref.WeakReference; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
45 |
import java.lang.reflect.Field; |
48941 | 46 |
import java.nio.file.Path; |
47 |
import java.util.ArrayDeque; |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
48 |
import java.util.HashSet; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
49 |
import java.util.concurrent.atomic.AtomicInteger; |
2 | 50 |
|
48534 | 51 |
import com.sun.management.UnixOperatingSystemMXBean; |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
52 |
|
50727
081b132c4dc0
8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents:
48941
diff
changeset
|
53 |
import jdk.test.lib.util.FileUtils; |
081b132c4dc0
8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents:
48941
diff
changeset
|
54 |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
55 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
56 |
* Tests for FIS unreferenced. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
57 |
* - Not subclassed - cleaner cleanup |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
58 |
* - Subclassed no finalize or close - cleaner cleanup |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
59 |
* - Subclassed close overridden - AltFinalizer cleanup |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
60 |
* - Subclasses finalize overridden - cleaner cleanup |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
61 |
* - Subclasses finalize and close overridden - AltFinalizer cleanup |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
62 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
63 |
public class UnreferencedFISClosesFd { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
64 |
|
2 | 65 |
static final String FILE_NAME = "empty.txt"; |
66 |
||
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
67 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
68 |
* Subclass w/ no overrides; not finalize or close. |
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
69 |
* Cleanup should be via the Cleaner. |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
70 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
71 |
public static class StreamOverrides extends FileInputStream { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
72 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
73 |
protected final AtomicInteger closeCounter; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
74 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
75 |
public StreamOverrides(String name) throws FileNotFoundException { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
76 |
super(name); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
77 |
closeCounter = new AtomicInteger(0); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
78 |
} |
2 | 79 |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
80 |
final AtomicInteger closeCounter() { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
81 |
return closeCounter; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
82 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
83 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
84 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
85 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
86 |
* Subclass overrides close. |
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
87 |
* Cleanup should be via the Cleaner. |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
88 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
89 |
public static class StreamOverridesClose extends StreamOverrides { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
90 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
91 |
public StreamOverridesClose(String name) throws FileNotFoundException { |
2 | 92 |
super(name); |
93 |
} |
|
94 |
||
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
95 |
public void close() throws IOException { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
96 |
closeCounter.incrementAndGet(); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
97 |
super.close(); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
98 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
99 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
100 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
101 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
102 |
* Subclass overrides finalize. |
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
103 |
* Cleanup should be via the Cleaner. |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
104 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
105 |
public static class StreamOverridesFinalize extends StreamOverrides { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
106 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
107 |
public StreamOverridesFinalize(String name) throws FileNotFoundException { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
108 |
super(name); |
2 | 109 |
} |
110 |
||
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
111 |
@SuppressWarnings({"deprecation","removal"}) |
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
112 |
protected void finalize() throws IOException, Throwable { |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
113 |
super.finalize(); |
2 | 114 |
} |
115 |
} |
|
116 |
||
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
117 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
118 |
* Subclass overrides finalize and close. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
119 |
* Cleanup should be via AltFinalizer calling close(). |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
120 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
121 |
public static class StreamOverridesFinalizeClose extends StreamOverridesClose { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
122 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
123 |
public StreamOverridesFinalizeClose(String name) throws FileNotFoundException { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
124 |
super(name); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
125 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
126 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
127 |
@SuppressWarnings({"deprecation","removal"}) |
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
128 |
protected void finalize() throws IOException, Throwable { |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
129 |
super.finalize(); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
130 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
131 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
132 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
133 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
134 |
* Main runs each test case and reports number of failures. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
135 |
*/ |
2 | 136 |
public static void main(String argv[]) throws Exception { |
137 |
||
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
138 |
File inFile = new File(System.getProperty("test.dir", "."), FILE_NAME); |
2 | 139 |
inFile.createNewFile(); |
140 |
inFile.deleteOnExit(); |
|
141 |
||
142 |
String name = inFile.getPath(); |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
143 |
|
50727
081b132c4dc0
8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents:
48941
diff
changeset
|
144 |
FileUtils.listFileDescriptors(System.out); |
48534 | 145 |
long fdCount0 = getFdCount(); |
146 |
||
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
147 |
int failCount = 0; |
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
148 |
failCount += test(new FileInputStream(name)); |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
149 |
|
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
150 |
failCount += test(new StreamOverrides(name)); |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
151 |
|
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
152 |
failCount += test(new StreamOverridesClose(name)); |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
153 |
|
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
154 |
failCount += test(new StreamOverridesFinalize(name)); |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
155 |
|
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
156 |
failCount += test(new StreamOverridesFinalizeClose(name)); |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
157 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
158 |
if (failCount > 0) { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
159 |
throw new AssertionError("Failed test count: " + failCount); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
160 |
} |
48534 | 161 |
|
162 |
// Check the final count of open file descriptors |
|
163 |
long fdCount = getFdCount(); |
|
164 |
if (fdCount != fdCount0) { |
|
50727
081b132c4dc0
8202292: java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
rriggs
parents:
48941
diff
changeset
|
165 |
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:
48941
diff
changeset
|
166 |
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:
48941
diff
changeset
|
167 |
FileUtils.listFileDescriptors(System.out); |
48534 | 168 |
} |
169 |
} |
|
170 |
||
171 |
// Get the count of open file descriptors, or -1 if not available |
|
172 |
private static long getFdCount() { |
|
173 |
OperatingSystemMXBean mxBean = ManagementFactory.getOperatingSystemMXBean(); |
|
174 |
return (mxBean instanceof UnixOperatingSystemMXBean) |
|
175 |
? ((UnixOperatingSystemMXBean) mxBean).getOpenFileDescriptorCount() |
|
176 |
: -1L; |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
177 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
178 |
|
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
179 |
private static int test(FileInputStream fis) throws Exception { |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
180 |
|
2 | 181 |
try { |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
182 |
System.out.printf("%nTesting %s%n", fis.getClass().getName()); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
183 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
184 |
// Prepare to wait for FIS to be reclaimed |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
185 |
ReferenceQueue<Object> queue = new ReferenceQueue<>(); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
186 |
HashSet<Reference<?>> pending = new HashSet<>(); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
187 |
WeakReference<FileInputStream> msWeak = new WeakReference<>(fis, queue); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
188 |
pending.add(msWeak); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
189 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
190 |
FileDescriptor fd = fis.getFD(); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
191 |
WeakReference<FileDescriptor> fdWeak = new WeakReference<>(fd, queue); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
192 |
pending.add(fdWeak); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
193 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
194 |
Field fdField = FileDescriptor.class.getDeclaredField("fd"); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
195 |
fdField.setAccessible(true); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
196 |
int ffd = fdField.getInt(fd); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
197 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
198 |
Field cleanupField = FileDescriptor.class.getDeclaredField("cleanup"); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
199 |
cleanupField.setAccessible(true); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
200 |
Object cleanup = cleanupField.get(fd); |
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
201 |
System.out.printf(" cleanup: %s, ffd: %d, cf: %s%n", cleanup, ffd, cleanupField); |
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
202 |
if (cleanup == null) { |
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
203 |
throw new RuntimeException("cleanup should not be null"); |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
204 |
} |
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
205 |
|
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
206 |
WeakReference<Object> cleanupWeak = new WeakReference<>(cleanup, queue); |
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
207 |
pending.add(cleanupWeak); |
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
208 |
System.out.printf(" fdWeak: %s%n msWeak: %s%n cleanupWeak: %s%n", |
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
209 |
fdWeak, msWeak, cleanupWeak); |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
210 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
211 |
AtomicInteger closeCounter = fis instanceof StreamOverrides |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
212 |
? ((StreamOverrides)fis).closeCounter() : null; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
213 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
214 |
Reference<?> r; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
215 |
while (((r = queue.remove(1000L)) != null) |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
216 |
|| !pending.isEmpty()) { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
217 |
System.out.printf(" r: %s, pending: %d%n", |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
218 |
r, pending.size()); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
219 |
if (r != null) { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
220 |
pending.remove(r); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
221 |
} else { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
222 |
fis = null; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
223 |
fd = null; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
224 |
cleanup = null; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
225 |
System.gc(); // attempt to reclaim them |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
226 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
227 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
228 |
Reference.reachabilityFence(fd); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
229 |
Reference.reachabilityFence(fis); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
230 |
Reference.reachabilityFence(cleanup); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
231 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
232 |
// Confirm the correct number of calls to close depending on the cleanup type |
52149
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
233 |
if (closeCounter != null && closeCounter.get() > 0) { |
0edbbc64393c
8192939: Remove Finalize methods from FileInputStream and FileOutputStream
rriggs
parents:
50727
diff
changeset
|
234 |
throw new RuntimeException("Close should not have been called: count: " + closeCounter); |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
235 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
236 |
} catch (Exception ex) { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
237 |
ex.printStackTrace(System.out); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
238 |
return 1; |
2 | 239 |
} |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47216
diff
changeset
|
240 |
return 0; |
2 | 241 |
} |
242 |
} |