Hi Staffan,
thanks for reviewing this!
On 01/24/2013 01:27 PM, Staffan Larsen wrote:
The JDK changes look good, but I think there is change needed in
test/sun/tools/jstatd/jstatGcutilOutput1.awk as well.
Yes, you are right. Please see an updated webrev at:
http://cr.openjdk.java.net/~ehelin/8004172/jdk/webrev.01/
Thanks,
Erik
Thanks,
/Staffan
On 24 jan 2013, at 11:51, Erik Helin <[email protected]> wrote:
Hi all,
here are the JDK changes for fixing JDK-8004172. This change updates the jstat
resource file to use the new metaspace performance counters and also updates
the tests.
Note that the fixed tests will continue to fail until the HotSpot part of the
change finds it way into the tl forest.
The HotSpot part of the change is also out for review on
[email protected].
Webrev:
JDK: http://cr.openjdk.java.net/~ehelin/8004172/jdk/webrev.00/
HotSpot: http://cr.openjdk.java.net/~ehelin/8004172/hotspot/webrev.00/
Bug:
http://bugs.sun.com/view_bug.do?bug_id=8004172
Testing:
Run the jstat jtreg tests locally on my machine on a repository where I've
applied both the JDK changes and the HotSpot changes.
Thanks,
Erik
