Looks good. (As in earlier review, I wish the duplication of comment and code could be minimized via consolidation into a suitable new method definition ...)<br><br>- ramki<br><br><div class="gmail_quote">On Wed, Jan 18, 2012 at 11:38 AM, John Cuthbertson <span dir="ltr"><<a href="mailto:john.cuthbertson@oracle.com">john.cuthbertson@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Everyone,<br>
<br>
I forgot to include the webrev link: <a href="http://cr.openjdk.java.net/%7Ejohnc/7129514/webrev.0/" target="_blank">http://cr.openjdk.java.net/~<u></u>johnc/7129514/webrev.0/</a><br>
<br>
Thanks to Ramki for pointing this out.<br>
<br>
JohnC<div class="HOEnZb"><div class="h5"><br>
<br>
On 1/18/2012 10:08 AM, John Cuthbertson wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Everyone,<br>
<br>
While making the changes for 711303, I missed a few calls to os::javaTimeMillis() (specifically with updating the time since the last GC). As a consequence we can still see the occasional time-warp warning. The issue is that os::javaTimeMillis() returns values that are not guaranteed to be monotonically non-decreasing and so they can go backwards.  I've replaced these calls to an equivalent that uses os::javaTimeNanos(), which will return values that are monotonically non-decreasing if the underlying system time source supports such a mode.<br>

<br>
Many thanks to David Holmes for diagnosing the issue.<br>
<br>
Thanks,<br>
<br>
JohnC<br>
</blockquote>
<br>
</div></div></blockquote></div><br>