Thanks, Dan!
Serguei

On 6/26/18 12:21, Daniel D. Daugherty wrote:
Thumbs up and the Trivial Change rule applies.

Dan


On 6/26/18 3:14 PM, [email protected] wrote:
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


Reply via email to