JDK-8207765 covers two different tests as of yesterday:
serviceability/jvmti/HeapMonitor/MyPackage/HeapMonitorStatRateTest.java
and
serviceability/jvmti/HeapMonitor/MyPackage/HeapMonitorStatIntervalTest.java
I updated it to add a similar failure mode sighting for
HeapMonitorStatIntervalTest.java
Does your fix address both test failures?
Dan
On 7/19/18 12:39 PM, JC Beyler wrote:
Hi all,
Could I have a few reviews of:
http://cr.openjdk.java.net/~jcbeyler/8207765/webrev.00/
<http://cr.openjdk.java.net/%7Ejcbeyler/8207765/webrev.00/>
The test assumed the size of a 1-element array but ZGC changes that
assumption. The test now first allocates a bit of memory and gets the
average size of the samples before assuming the size. This works
with/without ZGC.
Webrev: http://cr.openjdk.java.net/~jcbeyler/8207765/webrev.00/
<http://cr.openjdk.java.net/%7Ejcbeyler/8207765/webrev.00/>
Bug: https://bugs.openjdk.java.net/browse/JDK-8207765
Thanks!
Jc
