Re: (S) RFR: 8154710: [Solaris] Investigate use of in-memory low-resolution timestamps for Java and internal time API's

[Aleksey Shipilev]

On 04/29/2016 02:09 AM, David Holmes wrote:
> bug: https://bugs.openjdk.java.net/browse/JDK-8154710
> webrev: http://cr.openjdk.java.net/~dholmes/8154710/webrev/

Looks good.

Is hrtime_t always integral, so you can "(hrtime_t)now /
NANOSECS_PER_MILLISEC" it?

> This change is small in nature but somewhat broad in scope. It "affects"
> the implementation of System.currentTimeMillis() in the Java space, and
> os::javaTimeMillis() in the VM. But on Solaris only.
> 
> I say "affects" but the change will be unobservable other than in terms
> of performance.

Observable enough to me.

Thanks,
-Aleksey