Looks good, but why are you not using a newer version of webrev with "next" links??
-- - Keith On Fri, Apr 11, 2014 at 10:48 AM, Coleen Phillimore < coleen.phillim...@oracle.com> wrote: > Summary: Dtrace monitoring uses size before mirror size is set. > > The refactoring I did for bug https://bugs.openjdk.java.net/ > browse/JDK-8028497 caused this bug. The size of the mirror is filled in > by the InstanceMirrorKlass allocation but was used for dtrace probes before > it in the normal allocation pass. Pass the allocated size to dtrace > function instead. > > Tested by dtrace tests on solaris sparcv9, testbase vm.quick.testlist also. > > open webrev at http://cr.openjdk.java.net/~coleenp/8039904/ > bug link https://bugs.openjdk.java.net/browse/JDK-8039904 > > Thanks, > Coleen >
