Review request for 8028647: Add instrumentation in GetSafepointSyncTime.java and remove it from ProblemList.txt
mandy.chung at oracle.com
Wed Nov 20 08:14:49 PST 2013
Thanks for the review.
On 11/20/2013 3:59 AM, Chris Hegarty wrote:
> The changes look ok to me, and the approach of adding additional
> diagnostic information makes sense.
> Trivially, you can remove 'trace', as I don't think it is being used
> any more.
Will remove it before I push.
> On 19/11/13 23:12, Mandy Chung wrote:
>> Webrev at:
>> This patch turns the trace on by default and also add the
>> instrumentation to print the safepoint count to help troubleshooting the
>> test failure.
>> test/sun/management/HotspotRuntimeMBean/GetSafepointSyncTime.java is no
>> longer reproducible on jdk8 b109. GetSafepointSyncTime.java should be
>> removed from the ProblemList.txt. This still fails with 7u45 and so I
>> want to backport this change to 7u-dev. There might be a hotspot fix
>> in jdk8 that resolves the zero safepoint sync time issue for further
More information about the serviceability-dev