Bug 98431 - JSC should have a way to gather and log Heap memory use and pause times
Summary: JSC should have a way to gather and log Heap memory use and pause times
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mark Hahnenberg
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-04 11:21 PDT by Mark Hahnenberg
Modified: 2012-10-05 10:35 PDT (History)
6 users (show)

See Also:


Attachments
Patch (28.32 KB, patch)
2012-10-04 11:37 PDT, Mark Hahnenberg
no flags Details | Formatted Diff | Diff
Patch (30.61 KB, patch)
2012-10-04 12:11 PDT, Mark Hahnenberg
no flags Details | Formatted Diff | Diff
Patch (29.79 KB, patch)
2012-10-04 13:49 PDT, Mark Hahnenberg
no flags Details | Formatted Diff | Diff
Patch (30.34 KB, patch)
2012-10-04 15:14 PDT, Mark Hahnenberg
no flags Details | Formatted Diff | Diff
Patch (30.28 KB, patch)
2012-10-04 16:33 PDT, Mark Hahnenberg
no flags Details | Formatted Diff | Diff
Patch (31.00 KB, patch)
2012-10-04 18:55 PDT, Mark Hahnenberg
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Hahnenberg 2012-10-04 11:21:31 PDT
In order to improve our infrastructure for benchmark-driven development, we should have a centralized method of gathering and logging various statistics about the state of the JS heap. This would allow us to create and to use other tools to analyze the output of the VM after running various workloads.

The first two statistics that might be interesting is memory use by JSC and GC pause times. We can control whether this recording happens through the use of the Options class, allowing us to either use environment variables or command line flags.
Comment 1 Mark Hahnenberg 2012-10-04 11:37:17 PDT
Created attachment 167148 [details]
Patch
Comment 2 WebKit Review Bot 2012-10-04 11:40:44 PDT
Attachment 167148 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1
Source/JavaScriptCore/runtime/Options.h:178:  size_tVal is incorrectly named. Don't use underscores in your identifier names.  [readability/naming/underscores] [4]
Total errors found: 1 in 12 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Mark Hahnenberg 2012-10-04 12:11:52 PDT
Created attachment 167155 [details]
Patch
Comment 4 WebKit Review Bot 2012-10-04 12:13:51 PDT
Attachment 167155 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/CMakeLists.txt', u'S..." exit_code: 1
Source/JavaScriptCore/runtime/Options.h:178:  size_tVal is incorrectly named. Don't use underscores in your identifier names.  [readability/naming/underscores] [4]
Total errors found: 1 in 16 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 Gyuyoung Kim 2012-10-04 12:55:03 PDT
Comment on attachment 167155 [details]
Patch

Attachment 167155 [details] did not pass efl-ews (efl):
Output: http://queues.webkit.org/results/14180136
Comment 6 Early Warning System Bot 2012-10-04 12:57:45 PDT
Comment on attachment 167155 [details]
Patch

Attachment 167155 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/14152623
Comment 7 Build Bot 2012-10-04 13:20:24 PDT
Comment on attachment 167155 [details]
Patch

Attachment 167155 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14182088
Comment 8 Build Bot 2012-10-04 13:30:00 PDT
Comment on attachment 167155 [details]
Patch

Attachment 167155 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/14173275
Comment 9 Early Warning System Bot 2012-10-04 13:38:22 PDT
Comment on attachment 167155 [details]
Patch

Attachment 167155 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/14183075
Comment 10 Mark Hahnenberg 2012-10-04 13:49:26 PDT
Created attachment 167169 [details]
Patch
Comment 11 Build Bot 2012-10-04 14:42:14 PDT
Comment on attachment 167169 [details]
Patch

Attachment 167169 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/14171396
Comment 12 Build Bot 2012-10-04 14:57:04 PDT
Comment on attachment 167169 [details]
Patch

Attachment 167169 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14183108
Comment 13 Mark Hahnenberg 2012-10-04 15:14:42 PDT
Created attachment 167184 [details]
Patch
Comment 14 Build Bot 2012-10-04 15:52:15 PDT
Comment on attachment 167184 [details]
Patch

Attachment 167184 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/14181126
Comment 15 Mark Hahnenberg 2012-10-04 16:33:31 PDT
Created attachment 167202 [details]
Patch
Comment 16 Build Bot 2012-10-04 18:48:04 PDT
Comment on attachment 167202 [details]
Patch

Attachment 167202 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/14178280
Comment 17 Mark Hahnenberg 2012-10-04 18:55:44 PDT
Created attachment 167221 [details]
Patch
Comment 18 Geoffrey Garen 2012-10-04 21:24:47 PDT
Comment on attachment 167221 [details]
Patch

r=me
Comment 19 WebKit Review Bot 2012-10-05 10:35:42 PDT
Comment on attachment 167221 [details]
Patch

Clearing flags on attachment: 167221

Committed r130520: <http://trac.webkit.org/changeset/130520>
Comment 20 WebKit Review Bot 2012-10-05 10:35:46 PDT
All reviewed patches have been landed.  Closing bug.