Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(320)

Issue 382043003: Add ExternalTimerEventScope to log non internal timer events scopes (Closed)

Created:
6 years, 5 months ago by fmeawad
Modified:
5 years, 6 months ago
Reviewers:
Yang
CC:
v8-dev
Base URL:
git://github.com/v8/v8.git@master
Visibility:
Public.

Description

Add ExternalTimerEventScope to log non internal timer events scopes BUG=3432 LOG=N

Patch Set 1 #

Patch Set 2 : Minor fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -17 lines) Patch
M src/ic.cc View 17 chunks +17 lines, -17 lines 0 comments Download
M src/log.h View 1 1 chunk +18 lines, -0 lines 0 comments Download
M src/log.cc View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
fmeawad
After the addition of the IcMiss timer events, the buffers gets filed rather quickly, and ...
6 years, 5 months ago (2014-07-10 21:35:01 UTC) #1
Yang
6 years, 5 months ago (2014-07-11 07:11:27 UTC) #2
On 2014/07/10 21:35:01, fmeawad-cr wrote:
> After the addition of the IcMiss timer events, the buffers gets filed rather
> quickly, and the logs becomes un-proccessable for many websites and
benchmarks.
> 
> Attempting to allow to log but exclude the IcMisses, feel free to suggest a
> different approach.

There is nothing external to IC misses. Let me come up with a CL.

Powered by Google App Engine
This is Rietveld 408576698