<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hi John,<div><br></div><div>Thanks, that one fails also.</div><div><br></div><div>Regards,</div><div>Kirk</div><div><br><div><div>On Dec 12, 2011, at 8:48 PM, John Cuthbertson wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">

  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">

<div bgcolor="#ffffff" text="#000000">

Hi Kirk,<br>

<br>

Can you please try <a href="http://monaco.us.oracle.com">monaco.us.oracle.com</a>? I believe the link is supplied

by the webrev tool automatically when it sees the CR# in the mercurial

queue patch. It could be that my path references an old version.<br>

<br>

Thanks,<br>

<br>

JohnC<br>

<br>

On 12/09/11 23:59, Charles K Pepperdine wrote:

<blockquote cite="mid:B13CEB5F-4ADD-4BAB-9783-23C60A4A6A4F@kodewerk.com" type="cite">

  <pre wrap="">Hi John,

The link <a class="moz-txt-link-freetext" href="http://monaco.sfbay.sun.com/detail.jsp?cr=7117303">http://monaco.sfbay.sun.com/detail.jsp?cr=7117303</a> is unreachable.

Regards,

Kirk

On Dec 9, 2011, at 8:30 PM, John Cuthbertson wrote:

  </pre>

  <blockquote type="cite">

    <pre wrap="">Hi Everyone,

I have updated the comments based upon feedback from David Holmes. A new webrev can be found at: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~johnc/7117303/webrev.1/">http://cr.openjdk.java.net/~johnc/7117303/webrev.1/</a>

Thanks,

JohnC

On 12/7/2011 9:59 AM, John Cuthbertson wrote:

    </pre>

    <blockquote type="cite">

      <pre wrap="">Hi Everyone,

Can I have a couple of volunteers review the changes for this CR? The webrev can be found at: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~johnc/7117303/webrev.0/">http://cr.openjdk.java.net/~johnc/7117303/webrev.0/</a>.

Summary:

I replaced the calls to os::javaTimeMillis() in the GC where we expect monotonicity with calls os::javaTimeNanos(), converting the result to milliseconds. os::javaTimeNanos(), at least on some configurations, does guarantee monotonicity and so is a better alternative. The changes in the os_<*> files are to make use of the named conversion constants I added/moved to globalDefinitions.hpp - we seemed to have multiple names for the same two constants.

Testing: GC test suite on solaris and Linux, NSK tests on solaris, and jprt.

Thanks,

JohnC

      </pre>

    </blockquote>

  </blockquote>

  <pre wrap=""><!---->

  </pre>

</blockquote>

<br>

</div>

</blockquote></div><br></div></body></html>