Commit 45d6ef06 authored by yurys@chromium.org's avatar yurys@chromium.org

Use 1ms CPU profiler sampling interval on Android by default

This CL changes default sampling interval on Android from 5ms to 1ms so that it is the same on all platforms. All perf tests on modern devices (Galaxy Nexus, Nexus 7) show that 1ms should be long enough for profiling on mobile devices.

BUG=None
R=bmeurer@chromium.org, loislo@chromium.org

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

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16526 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent c034bb48
......@@ -455,16 +455,8 @@ DEFINE_bool(compilation_cache, true, "enable compilation cache")
DEFINE_bool(cache_prototype_transitions, true, "cache prototype transitions")
// cpu-profiler.cc
#if defined(ANDROID)
// Phones and tablets have processors that are much slower than desktop
// and laptop computers for which current heuristics are tuned.
#define DEFAULT_INTERVAL 5000
#else
#define DEFAULT_INTERVAL 1000
#endif
DEFINE_int(cpu_profiler_sampling_interval, DEFAULT_INTERVAL,
DEFINE_int(cpu_profiler_sampling_interval, 1000,
"CPU profiler sampling interval in microseconds")
#undef DEFAULT_INTERVAL
// debug.cc
DEFINE_bool(trace_debug_json, false, "trace debugging JSON request/response")
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment