Use CLOCK_MONOTONIC_COARSE and CLOCK_REALTIME_COARSE on Linux if available.

src/platform/time.cc

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 25 matching lines @@
 
 #include <cstring>
 
 #include "checks.h"
 #include "cpu.h"
 #include "platform.h"
 #if V8_OS_WIN
 #include "win32-headers.h"
 #endif
 
+#if V8_OS_LINUX && !defined(CLOCK_MONOTONIC_COARSE)
+#define CLOCK_MONOTONIC_COARSE 6  // 2.6.32 and up.
+#endif
+
 namespace v8 {
 namespace internal {
 
 TimeDelta TimeDelta::FromDays(int days) {
   return TimeDelta(days * Time::kMicrosecondsPerDay);
 }
 
 
 TimeDelta TimeDelta::FromHours(int hours) {
   return TimeDelta(hours * Time::kMicrosecondsPerHour);
@@ skipping 507 matching lines @@
   // TODO(bmeurer): This is a temporary hack to support cross-compiling
   // Chrome for Android in AOSP. Remove this once AOSP is fixed, also
   // cleanup the tools/gyp/v8.gyp file.
   struct timeval tv;
   int result = gettimeofday(&tv, NULL);
   ASSERT_EQ(0, result);
   USE(result);
   ticks = (tv.tv_sec * Time::kMicrosecondsPerSecond + tv.tv_usec);
 #elif V8_OS_POSIX
   struct timespec ts;
+#if V8_OS_LINUX
+  // Prefer CLOCK_MONOTONIC_COARSE, a high-res clock that's available in
+  // 2.6.32 and can be fully serviced from the vDSO, even on virtualized
+  // systems.  Check that it's not *too* coarse though.
+  static clock_t clock_id = -1;

[Benedikt Meurer, 2013/10/31 06:44:50]

I think we can safely use zero here, i.e. no explicit initialization (as Xorg
does).

[bnoordhuis, 2013/10/31 12:22:24]

I wrote it that way because 0 is a valid clock (CLOCK_REALTIME.)  I'm fine with
either though, just say the word.

[Benedikt Meurer, 2013/11/04 09:09:57]

Please just use 0 here.

+  if (clock_id == -1) {
+    if (clock_getres(CLOCK_MONOTONIC_COARSE, &ts) == 0 &&

[Benedikt Meurer, 2013/10/31 06:44:50]

This won't compile on non-Linux systems that miss the CLOCK_MONOTONIC_COARSE
define.

[bnoordhuis, 2013/10/31 12:22:24]

If it weren't for the #if V8_OS_LINUX a few lines up, you mean? :-)

[Benedikt Meurer, 2013/11/04 09:09:57]

Ups, right. Ok, that's fine.

+        ts.tv_nsec <= 1 * 1000 * 1000) {  // 1 ms granularity or less.
+      clock_id = CLOCK_MONOTONIC_COARSE;
+    } else {
+      clock_id = CLOCK_MONOTONIC;
+    }
+  }
+  int result = clock_gettime(clock_id, &ts);
+#else
   int result = clock_gettime(CLOCK_MONOTONIC, &ts);
+#endif  // V8_OS_LINUX
   ASSERT_EQ(0, result);
   USE(result);
   ticks = (ts.tv_sec * Time::kMicrosecondsPerSecond +
            ts.tv_nsec / Time::kNanosecondsPerMicrosecond);
 #endif  // V8_OS_MACOSX
   // Make sure we never return 0 here.
   return TimeTicks(ticks + 1);
 }
 
 
 // static
 bool TimeTicks::IsHighResolutionClockWorking() {
   return true;
 }
 
 #endif  // V8_OS_WIN
 
 } }  // namespace v8::internal