RFR (S): 8211175: Remove temporary clock initialization duplication (original) (raw)

David Holmes david.holmes at oracle.com
Tue Oct 2 19:57:11 UTC 2018

Thanks Robbin.

David

On 3/10/2018 1:38 AM, Robbin Ehn wrote:

Thanks, looks good.

/Robbin On 10/2/18 8:08 AM, David Holmes wrote: webrev: http://cr.openjdk.java.net/~dholmes/8211175/webrev/ bug: https://bugs.openjdk.java.net/browse/JDK-8211175

This cleans up some code duplication between os::Linux and os::Posix. os::Posix now exports the necessary clock functions so the Linux code can use those. Also cleaned up the clockgetres syscall as that is already provided via the direct clockgetres (no need for a syscall). Finally, as code in os::Linux already assumed clockgettime etc was always available at build-time (and run-time for that matter) this is now explicitly checked via: + #ifndef SUPPORTSCLOCKMONOTONIC + #error "Build platform doesn't support clockgettime and related functionality" + #endif Testing: tiers 1-3 (mach5) Thanks, David

More information about the hotspot-runtime-dev mailing list