Skip to content

Commit

Permalink
Revert of Use CLOCK_REALTIME_COARSE when available. (patchset v8#3 id…
Browse files Browse the repository at this point in the history
…:40001 of https://codereview.chromium.org/1125003002/)

Reason for revert:
[Sheriff] This causes layout test crashes:
http://build.chromium.org/p/client.v8/builders/V8-Blink%20Linux%2064%20%28dbg%29/builds/2735

You can add the trybot v8_linux_layout_dbg on reland.

Original issue's description:
> Use CLOCK_REALTIME_COARSE when available.
>
> On systems that have CLOCK_REALTIME_COARSE with good enough resolution, we can
> avoid making a system call to get the current time; it's serviced from the vDSO.
>
> BUG=
> LOG=N
>
> Committed: https://crrev.com/537ed7500c087786f28f51ff5222f1c2113776d3
> Cr-Commit-Position: refs/heads/master@{#28280}

TBR=jochen@chromium.org,deanm@chromium.org,bmeurer@chromium.org,ben@strongloop.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Review URL: https://codereview.chromium.org/1130083003

Cr-Commit-Position: refs/heads/master@{#28288}
  • Loading branch information
mi-ac authored and Commit bot committed May 7, 2015
1 parent 2f9411d commit e137650
Showing 1 changed file with 0 additions and 38 deletions.
38 changes: 0 additions & 38 deletions src/base/platform/time.cc
Expand Up @@ -24,11 +24,6 @@
#include "src/base/logging.h"
#include "src/base/platform/platform.h"

// CLOCK_REALTIME_COARSE was added in Linux 2.6.32.
#if V8_OS_LINUX && !defined(CLOCK_REALTIME_COARSE)
#define CLOCK_REALTIME_COARSE 5
#endif

namespace v8 {
namespace base {

Expand Down Expand Up @@ -257,44 +252,11 @@ FILETIME Time::ToFiletime() const {
#elif V8_OS_POSIX

Time Time::Now() {
#ifdef CLOCK_REALTIME_COARSE
struct timespec ts;
// clockid_t is an int32_t; loads and stores are atomic.
static const clockid_t kInvalidClockId = static_cast<clockid_t>(-1);
static clockid_t clock_id = kInvalidClockId;
if (V8_UNLIKELY(clock_id == kInvalidClockId)) {
// CLOCK_REALTIME_COARSE is not supported on Linux kernels < 2.6.32.
// Probe the kernel to see if it's available and has <= 1 ms resolution.
//
// CLOCK_REALTIME_COARSE, unlike CLOCK_REALTIME, can often be serviced
// entirely from the vDSO without the need to make a system call.
// It can have a dramatic impact on applications that frequently
// query the current time.
//
// One caveat is that CLOCK_REALTIME_COARSE is tied to CONFIG_HZ, the
// number of ticks per second that the kernel runs at. Its granularity
// can be as low as one update every 300 ms so we need to make sure that
// it is accurate enough. Fortunately, many if not most kernels are built
// with CONFIG_HZ=1000, giving it a one millisecond precision and that is
// good enough for our purposes.
if (clock_getres(CLOCK_REALTIME_COARSE, &ts) < 0 || ts.tv_sec > 0 ||
ts.tv_nsec > 1000 * 1000) {
clock_id = CLOCK_REALTIME; // Not available or not suitable.
} else {
clock_id = CLOCK_REALTIME_COARSE;
}
}
int result = clock_gettime(clock_id, &ts);
DCHECK_EQ(0, result);
USE(result);
return FromTimespec(ts);
#else
struct timeval tv;
int result = gettimeofday(&tv, NULL);
DCHECK_EQ(0, result);
USE(result);
return FromTimeval(tv);
#endif
}


Expand Down

0 comments on commit e137650

Please sign in to comment.