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

Issue 10440073: Reduce the amount of log generated with dtrace. (Closed)

Created:
8 years, 6 months ago by M-A Ruel
Modified:
8 years, 5 months ago
Reviewers:
cmp, MAD
CC:
chromium-reviews, pam+watch_chromium.org, csharp
Visibility:
Public.

Description

Reduce the amount of log generated with dtrace. Remove extraneous ppid and errno logging. Have better variable cleanup characteristics to reduce the amount of kernel memory leaked while tracing. Add more comments explaining how the process tracking is done NOTRY=true R=cmp@chromium.org BUG=98636 TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=140425

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+182 lines, -90 lines) Patch
M tools/isolate/trace_inputs.py View 15 chunks +182 lines, -90 lines 3 comments Download

Messages

Total messages: 6 (0 generated)
M-A Ruel
8 years, 6 months ago (2012-05-29 13:58:30 UTC) #1
cmp
lgtm, parts were opaque to me but mostly made sense
8 years, 6 months ago (2012-06-04 22:21:32 UTC) #2
M-A Ruel
On 2012/06/04 22:21:32, cmp wrote: > lgtm, parts were opaque to me but mostly made ...
8 years, 6 months ago (2012-06-04 23:28:28 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/10440073/1
8 years, 6 months ago (2012-06-04 23:28:52 UTC) #4
commit-bot: I haz the power
Change committed as 140425
8 years, 6 months ago (2012-06-04 23:31:03 UTC) #5
MAD
8 years, 5 months ago (2012-06-28 13:04:57 UTC) #6
Just found a few old drafts, in case you care... :-)

http://codereview.chromium.org/10440073/diff/1/tools/isolate/trace_inputs.py
File tools/isolate/trace_inputs.py (right):

http://codereview.chromium.org/10440073/diff/1/tools/isolate/trace_inputs.py#...
tools/isolate/trace_inputs.py:984: errno is not printed in the log since this
implementation currently care only
care only -> only cares

http://codereview.chromium.org/10440073/diff/1/tools/isolate/trace_inputs.py#...
tools/isolate/trace_inputs.py:1098: arg1, sizeof(user_addr_t) * (self->argc +
1));
I guess you can't use a loop for this... A bit sad... :-(

http://codereview.chromium.org/10440073/diff/1/tools/isolate/trace_inputs.py#...
tools/isolate/trace_inputs.py:1184: /* These are a good learning experience,
since it traces a lot of things
These are -> This is

Powered by Google App Engine
This is Rietveld 408576698