changeset 5616:97eb7a4b1fdd

6948101: java/rmi/transport/pinLastArguments/PinLastArguments.java failing intermittently Reviewed-by: dholmes, smarks Contributed-by: Eric Wang <yiming.wang@oracle.com>
author smarks
date Thu, 05 Jul 2012 15:12:10 -0700
parents 3ae91286f313
children 4ad204cc7433
files test/ProblemList.txt test/java/rmi/transport/pinLastArguments/PinLastArguments.java
diffstat 2 files changed, 9 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/test/ProblemList.txt	Tue Jul 03 20:29:16 2012 -0700
+++ b/test/ProblemList.txt	Thu Jul 05 15:12:10 2012 -0700
@@ -271,9 +271,6 @@
 # 7140992
 java/rmi/server/Unreferenced/finiteGCLatency/FiniteGCLatency.java generic-all
 
-# 6948101
-java/rmi/transport/pinLastArguments/PinLastArguments.java	generic-all
-
 # 7146541
 java/rmi/transport/rapidExportUnexport/RapidExportUnexport.java	linux-all
 
--- a/test/java/rmi/transport/pinLastArguments/PinLastArguments.java	Tue Jul 03 20:29:16 2012 -0700
+++ b/test/java/rmi/transport/pinLastArguments/PinLastArguments.java	Thu Jul 05 15:12:10 2012 -0700
@@ -78,10 +78,15 @@
         }
         impl = null;
 
-        System.gc();
-
-        if (ref.get() != null) {
-            throw new Error("TEST FAILED: impl not garbage collected");
+        // Might require multiple calls to System.gc() for weak-references
+        // processing to be complete. If the weak-reference is not cleared as
+        // expected we will hang here until timed out by the test harness.
+        while (true) {
+            System.gc();
+            Thread.sleep(20);
+            if (ref.get() == null) {
+                break;
+            }
         }
 
         System.err.println("TEST PASSED");