|
Thanks, Dan!
Serguei
On 6/26/18 12:21, Daniel D. Daugherty wrote:
Thumbs up and the Trivial Change rule applies.
Dan
Please, review the fix for sub-task:
https://bugs.openjdk.java.net/browse/JDK-8205701
The test HeapMonitorGCCMSTest.java needs to be problem listed
until main bug is fixed:
https://bugs.openjdk.java.net/browse/JDK-8205643
The test failed with the Graal compiler but has a potential to
fail without it as well.
The patch is:
diff -r bba1deda9216 test/hotspot/jtreg/ProblemList.txt
--- a/test/hotspot/jtreg/ProblemList.txt Tue Jun 26 10:43:50
2018 +0800
+++ b/test/hotspot/jtreg/ProblemList.txt Tue Jun 26 12:06:17
2018 -0700
@@ -81,6 +81,7 @@
serviceability/sa/TestRevPtrsForInvokeDynamic.java 8191270
generic-all
serviceability/sa/sadebugd/SADebugDTest.java 8163805
generic-all
+serviceability/jvmti/HeapMonitor/MyPackage/HeapMonitorGCCMSTest.java
8205643 generic-all
#############################################################################
Thanks,
Serguei
|